Recursive rebuilding of url aliases

Allright, I managed to implement this as a seperate module in only a handfull lines of code, hooray :)

The module is called nodehierarchy_recursive_pathauto . It should be seen as a proof of concept. For actual implementation I think you should add this to the nodehierarchy module directly:

• Create a 'private' nodehierarchy function that wraps the necessary actions (call it nodehierarchy_pathauto_update_recursive or something and it should be in nodehierarchy_pathauto.inc file I think?).
• Call that function from within the nodehierarchyapi hook implementation in nodehierarchy itself (line 244, nodehierarchy.module, dev snapshot). But then you will have to check first if the pathauto module exists. OR
• If you don't want to check for the existence of the pathauto module in the middle of your code (you already do it at init, so it would be a bit ugly): convert the pathauto integration (nodehierarchy_pathauto.inc) to an actual submodule of nodehierarchy (like you do for views integration) and create a new nodehierarchyapi hook implementation in that module (where you don't need to check if pathauto module exists).

The last option looks best IMHO. What do you think Ronan?

Now for the actual code:

/**
 * This module does the following:
 *   Get invoked whenever a node under nodehierarchy control is updated
 *   Get all descendants of that node
 *   Invoke pathauto module on the descendant nodes
 */

/**
 * Implementation of hook_nodehierarchyapi().
 */
function nodehierarchy_recursive_pathauto_nodehierarchyapi($op, &$node) {
  if ($op === 'update_parent') {
    // Get all descendants of the node
    $node_descendants = nodehierarchy_get_descendant_list($node->nid);

    // Before recreating the descendant aliases, make sure this node has it's
    // new alias generated. Or else all hierarchy related pathauto replacement
    // patterns will derive their parent from the old alias.
    module_invoke('pathauto', 'nodeapi', $node, 'update', null, null);

    // Now have the descendant aliases updated by invoking the pathauto nodeapi
    // hook on them.
    foreach ($node_descendants as $node_descendant_id) {
      $node_descendant = node_load($node_descendant_id);
      module_invoke('pathauto', 'nodeapi', $node_descendant, 'update', null, null);
    }
  }
}

Hi Ronan,

Yes, your foresight to develop an api and hook system certainly made this task a LOT easier :) I wish I had such experienced development skills!

If you can't come up with 'the best' solution right now, I suggest you implement it in whichever form anyway. Because, even though it's only a handful lines of code, it's a pretty valuable addition I think? And then you can always re-implement it in some other form later on(?).

Hopla

Hi luron,

I don't know when ronan is planning to integrate this, but you can easily include this yourself. Just create a new module 'nodehierarchy_recursive_pathauto' and copy the code as pasted above in the .module file.

We have been using this same code for months now, in production and so far it never failed :)

Of course it would be nice if this can be included officially, but if you need it here and now: it's pretty easy to do.