Ori.livneh has submitted this change and it was merged.

Change subject: emergency spam prevention
......................................................................


emergency spam prevention

disable reply, edit, edit-title, and new-topic

Change-Id: Ic096bfca35f3da5ebe508e224fecac6361753061
(cherry picked from commit a3bf4c401a2ce14f808a6b4964cbb1803a3da518)
---
M includes/Block/Topic.php
M includes/Block/TopicList.php
2 files changed, 12 insertions(+), 0 deletions(-)

Approvals:
  Ori.livneh: Verified; Looks good to me, approved



diff --git a/includes/Block/Topic.php b/includes/Block/Topic.php
index f223aa9..1fe9c14 100644
--- a/includes/Block/Topic.php
+++ b/includes/Block/Topic.php
@@ -70,10 +70,16 @@
        protected function validate() {
                switch( $this->action ) {
                case 'edit-title':
+                       // hard-code fail during emergency
+                       $this->addError( 'readonly', wfMessage( 
'flow-error-other' ) );
+
                        $this->validateEditTitle();
                        break;
 
                case 'reply':
+                       // hard-code fail during emergency
+                       $this->addError( 'readonly', wfMessage( 
'flow-error-other' ) );
+
                        $this->validateReply();
                        break;
 
@@ -102,6 +108,9 @@
                        break;
 
                case 'edit-post':
+                       // hard-code fail during emergency
+                       $this->addError( 'readonly', wfMessage( 
'flow-error-other' ) );
+
                        $this->validateEditPost();
                        break;
 
diff --git a/includes/Block/TopicList.php b/includes/Block/TopicList.php
index 0d54b80..e5d94f7 100644
--- a/includes/Block/TopicList.php
+++ b/includes/Block/TopicList.php
@@ -40,6 +40,9 @@
        }
 
        protected function validate() {
+               // hard-code fail during emergency
+               $this->addError( 'readonly', wfMessage( 'flow-error-other' ) );
+
                // for now, new topic is considered a new post; perhaps some 
day topic creation should get it's own permissions?
                if ( !$this->permissions->isAllowed( null, 'new-post' ) ) {
                        $this->addError( 'permissions', wfMessage( 
'flow-error-not-allowed' ) );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic096bfca35f3da5ebe508e224fecac6361753061
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: wmf/1.23wmf9
Gerrit-Owner: Ori.livneh <o...@wikimedia.org>
Gerrit-Reviewer: EBernhardson <ebernhard...@wikimedia.org>
Gerrit-Reviewer: Ori.livneh <o...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot

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

Reply via email to