Merge branch 'MDL-56372_static_bind_clean_return_val' of https://github.com/BenKelada...
authorDan Poltawski <dan@moodle.com>
Wed, 2 Nov 2016 15:14:46 +0000 (15:14 +0000)
committerDan Poltawski <dan@moodle.com>
Wed, 2 Nov 2016 15:14:46 +0000 (15:14 +0000)
1  2 
lib/externallib.php

diff --combined lib/externallib.php
@@@ -316,7 -316,7 +316,7 @@@ class external_api 
                      }
                      if ($subdesc->required == VALUE_DEFAULT) {
                          try {
-                             $result[$key] = self::validate_parameters($subdesc, $subdesc->default);
+                             $result[$key] = static::validate_parameters($subdesc, $subdesc->default);
                          } catch (invalid_parameter_exception $e) {
                              //we are only interested by exceptions returned by validate_param() and validate_parameters()
                              //(in order to build the path to the faulty attribut)
                      }
                  } else {
                      try {
-                         $result[$key] = self::validate_parameters($subdesc, $params[$key]);
+                         $result[$key] = static::validate_parameters($subdesc, $params[$key]);
                      } catch (invalid_parameter_exception $e) {
                          //we are only interested by exceptions returned by validate_param() and validate_parameters()
                          //(in order to build the path to the faulty attribut)
              }
              $result = array();
              foreach ($params as $param) {
-                 $result[] = self::validate_parameters($description->content, $param);
+                 $result[] = static::validate_parameters($description->content, $param);
              }
              return $result;
  
                      if ($subdesc instanceof external_value) {
                          if ($subdesc->required == VALUE_DEFAULT) {
                              try {
-                                     $result[$key] = self::clean_returnvalue($subdesc, $subdesc->default);
+                                     $result[$key] = static::clean_returnvalue($subdesc, $subdesc->default);
                              } catch (invalid_response_exception $e) {
                                  //build the path to the faulty attribut
                                  throw new invalid_response_exception($key." => ".$e->getMessage() . ': ' . $e->debuginfo);
                      }
                  } else {
                      try {
-                         $result[$key] = self::clean_returnvalue($subdesc, $response[$key]);
+                         $result[$key] = static::clean_returnvalue($subdesc, $response[$key]);
                      } catch (invalid_response_exception $e) {
                          //build the path to the faulty attribut
                          throw new invalid_response_exception($key." => ".$e->getMessage() . ': ' . $e->debuginfo);
              }
              $result = array();
              foreach ($response as $param) {
-                 $result[] = self::clean_returnvalue($description->content, $param);
+                 $result[] = static::clean_returnvalue($description->content, $param);
              }
              return $result;
  
@@@ -724,7 -724,6 +724,7 @@@ function external_generate_token($token
      if (!empty($iprestriction)) {
          $newtoken->iprestriction = $iprestriction;
      }
 +    $newtoken->privatetoken = null;
      $DB->insert_record('external_tokens', $newtoken);
      return $newtoken->token;
  }
@@@ -816,14 -815,11 +816,14 @@@ class external_format_value extends ext
       *
       * @param string $textfieldname Name of the text field
       * @param int $required if VALUE_REQUIRED then set standard default FORMAT_HTML
 +     * @param int $default Default value.
       * @since Moodle 2.3
       */
 -    public function __construct($textfieldname, $required = VALUE_REQUIRED) {
 +    public function __construct($textfieldname, $required = VALUE_REQUIRED, $default = null) {
  
 -        $default = ($required == VALUE_DEFAULT) ? FORMAT_HTML : null;
 +        if ($default == null && $required == VALUE_DEFAULT) {
 +            $default = FORMAT_HTML;
 +        }
  
          $desc = $textfieldname . ' format (' . FORMAT_HTML . ' = HTML, '
                  . FORMAT_MOODLE . ' = MOODLE, '
@@@ -1054,8 -1050,6 +1054,8 @@@ function external_generate_token_for_cu
              $token->externalserviceid = $service->id;
              // MDL-43119 Token valid for 3 months (12 weeks).
              $token->validuntil = $token->timecreated + 12 * WEEKSECS;
 +            // Generate the private token, it must be transmitted only via https.
 +            $token->privatetoken = random_string(64);
              $token->id = $DB->insert_record('external_tokens', $token);
  
              $params = array(