[gnome-software: 7/8] gs-odrs-provider: Drop unnecessary cached origin




commit a7afe002b790c5dc16dc20158a314b425b138bbc
Author: Philip Withnall <pwithnall endlessos org>
Date:   Tue Feb 22 16:59:43 2022 +0000

    gs-odrs-provider: Drop unnecessary cached origin
    
    It doesn’t add anything any more, apart from a prefix string on some
    error messages. That doesn’t seem worth the complexity.
    
    Helps: #1472
    
    Signed-off-by: Philip Withnall <pwithnall endlessos org>

 lib/gs-odrs-provider.c | 9 ---------
 1 file changed, 9 deletions(-)
---
diff --git a/lib/gs-odrs-provider.c b/lib/gs-odrs-provider.c
index 0286ddea4..e3bbd53ce 100644
--- a/lib/gs-odrs-provider.c
+++ b/lib/gs-odrs-provider.c
@@ -71,7 +71,6 @@ struct _GsOdrsProvider
        gchar           *review_server;  /* (not nullable) (owned) */
        GArray          *ratings;  /* (element-type GsOdrsRating) (mutex ratings_mutex) (owned) (nullable) */
        GMutex           ratings_mutex;
-       GsApp           *cached_origin;
        guint64          max_cache_age_secs;
        guint            n_results_max;
        SoupSession     *session;  /* (owned) (not nullable) */
@@ -807,7 +806,6 @@ gs_odrs_provider_fetch_for_app (GsOdrsProvider  *self,
                                     GS_ODRS_PROVIDER_ERROR,
                                     GS_ODRS_PROVIDER_ERROR_DOWNLOADING,
                                     "status code invalid");
-               gs_utils_error_add_origin_id (error, self->cached_origin);
                return NULL;
        }
        reviews = gs_odrs_provider_parse_reviews (self, downloaded_data, downloaded_data_length, error);
@@ -1019,11 +1017,6 @@ static void
 gs_odrs_provider_init (GsOdrsProvider *self)
 {
        g_mutex_init (&self->ratings_mutex);
-
-       /* add source */
-       self->cached_origin = gs_app_new ("odrs");
-       gs_app_set_kind (self->cached_origin, AS_COMPONENT_KIND_REPOSITORY);
-       gs_app_set_origin_hostname (self->cached_origin, self->review_server);
 }
 
 static void
@@ -1122,7 +1115,6 @@ gs_odrs_provider_dispose (GObject *object)
 {
        GsOdrsProvider *self = GS_ODRS_PROVIDER (object);
 
-       g_clear_object (&self->cached_origin);
        g_clear_object (&self->session);
 
        G_OBJECT_CLASS (gs_odrs_provider_parent_class)->dispose (object);
@@ -1712,7 +1704,6 @@ gs_odrs_provider_add_unvoted_reviews (GsOdrsProvider  *self,
                                     GS_ODRS_PROVIDER_ERROR,
                                     GS_ODRS_PROVIDER_ERROR_DOWNLOADING,
                                     "status code invalid");
-               gs_utils_error_add_origin_id (error, self->cached_origin);
                return FALSE;
        }
        g_debug ("odrs returned: %.*s", (gint) downloaded_data_length, (const gchar *) downloaded_data);


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]