Last modified: 2014-06-13 15:22:23 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 T68578, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66578 - NaN is shown as the notification count in the Persian Wikipedia
NaN is shown as the notification count in the Persian Wikipedia
Status: RESOLVED FIXED
Product: MobileFrontend
Classification: Unclassified
beta (Other open bugs)
unspecified
All All
: Unprioritized normal
: ---
Assigned To: Nobody - You can work on this!
: i18n
Depends on: 66577
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-13 10:55 UTC by Amir E. Aharoni
Modified: 2014-06-13 15:22 UTC (History)
6 users (show)

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


Attachments

Description Amir E. Aharoni 2014-06-13 10:55:49 UTC
To reproduce, log in to the Persian Wikipedia and click the notifications indicator box (or click this quick link: https://fa.m.wikipedia.org/wiki/Main_Page#/notifications ).

Whether the number is zero or more, the notification count in the heading is shown as "NaN".

I didn't look at the code, but I suspect that it may have something to do with the fact that Persian uses different digits, and the digits localization is handled incorrectly somewhere. For what it's worth, it is handled correctly for the main (red) notifications indicator box.
Comment 1 Bingle 2014-06-13 11:00:20 UTC
Prioritization and scheduling of this bug is tracked on Trello card https://trello.com/c/sxcl0blc
Comment 2 Florian 2014-06-13 11:55:38 UTC
Hi! Like said in this bug:
https://bugzilla.wikimedia.org/show_bug.cgi?id=66577

the notification badge isn't shown anymore on notifications page in actual git version. If i understand and see right, the other positions of the badge is correctly with numbers, yes?
Comment 3 Jon 2014-06-13 15:22:23 UTC
https://trello.com/c/xH4Fu82w/2-2-change-notification-icon removes this number so this shouldn't be a problem in future (the patch got deployed to mediawiki.org yesterday so will show up here in a week)

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


Navigation
Links