[wp-trac] Re: [WordPress Trac] #9244: Hyphen in Theme Directory Breaks Theme Preview

[WordPress Trac]

#9244: Hyphen in Theme Directory Breaks Theme Preview
-------------------------------------+--------------------------------------
 Reporter:  chrisbliss18             |        Owner:  chrisbliss18
     Type:  defect (bug)             |       Status:  closed      
 Priority:  normal                   |    Milestone:  2.8         
Component:  Themes                   |      Version:  2.8         
 Severity:  trivial                  |   Resolution:  fixed       
 Keywords:  theme,preview,has-patch  |  
-------------------------------------+--------------------------------------

Comment(by filosofo):

 Replying to [comment:9 chrisbliss18]:
 > Just to play devil's advocate here, I'm not sure how putting the hyphen
 at the end again is less confusing. It simply adds the potential of
 reintroducing this bug if someone adds yet another character at the end of
 the character class.

 A mere backslash won't prevent all the potential damage to a regex done by
 someone who modifies it without understanding.

 Moving the hyphen to the beginning or the end of the character class makes
 its ''purpose'' unambiguous to a knowledgeable person, and it eliminates
 an excess character.

 And now I must return to trimming my lawn with a nail clipper.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/9244#comment:10>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software