"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