Last modified: 2013-04-22 16:17:18 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T46253, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 44253 - Info action should not omit number of watchers row when $wgUnwatchedPageThreshold is not false
Info action should not omit number of watchers row when $wgUnwatchedPageThres...
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
1.21.x
All All
: Low normal (vote)
: ---
Assigned To: Brad Jorsch
: easy
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-01-22 19:17 UTC by MZMcBride
Modified: 2013-04-22 16:17 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description MZMcBride 2013-01-22 19:17:07 UTC
Currently, if $wgUnwatchedPageThreshold is not false, the info action will omit the "Number of page watchers" row if the threshold is not met and the user is unprivileged (missing the "unwatchedpages" user right). This can create confusion, however, when other users reference the number of page watchers (in text or via a link).

Instead of simply omitting the row, it should say "Number of page watchers: unavailable" or "Number of page watchers: restricted" or something.
Comment 1 Kunal Mehta (Legoktm) 2013-01-26 09:37:43 UTC
How about "<$wgUnwatchedPageThreshold"?

This would simply require an additional check to make sure it isn't set to false, and if it is, just don't add the watchers row.
Comment 2 Amir E. Aharoni 2013-01-27 19:10:17 UTC
'<' is very unreadable. It looks like an opening parenthesis or an arrow. If you want to do something like this, just write "less than" in a human language.
Comment 3 Redrose64 2013-01-28 12:08:13 UTC
(In reply to comment #1)
> How about "<$wgUnwatchedPageThreshold"?
> 
> This would simply require an additional check to make sure it isn't set to
> false, and if it is, just don't add the watchers row.

The point is not to determine the circumstances of when to add the watchers row - it is that users are expecting to see that row because they may be sent there from a link titled "Number of watchers", see http://en.wikipedia.org/wiki/MediaWiki:Histlegend

If the figure should not be shown to them (for whatever reason), the row entry should still be there, but with some short explanation instead of the actual figure. If the row is absent, they will suspect a bad link target.
Comment 4 MZMcBride 2013-01-28 18:57:53 UTC
(In reply to comment #3)
> The point is not to determine the circumstances of when to add the watchers
> row - it is that users are expecting to see that row because they may be sent
> there from a link titled "Number of watchers", see
> http://en.wikipedia.org/wiki/MediaWiki:Histlegend
> 
> If the figure should not be shown to them (for whatever reason), the row
> entry should still be there, but with some short explanation instead of the
> actual figure. If the row is absent, they will suspect a bad link target.

Right. That's what Legoktm is proposing. He's saying to always include the table row for number of watchers when this feature is enabled (i.e., when $wgUnwatchedPageThreshold is not false). In this case, he's proposing using "<$wgUnwatchedPageThreshold", which for the English Wikipedia would be "<30" as the value for this row when it is restricted.

As comment 2 suggests, this might be better written as "less than $wgUnwatchedPageThreshold", which would translate to "less than 30".
Comment 5 Brad Jorsch 2013-01-29 03:17:32 UTC
Gerrit change #46489.

I checked a few grammar websites, and apparently it's supposed to be "fewer than" since "watchers" is a countable noun. At any rate, it can easily be customized in the MediaWiki namespace.
Comment 6 MZMcBride 2013-01-29 07:29:16 UTC
(In reply to comment #5)
> Gerrit change #46489.

Thanks for this. :-)  Somewhat related is that it'd be nice to have the ability to directly link to this table row via an HTML id attribute (bug 44252), but that's tangential to this bug, of course.

> I checked a few grammar websites, and apparently it's supposed to be "fewer
> than" since "watchers" is a countable noun. At any rate, it can easily be
> customized in the MediaWiki namespace.

Hah, and we apparently even have an article about this on the English Wikipedia: <https://en.wikipedia.org/wiki/Fewer_vs._less>. I suppose "fewer than" is more correct, though I wonder, in this particular case, given that the noun is in fact arguably uncountable (when this message is shown), whether it should be "less than." ;-)  As you say, we'll leave that to local admins to discuss and debate. Thanks again for the changeset.
Comment 7 Brad Jorsch 2013-02-07 21:42:33 UTC
Changeset merged. Should make it into 1.21wmf10.

https://www.mediawiki.org/wiki/MediaWiki_1.21/Roadmap

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links