web service MDL-12886 trivial refactoring

moodle · Jan 30, 2009 · f4358aa · f4358aa
1 parent 965a0eb
commit f4358aa
Show file tree

Hide file tree

Showing 5 changed files with 27 additions and 38 deletions.
diff --git a/webservice/amf/server.php b/webservice/amf/server.php
@@ -27,22 +27,6 @@
 $server->setClass($classpath."_external");
 $response = $server->handle();
 echo $response;
-/*
- *
-
-varlog("-- The Moodle AMF server is running --");
-//basic test server
-$server = new Zend_Amf_Server();
-$server->addFunction('hello');
-$response = $server->handle();
-echo $response;
 
 
-function hello($name, $greeting = 'The Moodle server say Hi to')
-{
-    varlog($greeting . ', ' . $name);
-    return $greeting . ', ' . $name;
-}
-*/
-
 ?>
diff --git a/webservice/rest/testclient/index.php → .../rest/testclient/php_curl_rest_client.php b/webservice/rest/testclient/index.php → .../rest/testclient/php_curl_rest_client.php
diff --git a/webservice/rest/testclient/zend_rest_client.php b/webservice/rest/testclient/zend_rest_client.php
@@ -20,30 +20,30 @@
 $client = new Zend_Rest_Client($CFG->wwwroot."/webservice/rest/zend_rest_server.php");
 
 
-    $token = $client->tmp_get_token(array('username' => "wsuser", 'password' => "wspassword"))->get();
-   echo $token->response();
-   $token = $token->response();
-    printLastRequestResponse($client);
+$token = $client->tmp_get_token(array('username' => "wsuser", 'password' => "wspassword"))->get();
+echo $token->response();
+$token = $token->response();
+printLastRequestResponse($client);
 
 
 //2. test functions
 $client = new Zend_Rest_Client($CFG->wwwroot."/webservice/rest/zend_rest_server.php/?classpath=user&token=".$token);
-   
-    var_dump($client->tmp_get_users(array('search' => "admin"))->get());
-    printLastRequestResponse($client);
-    var_dump($client->tmp_create_user(array('username' => "mockuser66",'firstname' => "firstname6",'lastname' => "lastname6",'email' => "mockuser6@mockuser6.com",'password' => "password6"))->get());
-    printLastRequestResponse($client);
-    var_dump($client->tmp_update_user(array('username' => "mockuser66",'mnethostid' => 1,'newusername' => "mockuser6b",'firstname' => "firstname6b"))->get());
-   printLastRequestResponse($client);
-    var_dump($client->tmp_delete_user(array('username' => "mockuser6b",'mnethostid' => 1))->get());
-   printLastRequestResponse($client);
+
+var_dump($client->tmp_get_users(array('search' => "admin"))->get());
+printLastRequestResponse($client);
+var_dump($client->tmp_create_user(array('username' => "mockuser66",'firstname' => "firstname6",'lastname' => "lastname6",'email' => "mockuser6@mockuser6.com",'password' => "password6"))->get());
+printLastRequestResponse($client);
+var_dump($client->tmp_update_user(array('username' => "mockuser66",'mnethostid' => 1,'newusername' => "mockuser6b",'firstname' => "firstname6b"))->get());
+printLastRequestResponse($client);
+var_dump($client->tmp_delete_user(array('username' => "mockuser6b",'mnethostid' => 1))->get());
+printLastRequestResponse($client);
 
 
 
 function printLastRequestResponse($client) {
     print "<pre>\n";
-  //  print "Request :\n".htmlspecialchars($client->__getLastRequest()) ."\n";
-   // print "Response:\n".htmlspecialchars($client->__getLastResponse())."\n";
+    //  print "Request :\n".htmlspecialchars($client->__getLastRequest()) ."\n";
+    // print "Response:\n".htmlspecialchars($client->__getLastResponse())."\n";
     print "</pre>";
 }
 

diff --git a/webservice/rest/zend_rest_server.php b/webservice/rest/zend_rest_server.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * Main script - SOAP server
+ * Main script - REST server
  *
  * @author Jerome Mouneyrac <jerome@moodle.com>
  * @version 1.0
@@ -38,7 +38,7 @@
     //retrieve the api name
     $classpath = optional_param(classpath,null,PARAM_ALPHA);
     require_once(dirname(__FILE__) . '/../../'.$classpath.'/external.php');
- 
+
     /// run the server
     $server = new Zend_Rest_Server(); //TODO: need to call the wsdl generation on the fly
     $server->setClass($classpath."_external"); //TODO: pass $user as parameter
@@ -64,12 +64,17 @@ function mock_check_token($token) {
 }
 
 class soap_authentication {
+    /**
+     *
+     * @param array $params
+     * @return integer
+     */
     function tmp_get_token($params) {
         if ($params['username'] == 'wsuser' && $params['password'] == 'wspassword') {
-                return '465465465468468464';
-            } else {
-                throw new moodle_exception('wrongusernamepassword');
-            }
+            return '465465465468468464';
+        } else {
+            throw new moodle_exception('wrongusernamepassword');
+        }
     }
 }
 ?>
diff --git a/webservice/soap/testclient/index.php → ...rvice/soap/testclient/php_soap_client.php b/webservice/soap/testclient/index.php → ...rvice/soap/testclient/php_soap_client.php
@@ -21,7 +21,7 @@
 try {
     $token = $client->tmp_get_token(array('username' => "wsuser", 'password' => "wspassword"));
     printLastRequestResponse($client);
-   
+
 } catch (SoapFault $exception) {
     echo $exception;
 }