Oops. Roll-back of #328110. Apologies

1 files changed, 2 insertions, 2 deletions

diff --git a/includes/menu.inc b/includes/menu.inc
index d80b6dce2..f4f8ecd29 100644
--- a/includes/menu.inc
+++ b/includes/menu.inc
@@ -1913,7 +1913,6 @@ function menu_link_save(&$item) {
   $existing_item = FALSE;
   if (isset($item['mlid'])) {
     $existing_item = db_fetch_array(db_query("SELECT * FROM {menu_links} WHERE mlid = %d", $item['mlid']));
-    $existing_item['options'] = unserialize($existing_item['options']);
   }
 
   if (isset($item['plid'])) {
@@ -2018,6 +2017,7 @@ function menu_link_save(&$item) {
       $item['router_path'] = _menu_find_router_path($menu, $item['link_path']);
     }
   }
+  $item['options'] = serialize($item['options']);
   // If every value in $existing_item is the same in the $item, there is no
   // reason to run the update queries or clear the caches. We use
   // array_diff_assoc() with the $existing_item as the first parameter
@@ -2033,7 +2033,7 @@ function menu_link_save(&$item) {
       $item['router_path'], $item['hidden'], $item['external'], $item['has_children'],
       $item['expanded'], $item['weight'],  $item['depth'],
       $item['p1'], $item['p2'], $item['p3'], $item['p4'], $item['p5'], $item['p6'], $item['p7'], $item['p8'], $item['p9'],
-      $item['module'],  $item['link_title'], serialize($item['options']), $item['customized'], $item['mlid']);
+      $item['module'],  $item['link_title'], $item['options'], $item['customized'], $item['mlid']);
     // Check the has_children status of the parent.
     _menu_update_parental_status($item);
     menu_cache_clear($menu_name);