Last modified: 2014-09-08 12:25:56 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 T71879, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 69879 - Flow is creating notification links that go to ?action=edit, which cause exceptions
Flow is creating notification links that go to ?action=edit, which cause exce...
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Flow (Other open bugs)
unspecified
All All
: Highest blocker (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-08-22 02:10 UTC by Helder
Modified: 2014-09-08 12:25 UTC (History)
10 users (show)

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


Attachments
Screencast showing both bugs (1.19 MB, video/ogg)
2014-08-22 02:19 UTC, Helder
Details

Description Helder 2014-08-22 02:10:04 UTC
When I click in one of my (Echo) notifications on mw.org it shows this:
[3b4db816] 2014-08-22 02:00:04: Fatal exception of type MWException
Any idea what that could be?

Also, the red icon for new notifications doesn't change to gray until I open Special:Notifications

Maybe it is related to Flow, since the notification in question is about someone creating a new topic.
Comment 1 Helder 2014-08-22 02:19:25 UTC
Created attachment 16263 [details]
Screencast showing both bugs
Comment 2 PiRSquared17 2014-08-22 02:25:50 UTC
This blocks deployment.
Comment 3 Kunal Mehta (Legoktm) 2014-08-22 03:34:43 UTC
2014-08-22 02:00:04 mw1174 mediawikiwiki: [3b4db816] /w/index.php?title=Topic:Rs49btpjdntat0ar&fromnotif=1&action=edit&redlink=1   Exception from line 2238 of /usr/local/apache/common-local/php-1.24wmf$
#0 /usr/local/apache/common-local/php-1.24wmf18/includes/EditPage.php(938): EditPage->toEditText(Object(Flow\Content\BoardContent))
#1 /usr/local/apache/common-local/php-1.24wmf18/includes/EditPage.php(532): EditPage->initialiseForm()
#2 /usr/local/apache/common-local/php-1.24wmf18/includes/actions/EditAction.php(56): EditPage->edit()
#3 /usr/local/apache/common-local/php-1.24wmf18/includes/MediaWiki.php(439): EditAction->show()
#4 /usr/local/apache/common-local/php-1.24wmf18/includes/MediaWiki.php(307): MediaWiki->performAction(Object(Article), Object(Title))
#5 /usr/local/apache/common-local/php-1.24wmf18/includes/MediaWiki.php(609): MediaWiki->performRequest()
#6 /usr/local/apache/common-local/php-1.24wmf18/includes/MediaWiki.php(460): MediaWiki->main()
#7 /usr/local/apache/common-local/php-1.24wmf18/index.php(46): MediaWiki->run()
#8 /usr/local/apache/common-local/w/index.php(3): require('/usr/local/apac...')
#9 {main}
Comment 4 Kunal Mehta (Legoktm) 2014-08-22 03:37:53 UTC
Going to https://www.mediawiki.org/w/index.php?title=Topic:Rs49btpjdntat0ar directly works. Adding action=edit to it fails.
Comment 5 Kunal Mehta (Legoktm) 2014-08-22 03:57:58 UTC
Two issues here: 1) Flow is linking to notifications with ?action=edit. 2) Visiting a Flow page with ?action=edit throws an exception.

Lets keep this bug about #1, which is the real breakage. #2 has been around for a while and is tracked as bug 62173.
Comment 6 bsitu 2014-08-22 04:09:09 UTC
Flow notification in the overlay doesn't get automatically marked as read, it can be marked as read by the following:

1. click on the X on the upper right hand corner of the overlay

2. click on 'mark all read'

3. visit the page

I notice that the notification that throws exception was created 4 months ago, which was before we introduced the topic namespace.  On top of my head, This is likely because it doesn't have a revision in revision table which generates action=edit in the url. We could write something to fix old data but I could be wrong in here
Comment 7 bsitu 2014-08-22 04:39:15 UTC
Since that Flow topic page has been visited directly without action=edit, it should have automatically created a revision and you should not see the error anymore when clicking on it again from the overlay
Comment 8 Helder 2014-08-22 12:32:45 UTC
(In reply to Kunal Mehta (Legoktm) from comment #5)
> Two issues here:
> ...
Actually three.

(In reply to bsitu from comment #6)
> Flow notification in the overlay doesn't get automatically marked as read,
> it can be marked as read by the following:
> ...
> 3. visit the page

As you can see in the screencast, when I visit the page of the first notification
https://www.mediawiki.org/w/index.php?title=Topic:S0vayt0hge34g5t3&fromnotif=1
the red icon is not changed to gray (it is still red today, and there is another notification with the same behavior - so, a red icon "[ 2 ]" ).

I'll split this into another bug later.
Comment 9 Quiddity 2014-08-22 17:50:16 UTC
This bug is on trello at https://trello.com/c/p4xeXeji/

(In reply to Helder from bug 69879 comment 0)
> Also, the red icon for new notifications doesn't change to gray until I open
> Special:Notifications

I've split this to bug 69913

Thanks, Helder :)
Comment 10 Gerrit Notification Bot 2014-08-25 20:55:53 UTC
Change 156183 had a related patch set uploaded by Bsitu:
Redirect to Flow view action when url action is set to edit

https://gerrit.wikimedia.org/r/156183
Comment 11 Kunal Mehta (Legoktm) 2014-08-26 23:08:56 UTC
Patch hasn't been merged yet?
Comment 12 Gerrit Notification Bot 2014-08-27 00:02:03 UTC
Change 156183 merged by jenkins-bot:
Redirect to Flow view action when url action is set to edit

https://gerrit.wikimedia.org/r/156183
Comment 13 Jon 2014-09-08 12:25:56 UTC
Patch was merged. I suspect Jenkins bot was a bit slow that day.

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


Navigation
Links