From 441ea23bbd2447e75a06622239bf35fe137203de Mon Sep 17 00:00:00 2001
From: Dries Buytaert <dries@buytaert.net>
Date: Mon, 1 Feb 2010 19:19:35 +0000
Subject: - Patch #682784 by andrewfn: once created, menu item query strings
 could never be deleted.

---
 modules/menu/menu.admin.inc | 8 ++++++++
 1 file changed, 8 insertions(+)

(limited to 'modules/menu')

diff --git a/modules/menu/menu.admin.inc b/modules/menu/menu.admin.inc
index da28e8fb3..56d9f747b 100644
--- a/modules/menu/menu.admin.inc
+++ b/modules/menu/menu.admin.inc
@@ -348,9 +348,17 @@ function menu_edit_item_validate($form, &$form_state) {
     if (isset($parsed_link['query'])) {
       $item['options']['query'] = drupal_get_query_array($parsed_link['query']);
     }
+    else {
+      // Use unset() rather than setting to empty string
+      // to avoid redundant serialized data being stored.
+      unset($item['options']['query']);
+    }
     if (isset($parsed_link['fragment'])) {
       $item['options']['fragment'] = $parsed_link['fragment'];
     }
+    else {
+      unset($item['options']['fragment']);
+    }
     if ($item['link_path'] != $parsed_link['path']) {
       $item['link_path'] = $parsed_link['path'];
     }
-- 
cgit v1.2.3