On Sun, Jun 8, 2008 at 6:48 AM, <werdna(a)svn.wikimedia.org> wrote:
+// Define new autopromote condition
+define('APCOND_TOR', 'tor'); // Numbers won't work, we'll get
collisions
Speaking of which, is there any reason for us to ever use named
constants like APCOND_TOR rather than just strings? This isn't C,
after all. I think it would be wise to deprecate such things for the
future and just use strings, arrays of them if necessary, as with
wfMsgExt() and similar. (Although in this particular case a named
constant might be better, to be consistent with the
already-established core conditions.)