Download & Extend

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

Project:Install Profile API
Version:6.x-2.x-dev
Component:CRUD functions and includes
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

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.

Comments

#1

Assigned to:nestorconde» Anonymous
Status:active» needs review

The submitted patch should fix both issues.

AttachmentSize
install_profile_api-478576.patch 852 bytes

#2

Status:needs review» fixed

Fixed in CVS

#3

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

nobody click here