Bugzilla – Bug 944
skin override falls to EN on a case mismatch
Last modified: 2008-09-15 14:37:04 UTC
It seems that most skins will work for override without a trailing slash, except Exbrowse2. It returns a 404, so I guess the trailing numeric is causeing a problem with the URL regex matching.
r2314 was a partial fix to allow ExBrowse2 work without a trailing slash, but it now returns the EN skin instead of a 404. All skins done by url override seem to render using EN if the case isn't an exact match.
Created attachment 306 [details] change isSkin to return with proper case it seems that a regex match that is case insensitive, puts an all lowercase match into $1, instead of the actual matched contents. Thus, thre is a skin match, but it doesnt' end up with the right urls to files. instead, step through each skinname and return the one matched. I can commit this after it passes a review.
Looks ok.
There are 536 bugs in the database with targets of '---' that were fixed prior to new year 2006. I am setting them to targets of 6.2.1 to keep them from showing up in my queries.