e/desk_wallpaper: Fix setting wallpaper from within Virtual Desk dialog
authorStafford Horne <shorne@gmail.com>
Sun, 8 Nov 2015 03:52:21 +0000 (22:52 -0500)
committerMike Blumenkrantz <zmike@osg.samsung.com>
Sun, 8 Nov 2015 03:52:21 +0000 (22:52 -0500)
Summary:
It seems this was missed when doing the manager/container id removal
back in march.  The param validation was failing and causing the "Set"
button to not work in "Screen > Virtual Desktops > Click to Change
wallpaper".

Test Plan:
1. Open "Screen > Virtual Desktops > Click to Change wallpaper"
2. Click "Set"
  - Expect that a wallpaper dialog be shown
  - Before this patch nothing happens

Reviewers: zmike

Subscribers: cedric, seoz

Differential Revision: https://phab.enlightenment.org/D3298

src/modules/conf_theme/e_int_config_wallpaper.c

index 9b306c2..9b0f1cb 100644 (file)
@@ -64,7 +64,7 @@ e_int_config_wallpaper_desk(Evas_Object *parent EINA_UNUSED, const char *params)
 
    if (!params) return NULL;
    zone_num = desk_x = desk_y = -1;
-   if (sscanf(params, "%i %i %i", &zone_num, &desk_x, &desk_y) != 4)
+   if (sscanf(params, "%i %i %i", &zone_num, &desk_x, &desk_y) != 3)
      return NULL;
    return _e_int_config_wallpaper_desk(NULL, zone_num, desk_x, desk_y);
 }