OpencachingDeutschland / oc-server3

htdocs/okapi/services/caches/mark.php

Braces +21 added lines, -18 removed lines

@@ -31,55 +31,58 @@
         # on it - this will also throw a proper exception if it doesn't exist.
 
         $cache_code = $request->get_parameter('cache_code');
-        if ($cache_code == null)
-            throw new ParamMissing('cache_code');
+        if ($cache_code == null) {
+                    throw new ParamMissing('cache_code');
+        }
         $geocache = OkapiServiceRunner::call('services/caches/geocache', new OkapiInternalRequest(
             $request->consumer, $request->token, array('cache_code' => $cache_code, 'fields' => 'internal_id')));
 
         # watched
 
-        if ($tmp = $request->get_parameter('watched'))
-        {
-            if (!in_array($tmp, array('true', 'false', 'unchanged')))
-                throw new InvalidParam('watched', $tmp);
-            if ($tmp == 'true')
-                Db::execute("
+        if ($tmp = $request->get_parameter('watched')) {
+            if (!in_array($tmp, array('true', 'false', 'unchanged'))) {
+                            throw new InvalidParam('watched', $tmp);
+            }
+            if ($tmp == 'true') {
+                            Db::execute("
                     insert ignore into cache_watches (cache_id, user_id)
                     values (
                         '".Db::escape_string($geocache['internal_id'])."',
                         '".Db::escape_string($request->token->user_id)."'
                     );
                 ");
-            elseif ($tmp == 'false')
-                Db::execute("
+            } elseif ($tmp == 'false') {
+                            Db::execute("
                     delete from cache_watches
                     where
                         cache_id = '".Db::escape_string($geocache['internal_id'])."'
                         and user_id = '".Db::escape_string($request->token->user_id)."';
                 ");
+            }
         }
 
         # ignored
 
-        if ($tmp = $request->get_parameter('ignored'))
-        {
-            if (!in_array($tmp, array('true', 'false', 'unchanged')))
-                throw new InvalidParam('ignored', $tmp);
-            if ($tmp == 'true')
-                Db::execute("
+        if ($tmp = $request->get_parameter('ignored')) {
+            if (!in_array($tmp, array('true', 'false', 'unchanged'))) {
+                            throw new InvalidParam('ignored', $tmp);
+            }
+            if ($tmp == 'true') {
+                            Db::execute("
                     insert ignore into cache_ignore (cache_id, user_id)
                     values (
                         '".Db::escape_string($geocache['internal_id'])."',
                         '".Db::escape_string($request->token->user_id)."'
                     );
                 ");
-            elseif ($tmp == 'false')
-                Db::execute("
+            } elseif ($tmp == 'false') {
+                            Db::execute("
                     delete from cache_ignore
                     where
                         cache_id = '".Db::escape_string($geocache['internal_id'])."'
                         and user_id = '".Db::escape_string($request->token->user_id)."'
                 ");
+            }
         }
 
         $result = array(

htdocs/okapi/services/users/by_internal_ids.php

Braces +16 added lines, -13 removed lines

@@ -23,14 +23,18 @@ 
     public static function call(OkapiRequest $request)
     {
         $internal_ids = $request->get_parameter('internal_ids');
-        if (!$internal_ids) throw new ParamMissing('internal_ids');
+        if (!$internal_ids) {
+            throw new ParamMissing('internal_ids');
+        }
         $internal_ids = explode("|", $internal_ids);
-        if (count($internal_ids) > 500)
-            throw new InvalidParam('internal_ids', "Maximum allowed number of referenced users ".
+        if (count($internal_ids) > 500) {
+                    throw new InvalidParam('internal_ids', "Maximum allowed number of referenced users ".
                 "is 500. You provided ".count($internal_ids)." references.");
+        }
         $fields = $request->get_parameter('fields');
-        if (!$fields)
-            throw new ParamMissing('fields');
+        if (!$fields) {
+                    throw new ParamMissing('fields');
+        }
 
         # There's no need to validate the fields parameter as the 'users'
         # method does this (it will raise a proper exception on invalid values).
@@ -41,8 +45,7 @@ 
             where user_id in ('".implode("','", array_map('\okapi\Db::escape_string', $internal_ids))."')
         ");
         $internalid2useruuid = array();
-        while ($row = Db::fetch_assoc($rs))
-        {
+        while ($row = Db::fetch_assoc($rs)) {
             $internalid2useruuid[$row['user_id']] = $row['uuid'];
         }
         Db::free_result($rs);
@@ -55,12 +58,12 @@ 
         # Map user_uuids to internal_ids. Also check which internal_ids were not found
         # and mark them with null.
         $results = array();
-        foreach ($internal_ids as $internal_id)
-        {
-            if (!isset($internalid2useruuid[$internal_id]))
-                $results[$internal_id] = null;
-            else
-                $results[$internal_id] = $id_results[$internalid2useruuid[$internal_id]];
+        foreach ($internal_ids as $internal_id) {
+            if (!isset($internalid2useruuid[$internal_id])) {
+                            $results[$internal_id] = null;
+            } else {
+                            $results[$internal_id] = $id_results[$internalid2useruuid[$internal_id]];
+            }
         }
 
         return Okapi::formatted_response($request, $results);

htdocs/okapi/services/users/user.php

Braces +6 added lines, -9 removed lines

@@ -26,16 +26,12 @@ 
     public static function call(OkapiRequest $request)
     {
         $user_uuid = $request->get_parameter('user_uuid');
-        if (!$user_uuid)
-        {
-            if ($request->token)
-            {
+        if (!$user_uuid) {
+            if ($request->token) {
                 $tmp = OkapiServiceRunner::call('services/users/by_internal_id', new OkapiInternalRequest(
                     $request->consumer, null, array('internal_id' => $request->token->user_id, 'fields' => 'uuid')));
                 $user_uuid = $tmp['uuid'];
-            }
-            else
-            {
+            } else {
                 throw new BadRequest("You must either: 1. supply the user_uuid argument, or "
                     ."2. sign your request with an Access Token.");
             }
@@ -49,8 +45,9 @@ 
             $request->consumer, $request->token, array('user_uuids' => $user_uuid,
             'fields' => $fields)));
         $result = $results[$user_uuid];
-        if ($result == null)
-            throw new InvalidParam('user_uuid', "There is no user by this ID.");
+        if ($result == null) {
+                    throw new InvalidParam('user_uuid', "There is no user by this ID.");
+        }
         return Okapi::formatted_response($request, $result);
     }
 }

htdocs/okapi/services/users/by_usernames.php

Braces +17 added lines, -15 removed lines

@@ -24,14 +24,18 @@ 
     public static function call(OkapiRequest $request)
     {
         $usernames = $request->get_parameter('usernames');
-        if (!$usernames) throw new ParamMissing('usernames');
+        if (!$usernames) {
+            throw new ParamMissing('usernames');
+        }
         $usernames = explode("|", $usernames);
-        if (count($usernames) > 500)
-            throw new InvalidParam('usernames', "Maximum allowed number of referenced users ".
+        if (count($usernames) > 500) {
+                    throw new InvalidParam('usernames', "Maximum allowed number of referenced users ".
                 "is 500. You provided ".count($usernames)." usernames.");
+        }
         $fields = $request->get_parameter('fields');
-        if (!$fields)
-            throw new ParamMissing('fields');
+        if (!$fields) {
+                    throw new ParamMissing('fields');
+        }
 
         # There's no need to validate the fields parameter as the 'users'
         # method does this (it will raise a proper exception on invalid values).
@@ -42,16 +46,14 @@ 
             where username collate ".Settings::get('DB_CHARSET')."_general_ci in ('".implode("','", array_map('\okapi\Db::escape_string', $usernames))."')
         ");
         $lower_username2useruuid = array();
-        while ($row = Db::fetch_assoc($rs))
-        {
+        while ($row = Db::fetch_assoc($rs)) {
             $lower_username2useruuid[mb_strtolower($row['username'], 'utf-8')] = $row['uuid'];
         }
         Db::free_result($rs);
 
         # Retrieve data for the found user_uuids.
 
-        if (count($lower_username2useruuid) > 0)
-        {
+        if (count($lower_username2useruuid) > 0) {
             $id_results = OkapiServiceRunner::call('services/users/users', new OkapiInternalRequest(
                 $request->consumer, $request->token, array('user_uuids' => implode("|", array_values($lower_username2useruuid)),
                 'fields' => $fields)));
@@ -63,12 +65,12 @@ 
         # and mark them with null.
 
         $results = array();
-        foreach ($usernames as $username)
-        {
-            if (!isset($lower_username2useruuid[mb_strtolower($username, 'utf-8')]))
-                $results[$username] = null;
-            else
-                $results[$username] = $id_results[$lower_username2useruuid[mb_strtolower($username, 'utf-8')]];
+        foreach ($usernames as $username) {
+            if (!isset($lower_username2useruuid[mb_strtolower($username, 'utf-8')])) {
+                            $results[$username] = null;
+            } else {
+                            $results[$username] = $id_results[$lower_username2useruuid[mb_strtolower($username, 'utf-8')]];
+            }
         }
 
         return Okapi::formatted_response($request, $results);

htdocs/okapi/services/apiref/issue.php

Braces +9 added lines, -11 removed lines

@@ -26,19 +26,19 @@ 
     public static function call(OkapiRequest $request)
     {
         $issue_id = $request->get_parameter('issue_id');
-        if (!$issue_id)
-            throw new ParamMissing('issue_id');
-        if ((!preg_match("/^[0-9]+$/", $issue_id)) || (strlen($issue_id) > 6))
-            throw new InvalidParam('issue_id');
+        if (!$issue_id) {
+                    throw new ParamMissing('issue_id');
+        }
+        if ((!preg_match("/^[0-9]+$/", $issue_id)) || (strlen($issue_id) > 6)) {
+                    throw new InvalidParam('issue_id');
+        }
 
         $cache_key = "apiref/issue#".$issue_id;
         $result = Cache::get($cache_key);
-        if ($result == null)
-        {
+        if ($result == null) {
             # Download the number of comments from GitHub Issue Tracker.
 
-            try
-            {
+            try {
                 $extra_headers = array();
                 $extra_headers[] = "Accept: application/vnd.github.v3.html+json";
                 $extra_headers[] = "User-Agent: https://github.com/opencaching/okapi/";
@@ -55,9 +55,7 @@ 
                 $context = stream_context_create($opts);
                 $json = file_get_contents("https://api.github.com/repos/opencaching/okapi/issues/$issue_id",
                     false, $context);
-            }
-            catch (ErrorException $e)
-            {
+            } catch (ErrorException $e) {
                 throw new BadRequest("Sorry, we could not retrieve issue stats from the GitHub site. ".
                     "This is probably due to a temporary connection problem. Try again later or contact ".
                     "us if this seems permanent.");

htdocs/okapi/services/apiref/method.php

Spacing +2 added lines, -2 removed lines

@@ -185,14 +185,14 @@
                 if (($include_list === null) && (count($exclude_list) == 0))
                 {
                     $arg['description'] = "<i>Inherited from <a href='".$referenced_method_info['ref_url'].
-                        "#arg_". $arg['name'] . "'>".$referenced_method_info['name']."</a> method.</i>";
+                        "#arg_".$arg['name']."'>".$referenced_method_info['name']."</a> method.</i>";
                 }
                 elseif (
                     (($include_list === null) || in_array($arg['name'], $include_list))
                     && (!in_array($arg['name'], $exclude_list))
                 ) {
                     $arg['description'] = "<i>Same as in the <a href='".$referenced_method_info['ref_url'].
-                        "#arg_". $arg['name'] . "'>".$referenced_method_info['name']."</a> method.</i>";
+                        "#arg_".$arg['name']."'>".$referenced_method_info['name']."</a> method.</i>";
                 } else {
                     continue;
                 }

Braces +51 added lines, -44 removed lines

@@ -102,12 +102,10 @@ 
             case 'methodretref':
                 $elements = explode('#', $arr[1]);
                 $result = '';
-                if ($elements[0] != '')
-                {
+                if ($elements[0] != '') {
                     $result .= Settings::get('SITE_URL')."okapi/".$elements[0].'.html';
                 }
-                if (count($elements) > 1)
-                {
+                if (count($elements) > 1) {
                     $result .= '#';
                     switch ($plugin_name) {
                         case 'methodargref':
@@ -128,15 +126,19 @@ 
     public static function call(OkapiRequest $request)
     {
         $methodname = $request->get_parameter('name');
-        if (!$methodname)
-            throw new ParamMissing('name');
-        if (!preg_match("#^services/[0-9a-z_/]*$#", $methodname))
-            throw new InvalidParam('name');
-        if (!OkapiServiceRunner::exists($methodname))
-            throw new InvalidParam('name', "Method does not exist: '$methodname'.");
+        if (!$methodname) {
+                    throw new ParamMissing('name');
+        }
+        if (!preg_match("#^services/[0-9a-z_/]*$#", $methodname)) {
+                    throw new InvalidParam('name');
+        }
+        if (!OkapiServiceRunner::exists($methodname)) {
+                    throw new InvalidParam('name', "Method does not exist: '$methodname'.");
+        }
         $options = OkapiServiceRunner::options($methodname);
-        if (!isset($options['min_auth_level']))
-            throw new Exception("Method $methodname is missing a required 'min_auth_level' option!");
+        if (!isset($options['min_auth_level'])) {
+                    throw new Exception("Method $methodname is missing a required 'min_auth_level' option!");
+        }
         $docs = simplexml_load_string(OkapiServiceRunner::docs($methodname));
         $exploded = explode("/", $methodname);
         $result = array(
@@ -149,45 +151,49 @@ 
                 'oauth_token' => $options['min_auth_level'] >= 3,
             )
         );
-        if (!$docs->brief)
-            throw new Exception("Missing <brief> element in the $methodname.xml file.");
-        if ($docs->brief != self::get_inner_xml($docs->brief))
-            throw new Exception("The <brief> element may not contain HTML markup ($methodname.xml).");
-        if (strlen($docs->brief) > 80)
-            throw new Exception("The <brief> description may not be longer than 80 characters ($methodname.xml).");
-        if (strpos($docs->brief, "\n") !== false)
-            throw new Exception("The <brief> element may not contain new-lines ($methodname.xml).");
-        if (substr(trim($docs->brief), -1) == '.')
-            throw new Exception("The <brief> element should not end with a dot ($methodname.xml).");
+        if (!$docs->brief) {
+                    throw new Exception("Missing <brief> element in the $methodname.xml file.");
+        }
+        if ($docs->brief != self::get_inner_xml($docs->brief)) {
+                    throw new Exception("The <brief> element may not contain HTML markup ($methodname.xml).");
+        }
+        if (strlen($docs->brief) > 80) {
+                    throw new Exception("The <brief> description may not be longer than 80 characters ($methodname.xml).");
+        }
+        if (strpos($docs->brief, "\n") !== false) {
+                    throw new Exception("The <brief> element may not contain new-lines ($methodname.xml).");
+        }
+        if (substr(trim($docs->brief), -1) == '.') {
+                    throw new Exception("The <brief> element should not end with a dot ($methodname.xml).");
+        }
         $result['brief_description'] = self::get_inner_xml($docs->brief);
-        if ($docs->{'issue-id'})
-            $result['issue_id'] = (string)$docs->{'issue-id'};
-        else
-            $result['issue_id'] = null;
-        if (!$docs->desc)
-            throw new Exception("Missing <desc> element in the $methodname.xml file.");
+        if ($docs->{'issue-id'}) {
+                    $result['issue_id'] = (string)$docs->{'issue-id'};
+        } else {
+                    $result['issue_id'] = null;
+        }
+        if (!$docs->desc) {
+                    throw new Exception("Missing <desc> element in the $methodname.xml file.");
+        }
         $result['description'] = self::get_inner_xml($docs->desc);
         $result['arguments'] = array();
         foreach ($docs->req as $arg) { $result['arguments'][] = self::arg_desc($arg); }
         foreach ($docs->opt as $arg) { $result['arguments'][] = self::arg_desc($arg); }
-        foreach ($docs->{'import-params'} as $import_desc)
-        {
+        foreach ($docs->{'import-params'} as $import_desc) {
             $attrs = $import_desc->attributes();
             $referenced_methodname = $attrs['method'];
             $referenced_method_info = OkapiServiceRunner::call('services/apiref/method',
                 new OkapiInternalRequest(new OkapiInternalConsumer(), null, array('name' => $referenced_methodname)));
             $include_list = isset($attrs['params']) ? explode("|", $attrs['params']) : null;
             $exclude_list = isset($attrs['except']) ? explode("|", $attrs['except']) : array();
-            foreach ($referenced_method_info['arguments'] as $arg)
-            {
-                if ($arg['class'] == 'common-formatting')
-                    continue;
-                if (($include_list === null) && (count($exclude_list) == 0))
-                {
+            foreach ($referenced_method_info['arguments'] as $arg) {
+                if ($arg['class'] == 'common-formatting') {
+                                    continue;
+                }
+                if (($include_list === null) && (count($exclude_list) == 0)) {
                     $arg['description'] = "<i>Inherited from <a href='".$referenced_method_info['ref_url'].
                         "#arg_". $arg['name'] . "'>".$referenced_method_info['name']."</a> method.</i>";
-                }
-                elseif (
+                } elseif (
                     (($include_list === null) || in_array($arg['name'], $include_list))
                     && (!in_array($arg['name'], $exclude_list))
                 ) {
@@ -200,8 +206,7 @@ 
                 $result['arguments'][] = $arg;
             }
         }
-        if ($docs->{'common-format-params'})
-        {
+        if ($docs->{'common-format-params'}) {
             $result['arguments'][] = array(
                 'name' => 'format',
                 'is_required' => false,
@@ -217,12 +222,14 @@ 
                 'description' => "<i>Standard <a href='".Settings::get('SITE_URL')."okapi/introduction.html#common-formatting'>common formatting</a> argument.</i>"
             );
         }
-        foreach ($result['arguments'] as &$arg_ref)
-            if ($arg_ref['is_deprecated'])
+        foreach ($result['arguments'] as &$arg_ref) {
+                    if ($arg_ref['is_deprecated'])
                 $arg_ref['class'] .= " deprecated";
-        if (!$docs->returns)
-            throw new Exception("Missing <returns> element in the $methodname.xml file. ".
+        }
+        if (!$docs->returns) {
+                    throw new Exception("Missing <returns> element in the $methodname.xml file. ".
                 "If your method does not return anything, you should document in nonetheless.");
+        }
         $result['returns'] = self::get_inner_xml($docs->returns);
         return Okapi::formatted_response($request, $result);
     }

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -136,7 +136,7 @@
             throw new InvalidParam('name', "Method does not exist: '$methodname'.");
         $options = OkapiServiceRunner::options($methodname);
         if (!isset($options['min_auth_level']))
-            throw new Exception("Method $methodname is missing a required 'min_auth_level' option!");
+            throw new Exception("method $methodname is missing a required 'min_auth_level' option!");
         $docs = simplexml_load_string(OkapiServiceRunner::docs($methodname));
         $exploded = explode("/", $methodname);
         $result = array(

htdocs/okapi/services/apiref/method_index.php

Braces +2 added lines, -4 removed lines

@@ -28,11 +28,9 @@
         sort($methodnames);
         $cache_key = "api_ref/method_index#".md5(implode("#", $methodnames));
         $results = Cache::get($cache_key);
-        if ($results == null)
-        {
+        if ($results == null) {
             $results = array();
-            foreach ($methodnames as $methodname)
-            {
+            foreach ($methodnames as $methodname) {
                 $info = OkapiServiceRunner::call('services/apiref/method', new OkapiInternalRequest(
                     new OkapiInternalConsumer(), null, array('name' => $methodname)));
                 $results[] = array(

htdocs/okapi/services/apisrv/stats.php

Braces +1 added lines, -2 removed lines

@@ -26,8 +26,7 @@
     {
         $cachekey = "apisrv/stats";
         $result = Cache::get($cachekey);
-        if (!$result)
-        {
+        if (!$result) {
             $result = array(
                 'cache_count' => 0 + Db::select_value("
                     select count(*) from caches where status in (1,2,3)

htdocs/okapi/services/apisrv/installations.php

Spacing +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
             # Cache it for one day. Also, save a backup (valid for 30 days).
 
             Cache::set($cachekey, $results, 86400);
-            Cache::set($backupkey, $results, 86400*30);
+            Cache::set($backupkey, $results, 86400 * 30);
         }
 
         return Okapi::formatted_response($request, $results);

Braces +19 added lines, -23 removed lines

@@ -30,12 +30,10 @@ 
         $cachekey = 'apisrv/installations';
         $backupkey = 'apisrv/installations-backup';
         $results = Cache::get($cachekey);
-        if (!$results)
-        {
+        if (!$results) {
             # Download the current list of OKAPI servers.
 
-            try
-            {
+            try {
                 $opts = array(
                     'http' => array(
                         'method' => "GET",
@@ -49,14 +47,11 @@ 
                 if (!$doc) {
                     throw new ErrorException(); # just to get to the catch block
                 }
-            }
-            catch (ErrorException $e)
-            {
+            } catch (ErrorException $e) {
                 # GitHub failed on us. Try to respond with a backup list.
 
                 $results = Cache::get($backupkey);
-                if ($results)
-                {
+                if ($results) {
                     Cache::set($cachekey, $results, 12 * 3600); # so to retry no earlier than after 12 hours
                     return Okapi::formatted_response($request, $results);
                 }
@@ -77,31 +72,32 @@ 
 
             $results = array();
             $i_was_included = false;
-            foreach ($doc->installation as $inst)
-            {
+            foreach ($doc->installation as $inst) {
                 $site_url = (string)$inst[0]['site_url'];
-                if ($inst[0]['okapi_base_url'])
-                    $okapi_base_url = (string)$inst[0]['okapi_base_url'];
-                else
-                    $okapi_base_url = $site_url."okapi/";
-                if ($inst[0]['site_name'])
-                    $site_name = (string)$inst[0]['site_name'];
-                else
-                    $site_name = Okapi::get_normalized_site_name($site_url);
+                if ($inst[0]['okapi_base_url']) {
+                                    $okapi_base_url = (string)$inst[0]['okapi_base_url'];
+                } else {
+                                    $okapi_base_url = $site_url."okapi/";
+                }
+                if ($inst[0]['site_name']) {
+                                    $site_name = (string)$inst[0]['site_name'];
+                } else {
+                                    $site_name = Okapi::get_normalized_site_name($site_url);
+                }
                 $results[] = array(
                     'site_url' => $site_url,
                     'site_name' => $site_name,
                     'okapi_base_url' => $okapi_base_url,
                 );
-                if ($site_url == Settings::get('SITE_URL'))
-                    $i_was_included = true;
+                if ($site_url == Settings::get('SITE_URL')) {
+                                    $i_was_included = true;
+                }
             }
 
             # If running on a local development installation, then include the local
             # installation URL.
 
-            if (!$i_was_included)
-            {
+            if (!$i_was_included) {
                 $results[] = array(
                     'site_url' => Settings::get('SITE_URL'),
                     'site_name' => "DEVELSITE",