[gnome-software/wip/rancell/featured-snap: 2/3] Use a specially named screenshot as the featured banner

[Robert Ancell <rancell src gnome org>]

commit 819272e92a5b660a19b67fa433e96de06a09bef0
Author: Robert Ancell <robert ancell canonical com>
Date:   Wed Aug 2 13:31:04 2017 +0100

    Use a specially named screenshot as the featured banner

 plugins/snap/gs-plugin-snap.c |   29 +++++++++++++++++++++++++++++
 1 files changed, 29 insertions(+), 0 deletions(-)
---
diff --git a/plugins/snap/gs-plugin-snap.c b/plugins/snap/gs-plugin-snap.c
index bd52822..aea96a5 100644
--- a/plugins/snap/gs-plugin-snap.c
+++ b/plugins/snap/gs-plugin-snap.c
@@ -311,6 +311,9 @@ gs_plugin_add_featured (GsPlugin *plugin,
        g_autoptr(GPtrArray) snaps = NULL;
        SnapdSnap *snap;
        g_autoptr(GsApp) app = NULL;
+       GPtrArray *screenshots;
+       guint i;
+       const gchar *banner_url = NULL;
        g_autofree gchar *css = NULL;
 
        snaps = find_snaps (plugin, SNAPD_FIND_FLAGS_NONE, "featured", NULL, cancellable, error);
@@ -323,6 +326,23 @@ gs_plugin_add_featured (GsPlugin *plugin,
        /* use first snap as the featured app */
        snap = snaps->pdata[0];
        app = snap_to_app (plugin, snap);
+
+       /* if has a sceenshot called 'banner.png' then use that as the banner */
+       screenshots = snapd_snap_get_screenshots (snap);
+       for (i = 0; i < screenshots->len; i++) {
+               SnapdScreenshot *screenshot = screenshots->pdata[i];
+               const gchar *url;
+               g_autofree gchar *filename = NULL;
+
+               url = snapd_screenshot_get_url (screenshot);
+               filename = g_path_get_basename (url);
+               if (g_strcmp0 (filename, "banner.png") == 0 ||
+                   g_strcmp0 (filename, "banner.jpg") == 0) {
+                       banner_url = url;
+                       break;
+               }
+       }
+
        css = g_strdup_printf ("border-color: #000000;\n"
                               "text-shadow: 0 1px 1px rgba(0,0,0,0.5);\n"
                               "color: #ffffff;\n"
@@ -592,9 +612,18 @@ gs_plugin_refine_app (GsPlugin *plugin,
 
                        for (i = 0; i < screenshots->len; i++) {
                                SnapdScreenshot *screenshot = screenshots->pdata[i];
+                               const gchar *url;
+                               g_autofree gchar *filename = NULL;
                                g_autoptr(AsScreenshot) ss = NULL;
                                g_autoptr(AsImage) image = NULL;
 
+                               /* skip sceenshots used for banner when app is featured */
+                               url = snapd_screenshot_get_url (screenshot);
+                               filename = g_path_get_basename (url);
+                               if (g_strcmp0 (filename, "banner.png") == 0 ||
+                                   g_strcmp0 (filename, "banner.jpg") == 0)
+                                       continue;
+
                                ss = as_screenshot_new ();
                                as_screenshot_set_kind (ss, AS_SCREENSHOT_KIND_NORMAL);
                                image = as_image_new ();