since Pathauto is installed on a LOT of systems. And right now, you get the Pathauto "hide automatic alias" checkbox showing up (by itself, instead of inside the 'path' fieldset), for content types that have "Remove the path options on content types:" selected.

CommentFileSizeAuthor
#1 jammer-pathauto-fixed.patch970 bytesroderik
jammer-pathauto.patch972 bytesroderik

Comments

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
StatusFileSize
new jammer-pathauto-fixed.patch970 bytes

Fixing deadly stupid mistake... :-x

greggles’s picture

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

2 is a good number. I feel like Jammer's purpose makes it likely that a much higher number (100?) would be even better.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Actually the whole module needs to be rewritten. Using unset($form['stuff']); is BAD and will cause things to go wrong. If you want to hide things, you need to use $form['stuff']['#access'] = FALSE;. Every time someone uses the former a kitten dies. Seriously.

ergophobe’s picture

ergophobe commented

Deleted [sorry, thought this was related to another issue, but hadn't read that issue carefully enough]

Bastlynn’s picture

Bastlynn commented
Status: Needs review » Fixed

Fixed with release 1.3: http://drupal.org/node/974554

@roderik Thank you for the patch, but Dave made a really good point. The switch to using ['#access'] seems to have taken care of the pathauto checkbox display as well. Thanks!

roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented

Absolutely, Dave was right. Glad to see this in.

Status: Fixed » Closed (fixed)

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