Trying to do a bulk URL update

Fatal error: Call to undefined function pathauto_get_placeholders() in .../sites/all/modules/pathauto/pathauto.module on line 324
CommentFileSizeAuthor
#2 428332_undefined_func.patch649 bytesgreggles
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gbrussel’s picture

gbrussel CreditAttribution: gbrussel commented

Confirmed.

Drupal 6.10
Pathauto 6.x-2.x-dev
Latest dev of Token module

Line 73 of pathauto.module calls: module_load_include('inc', 'pathauto'); to include the pathauto.inc file, where the function pathauto_get_placeholders() is defined.

Line 321 is the function pathauto_node_operations_update(), which doesn't appear to call _pathauto_include();, whereas pathauto_taxonomy($op, $type, $object = NULL) on line 335 does call it.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Active » Needs review
FileSize
428332_undefined_func.patch649 bytes

Thanks for reporting this issue and for the research, gbrussel.

Does the attached patch do what you imagined?

gbrussel’s picture

gbrussel CreditAttribution: gbrussel commented

Patch works as intended. Aliases do update properly. Thanks greggles!

introfini’s picture

introfini CreditAttribution: introfini commented

It also fixes the bug when called trough admin>content node operations (hook_node_operations).

introfini

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Assigned: Unassigned » greggles
Status: Needs review » Fixed

Thanks introfini - fixed in all three branches:

http://drupal.org/cvs?commit=198440
http://drupal.org/cvs?commit=198442
http://drupal.org/cvs?commit=198444

Status: Fixed » Closed (fixed)

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