From fe4cd157823be2618f14a0d2415c33496e5d8142 Mon Sep 17 00:00:00 2001 From: Stafford Horne Date: Sat, 7 Nov 2015 22:52:21 -0500 Subject: [PATCH] e/desk_wallpaper: Fix setting wallpaper from within Virtual Desk dialog 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 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/modules/conf_theme/e_int_config_wallpaper.c b/src/modules/conf_theme/e_int_config_wallpaper.c index 9b306c2..9b0f1cb 100644 --- a/src/modules/conf_theme/e_int_config_wallpaper.c +++ b/src/modules/conf_theme/e_int_config_wallpaper.c @@ -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); } -- 2.7.4