GEANT / CAT

core/User.php

Doc Comments +4 added lines, -1 removed lines

@@ -80,7 +80,7 @@ 
      * user is a federation administrator of *any* federation. When given a parameter (ISO shortname of federation), it checks
      * if the user administers this particular federation.
      * 
-     * @param string $federation optional: federation to be checked
+     * @param integer $federation optional: federation to be checked
      * @return boolean TRUE if the user is federation admin, FALSE if not 
      */
     public function isFederationAdmin($federation = 0) {
@@ -124,6 +124,9 @@ 
         return FALSE;
     }
 
+    /**
+     * @param string $content
+     */
     public function sendMailToUser($subject, $content) {
         $mailaddr = $this->getAttributes("user:email");
         if (count($mailaddr) == 0) { // we don't know user's mail address

Spacing +3 added lines, -3 removed lines

@@ -186,7 +186,7 @@ 
                 }
                 $lookFor .= "$name";
             }
-            $finding = preg_match("/^(".$lookFor."):(.*)/", $oneRow->user_id, $matches);
+            $finding = preg_match("/^(" . $lookFor . "):(.*)/", $oneRow->user_id, $matches);
             if ($finding === 0 || $finding === FALSE) {
                 return FALSE;
             }
@@ -207,7 +207,7 @@ 
                             $url = CONFIG_DIAGNOSTICS['eduGainResolver']['url'] . "?action=get_entity_name&type=idp&e_id=$idp&lang=pl";
                             $ch = curl_init($url);
                             curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
-                            curl_setopt($ch,CURLOPT_TIMEOUT, CONFIG_DIAGNOSTICS['eduGainResolver']['timeout']);
+                            curl_setopt($ch, CURLOPT_TIMEOUT, CONFIG_DIAGNOSTICS['eduGainResolver']['timeout']);
                             $response = curl_exec($ch);
                             if ($response == FALSE) {
                                 $skipCurl = 1;
@@ -224,7 +224,7 @@ 
                 case $providerStrings[3]:
                 case $providerStrings[4]:
                 case $providerStrings[5]:
-                    if (!in_array(User::PROVIDER_STRINGS[$matches[1]],$listOfProviders)) {
+                    if (!in_array(User::PROVIDER_STRINGS[$matches[1]], $listOfProviders)) {
                         $providerName = $this->getEdugainName(User::PROVIDER_STRINGS[$matches[1]]);
                         $listOfProviders[] = $providerName . "xx";
 //                        $listOfProviders[] = User::PROVIDER_STRINGS[$matches[1]];