Deleted wikitext Talk page cannot be restored after EnableFlow creates the same page
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Etonkovidova
	Jul 2 2015, 5:12 PM

Description

In beta on cawiki

Created a new Talk:ET10 with simple wikitext --
MariaDB [cawiki]> select page_id, page_namespace, page_title, page_content_model from page where page_title like 'ET10' and page_namespace=1;

select page_id,page_namespace, page_title, page_content_model from page where page_title like 'ET10' and page_namespace=1;
+---------+----------------+------------+--------------------+
| page_id | page_namespace | page_title | page_content_model |
+---------+----------------+------------+--------------------+
|   23630 |              1 | ET10       | wikitext        |
+---------+----------------+------------+--------------------+

Delete page_id 23630

MariaDB [cawiki]> select ar_rev_id, ar_title, ar_content_model from archive where ar_title='ET10';
+-----------+----------+------------------+
| ar_rev_id | ar_title | ar_content_model |
+-----------+----------+------------------+
|     23782 | ET10     | NULL             |
+-----------+----------+------------------+

Create a Flow page ET10 with http://ca.wikipedia.beta.wmflabs.org/wiki/Especial:EnableFlow

select page_id,page_namespace, page_title, page_content_model from page where page_title like 'ET10' and page_namespace=1;
+---------+----------------+------------+--------------------+
| page_id | page_namespace | page_title | page_content_model |
+---------+----------------+------------+--------------------+
|   23631 |              1 | ET10       | flow-board         |
+---------+----------------+------------+--------------------+

Go to deletion log: http://ca.wikipedia.beta.wmflabs.org/wiki/Especial:Registre/delete and Restore wikitext page Talk:ET10 -Error undeleting page

Undelete failed: No matching pages found in the deletion archive.

Error

An error has occurred.

Return to Portada.

[7d745e55] /w/index.php?title=Especial:Restaura&action=submit Flow\Exception\FlowException from line 76 of /srv/mediawiki/php-master/extensions/Flow/includes/BoardMover.php: Could not locate workflow for 23630

Backtrace:

#0 /srv/mediawiki/php-master/extensions/Flow/Hooks.php(1463): Flow\BoardMover->prepareMove(integer, Title)
#1 /srv/mediawiki/php-master/includes/Hooks.php(204): FlowHooks::onArticleUndelete(Title, boolean, string, integer)
#2 /srv/mediawiki/php-master/includes/specials/SpecialUndelete.php(635): Hooks::run(string, array)
#3 /srv/mediawiki/php-master/includes/specials/SpecialUndelete.php(384): PageArchive->undeleteRevisions(array, boolean, string)
#4 /srv/mediawiki/php-master/includes/specials/SpecialUndelete.php(1652): PageArchive->undelete(array, string, array, boolean, User)
#5 /srv/mediawiki/php-master/includes/specials/SpecialUndelete.php(822): SpecialUndelete->undelete()
#6 /srv/mediawiki/php-master/includes/specialpage/SpecialPage.php(384): SpecialUndelete->execute(NULL)
#7 /srv/mediawiki/php-master/includes/specialpage/SpecialPageFactory.php(553): SpecialPage->run(NULL)
#8 /srv/mediawiki/php-master/includes/MediaWiki.php(249): SpecialPageFactory::executePath(Title, RequestContext)
#9 /srv/mediawiki/php-master/includes/MediaWiki.php(678): MediaWiki->performRequest()
#10 /srv/mediawiki/php-master/includes/MediaWiki.php(475): MediaWiki->main()
#11 /srv/mediawiki/php-master/index.php(41): MediaWiki->run()
#12 /srv/mediawiki/w/index.php(3): include(string)
#13 {main}

Details

	Subject	Repo	Branch	Lines +/-
	Block undeleting an article over an existing Flow board	mediawiki/extensions/Flow	master	+26 -2

Customize query in gerrit

Related Objects

Mentioned In: T135166: More accurate UI text when you start doing a disallowed undeletion (over a Flow board)
T122262: Improve Flow deletion/undeletion resilience
T117812: Fix en:Wikipedia_talk:Flow/Developer_test_page
T122261: Come up with scenarios for deletion/undeletion of Flow boards and figure out how to handle them
T110696: Can no longer view deleted board in Special:Undelete
Mentioned Here: rEFLWd569f06d016e: WIP: Use ArticleRevisionUndeleted instead of ArticleUndelete
T135166: More accurate UI text when you start doing a disallowed undeletion (over a Flow board)
T117812: Fix en:Wikipedia_talk:Flow/Developer_test_page
T122261: Come up with scenarios for deletion/undeletion of Flow boards and figure out how to handle them

Event Timeline

Etonkovidova created this task.Jul 2 2015, 5:12 PM

Etonkovidova raised the priority of this task from to Needs Triage.

Etonkovidova updated the task description. (Show Details)

Etonkovidova added projects: StructuredDiscussions, Collaboration-Team-Triage.

Etonkovidova subscribed.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 2 2015, 5:12 PM

After initial Error display(as in the ticket), all subsequent attempts to Restore such pages from e.g. http://ca.wikipedia.beta.wmflabs.org/w/index.php?title=Especial:Restaura&target=Discussi%C3%B3%3AET10 do not do anything.

You may check http://ca.wikipedia.beta.wmflabs.org/wiki/Especial:Registre/delete for

Etonkovidova (Talk | contribs | block) deleted page Discussió:ET10
RoanKattouw (Talk | contribs | block) deleted page Discussió:Blah2

Catrope updated the task description. (Show Details)Aug 5 2015, 11:22 PM

Catrope set Security to None.

Catrope updated the task description. (Show Details)

Catrope added a subscriber: Legoktm.

We probably don't want to allow people to undelete wikitext pages on top of Flow boards, but the error message should make more sense.

Catrope triaged this task as Medium priority.Aug 5 2015, 11:25 PM

• DannyH moved this task from Untriaged to Near-Term Interest on the Collaboration-Team-Triage board.Aug 5 2015, 11:25 PM

• DannyH moved this task from Near-Term Interest to Freezer on the Collaboration-Team-Triage board.Aug 21 2015, 9:19 PM

Etonkovidova mentioned this in T110696: Can no longer view deleted board in Special:Undelete.Sep 25 2015, 9:03 PM

• Mattflaschen-WMF mentioned this in T122261: Come up with scenarios for deletion/undeletion of Flow boards and figure out how to handle them.Dec 24 2015, 11:52 PM

Re-checked while verifying T117812: Fix en:Wikipedia_talk:Flow/Developer_test_page

The issue is still happening - cannot restore wiki page after Flow page with the same name was created(via EnableFlow).
In this scenario, a page exists simultaneously in two tables:

Summary of the scenario:

create a wikitext page - Delete
Special: EnableFlow the page with the same title
Go to Special:Undelete - you'll see the page listed there
try Undelete - the above error is displayed.

There is also a follow up task: T122261: Come up with scenarios for deletion/undeletion of Flow boards and figure out how to handle them

Etonkovidova mentioned this in T117812: Fix en:Wikipedia_talk:Flow/Developer_test_page.Dec 25 2015, 1:25 AM

In T104591#1903858, @Etonkovidova wrote:

Re-checked while verifying T117812: Fix en:Wikipedia_talk:Flow/Developer_test_page

The issue is still happening - cannot restore wiki page after Flow page with the same name was created(via EnableFlow).
In this scenario, a page exists simultaneously in two tables:

Summary of the scenario:

create a wikitext page - Delete

Special: EnableFlow the page with the same title

Go to Special:Undelete - you'll see the page listed there

try Undelete - the above error is displayed.

It should work if you move the Flow board out of the way (without leaving a redirect) before the Special:Undelete step. Not tested, though.

Specific cause is that it sees there's currently a Flow board there, so triggers onArticleDelete hook, so it expects the page ID being undeleted to be Flow, but it's not (the already-existent one was Flow, the undeleted page was not).

But I agree with Roan we should probably just block this scenario.

Etonkovidova edited projects, added Collaboration-Team-Archive-2015-2016; removed Collaboration-Team-Triage.Dec 29 2015, 8:01 PM

Catrope moved this task from Untriaged to Ready for pickup on the Collaboration-Team-Archive-2015-2016 board.Jan 19 2016, 3:06 AM

Change 277536 had a related patch set uploaded (by Matthias Mullie):
Block undeleting an article over an existing Flow board

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

gerritbot added a project: Patch-For-Review.Mar 15 2016, 4:37 PM

matthiasmullie claimed this task.Mar 15 2016, 4:37 PM

matthiasmullie moved this task from Ready for pickup to Needs Review on the Collaboration-Team-Archive-2015-2016 board.

matthiasmullie mentioned this in T122262: Improve Flow deletion/undeletion resilience.Mar 16 2016, 3:42 PM

Change 277536 merged by jenkins-bot:
Block undeleting an article over an existing Flow board

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

ReleaseTaggerBot added a project: MW-1.27-release (WMF-deploy-2016-03-22_(1.27.0-wmf.18)).Mar 18 2016, 2:01 AM

• jmatazzoni added a project: Collab-Team-2016-Apr-Jun-Q4.Apr 6 2016, 10:24 PM

• jmatazzoni moved this task from Untriaged to Needs Review on the Collab-Team-2016-Apr-Jun-Q4 board.Apr 6 2016, 10:30 PM

SBisson moved this task from Needs Review to QA Review on the Collab-Team-2016-Apr-Jun-Q4 board.Apr 14 2016, 8:30 PM

Checked the fix in betalabs - the error message is not present anymore. Instead, the proper warning is displayed: "The article couldn't be restored because there is currently a discussion board in place."

Below is the summary of issues that still present (for future improvements consideration)

The proper warning to users is in place. But to find it - it's little confusing.

The below example uses Talk:Mavetuna2 page as an example.

create wikitext Talk:Mavetuna2 and then delete it.
via Special:EnableFlow create Talk:Mavetuna2
Special:Undelete informs that "The following page has been deleted but is still in the archive and can be restored."

Screen Shot 2016-04-19 at 3.27.31 PM.png (379×1 px, 72 KB)

Click on 'Restore revisions' button under 'Restore revisions' will give a proper warning:

No such action
The article couldn't be restored because there is currently a discussion board in place.

Screen Shot 2016-04-19 at 3.07.42 PM.png (272×1 px, 51 KB)

clicking on the link Talk:Mavetuna2 from Special:Undelete page will present 'View and restore deleted pages' page with several points of confusion, including Talk:Mavetuna2 active link (to a new Flow board) and the following text "If a new page with the same name has been created since the deletion, the restored revisions will be merged with the new page's history, and the current revision of the live page will not be automatically replaced." - not true if a new page with the same name is a Flow board:

Screen Shot 2016-04-19 at 3.34.34 PM.png (716×1 px, 189 KB)

Clicking on the link 'del/undel' link or on 'del/undel selected revision' button will display 'Delete/undelete revisions' page which has options only for changing the visibility of revisions.

Restricted Application added a subscriber: TerraCodes. · View Herald TranscriptApr 19 2016, 11:09 PM

Etonkovidova moved this task from QA Review to Product Review on the Collab-Team-2016-Apr-Jun-Q4 board.Apr 19 2016, 11:09 PM

@matthiasmullie @Mattflaschen @Catrope, before I close this, what about the confusing messaging Elena observes above. Does this need to be transferred to a new tag in "Interested"? (If so, if someone who understands the issue better than me could do it that would be helpful.)

Reposting this request for comment:

@matthiasmullie @Mattflaschen @Catrope, before I close this, what about the confusing messaging Elena observes above. Does this need to be transferred to a new ticket in "Interested"? (If so, if someone who understands the issue better than me could do it that would be helpful.)

Most of the things she mentioned are areas for possible improvement.

However, there is nothing wrong with "Talk:Mavetuna2 active link (to a new Flow board)" in my opinion. That is standard behavior for deleted pages where there is currently non-deleted content there (either due to a recreation or a partial deletion).

Filed the rest as T135166: More accurate UI text when you start doing a disallowed undeletion (over a Flow board)

Thanks Matt. Closing this now.

• jmatazzoni closed this task as Resolved.May 12 2016, 8:47 PM

@matthiasmullie and I think that this should be allowed (but handled properly), so rEFLWd569f06d016e: WIP: Use ArticleRevisionUndeleted instead of ArticleUndelete takes that approach. This works without error (and you can later delete the Flow board and restore the wikitext and that will also still work).

The restriction put in place was also causing false positives.

	F3896333: Screen Shot 2016-04-19 at 3.07.42 PM.png
	Apr 19 2016, 11:09 PM

	F3896412: Screen Shot 2016-04-19 at 3.27.31 PM.png
	Apr 19 2016, 11:09 PM

	F3896440: Screen Shot 2016-04-19 at 3.34.34 PM.png
	Apr 19 2016, 11:09 PM

Deleted wikitext Talk page cannot be restored after EnableFlow creates the same pageClosed, ResolvedPublicActions

Description

Details

Related Objects

Event Timeline

Deleted wikitext Talk page cannot be restored after EnableFlow creates the same page
Closed, ResolvedPublic
Actions