"Nikerabbit" posted a comment on MediaWiki.r110469.
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/110469#c30390

Commit summary for MediaWiki.r110469:

Fix bug 34046. If the same workflow state is already set, show an error 
message, don't touch the DB and don't log anything.

Nikerabbit's comment:

I'd just stick to the way it is done few lines above:
 $this->dieUsage( 'Permission denied', 'permissiondenied' );
And add hew new error to
 getPossibleErrors

_______________________________________________
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to