$curr_menu['options'] should be unserialized in install_menu_set_menu()

nestorconde - June 1, 2009 - 11:31
Project:Install Profile API
Version:6.x-2.x-dev
Component:CRUD functions and includes
Category:bug report
Priority:normal
Assigned:Unassigned
Status:needs review
Description

In function install_menu_set_menu()

The current implementation does not unserialize the $curr_menu['options'] after fetching it from the database. If the item is then saved with menu_link_save() it gets serialized resulting in a double serialization in the database.

There is too a loop on db_fetch_array for the results of the query. However since $mlid identifies uniquely the column this is loop not necessary.

#1

nestorconde - June 1, 2009 - 11:42
Assigned to:nestorconde» Anonymous
Status:active» needs review

The submitted patch should fix both issues.

AttachmentSize
install_profile_api-478576.patch 852 bytes
 
 

Drupal is a registered trademark of Dries Buytaert.