r94793 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r94792‎ | r94793 | r94794 >
Date:19:08, 17 August 2011
Author:reedy
Status:ok
Tags:
Comment:
Modified paths:
  • /branches/wmf/1.17wmf1/extensions/CodeReview/CodeReview.i18n.php (modified) (history)
  • /branches/wmf/1.17wmf1/extensions/CodeReview/backend/CodeRevision.php (modified) (history)

Diff [purge]

Index: branches/wmf/1.17wmf1/extensions/CodeReview/CodeReview.i18n.php
@@ -232,7 +232,7 @@
233233 'right-codereview-signoff' => 'Sign off on revisions',
234234 'right-codereview-link-user' => 'Link authors to wiki users',
235235 'right-codereview-associate' => 'Manage revision associations',
236 - 'right-codereview-review-own' => 'Mark your own revisions as OK',
 236+ 'right-codereview-review-own' => 'Mark your own revisions as OK or Resolved',
237237
238238 'specialpages-group-developer' => 'Developer tools',
239239
Index: branches/wmf/1.17wmf1/extensions/CodeReview/backend/CodeRevision.php
@@ -282,10 +282,10 @@
283283 throw new MWException( "Tried to save invalid code revision status" );
284284 }
285285
286 - // Don't allow the user account tied to the committer account mark their own revisions as ok
 286+ // Don't allow the user account tied to the committer account mark their own revisions as ok/resolved
287287 // Obviously only works if user accounts are tied!
288288 $wikiUser = $this->getWikiUser();
289 - if ( $status == 'ok' && $wikiUser && $user->getName() == $wikiUser->getName() ) {
 289+ if ( ( $status == 'ok' || $status == 'resolved' ) && $wikiUser && $user->getName() == $wikiUser->getName() ) {
290290 // allow the user to review their own code if required
291291 if ( !$wikiUser->isAllowed( 'codereview-review-own' ) ) {
292292 return false;
Property changes on: branches/wmf/1.17wmf1/extensions/CodeReview/backend/CodeRevision.php
___________________________________________________________________
Modified: svn:mergeinfo
293293 Merged /trunk/extensions/CodeReview/backend/CodeRevision.php:r93108-93109

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r93108Stop users marking their own revisions resolved (as well as ok)reedy20:26, 25 July 2011
r93109Followup r93108, add "or Resolved" to right-codereview-review-ownreedy20:27, 25 July 2011

Status & tagging log