$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
The submitted patch should fix both issues.