| Etonkovidova | |
| Dec 6 2017, 8:08 PM |
| F13155500: Screen Shot 2018-02-06 at 3.28.42 PM.png | |
| Feb 6 2018, 11:54 PM |
| F11225455: Screen Shot 2017-12-06 at 11.59.41 AM.png | |
| Dec 6 2017, 8:08 PM |
| F11225266: Screen Shot 2017-12-06 at 11.58.32 AM.png | |
| Dec 6 2017, 8:08 PM |
| F11225381: Screen Shot 2017-12-06 at 11.57.53 AM.png | |
| Dec 6 2017, 8:08 PM |
Previous to wmf.11 Number of edits to show by default: 1 was accommodated:
Note 1: Surprisingly, Number of edits to show by default: 0 is displayed fine:
Note 2: Watchlist has the same issue, of course.
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| RCFilters: Fix limit display for limit=1 | mediawiki/core | master | +1 -1 |
Yes, there's a funny (not) bug/feature in mw.msg regarding treating the number 0 as plural, so {{PLURAL:$1|change|$1 changes}} will be 'change' if it's only one, and '0 changes' if it's zero or bigger than 1. I'm not 100% sure, but I think we tried a variation {{PLURAL:$1|0=no changes|1=one change|$1 changes}} and that didn't work right, but I might be wrong, and this could be a solution.
We can rephrase the sentence, perhaps, to allow for showing "1 change" specifically for 1. Also, 0 changes should probably be normalized to 1, since showing nothing makes no sense,but if the previous system (and backend) allow for it, I guess we can leave that as is.
(Note: I was originally thinking of T170975: {{PLURAL}} behaves differently in PHP vs JS for decimal numbers but this is a different issue than the one in this ticket)
This is not a bug in mw.msg/jQueryMsg. It is correct English, and the same in the PHP message system. It turns out jQueryMsg/mw.language gets the plural rules directly from the server (not tested, but languages where 0 is not plural should work correctly).
I'm not 100% sure, but I think we tried a variation {{PLURAL:$1|0=no changes|1=one change|$1 changes}} and that didn't work right, but I might be wrong, and this could be a solution.
The general rule is to spell numbers "one" through "nine", then go to "10", etc. Of course, that's a pain for software, so I think it's best to just stick to "1", "2", etc.
I don't object to "No changes", but the description doesn't request that, so I'll just stick to "0 changes" for now.
We can rephrase the sentence, perhaps, to allow for showing "1 change" specifically for 1.
+1
Change 402177 had a related patch set uploaded (by Mattflaschen; owner: Mattflaschen):
[mediawiki/core@master] RCFilters: Fix 1 change
Change 402177 merged by jenkins-bot:
[mediawiki/core@master] RCFilters: Fix limit display for limit=1
