diff options
| author | Carlos Garnacho <carlosg@gnome.org> | 2020-11-28 18:53:10 +0100 |
|---|---|---|
| committer | Carlos Garnacho <carlosg@gnome.org> | 2020-12-01 13:07:12 +0100 |
| commit | 1733879a59bb1183062a7314c1f46f7a37cae5ae (patch) | |
| tree | f9aa07cb9c95b400ec866f8adf8a550c479943fc | |
| parent | 9a89c7c6b92bec114a85621dabe86605729f665e (diff) | |
| download | tracker-1733879a59bb1183062a7314c1f46f7a37cae5ae.tar.gz | |
libtracker-data: Prefer tracker_class_get_id() over queries
Drop one place where we did that, and some vestigial comment about it being
done elsewhere in the past. Classes should indeed always have an ID,
regardless of being parsed from ontology, retrieved from the database, or
retrieved from gvdb.
| -rw-r--r-- | src/libtracker-data/tracker-data-update.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/src/libtracker-data/tracker-data-update.c b/src/libtracker-data/tracker-data-update.c index a18bcff1c..0832a0b4d 100644 --- a/src/libtracker-data/tracker-data-update.c +++ b/src/libtracker-data/tracker-data-update.c @@ -970,7 +970,7 @@ tracker_data_resource_buffer_flush (TrackerData *data, if (stmt) { tracker_db_statement_bind_int (stmt, 0, resource->id); - tracker_db_statement_bind_int (stmt, 1, ensure_resource_id (data, tracker_class_get_uri (table->class), NULL)); + tracker_db_statement_bind_int (stmt, 1, tracker_class_get_id (table->class)); tracker_db_statement_execute (stmt, &actual_error); g_object_unref (stmt); } @@ -1244,9 +1244,6 @@ cache_create_service_decomposed (TrackerData *data, cache_insert_row (data, cl); - /* This is the original, no idea why tracker_class_get_id wasn't used here: - * class_id = ensure_resource_id (tracker_class_get_uri (cl), NULL); */ - class_id = tracker_class_get_id (cl); ontologies = tracker_data_manager_get_ontologies (data->manager); |