jenkins-bot has submitted this change and it was merged.

Change subject: Remove use of deprecated getPossibleErrors (since 1.24)
......................................................................


Remove use of deprecated getPossibleErrors (since 1.24)

Change-Id: I34111073384f0ccec2186bdd70ad308fbf800a37
---
M includes/api/ApiAddStudents.php
M includes/api/ApiDeleteEducation.php
M includes/api/ApiEnlist.php
M includes/api/ApiListStudents.php
M includes/api/ApiRefreshEducation.php
5 files changed, 0 insertions(+), 36 deletions(-)

Approvals:
  Legoktm: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/api/ApiAddStudents.php b/includes/api/ApiAddStudents.php
index 7e20865..2316c6b 100644
--- a/includes/api/ApiAddStudents.php
+++ b/includes/api/ApiAddStudents.php
@@ -177,13 +177,6 @@
                );
        }
 
-       public function getPossibleErrors() {
-               return array_merge( parent::getPossibleErrors(), array(
-                       array( 'code' => 'invalid-course', 'info' => 'There is 
no course with the provided ID.' ),
-                       array( 'code' => 'internal-error', 'info' => 'There was 
a problem enrolling students in this course.' ),
-                       array( 'code' => 'no-rights', 'info' => 'The user is 
not authorized to perform this action.' ),
-               ) );
-       }
 
        protected function getExamples() {
                return array(
diff --git a/includes/api/ApiDeleteEducation.php 
b/includes/api/ApiDeleteEducation.php
index 0e0e63b..b8b0e35 100644
--- a/includes/api/ApiDeleteEducation.php
+++ b/includes/api/ApiDeleteEducation.php
@@ -150,12 +150,6 @@
                );
        }
 
-       public function getPossibleErrors() {
-               return array_merge( parent::getPossibleErrors(), array(
-                       array( 'badaccess-groups' ),
-               ) );
-       }
-
        protected function getExamples() {
                return array(
                        'api.php?action=deleteeducation&ids=42&type=course',
diff --git a/includes/api/ApiEnlist.php b/includes/api/ApiEnlist.php
index 1cac40d..cf730ce 100644
--- a/includes/api/ApiEnlist.php
+++ b/includes/api/ApiEnlist.php
@@ -183,14 +183,6 @@
                );
        }
 
-       public function getPossibleErrors() {
-               return array_merge( parent::getPossibleErrors(), array(
-                       array( 'code' => 'username-xor-userid', 'info' => 'You 
need to either provide the username or the userid parameter' ),
-                       array( 'code' => 'invalid-user', 'info' => 'An invalid 
user name or id was provided' ),
-                       array( 'code' => 'invalid-course', 'info' => 'There is 
no course with the provided ID' ),
-               ) );
-       }
-
        protected function getExamples() {
                return array(
                        
'api.php?action=instructor&subaction=add&courseid=42&userid=9001',
diff --git a/includes/api/ApiListStudents.php b/includes/api/ApiListStudents.php
index 3ec2e61..3b973c8 100644
--- a/includes/api/ApiListStudents.php
+++ b/includes/api/ApiListStudents.php
@@ -445,15 +445,6 @@
                );
        }
 
-       public function getPossibleErrors() {
-               return array_merge( parent::getPossibleErrors(), array(
-                       array( 'code' => 'invalid-course',
-                               'info' => 'There is no course with the provided 
ID.' ),
-                       array( 'code' => 'over-query-limit',
-                               'info' => 'This query would return more 
usernames than allowed.' ),
-               ));
-       }
-
        protected function getExamples() {
                return array(
                        'api.php?action=liststudents&courseids=3',
diff --git a/includes/api/ApiRefreshEducation.php 
b/includes/api/ApiRefreshEducation.php
index 6ac9623..969fff9 100644
--- a/includes/api/ApiRefreshEducation.php
+++ b/includes/api/ApiRefreshEducation.php
@@ -84,12 +84,6 @@
                );
        }
 
-       public function getPossibleErrors() {
-               return array_merge( parent::getPossibleErrors(), array(
-                       array( 'badaccess-groups' ),
-               ) );
-       }
-
        protected function getExamples() {
                return array(
                        'api.php?action=refresheducation&ids=42&type=course',

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I34111073384f0ccec2186bdd70ad308fbf800a37
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/EducationProgram
Gerrit-Branch: master
Gerrit-Owner: Addshore <addshorew...@gmail.com>
Gerrit-Reviewer: AndyRussG <andrew.green...@gmail.com>
Gerrit-Reviewer: Awight <awi...@wikimedia.org>
Gerrit-Reviewer: Legoktm <legoktm.wikipe...@gmail.com>
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