Page MenuHomePhabricator

Mysterious missingrev failure during save
Closed, ResolvedPublic

Description

Ori reported that when he was trying to save an article, he repeatedly got errors like {"error":{"code":"missingrev","info":"There is no revision ID 654089716"}. But in reality, his edit was saved, and the revid in the error message was the ID of the new revision he'd created.

It seems like this error is coming from ApiParse.php, probably from the somewhat strange code path that leads to $this->dieUsage( "There is no revision ID {$page->getLatest()}", 'missingrev' );

Event Timeline

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

Additionally, errors thrown from within a FauxRequest-based API request probably shouldn't kill the main API request.

Probably the same underlying issue as T95466

Change 202911 had a related patch set uploaded (by Catrope):
Explicitly pass oldid to ApiParse after saving

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

Change 202911 merged by jenkins-bot:
Explicitly pass oldid to ApiParse after saving

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

Change 202920 had a related patch set uploaded (by Catrope):
Explicitly pass oldid to ApiParse after saving

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

Change 202921 had a related patch set uploaded (by Catrope):
Explicitly pass oldid to ApiParse after saving

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

Change 202920 merged by jenkins-bot:
Explicitly pass oldid to ApiParse after saving

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

Change 202921 merged by jenkins-bot:
Explicitly pass oldid to ApiParse after saving

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

@Catrope: All three patches merged. What's left to do here?