https://bugzilla.wikimedia.org/show_bug.cgi?id=5125

--- Comment #12 from Marcus Bointon <[email protected]> 2010-07-10 
08:40:34 UTC ---
Goodness, I'd forgotten I'd written all this.

I find your conclusion nonsensical - My changes for 1 above does not amount to
customization, merely that making the definition only in one place (aka DRY),
which is just good coding practice IMHO. WHEN you decide to change it, chances
are you'll write code that does pretty much the same thing. Either that or
invite disaster by making changes in multiple locations.

3 directly conflicts with 2: you say that writing to config is dangerous (which
is why SF doesn't allow it), but then refuse to provide a workaround. Also,
again my changes really only amount to making this so it's set only in once
place, it doesn't represent any change in policy. Allowing someone to hack the
path simply (for example so it writes somewhere that's writable but not under
the web root) seems like a handy side effect.

To summarise: my patch would:

a) simplify changing paths used by mediawiki and make doing so safer and more
reliable, pre-emptively fixing future bugs in that area
b) have a side effect of making changing them again easier, even though that
may not be the primary aim
c) make mediawiki installations on sourceforge easier

I'm mystified why you'd not want this fix; the alternative means you're
deliberately choosing to live with bad code for no good reason.

-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
You are on the CC list for the bug.

_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to