I noticed that on line 2044 of acidfree.module you implement hook_pathauto_node. As of Pathauto 5.x-2.x that hook is no longer supported and you should move to using the token module's functions. I imagine that in the next couple of weeks that version will become stable enough that it will be the default release at which point Acidfree will need to implement the token hooks instead of pathauto in order to work.

Instead of that function, you should now implement two new functions:

hook_token_list
hook_token_values

For more information see the API.txt in the token module http://drupal.org/project/token

During a migration period when some users may still be using Pathauto5.x-1.x it is probably best to leave the hook_pathauto_node in your module so that your module is compatible with both versions.

If you would like more assistance, you can ask questions in the tokens group: http://groups.drupal.org/tokens so that they are around for other folks making the crossover.

For an example you can look at the change in pathauto_node_event.inc
http://cvs.drupal.org/viewcvs/drupal/contributions/modules/pathauto/cont...

Comments

vhmauery’s picture

vhmauery commented
Status: Active » Closed (won't fix)

Thanks for bringing this up. I am a user of pathauto on my sites as well. However, the more I think about it, the more this is unnecessary. Because albums in Acidfree 5.0 are each linked to a taxonomy term, it should be possible to just specify 'albums/[catpath]/[title]' instead of the old 'albums/[acidfreepath]/[title]'.

However, that said, I just tried to do that and found that pathauto just returned '' for the [catpath] token. Oh well. you can't win 'em all. :)

greggles’s picture

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

Well that's not surprising. Catpath has worked sporadically in pathauto - that's part of why the move to token is so exciting.

If you find that bug in the 5.x-2 version of pathauto then please file a bug for it in the pathauto issue queue.