#682784 by andrewfn, naxoc, Island Usurper: make it possible to remove query strings...
authorGábor Hojtsy
Wed, 2 Jun 2010 12:56:20 +0000 (12:56 +0000)
committerGábor Hojtsy
Wed, 2 Jun 2010 12:56:20 +0000 (12:56 +0000)
modules/menu/menu.admin.inc

index a8e8b49..1092afb 100644 (file)
@@ -344,9 +344,17 @@ function menu_edit_item_validate($form, &$form_state) {
     if (isset($parsed_link['query'])) {
       $item['options']['query'] = $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'];
     }