Last modified: 2014-08-20 21:39:21 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 T68891, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 66891 - Until Flow pages can be moved, prevent someone from trying
Until Flow pages can be moved, prevent someone from trying
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Flow (Other open bugs)
unspecified
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-06-20 21:09 UTC by Greg Grossmeier
Modified: 2014-08-20 21:39 UTC (History)
10 users (show)

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


Attachments

Description Greg Grossmeier 2014-06-20 21:09:02 UTC
I was tidying up (problem one) on mw.org.

There's a new BF going out. The page about the feature was at mw:Wikibase/Beta_Features/Other_projects_sidebar

I moved it to mw:Beta_Features/Other_projects_sidebar

I also moved the Talk: page, which was flowified.

Apparently that's not OK.

Please don't let me do that again. :) Erroring out or something.
Comment 1 spage 2014-06-21 23:30:43 UTC
Yes, Flow should block moving a Flow board in the AbortMove hook since the board's header and topics don't follow the move. Details follow.


Flow blocks most page actions, but move is handled by DB updates within Special:MoveTo. Moving a Flow board to a new page name appears to succeed.  But $wgFlowOccupyPages still tells Flow to occupy the old page name. (If you checked [Leave a redirect behind], Flow occupies the page despite page.page_is_redirect being true.)

The page "content" is at the new page name, but it is just the placeholder text "This talk page has been taken over by a _Flow board_.", and the new page probably isn't a Flow board.

Even if you enable Flow on the new page, the board header and any topics are still associated with the old page name in Flow DB tables, so they don't appear on the new Flow board name. If you disable Flow on the old page name, you can't access its items, even if you use their Permalinks URLs


The temporary fix is for Flow to hook AbortMove  and return false and an err message if the page is Flow-enabled, (as e.g. EducationProgram.hooks.php does).  This may change when a Flow board uses ContentHandler, since Title->isValidMoveOperation() already has checks for changing content model.

To actually support page moves, Flow could hook TitleMoveComplete and update the Flow items in its database to associate them with the new board name (update flow_workflow.workflow_page_id ?).
Comment 2 Quiddity 2014-06-28 18:28:29 UTC
Tracked at https://trello.com/c/V9va2kmj/
and https://trello.com/c/0ALdL2lg/
Comment 3 Gerrit Notification Bot 2014-08-20 14:13:35 UTC
Change 155262 had a related patch set uploaded by Matthias Mullie:
(bug 66891) Disallow moving a talk page

https://gerrit.wikimedia.org/r/155262
Comment 4 Gerrit Notification Bot 2014-08-20 20:15:21 UTC
Change 155262 merged by jenkins-bot:
(bug 66891) Disallow moving a Flow board

https://gerrit.wikimedia.org/r/155262

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


Navigation
Links