$disable_text set but not used anywhere

Freso - May 12, 2008 - 11:32
Project:Pathauto
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:minor
Assigned:Unassigned
Status:closed
Description

$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... ?

AttachmentSize
pathauto_no_disable_text.patch949 bytes

#1

Freso - May 12, 2008 - 11:35

And a patch for 5.x which also doesn't use this.

AttachmentSize
pathauto_no_disable_text.d5.patch 987 bytes

#2

greggles - May 12, 2008 - 11:55
Status:needs review» reviewed & tested by the community

Yep, seems like cruft to me.

#3

Freso - May 12, 2008 - 12:18
Status:reviewed & tested by the community» fixed

Committed: 6.x, 5.x. Yay for removing cruft! :)

#4

Anonymous (not verified) - May 26, 2008 - 12:23
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.