MediaWiki r46822 - Code Review

Jump to: navigation, search
Repository:MediaWiki
Revision:r46821‎ | r46822 (on ViewVC)‎ | r46823 >
Date:22:16, 4 February 2009
Author:werdna
Status:resolved (Comments)
Tags:
Comment:
Add ACTIVEUSERS magic-word, complement to NUMBEROFUSERS.
Modified paths:

Diff [purge]

Loading diff…

Follow-up revisions

Rev.Commit summaryAuthorDate
r47170RELEASE-NOTES and alias NUMBEROFACTIVEUSERS for magic word ACTIVEUSERS introd...werdna23:44, 11 February 2009

Comments

#Comment by Aaron Schulz (talk | contribs)   04:44, 5 February 2009

Should be 'numberofactiveusers'

#Comment by Werdna (talk | contribs)   04:50, 5 February 2009

Too long, and our naming of magic words is far from consistent anyway.

We have NUMBEROFARTICLES and NUMBEROFUSERS, but PAGESINCATEGORY.

#Comment by Aaron Schulz (talk | contribs)   05:03, 5 February 2009

Is it really that long? Anyway, I'd rather not justify crapiness by the existence of other crapiness :)

#Comment by Gurch (talk | contribs)   14:48, 5 February 2009

If it matters, make "numberofactiveusers" an alias (that can be done with magic words I assume?)

#Comment by Remember the dot (talk | contribs)   02:04, 6 February 2009

I'd also prefer the more accurate "numberofactiveusers"...if a wiki community really finds this too long then they could make an ACTIVEUSERS template to alias NUMBEROFACTIVEUSERS.

#Comment by Werdna (talk | contribs)   23:44, 11 February 2009

Added RELEASE-NOTES and alias NUMBEROFACTIVEUSERS in r47170.

No need to change the canonical name of the magic word, as that would break existing localisation.

#Comment by Remember the dot (talk | contribs)   06:52, 12 February 2009

This magic word has been translated into a grand total of two languages: Persian and Hebrew. Furthermore, the code has not yet been deployed, so the number of uses of the extra alias "ACTIVEUSERS" and its two translations is zero or close to it. The unnecessary alias can and should be eliminated to prevent confusion and remove the need to make two translations per language for one magic word.

#Comment by Werdna (talk | contribs)   17:36, 12 February 2009

I don't think it's hurting anything to have an extra alias.

If people don't like the extra 'unnecessary' alias, they're welcome to, you know, use the other one.

#Comment by Remember the dot (talk | contribs)   18:47, 12 February 2009

Doubling the number of aliases doubles the amount of translation work that will have to be done. It will also have editors scratching their heads wondering what the difference between ACTIVEUSERS and NUMBEROFACTIVEUSERS is.

#Comment by Aaron Schulz (talk | contribs)   19:10, 12 February 2009

I'd say that ACTIVEUSERS should just be removed. Not worth the hassle.

Status & tagging log

  • 23:44, 11 February 2009 Werdna (talk | contribs) changed the status of r46822 [removed: fixme added: resolved]
  • 04:44, 5 February 2009 Aaron Schulz (talk | contribs) changed the status of r46822 [removed: new added: fixme]