Last modified: 2014-03-08 08:13:27 UTC
Created attachment 7637 [details] 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.
Using the message "Cannot revert edit; last contributor is only author of this page." is bad. A new message should be added.
Also, You should really have it with braces as per the coding style guidelines if( !$status->isOK() ) { return array( array( 'cantrollback' ) ); }
Marking as reviewed per comment #2.
Check for $status->isOk was added with Gerrit change #21662