Rollbacks improperly reported under race condition
Closed, ResolvedPublic

Description

Author: Winbots

Description:
Fixes bug.

Rollbacks can be improperly reported under certain race conditions. Mainly because the doEdit call's status is never checked before notifying of a successful rollback.

The diff should fix this.

--Cobi.


Version: 1.17.x
Severity: enhancement

Attached: racecondition.diff

bzimport added a project: MediaWiki-Page-editing.Via ConduitNov 21 2014, 11:02 PM
bzimport added a subscriber: wikibugs-l.
bzimport set Reference to bz24866.
bzimport created this task.Via LegacyAug 19 2010, 5:23 AM
liangent added a comment.Via ConduitAug 19 2010, 5:32 AM

Using the message "Cannot revert edit; last contributor is only author of this page." is bad. A new message should be added.

Reedy added a comment.Via ConduitAug 19 2010, 9:27 AM

Also,
You should really have it with braces as per the coding style guidelines

if( !$status->isOK() ) {
return array( array( 'cantrollback' ) );
}

bzimport added a comment.Via ConduitMay 14 2011, 2:20 PM

Bryan.TongMinh wrote:

Marking as reviewed per comment #2.

duplicatebug added a comment.Via ConduitMar 8 2014, 8:13 AM

Check for $status->isOk was added with gerrit 21662

Add Comment

Column Prototype
This is a very early prototype of a persistent column. It is not expected to work yet, and leaving it open will activate other new features which will break things. Press "\" (backslash) on your keyboard to close it now.