[gnome-photos/wip/rishi/fix-tracker-assertions: 1/2] item-manager: Avoid CRITICALs when adding an item to a mode
- From: Debarshi Ray <debarshir src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-photos/wip/rishi/fix-tracker-assertions: 1/2] item-manager: Avoid CRITICALs when adding an item to a mode
- Date: Thu, 10 Sep 2020 18:23:14 +0000 (UTC)
commit 6dd26bd6022d4aa4dc40cdfad2e1e32aa0cb0d57
Author: Sam Thursfield <sam afuera me uk>
Date: Thu Sep 10 19:02:32 2020 +0200
item-manager: Avoid CRITICALs when adding an item to a mode
The mtime is stored as nfo:fileLastModified or nie:contentLastModified
in the Tracker database. These properties are of type xsd:dateTime,
and are returned as strings, not booleans. Hence it led to:
Tracker-CRITICAL **: tracker_sparql_cursor_real_get_boolean:
assertion '_tmp0_ == TRACKER_SPARQL_VALUE_TYPE_BOOLEAN' failed
These CRITICALs were more often (always?) triggered when using
TrackerSparqlConnection's 'bus' backend that uses D-Bus to access the
Tracker database. eg., the Flatpak builds that don't bundle the Tracker
D-Bus services.
Fallout from 3ad413e3dc7b5375efebf4e32bce6c66be24d6a5
https://gitlab.gnome.org/GNOME/gnome-photos/-/merge_requests/149
src/photos-item-manager.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
---
diff --git a/src/photos-item-manager.c b/src/photos-item-manager.c
index c44fbbfc..337891c1 100644
--- a/src/photos-item-manager.c
+++ b/src/photos-item-manager.c
@@ -181,7 +181,7 @@ photos_item_manager_can_add_cursor_for_mode (PhotosItemManager *self,
gboolean ret_val = TRUE;
gint64 mtime;
- mtime = tracker_sparql_cursor_get_boolean (cursor, PHOTOS_QUERY_COLUMNS_MTIME);
+ mtime = photos_utils_get_mtime_from_sparql_cursor (cursor);
ret_val = photos_item_manager_can_add_mtime_for_mode (self, mtime, mode);
return ret_val;
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]