On 20/05/14 23:05, Daniel Friesen wrote:
On 2014-05-20, 3:48 PM, Isarra Yos wrote:
Out of curiosity, what are your reasons for
advocating lowercase skin
names over the standard camelcase format used throughout the rest of
MediaWiki?
I included those in my bullet points.
* The skinname you require is the same one assigned to $wgValidSkins
and set on $wgDefaultSkin and set on $skinname, and $stylepath.
* For all skins using the old autoload pattern the assets directory
remains them same, so nothing has to be re-cached.
But that sounds more like an oversight with the autoloader
implementation and class setup than an actual reason why it would be
good practice to use lowercase. Was there a particular design reason to
set those up to be that way?
There's also a third reason.
There are 3 names for a skin containing two words:
* foobar
* FooBar
* Foo Bar
Foo Bar is exposed to users in the interface as the human readable name.
And foobar is exposed in &useskin= $wgDefaultSkin and the current
directory asset structure, in essence it IS the canonical name of the skin.
FooBar however generally isn't exposed anywhere. It exists exclusively
for things like skin class names like SkinFooBar and optionally some
filenames, both of which are internal to the skin.
~Daniel Friesen (Dantman, Nadir-Seen-Fire) [
http://danielfriesen.name/]
Two of those also apply to extensions in general - FooBar and Foo Bar.
Has this sort of inconsistency between name presentations ever posed a
problem there? And with skins in particular, why would the internal
handling of the skin name be coupled to the filename at all? And why the
case-sensitivity, especially when not all platforms that mw supports
even use case-sensitive filesystems? Is this just a holdover from the
autoloader implementation?
Sorry for all the questions, but this all seems quite odd, and I never
really had the opportunity to put much thought into it before.
-I