20after4 has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/182979

Change subject: argh! reset the projects and security field values to override 
the saved values because pre-existing tasks behave differently from newly 
created ones.
......................................................................

argh! reset the projects and security field values to override the saved values
because pre-existing tasks behave differently from newly created ones.

Change-Id: I21ed17087db2a557fee6d7e8cca05e921b5f5726
---
M src/policy/SecurityPolicyListener.php
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/phabricator/extensions/security 
refs/changes/79/182979/1

diff --git a/src/policy/SecurityPolicyListener.php 
b/src/policy/SecurityPolicyListener.php
index e6f4392..246eca6 100644
--- a/src/policy/SecurityPolicyListener.php
+++ b/src/policy/SecurityPolicyListener.php
@@ -51,6 +51,10 @@
         if ($t->getNewValue() == 'security-bug'
           && $t->getOldValue() != 'security-bug') {
           $switchToSecurity = true;
+          $security_setting = 'security_bug';
+          $project = WMFSecurityPolicy::getProjectByName('security');
+          $project_phids = array($project->getPHID() => $project->getPHID());
+          break;
         }
       }
       if (!$switchToSecurity) {

-- 
To view, visit https://gerrit.wikimedia.org/r/182979
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I21ed17087db2a557fee6d7e8cca05e921b5f5726
Gerrit-PatchSet: 1
Gerrit-Project: phabricator/extensions/security
Gerrit-Branch: master
Gerrit-Owner: 20after4 <mmod...@wikimedia.org>

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

Reply via email to