[gnome-software/misc-flatpak-fixes: 4/6] flatpak: Set app state to INSTALLED more consistently



commit 6089418147eaee8b9389cc2f9f4fc341f2299cca
Author: Matthew Leeds <matthew leeds endlessm com>
Date:   Mon Jan 27 12:51:18 2020 -0800

    flatpak: Set app state to INSTALLED more consistently
    
    In a couple places we were calling gs_flatpak_create_installed() but not
    setting the app to AS_APP_STATE_INSTALLED. Fix that.

 plugins/flatpak/gs-flatpak.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)
---
diff --git a/plugins/flatpak/gs-flatpak.c b/plugins/flatpak/gs-flatpak.c
index 59fe5f70..1188cf6f 100644
--- a/plugins/flatpak/gs-flatpak.c
+++ b/plugins/flatpak/gs-flatpak.c
@@ -1064,6 +1064,8 @@ gs_flatpak_create_installed (GsFlatpak *self,
        /* create new object */
        origin = flatpak_installed_ref_get_origin (xref);
        app = gs_flatpak_create_app (self, origin, FLATPAK_REF (xref));
+       if (gs_app_get_state (app) == AS_APP_STATE_UNKNOWN)
+               gs_app_set_state (app, AS_APP_STATE_INSTALLED);
        gs_flatpak_set_metadata_installed (self, app, xref);
        return g_steal_pointer (&app);
 }
@@ -1085,8 +1087,6 @@ gs_flatpak_add_installed (GsFlatpak *self, GsAppList *list,
        for (guint i = 0; i < xrefs->len; i++) {
                FlatpakInstalledRef *xref = g_ptr_array_index (xrefs, i);
                g_autoptr(GsApp) app = gs_flatpak_create_installed (self, xref);
-               if (gs_app_get_state (app) == AS_APP_STATE_UNKNOWN)
-                       gs_app_set_state (app, AS_APP_STATE_INSTALLED);
                gs_app_list_add (list, app);
        }
 
@@ -1147,8 +1147,6 @@ gs_flatpak_add_sources (GsFlatpak *self, GsAppList *list,
                                       flatpak_remote_get_name (xremote)) != 0)
                                continue;
                        related = gs_flatpak_create_installed (self, xref);
-                       if (gs_app_get_state (related) == AS_APP_STATE_UNKNOWN)
-                               gs_app_set_state (related, AS_APP_STATE_INSTALLED);
                        gs_app_add_related (app, related);
                }
        }


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