The function _pathauto_user_insert_update() throws warnings when the blog module is not installed which is because the variable $new_user only gets defined when the blog module exists.

The attached patch is fixing that problem.

CommentFileSizeAuthor
#3 788540_pathauto_clone_unconditioned.patch646 bytesjurgenhaas
pathauto_bugfix.patch751 bytesjurgenhaas

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Needs work

I think some other modules create the problem as well (as you found, tracker, contact). I suggest we just clone always and get it out of the conditional.

While we're fixing this, would http://api.drupal.org/api/function/drupal_clone make more sense here?

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Upon reading it, I guess drupal_clone is only necessary for the 6.x version of Pathauto since it has been dropped in Drupal 7.x.

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented
Status: Needs work » Needs review
StatusFileSize
new 788540_pathauto_clone_unconditioned.patch646 bytes

OK, attached is the modified version.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Committed a modified version that just uses the non-cloned user account variable instead in the non-blog module conditions.
http://drupal.org/cvs?commit=377188

Status: Fixed » Closed (fixed)

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