I have reverted to using pathauto 6.x-1.dev (2009 Oct 08) to solve this problem.
In 6.x-1.2 and the current version the auto alias is not active when new content is created. This does not occur with all content types. In many the auto alias is active. However, with Support module tickets it was not the case.
Testing different versions of the Support module makes no difference. However, reverting to the earlier pathauto dev solves the problem.
This problem is significant because new tickets and possibly other content created will require url alias to be updated.
Thanks,
Izzy
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | pathauto.isset_.patch | 597 bytes | brad.bulger |
Comments
Comment #1
brad.bulger commentedit looks to me that this is caused by the way it tests to see if this is a new node:
if (isset($node->nid)) {this will test as TRUE if $node->nid is 0 (only NULL fails isset()). changing it to
if (!empty($node->nid)) {makes the checkbox come up checked when creating a node.
zero vs NULL in nid may be something about the content type - in my case, it's a CCK content type that comes from a feature, that may be an issue.
Comment #2
izmeez commentedBrad,
Thanks. The patch works. I have changed status to needs review. Hope that's the right thing to do.
Izzy
Comment #3
izmeez commentedok, as I novice it is probably my own fault this has not moved forward to a commit.
I have tested the latest pathauto dev 2009-12-06 and the same problem exists.
I applied the patch and it works.
I am changing the status to "reviewed and tested by the community" to move this forward to commit.
Thanks,
Izzy
Comment #4
dave reidMakes sense. I agree with the RTBC. Should also be applied to the 6.x-2.x branch.
Comment #5
pepemty commentedHello everybody!
I'm using 6.x-2.x-dev (2009-Dic-05) and have the same problem, only it's happening with all content types.
Guess that patch won't work with my version, right?
Warm regards from sunny México!
:-)
Pepe
Comment #6
izmeez commentedTry it and report back. It's only one line, you can do it manually. Thanks,
Izzy
Comment #7
lelizondo commentedis not working for me, I'm using 6.x-1.2.
The checkbox is selected, but pathauto won't generate a new alias.
Comment #8
izmeez commented@lelizondob
Can you please clarify. Is the patch not working for you on 6.x-1.2 ?
Thanks,
Izzy
Comment #9
lelizondo commentedyes, the patch didn't work with 6.x-1.2. I just didn't want to move the issue's version
Comment #10
izmeez commentedHave you tried testing with the latest dev release and the patch? It has worked for us.
Thanks,
Izzy
Comment #11
lelizondo commentedNo, I will do that.
Comment #12
lelizondo commentedI've tried with 6.x-1.x-dev and it's not working, the checkbox is selected but pathauto isn't generating the alias, not even on bulk update.
Maybe this has something to do with something else.
Moving the status till someone can confirm this is actually working and I'm just a weird case.
Comment #13
lelizondo commentedI'm using 6.x-1.1 now and is working.
Comment #14
izmeez commentedDid you apply the patch automagically or manually? It may be that the patch needs a new diff to identify the correct line number for the change.
Comment #15
pepemty commentedRe #5 & #6
Yep! I did it manually, and it did the trick... ;-)
Thanks so much, really!
Warmest regards from sunny México!
:-)
Pepe
Comment #16
greggles@lelizondob it sounds like your problem is somewhat unrelated.
Comment #17
izmeez commentedCan the status of this be changed back to RTC ?
Comment #18
lelizondo commented@greggles Probably, but I don't know where to start looking since pathauto 6.x-1.1 is working
Comment #19
izmeez commented@lelizondob - Can you describe your problem in more specific detail? Thanks,
Comment #20
lelizondo commented@izmeez I don't know what else to tell you, the 'Create automatic alias' checkbox is selected, pathauto is properly configured, but pathauto won't generate the alias, when using pathauto 6.x-1.1 everything works smoothly.
I'll try to path the file manually and see what happens. I saw the release notes of 6.x-1.2 and I saw most of the changes were translations, any particular issue or change that could be affecting this so I could check it out, please let me know. I'll be willing to test since this is such an important module for many sites.
Comment #21
summit commentedHi,
Confirmation this is working also on version Pathauto 6.x-2.x-dev (2009-Dec-06).
After changing this one line of code the automatic alias was checked again!
So please also change this line in new Pathauto 6.x-2.x-dev version.
Thanks a lot in advance.
greetings,
Martijn
Comment #22
izmeez commentedI am not a maintainer of this module but found this fix essential so I have again changed the status back to RTBC.
Thanks,
Izzy
Comment #23
garbo commentedI noticed that pathauto and the "node-as-block" module don't work well together.
see: http://drupal.org/node/707956
Comment #24
gregglesNow fixed in 6.x-1.x-dev, 6.x-2.x-dev, 7.x-1.x-dev
http://drupal.org/cvs?commit=326996
http://drupal.org/cvs?commit=326998
http://drupal.org/cvs?commit=327000
Thanks everyone!
Comment #26
interestingaftermath commentedHas this item been fixed in the 6.x-1.4 release? I am experiencing this exact issue in 6.x-1.4. When I edit a node that has not been aliased, the Automatic alias option is not selected by default.
Sorry for opening the issue back up... I'm not sure if that's what you're supposed to do or not.
Comment #27
dave reid@interestingaftermath: Can you please file a new issue detailing exactly what's going wrong for you instead of opening tickets that have been closed for several months or years?
Comment #28
interestingaftermath commentedSigh... I sure will. It's hard to know what format each dev wants their issues organized. Some want to keep all issues grouped, some want new issues for issues related this items from months ago. A bit confusing indeed :)