Last modified: 2014-01-12 15:30:19 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 T53640, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 51640 - wgUserNewMsgRevisionId null when user talk page has only one revision
wgUserNewMsgRevisionId null when user talk page has only one revision
Status: NEW
Product: MediaWiki
Classification: Unclassified
JavaScript (Other open bugs)
1.22.0
All All
: High major (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-07-18 21:23 UTC by Bartosz Dziewoński
Modified: 2014-01-12 15:30 UTC (History)
5 users (show)

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


Attachments

Description Bartosz Dziewoński 2013-07-18 21:23:13 UTC
wgUserNewMsgRevisionId is apparently null when user talk page has only one revision, even if the user hasn't viewed it yet.

May be caused by Notifications interfering with something (I only tested on test.wp); certainly matters for it, though, as this causes the orange alert not to appear for new users ([[File:Echo Notifications new message indicator.png|this one]]). Therefore marking as 'major'.
Comment 1 Ryan Kaldari 2013-07-18 21:32:26 UTC
Confirmed in master branch.
Comment 2 Bartosz Dziewoński 2013-07-18 21:42:23 UTC
[23:39]	<kaldari> looks like $user->getNewMessageRevisionId() is returning null even after the first post for some reason.
Comment 3 Gerrit Notification Bot 2013-07-18 22:47:53 UTC
Change 74547 had a related patch set uploaded by Kaldari:
Making sure wgUserNewMsgRevisionId works even for the first post

https://gerrit.wikimedia.org/r/74547
Comment 4 Gerrit Notification Bot 2013-07-19 02:10:15 UTC
Change 74547 abandoned by Kaldari:
Making sure wgUserNewMsgRevisionId works even for the first post

Reason:
wrong approach

https://gerrit.wikimedia.org/r/74547
Comment 5 Gerrit Notification Bot 2013-07-19 18:54:35 UTC
Change 74666 had a related patch set uploaded by Bsitu:
(bug 51640) wgUserNewMsgRevisionId is null with only one rev

https://gerrit.wikimedia.org/r/74666
Comment 6 Gerrit Notification Bot 2013-07-19 19:58:36 UTC
Change 74666 merged by jenkins-bot:
(bug 51640) wgUserNewMsgRevisionId is null with only one rev

https://gerrit.wikimedia.org/r/74666
Comment 7 Gerrit Notification Bot 2013-07-30 20:45:04 UTC
Change 76824 had a related patch set uploaded by Bsitu:
Revert "(bug 51640) wgUserNewMsgRevisionId is null with only one rev"

https://gerrit.wikimedia.org/r/76824
Comment 8 Gerrit Notification Bot 2013-07-30 20:47:18 UTC
Change 76824 merged by jenkins-bot:
Revert "(bug 51640) wgUserNewMsgRevisionId is null with only one rev"

https://gerrit.wikimedia.org/r/76824
Comment 9 Andre Klapper 2013-09-25 09:50:46 UTC
All patches got merged - can this bug report be closed as RESOLVED FIXED or is more work required?
Comment 10 Bartosz Dziewoński 2013-09-25 11:51:12 UTC
The patch was merged, but then reverted.
Comment 11 Bartosz Dziewoński 2014-01-12 15:27:40 UTC
Echo's alert bar now doesn't require JavaScript, let's try doing this once again.

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


Navigation
Links