Page MenuHomePhabricator

Infinite redirect loop when trying to re-translate article after deleting draft
Closed, ResolvedPublic

Description

Steps to reproduce:

  1. Start translating an article
  2. Go to dashboard
  3. Delete the draft
  4. Start again translating the same article

Observe: infinite redirect loop. Should allow starting translation from scratch.

Event Timeline

Nikerabbit raised the priority of this task from to Needs Triage.
Nikerabbit updated the task description. (Show Details)
Nikerabbit added a project: ContentTranslation.
Nikerabbit added a subscriber: Nikerabbit.

Change 258420 had a related patch set uploaded (by Santhosh):
Avoid fetching the draft if it was deleted

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

santhosh triaged this task as Medium priority.
santhosh edited projects, added LE-CX7-Sprint 4; removed Patch-For-Review.
santhosh set Security to None.
santhosh added a project: Patch-For-Review.
santhosh moved this task from Backlog to In Review on the LE-CX7-Sprint 4 board.
santhosh removed a subscriber: gerritbot.
Arrbee renamed this task from Infinite redirect look when trying to re-translate article after deleting draft to Infinite redirect loop when trying to re-translate article after deleting draft.Dec 11 2015, 7:02 AM
Arrbee added a project: Unplanned-Sprint-Work.

Change 258420 merged by Umherirrender:
Avoid fetching the draft if it was deleted

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

Verified. This works as expected.

@santhosh I am moving this out of done and back into QA for confirmation from you. This problem still exists if the original article has been published.

Steps to reproduce:

  1. Start translating an article and then publish it
  2. Go to dashboard and check that it is under the 'published articles' tab
  3. Start translating the same article again

I tried the above steps and thre was no redirect also, draft from my previous translation was restored.

Change 258929 had a related patch set uploaded (by Santhosh):
Do not pass draft id while resuming, remove references to draft id

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

Arrbee raised the priority of this task from Medium to High.Jan 18 2016, 6:53 AM

Change 258929 merged by Nikerabbit:
Do not pass draft id while resuming, remove references to draft id

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

Verified, Not happening any more.