Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
D6's implementation of url() takes an array rather than a string as was previously the case. Additionally, the API documentation for url() states that l() (for formatting an internal link) may be a better choice than url() as it apparently also correctly handles aliased paths.
Existing calls to URL are causing a PHP Fatal Error, most notably when editing a node, as the following stack trace indicates:
- url(node/2, , , 1) called at [/var/www/html/modules/akismet/akismet.module:1034]
- akismet_notify_moderators(node, stdClass Object) called at [/var/www/html/modules/akismet/akismet.module:474]
- akismet_nodeapi(stdClass Object) called at [/var/www/html/modules/node/node.module:673]
- node_invoke_nodeapi...
- [more drupal core calls]
Comments
Comment #1
alastair CreditAttribution: alastair commentedThe attached patchfile seems to fix the failures caused by this issue. I used url() rather than l() as this seemed the more appropriate solution after studying the documentation on the two functions in more detail.
Comment #2
drewish CreditAttribution: drewish commentedcommitted to HEAD. thanks.
Comment #3
Anonymous (not verified) CreditAttribution: Anonymous commentedAutomatically closed -- issue fixed for two weeks with no activity.