Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
In menu.inc, function menu_execute_active_handler() I changed:
if (strlen($arg)) {
To:
if (strlen($arg) && is_array($arguments)) {
That did the trick to get rid of the annoying error messages in the log.
Comment | File | Size | Author |
---|---|---|---|
#5 | callback_0.patch | 1.24 KB | killes@www.drop.org |
#3 | callback.patch | 2.05 KB | killes@www.drop.org |
#2 | menu_19.patch | 732 bytes | Uwe Hermann |
Comments
Comment #1
killes@www.drop.org CreditAttribution: killes@www.drop.org commentednot a patch
Comment #2
Uwe Hermann CreditAttribution: Uwe Hermann commentedHere's a patch (untested).
Comment #3
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedAccording to
http://drupaldocs.org/api/head/function/hook_menu
the callback arguments need to be an array.
The only occurrence in core that needs to be changed is in commment module, patch attached. There are several contrib modules that need patching, though. I've updated the docs for menu.inc a bit.
Comment #4
moshe weitzman CreditAttribution: moshe weitzman commentedis this still an open issue?
Comment #5
killes@www.drop.org CreditAttribution: killes@www.drop.org commentedThe patch to comment module got applied, the docs update for menu.inc didn't. Here is that second part again.
Comment #6
Steven CreditAttribution: Steven commentedCommitted to HEAD, thanks.
Comment #7
Steven CreditAttribution: Steven commentedComment #8
(not verified) CreditAttribution: commented