After hitting the 'Submit Menu Updates' button, I get the yellow message at the top, but when I go to any other pages, the error message

"warning: Invalid argument supplied for foreach() in /sites/all/modules/quick_admin_menu/qam.module on line 460."

appears multiple times. Once for each menu item that was in my menu.

However, the menu is successfully changed upon checking it in the normal /admin/build/menu

CommentFileSizeAuthor
#7 qam.module.patch914 bytesstarbow

Comments

starbow’s picture

starbow commented

Hmm, are you using the lastest -dev? I am pretty sure I got rid of the yellow message box a week ago or so.

sirkitree’s picture

sirkitree commented

I tried the beta one just now because the date stamp on it is more recent then the -dev and got the same thing. No 'yellow' message, alert instead, but still got the original error on line 460. I then tried the -dev one and that on had even more errors.

warning: Missing argument 1 for qam_admin() in /Users/jbitner/Sites/vhq/sites/all/modules/quick_admin_menu/qam.module on line 104.

click a menu

warning: Missing argument 2 for qam_admin_menu(), called in /Users/jbitner/Sites/vhq/sites/all/modules/quick_admin_menu/qam.module on line 134 and defined in /Users/jbitner/Sites/vhq/sites/all/modules/quick_admin_menu/qam.module on line 156.

I believe I've described these in the other issue i posted: http://drupal.org/node/143211

starbow’s picture

starbow commented

Ok, looks like the -dev was lagging behind the -beta. I just bumped the -dev so it will catch up. I have not been able to replicate your issue on my test system using the beta (line 460 is a drupal_set_message call). Can you start from scratch, using the beta and see if this (and your other issue) still exist? If so, please send me step-by-step instructions on how to trigger the errors.
thanks,
-tao

starbow’s picture

starbow commented

Don't worry about the followup, I have figured out how to see the bug...but only on my laptop, not on my main dev system. Weird. Now I just need to figure out what the difference is.

starbow’s picture

starbow commented

Ok, this looks like it is a PHP version issue. My dev machine is running 5.1.4, and the laptop runs 5.1.1. Looking at the PHP changes file, it looks like there were a bunch of SimpleXMLElement fixes in between those two version. I updated the laptop to PHP 5.2.1 and it seems to have fixed the issues. Are you able to update your version of PHP and tell me if it fixes the problem for you?

sirkitree’s picture

sirkitree commented

I'm running php 5.2 on my local machine where I'm testing out your module. I've downloaded the latest beta dated may-11. I do not receive any other errors now that are outlined in my other issue, but I'm still getting this one (warning: Invalid argument supplied for foreach() in [...]/sites/all/modules/quick_admin_menu/qam.module on line 475) after hitting the 'Submit Menu Updates', clicking 'OK' on the success alert, and then going to another page. On that next page is where I receive the errors.

Hope this helps you to troubleshoot.

starbow’s picture

starbow commented
Assigned: Unassigned » starbow
Status: Active » Needs review
StatusFileSize
new qam.module.patch914 bytes

Great, we have forward progress. Can you try out this patch and see if it fixes it for you?

sirkitree’s picture

sirkitree commented

problem solved! nice job.

starbow’s picture

starbow commented
Status: Needs review » Fixed
Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

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