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:

  1. url(node/2, , , 1) called at [/var/www/html/modules/akismet/akismet.module:1034]
  2. akismet_notify_moderators(node, stdClass Object) called at [/var/www/html/modules/akismet/akismet.module:474]
  3. akismet_nodeapi(stdClass Object) called at [/var/www/html/modules/node/node.module:673]
  4. node_invoke_nodeapi...
  5. [more drupal core calls]
CommentFileSizeAuthor
#1 url.patch650 bytesalastair
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alastair’s picture

FileSize
650 bytes

The 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.

drewish’s picture

Status: Active » Fixed

committed to HEAD. thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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