[gnome-software] trivial: Factor out gs_utils_get_content_type()



commit 3b0940404d34d54624c4f630dab3bd2ae446fff9
Author: Richard Hughes <richard hughsie com>
Date:   Thu Apr 7 08:51:26 2016 +0100

    trivial: Factor out gs_utils_get_content_type()

 src/gs-utils.c                             |   27 ++++++++++++++
 src/gs-utils.h                             |    3 ++
 src/plugins/gs-plugin-fwupd.c              |   52 ++++-----------------------
 src/plugins/gs-plugin-packagekit-refresh.c |   53 ++++++----------------------
 src/plugins/gs-plugin-xdg-app.c            |   45 ++++--------------------
 5 files changed, 55 insertions(+), 125 deletions(-)
---
diff --git a/src/gs-utils.c b/src/gs-utils.c
index a7ff553..b809e57 100644
--- a/src/gs-utils.c
+++ b/src/gs-utils.c
@@ -711,4 +711,31 @@ gs_utils_get_content_rating (void)
 }
 #endif
 
+/**
+ * gs_utils_get_content_type:
+ */
+gchar *
+gs_utils_get_content_type (const gchar *filename,
+                          GCancellable *cancellable,
+                          GError **error)
+{
+       const gchar *tmp;
+       g_autoptr(GFile) file = NULL;
+       g_autoptr(GFileInfo) info = NULL;
+
+       /* get content type */
+       file = g_file_new_for_path (filename);
+       info = g_file_query_info (file,
+                                 G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE,
+                                 G_FILE_QUERY_INFO_NONE,
+                                 cancellable,
+                                 error);
+       if (info == NULL)
+               return NULL;
+       tmp = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE);
+       if (tmp == NULL)
+               return NULL;
+       return g_strdup (tmp);
+}
+
 /* vim: set noexpandtab: */
diff --git a/src/gs-utils.h b/src/gs-utils.h
index 9b6bb3c..eea7686 100644
--- a/src/gs-utils.h
+++ b/src/gs-utils.h
@@ -35,6 +35,9 @@ void   gs_container_remove_all        (GtkContainer   *container);
 void    gs_grab_focus_when_mapped      (GtkWidget      *widget);
 
 guint   gs_utils_get_file_age          (GFile          *file);
+gchar  *gs_utils_get_content_type      (const gchar    *filename,
+                                        GCancellable   *cancellable,
+                                        GError         **error);
 
 void    gs_app_notify_installed        (GsApp          *app);
 void    gs_app_notify_failed_modal     (GsApp          *app,
diff --git a/src/plugins/gs-plugin-fwupd.c b/src/plugins/gs-plugin-fwupd.c
index 8355ca5..c5dfdb2 100644
--- a/src/plugins/gs-plugin-fwupd.c
+++ b/src/plugins/gs-plugin-fwupd.c
@@ -766,45 +766,6 @@ gs_plugin_app_update (GsPlugin *plugin,
 }
 
 /**
- * gs_plugin_fwupd_content_type_matches:
- */
-static gboolean
-gs_plugin_fwupd_content_type_matches (const gchar *filename,
-                                     gboolean *matches,
-                                     GCancellable *cancellable,
-                                     GError **error)
-{
-       const gchar *tmp;
-       guint i;
-       g_autoptr(GFile) file = NULL;
-       g_autoptr(GFileInfo) info = NULL;
-       const gchar *mimetypes[] = {
-               "application/vnd.ms-cab-compressed",
-               NULL };
-
-       /* get content type */
-       file = g_file_new_for_path (filename);
-       info = g_file_query_info (file,
-                                 G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE,
-                                 G_FILE_QUERY_INFO_NONE,
-                                 cancellable,
-                                 error);
-       if (info == NULL)
-               return FALSE;
-       tmp = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE);
-
-       /* match any */
-       *matches = FALSE;
-       for (i = 0; mimetypes[i] != NULL; i++) {
-               if (g_strcmp0 (tmp, mimetypes[i]) == 0) {
-                       *matches = TRUE;
-                       break;
-               }
-       }
-       return TRUE;
-}
-
-/**
  * gs_plugin_filename_to_app:
  */
 gboolean
@@ -814,17 +775,18 @@ gs_plugin_filename_to_app (GsPlugin *plugin,
                           GCancellable *cancellable,
                           GError **error)
 {
-       gboolean supported;
+       g_autofree gchar *content_type = NULL;
        g_autoptr(FwupdResult) res = NULL;
        g_autoptr(GsApp) app = NULL;
+       const gchar *mimetypes[] = {
+               "application/vnd.ms-cab-compressed",
+               NULL };
 
        /* does this match any of the mimetypes we support */
-       if (!gs_plugin_fwupd_content_type_matches (filename,
-                                                  &supported,
-                                                  cancellable,
-                                                  error))
+       content_type = gs_utils_get_content_type (filename, cancellable, error);
+       if (content_type == NULL)
                return FALSE;
-       if (!supported)
+       if (!g_strv_contains (mimetypes, content_type))
                return TRUE;
 
        /* get results */
diff --git a/src/plugins/gs-plugin-packagekit-refresh.c b/src/plugins/gs-plugin-packagekit-refresh.c
index 2b1bd92..2828405 100644
--- a/src/plugins/gs-plugin-packagekit-refresh.c
+++ b/src/plugins/gs-plugin-packagekit-refresh.c
@@ -26,6 +26,7 @@
 #include <glib/gi18n.h>
 
 #include <gs-plugin.h>
+#include <gs-utils.h>
 
 #include "packagekit-common.h"
 
@@ -197,42 +198,6 @@ gs_plugin_packagekit_refresh_set_text (GsApp *app, const gchar *text)
 }
 
 /**
- * gs_plugin_packagekit_refresh_content_type_matches:
- */
-static gboolean
-gs_plugin_packagekit_refresh_content_type_matches (const gchar *filename,
-                                                  gboolean *matches,
-                                                  GCancellable *cancellable,
-                                                  GError **error)
-{
-       const gchar *tmp;
-       guint i;
-       g_autoptr(GFile) file = NULL;
-       g_autoptr(GFileInfo) info = NULL;
-       const gchar *mimetypes[] = {
-               "application/x-app-package",
-               "application/x-deb",
-               "application/x-redhat-package-manager",
-               "application/x-rpm",
-               NULL };
-
-       /* get content type */
-       file = g_file_new_for_path (filename);
-       info = g_file_query_info (file,
-                                 G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE,
-                                 G_FILE_QUERY_INFO_NONE,
-                                 cancellable,
-                                 error);
-       if (info == NULL)
-               return FALSE;
-       tmp = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE);
-
-       /* match any */
-       *matches = tmp != NULL && g_strv_contains (mimetypes, tmp);
-       return TRUE;
-}
-
-/**
  * gs_plugin_packagekit_refresh_guess_app_id:
  */
 static gboolean
@@ -305,24 +270,28 @@ gs_plugin_filename_to_app (GsPlugin *plugin,
                           GError **error)
 {
        const gchar *package_id;
-       gboolean supported;
        PkDetails *item;
        ProgressData data;
        g_autoptr (PkResults) results = NULL;
        g_autofree gchar *basename = NULL;
+       g_autofree gchar *content_type = NULL;
        g_autofree gchar *license_spdx = NULL;
        g_auto(GStrv) files = NULL;
        g_auto(GStrv) split = NULL;
        g_autoptr(GPtrArray) array = NULL;
        g_autoptr(GsApp) app = NULL;
+       const gchar *mimetypes[] = {
+               "application/x-app-package",
+               "application/x-deb",
+               "application/x-redhat-package-manager",
+               "application/x-rpm",
+               NULL };
 
        /* does this match any of the mimetypes we support */
-       if (!gs_plugin_packagekit_refresh_content_type_matches (filename,
-                                                               &supported,
-                                                               cancellable,
-                                                               error))
+       content_type = gs_utils_get_content_type (filename, cancellable, error);
+       if (content_type == NULL)
                return FALSE;
-       if (!supported)
+       if (!g_strv_contains (mimetypes, content_type))
                return TRUE;
 
        data.plugin = plugin;
diff --git a/src/plugins/gs-plugin-xdg-app.c b/src/plugins/gs-plugin-xdg-app.c
index bf25be4..3d573f1 100644
--- a/src/plugins/gs-plugin-xdg-app.c
+++ b/src/plugins/gs-plugin-xdg-app.c
@@ -1378,38 +1378,6 @@ gs_plugin_app_update (GsPlugin *plugin,
 }
 
 /**
- * gs_plugin_xdg_app_content_type_matches:
- */
-static gboolean
-gs_plugin_xdg_app_content_type_matches (const gchar *filename,
-                                     gboolean *matches,
-                                     GCancellable *cancellable,
-                                     GError **error)
-{
-       const gchar *tmp;
-       g_autoptr(GFile) file = NULL;
-       g_autoptr(GFileInfo) info = NULL;
-       const gchar *mimetypes[] = {
-               "application/vnd.xdgapp",
-               NULL };
-
-       /* get content type */
-       file = g_file_new_for_path (filename);
-       info = g_file_query_info (file,
-                                 G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE,
-                                 G_FILE_QUERY_INFO_NONE,
-                                 cancellable,
-                                 error);
-       if (info == NULL)
-               return FALSE;
-       tmp = g_file_info_get_attribute_string (info, G_FILE_ATTRIBUTE_STANDARD_CONTENT_TYPE);
-
-       /* match any */
-       *matches = tmp != NULL && g_strv_contains (mimetypes, tmp);
-       return TRUE;
-}
-
-/**
  * gs_plugin_filename_to_app:
  */
 gboolean
@@ -1419,7 +1387,7 @@ gs_plugin_filename_to_app (GsPlugin *plugin,
                           GCancellable *cancellable,
                           GError **error)
 {
-       gboolean supported;
+       g_autofree gchar *content_type = NULL;
        g_autofree gchar *id_prefixed = NULL;
        g_autoptr(GBytes) appstream_gz = NULL;
        g_autoptr(GBytes) icon_data = NULL;
@@ -1427,14 +1395,15 @@ gs_plugin_filename_to_app (GsPlugin *plugin,
        g_autoptr(GFile) file = NULL;
        g_autoptr(GsApp) app = NULL;
        g_autoptr(XdgAppBundleRef) xref_bundle = NULL;
+       const gchar *mimetypes[] = {
+               "application/vnd.xdgapp",
+               NULL };
 
        /* does this match any of the mimetypes we support */
-       if (!gs_plugin_xdg_app_content_type_matches (filename,
-                                                    &supported,
-                                                    cancellable,
-                                                    error))
+       content_type = gs_utils_get_content_type (filename, cancellable, error);
+       if (content_type == NULL)
                return FALSE;
-       if (!supported)
+       if (!g_strv_contains (mimetypes, content_type))
                return TRUE;
 
        /* load bundle */


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