From 437d9ff54c849b0bfe6d049c47f2288a15368902 Mon Sep 17 00:00:00 2001
From: jerome <jerome>
Date: Tue, 24 Feb 2009 08:26:25 +0000
Subject: [PATCH] web service MDL-12886 modify phpdoc for group_external class

---
 group/external.php | 73 +++++++++++++++++++++++-----------------------
 webservice/lib.php |  4 +--
 2 files changed, 39 insertions(+), 38 deletions(-)

diff --git a/group/external.php b/group/external.php
index 358e56648f..98a9dc3a82 100644
--- a/group/external.php
+++ b/group/external.php
@@ -19,42 +19,12 @@ require_once(dirname(dirname(__FILE__)) . '/lib/grouplib.php');
  */
 final class group_external {
 
-    /**
-     * Constructor - We set the description of this API in order to be access by Web service
-     */
-    /*
-    function __construct () {
-          $this->descriptions = array();
-
-
-          $this->descriptions['tmp_create_group']   = array( 'params' => array('groupname'=> PARAM_RAW, 'courseid'=> PARAM_INT),
-                                                            'optionalparams' => array( ),
-                                                            'return' => array('groupid' => PARAM_INT));
-
-          $this->descriptions['tmp_get_group']     = array( 'params' => array('groupid'=> PARAM_INT),
-                                                            'optionalparams' => array( ),
-                                                            'return' => array('group' => array('id' => PARAM_RAW, 'courseid' => PARAM_RAW,
-																		'name' => PARAM_RAW, 'enrolmentkey' => PARAM_RAW)));
-		  $this->descriptions['tmp_delete_group']     = array( 'params' => array('groupid'=> PARAM_INT),
-                                                            'optionalparams' => array( ),
-                                                            'return' => array('result' => PARAM_BOOL));
-
-          $this->descriptions['tmp_add_groupmember']   = array( 'params' => array('groupid'=> PARAM_INT, 'userid'=> PARAM_INT),
-                                                            'optionalparams' => array( ),
-                                                            'return' => array('result' => PARAM_BOOL));
-
-		  $this->descriptions['tmp_delete_groupmember']   = array( 'params' => array('groupid'=> PARAM_INT, 'userid'=> PARAM_INT),
-                                                            'optionalparams' => array( ),
-                                                            'return' => array('result' => PARAM_BOOL));
-
-    }*/
-
     /**
      * Creates a group
      * @param array $params
-     *  ->courseid int
-     *  ->groupname string
-     * @return int userid
+     * @subparam string $params->groupname
+     * @subparam integer $params->courseid
+     * @return integer groupid
      */
     static function tmp_create_group($params) {
         global $USER;
@@ -75,9 +45,12 @@ final class group_external {
     /**
      * add a group member
      * @param array $params
-     *  ->groupid int
-     *  ->userid int
-     * @return boolean success
+     * @subparam integer $params->groupid
+     * @return array $group
+     * @subreturn integer $group->id
+     * @subreturn integer $group->courseid
+     * @subreturn string $group->name
+     * @subreturn string $group->enrolmentkey
      */
 	static function tmp_get_group($params){
 
@@ -94,6 +67,13 @@ final class group_external {
 
 	}
 
+
+    /**
+     *
+     * @param array $params
+     * @subparam integer $params->groupid
+     * @return boolean result
+     */
 	static function tmp_delete_group($params){
 
 		if (has_capability('moodle/course:managegroups', get_context_instance(CONTEXT_SYSTEM))) {
@@ -106,9 +86,23 @@ final class group_external {
         }
 	}
 
+    /**
+     *
+     * @param array $params
+     * @subparam integer $params->groupid
+     * @subparam integer $params->userid
+     * @return boolean result
+     */
 	static function tmp_get_groupmember($params){
 	}
 
+    /**
+     *
+     * @param array $params
+     * @subparam integer $params->groupid
+     * @subparam integer $params->userid
+     * @return boolean result
+     */
 	static function tmp_add_groupmember($params){
 
 		if (has_capability('moodle/course:managegroups', get_context_instance(CONTEXT_SYSTEM))) {
@@ -121,6 +115,13 @@ final class group_external {
         }
 	}
 
+    /**
+     *
+     * @param array $params
+     * @subparam integer $params->groupid
+     * @subparam integer $params->userid
+     * @return boolean result
+     */
 	static function tmp_delete_groupmember($params){
 		if (has_capability('moodle/course:managegroups', get_context_instance(CONTEXT_SYSTEM))) {
 
diff --git a/webservice/lib.php b/webservice/lib.php
index 33794ad939..3e1153701a 100644
--- a/webservice/lib.php
+++ b/webservice/lib.php
@@ -164,7 +164,7 @@ final class webservice_lib {
                             break;
                     }
 
-                    if (empty($description)) {
+                    if (empty($description[$method->getName()])) {
                         $description[$method->getName()] = array();
                     }
 
@@ -202,7 +202,7 @@ final class webservice_lib {
 
                         $otherparam = substr($matches[3][$i],strpos($matches[3][$i], $separator)+$separatorsize);
                         $parsingdesc = $description[$method->getName()];
-
+                        
                         if (!empty($parsingdesc) && array_key_exists($descriptiontype, $parsingdesc)){
                             $parsingdesc = $parsingdesc[$descriptiontype];
                         }
-- 
2.39.5