Daniel Kinzler has uploaded a new change for review.

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


Change subject: Impl isWriteMode() for claim modules.
......................................................................

Impl isWriteMode() for claim modules.

All API modules that change the database should return true from isWriteMode().

Change-Id: If989c34bfd4ca3e50beb888507a774351e7c46b4
---
M repo/includes/api/ModifyClaim.php
M repo/includes/api/RemoveClaims.php
M repo/includes/api/SetClaim.php
3 files changed, 23 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase 
refs/changes/16/70816/1

diff --git a/repo/includes/api/ModifyClaim.php 
b/repo/includes/api/ModifyClaim.php
index a7892a8..624254d 100644
--- a/repo/includes/api/ModifyClaim.php
+++ b/repo/includes/api/ModifyClaim.php
@@ -58,4 +58,12 @@
                return $summary;
        }
 
+       /**
+        * @see ApiBase::isWriteMode
+        * @return bool true
+        */
+       public function isWriteMode() {
+               return true;
+       }
+
 }
diff --git a/repo/includes/api/RemoveClaims.php 
b/repo/includes/api/RemoveClaims.php
index 155db87..4883629 100644
--- a/repo/includes/api/RemoveClaims.php
+++ b/repo/includes/api/RemoveClaims.php
@@ -44,9 +44,13 @@
  */
 class RemoveClaims extends ApiWikibase {
 
-       // TODO: example
-       // TODO: rights
-       // TODO: conflict detection
+       /**
+        * @see ApiBase::isWriteMode
+        * @return bool true
+        */
+       public function isWriteMode() {
+               return true;
+       }
 
        /**
         * @see \ApiBase::execute
diff --git a/repo/includes/api/SetClaim.php b/repo/includes/api/SetClaim.php
index b724df3..bd4f131 100644
--- a/repo/includes/api/SetClaim.php
+++ b/repo/includes/api/SetClaim.php
@@ -48,8 +48,6 @@
  */
 class SetClaim extends ApiWikibase {
 
-       // TODO: rights
-
        /**
         * @var SnakValidationHelper
         */
@@ -74,6 +72,14 @@
        }
 
        /**
+        * @see ApiBase::isWriteMode
+        * @return bool true
+        */
+       public function isWriteMode() {
+               return true;
+       }
+
+       /**
         * @see ApiBase::execute
         *
         * @since 0.4

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If989c34bfd4ca3e50beb888507a774351e7c46b4
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Daniel Kinzler <daniel.kinz...@wikimedia.de>

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

Reply via email to