Posted by Freso on May 12, 2008 at 11:32am
| Project: | Pathauto |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | minor |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
$disable_text is set on line 112 of pathauto.admin.inc, but apparently not actually used anywhere.
freso@nayru /s/h/l/h/d/s/a/m/pathauto> grep "disable_text" **
pathauto.admin.inc: $disable_text = '';
pathauto_string_spring_cleaning.patch: $disable_text = '';
pathauto_using_transliterate_no_rm.patch: $disable_text = '';
freso@nayru /s/h/l/h/d/s/a/m/pathauto>
I'm not checking this in straight away, in case there's some obscure reason that it is needed... ?
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| pathauto_no_disable_text.patch | 949 bytes | Ignored: Check issue status. | None | None |
Comments
#1
And a patch for 5.x which also doesn't use this.
#2
Yep, seems like cruft to me.
#3
Committed: 6.x, 5.x. Yay for removing cruft! :)
#4
Automatically closed -- issue fixed for two weeks with no activity.