From 72c33033f84862215f25e04add12cad14bc6cc1f Mon Sep 17 00:00:00 2001
From: YJSoft <YJSoft@users.noreply.github.com>
Date: Tue, 7 Jul 2015 12:28:59 +0900
Subject: [PATCH] =?UTF-8?q?fix=20#1586=20=EB=A9=94=EB=89=B4=20=EB=85=B8?=
 =?UTF-8?q?=EC=B6=9C=20=EB=8C=80=EC=83=81=20=EA=B7=B8=EB=A3=B9=EC=9D=B4=20?=
 =?UTF-8?q?1=EA=B0=9C=20=EC=9D=B4=EC=83=81=EC=9D=BC=20=EA=B2=BD=EC=9A=B0?=
 =?UTF-8?q?=20=EB=A9=94=EB=89=B4=EB=AA=85=20=EC=88=98=EC=A0=95=EC=9D=B4=20?=
 =?UTF-8?q?=EC=95=88=EB=90=98=EB=8A=94=20=EB=AC=B8=EC=A0=9C=20=EC=88=98?=
 =?UTF-8?q?=EC=A0=95?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

`$args->group_srls`가 없는 경우에만 `$args->group_srls`를 unset하도록 되어 있습니다.
하지만 노출 권한 그룹이 두개 이상 지정된 경우 `$args->group_srls`값은 array인데, array를 별도 처리 없이 쿼리에 넣으면서 문제가 생깁니다.(실제 권한이 업데이트되는 `procMenuAdminUpdateAuth` act에서는 그룹 srl값을 `,`로 implode 해주고 있습니다)

메뉴 업데이트가 이루어지는 `procMenuAdminUpdateItem` act에서는 메뉴의 편집이 이루어지지 않고, 실제 쿼리에 들어가는 `$args->group_srls` 값도 원본 메뉴 그룹 그대로이기에 업데이트 쿼리에 포함시킬 이유가 없으므로, 해당 값을 무조건 unset하도록 변경했습니다.
---
 modules/menu/menu.admin.controller.php | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/modules/menu/menu.admin.controller.php b/modules/menu/menu.admin.controller.php
index ad66c74fe..bd4105a18 100644
--- a/modules/menu/menu.admin.controller.php
+++ b/modules/menu/menu.admin.controller.php
@@ -728,11 +728,8 @@ class menuAdminController extends menu
 		
 		if($request->menu_desc) $args->desc = $request->menu_desc;
 		else $args->desc = '';
-
-		if(count($args->group_srls) == 0)
-		{
-			unset($args->group_srls);
-		}
+		
+		unset($args->group_srls);
 		$args->open_window = $request->menu_open_window;
 		$args->expand = $request->menu_expand;
 		$output = executeQuery('menu.updateMenuItem', $args);