OpencachingDeutschland / oc-server3

htdocs/okapi/services/oauth/request_token.php

Braces +1 added lines, -2 removed lines

@@ -20,8 +20,7 @@
     public static function call(OkapiRequest $request)
     {
         $callback = $request->get_parameter('oauth_callback');
-        if (!$callback)
-        {
+        if (!$callback) {
             # We require the 1.0a flow (throw an error when there is no oauth_callback).
             throw new ParamMissing("oauth_callback");
         }

htdocs/okapi/services/oauth/access_token.php

Braces +1 added lines, -2 removed lines

@@ -21,8 +21,7 @@
     public static function call(OkapiRequest $request)
     {
         $verifier = $request->get_parameter('oauth_verifier');
-        if (!$verifier)
-        {
+        if (!$verifier) {
             # We require the 1.0a flow (throw an error when there is no oauth_verifier).
             throw new ParamMissing("oauth_verifier");
         }

htdocs/okapi/services/caches/formatters/ggzindex.tpl.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
                 <?php if (isset($c['ratings'])) {
                 ?><ratings>
                     <?php
-                        foreach ($c['ratings'] as $rating_key => $rating_val){
+                        foreach ($c['ratings'] as $rating_key => $rating_val) {
                             echo "<$rating_key>$rating_val</$rating_key>\n";
                         }
                     ?>

Braces +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
                 <?php if (isset($c['ratings'])) {
                 ?><ratings>
                     <?php
-                        foreach ($c['ratings'] as $rating_key => $rating_val){
+                        foreach ($c['ratings'] as $rating_key => $rating_val) {
                             echo "<$rating_key>$rating_val</$rating_key>\n";
                         }
                     ?>

htdocs/okapi/services/caches/formatters/ggz.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
         # Geocaching.com (groundspeak:) and Opencaching.com (ox:) extensions. It will
         # also include personal data (if the method was invoked using Level 3 Authentication).
 
-        $file_item_name = "data_".time()."_".rand(100000,999999).".gpx";
+        $file_item_name = "data_".time()."_".rand(100000, 999999).".gpx";
         $ggz_file = array(
             'name' => $file_item_name,
             'crc32' => sprintf('%08X', crc32($gpx_result['gpx'])),

htdocs/okapi/services/caches/formatters/gpxfile.tpl.php

Spacing +4 added lines, -4 removed lines

@@ -17,7 +17,7 @@ 
 http://www.gsak.net/xmlv1/5 http://www.gsak.net/xmlv1/5/gsak.xsd
 ">
     <name><?= $vars['installation']['site_name'] ?> Geocache Search Results</name>
-    <desc><?= $vars['installation']['site_name'] ?> Geocache Search Results, downloaded via OKAPI - <?= $vars['installation']['okapi_base_url'] . ($vars['alt_wpts'] && $vars['ns_gsak'] ? ' (HasChildren)' : '') ?></desc>
+    <desc><?= $vars['installation']['site_name'] ?> Geocache Search Results, downloaded via OKAPI - <?= $vars['installation']['okapi_base_url'].($vars['alt_wpts'] && $vars['ns_gsak'] ? ' (HasChildren)' : '') ?></desc>
     <author><?= $vars['installation']['site_name'] ?></author>
     <url><?= $vars['installation']['site_url'] ?></url>
     <urlname><?= $vars['installation']['site_name'] ?></urlname>
@@ -140,7 +140,7 @@ 
                         <?php if ($vars['protection_areas'] == 'desc:text' && count($c['protection_areas'])) { ?>
                             &lt;p&gt;<?= _("The cache probably is located in the following protection areas:") ?>&lt;/p&gt;
                             &lt;ul&gt;
-                            <?php foreach($c['protection_areas'] as $protection_area) { ?>
+                            <?php foreach ($c['protection_areas'] as $protection_area) { ?>
                                 &lt;li&gt;<?= Okapi::xmlescape($protection_area['type'])." - ".Okapi::xmlescape($protection_area['name']) ?>&lt;/li&gt;
                             <?php } ?>
                             &lt;/ul;&gt;
@@ -157,7 +157,7 @@ 
                                     <groundspeak:date><?= $log['date'] ?></groundspeak:date>
                                     <groundspeak:type><?= $log['type'] ?></groundspeak:type>
                                     <groundspeak:finder id="<?= $vars['user_uuid_to_internal_id'][$log['user']['uuid']] ?>"><?= Okapi::xmlescape($log['user']['username']) ?></groundspeak:finder>
-                                    <groundspeak:text encoded="False"><?= $log['was_recommended'] ? "(*) ": "" ?><?= Okapi::xmlescape($log['comment']) ?></groundspeak:text>
+                                    <groundspeak:text encoded="False"><?= $log['was_recommended'] ? "(*) " : "" ?><?= Okapi::xmlescape($log['comment']) ?></groundspeak:text>
                                 </groundspeak:log>
                             <?php } ?>
                         </groundspeak:logs>
@@ -220,7 +220,7 @@ 
                     <?php } ?>
                 </wpt>
                 <?php
-                    if (isset($wpt_ref['ggz_entry'])){
+                    if (isset($wpt_ref['ggz_entry'])) {
                         $wpt_ref['ggz_entry']['file_len'] = ob_get_length() - $wpt_ref['ggz_entry']['file_pos'];
                     }
                 ?>

Braces +35 added lines, -18 removed lines

@@ -40,14 +40,16 @@ 
             <urlname><?= Okapi::xmlescape($c['name']) ?></urlname>
             <sym><?= ($vars['mark_found'] && $c['is_found']) ? "Geocache Found" : "Geocache" ?></sym>
             <type>Geocache|<?= $vars['cache_GPX_types'][$c['type']] ?></type>
-            <?php if ($vars['ns_ground']) { /* Does user want us to include Groundspeak's <cache> element? */ ?>
+            <?php if ($vars['ns_ground']) {
+/* Does user want us to include Groundspeak's <cache> element? */ ?>
                 <groundspeak:cache archived="<?= ($c['status'] == 'Archived') ? "True" : "False" ?>" available="<?= ($c['status'] == 'Available') ? "True" : "False" ?>" id="<?= $c['internal_id'] ?>" xmlns:groundspeak="http://www.groundspeak.com/cache/1/0/1">
                     <groundspeak:name><?= Okapi::xmlescape(isset($c['name_2']) ? $c['name_2'] : $c['name']) ?></groundspeak:name>
                     <groundspeak:placed_by><?= Okapi::xmlescape($c['owner']['username']) ?></groundspeak:placed_by>
                     <groundspeak:owner id="<?= $vars['user_uuid_to_internal_id'][$c['owner']['uuid']] ?>"><?= Okapi::xmlescape($c['owner']['username']) ?></groundspeak:owner>
                     <groundspeak:type><?= $vars['cache_GPX_types'][$c['type']] ?></groundspeak:type>
                     <groundspeak:container><?= $vars['cache_GPX_sizes'][$c['size2']] ?></groundspeak:container>
-                    <?php if ($vars['gc_attrs'] || $vars['gc_ocde_attrs']) { /* Does user want us to include groundspeak:attributes? */ ?>
+                    <?php if ($vars['gc_attrs'] || $vars['gc_ocde_attrs']) {
+/* Does user want us to include groundspeak:attributes? */ ?>
                         <groundspeak:attributes>
                             <?php
                                 foreach ($c['gc_attrs'] as $gc_id => $gc_attr) {
@@ -71,23 +73,28 @@ 
                         &lt;p&gt;
                             &lt;a href="<?= $c['url'] ?>"&gt;<?= Okapi::xmlescape($c['name']) ?>&lt;/a&gt;
                             <?= _("hidden by") ?> &lt;a href='<?= $c['owner']['profile_url'] ?>'&gt;<?= Okapi::xmlescape($c['owner']['username']) ?>&lt;/a&gt;&lt;br/&gt;
-                            <?php if ($vars['recommendations'] == 'desc:count') { /* Does user want us to include recommendations count? */ ?>
+                            <?php if ($vars['recommendations'] == 'desc:count') {
+/* Does user want us to include recommendations count? */ ?>
                                 <?= sprintf(ngettext("%d recommendation", "%d recommendations", $c['recommendations']), $c['recommendations']) ?>
                                 (<?= sprintf(ngettext("found %d time", "found %d times", $c['founds']), $c['founds']) ?>).
                             <?php } ?>
-                            <?php if ($vars['trackables'] == 'desc:count') { /* Does user want us to include trackables count? */ ?>
+                            <?php if ($vars['trackables'] == 'desc:count') {
+/* Does user want us to include trackables count? */ ?>
                                 <?= sprintf(ngettext("%d trackable", "%d trackables", $c['trackables_count']), $c['trackables_count']) ?>.
                             <?php } ?>
                         &lt;/p&gt;
-                        <?php if ((in_array('desc:text', $vars['my_notes'])) && ($c['my_notes'] != null)) { /* Does user want us to include personal notes? */ ?>
+                        <?php if ((in_array('desc:text', $vars['my_notes'])) && ($c['my_notes'] != null)) {
+/* Does user want us to include personal notes? */ ?>
                             &lt;p&gt;&lt;b&gt;<?= _("Personal notes") ?>:&lt;/b&gt;&lt;br&gt;<?= Okapi::xmlescape(nl2br($c['my_notes'])) ?>&lt;/p&gt;
                         <?php } ?>
 
-                        <?php if (in_array('desc:text', $vars['attrs']) && count($c['attrnames']) > 0) { /* Does user want us to include attributes? */ ?>
+                        <?php if (in_array('desc:text', $vars['attrs']) && count($c['attrnames']) > 0) {
+/* Does user want us to include attributes? */ ?>
                             &lt;p&gt;<?= _("Attributes") ?>:&lt;/p&gt;
                             &lt;ul&gt;&lt;li&gt;<?= implode("&lt;/li&gt;&lt;li&gt;", $c['attrnames']) ?>&lt;/li&gt;&lt;/ul&gt;
                         <?php } ?>
-                        <?php if ($vars['trackables'] == 'desc:list' && count($c['trackables']) > 0) { /* Does user want us to include trackables list? */ ?>
+                        <?php if ($vars['trackables'] == 'desc:list' && count($c['trackables']) > 0) {
+/* Does user want us to include trackables list? */ ?>
                             &lt;p&gt;<?= _("Trackables") ?>:&lt;/p&gt;
                             &lt;ul&gt;
                             <?php foreach ($c['trackables'] as $t) { ?>
@@ -96,7 +103,8 @@ 
                             &lt;/ul&gt;
                         <?php } ?>
                         <?= Okapi::xmlescape($c['description']) ?>
-                        <?php if ((strpos($vars['images'], "descrefs:") === 0) && count($c['images']) > 0) { /* Does user want us to include <img> references in cache descriptions? */
+                        <?php if ((strpos($vars['images'], "descrefs:") === 0) && count($c['images']) > 0) {
+/* Does user want us to include <img> references in cache descriptions? */
                             if ($vars['images'] == "descrefs:thumblinks") { ?>
                                 &lt;h2&gt;<?= _("Images") ?> (<?= count($c['images']) ?>)&lt;/h2&gt;
                                 &lt;div&gt;
@@ -109,9 +117,12 @@ 
                                 # We will split images into two subcategories: spoilers and nonspoilers.
                                 $spoilers = array();
                                 $nonspoilers = array();
-                                foreach ($c['images'] as $img)
-                                    if ($img['is_spoiler']) $spoilers[] = $img;
-                                    else $nonspoilers[] = $img;
+                                foreach ($c['images'] as $img) {
+                                                                    if ($img['is_spoiler']) $spoilers[] = $img;
+                                }
+                                    else {
+                                        $nonspoilers[] = $img;
+                                    }
                                 ?>
                                 <?php if (count($nonspoilers) > 0) { ?>
                                     &lt;h2&gt;<?= _("Images") ?> (<?= count($nonspoilers) ?>)&lt;/h2&gt;
@@ -129,7 +140,8 @@ 
                                 <?php } ?>
                             <?php } ?>
                         <?php } ?>
-                        <?php if ((strpos($vars['images'], "ox:") === 0) && count($c['images']) > 0) { /* Include image descriptions (for ox:image numbers)? */ ?>
+                        <?php if ((strpos($vars['images'], "ox:") === 0) && count($c['images']) > 0) {
+/* Include image descriptions (for ox:image numbers)? */ ?>
                             &lt;p&gt;<?= _("Image descriptions") ?>:&lt;/p&gt;
                             &lt;ul&gt;
                                 <?php foreach ($c['images'] as $no => $img) { ?>
@@ -147,10 +159,12 @@ 
                         <?php } ?>
                     </groundspeak:long_description>
                     <groundspeak:encoded_hints><?= Okapi::xmlescape($c['hint2']) ?></groundspeak:encoded_hints>
-                    <?php if ((in_array('gc:personal_note', $vars['my_notes'])) && ($c['my_notes'] != null)) { /* Does user want us to include personal notes? -> Issue 294 */ ?>
+                    <?php if ((in_array('gc:personal_note', $vars['my_notes'])) && ($c['my_notes'] != null)) {
+/* Does user want us to include personal notes? -> Issue 294 */ ?>
                         <groundspeak:personal_note><?= Okapi::xmlescape($c['my_notes']) ?></groundspeak:personal_note>
                     <?php } ?>
-                    <?php if ($vars['latest_logs']) { /* Does user want us to include latest log entries? */ ?>
+                    <?php if ($vars['latest_logs']) {
+/* Does user want us to include latest log entries? */ ?>
                         <groundspeak:logs>
                             <?php foreach ($c['latest_logs'] as $log) { ?>
                                 <groundspeak:log id="<?= $log['internal_id'] ?>">
@@ -164,7 +178,8 @@ 
                     <?php } ?>
                 </groundspeak:cache>
             <?php } ?>
-            <?php if ($vars['ns_ox']) { /* Does user want us to include Garmin's <opencaching> element? */ ?>
+            <?php if ($vars['ns_ox']) {
+/* Does user want us to include Garmin's <opencaching> element? */ ?>
                 <ox:opencaching xmlns:ox="http://www.opencaching.com/xmlschemas/opencaching/1/0">
                     <ox:ratings>
                         <?php if ($c['rating'] !== null) { ?><ox:awesomeness><?= $c['rating'] ?></ox:awesomeness><?php } ?>
@@ -172,10 +187,12 @@ 
                         <?php if ($c['oxsize'] !== null) { ?><ox:size><?= $c['oxsize'] ?></ox:size><?php } ?>
                         <ox:terrain><?= $c['terrain'] ?></ox:terrain>
                     </ox:ratings>
-                    <?php if (in_array('ox:tags', $vars['attrs']) && count($c['attrnames']) > 0) { /* Does user want us to include ox:tags? */ ?>
+                    <?php if (in_array('ox:tags', $vars['attrs']) && count($c['attrnames']) > 0) {
+/* Does user want us to include ox:tags? */ ?>
                         <ox:tags><ox:tag><?= implode("</ox:tag><ox:tag>", $c['attrnames']) ?></ox:tag></ox:tags>
                     <?php } ?>
-                    <?php if ((strpos($vars['images'], "ox:") === 0) && count($c['images']) > 0) { /* Does user want us to include ox:image references? */ ?>
+                    <?php if ((strpos($vars['images'], "ox:") === 0) && count($c['images']) > 0) {
+/* Does user want us to include ox:image references? */ ?>
                         <ox:images>
                             <?php foreach ($c['images'] as $no => $img) { ?>
                                 <ox:image>
@@ -220,7 +237,7 @@ 
                     <?php } ?>
                 </wpt>
                 <?php
-                    if (isset($wpt_ref['ggz_entry'])){
+                    if (isset($wpt_ref['ggz_entry'])) {
                         $wpt_ref['ggz_entry']['file_len'] = ob_get_length() - $wpt_ref['ggz_entry']['file_pos'];
                     }
                 ?>

htdocs/okapi/services/caches/geocache.php

Braces +28 added lines, -13 removed lines

@@ -24,19 +24,33 @@ 
     public static function call(OkapiRequest $request)
     {
         $cache_code = $request->get_parameter('cache_code');
-        if (!$cache_code) throw new ParamMissing('cache_code');
-        if (strpos($cache_code, "|") !== false) throw new InvalidParam('cache_code');
+        if (!$cache_code) {
+            throw new ParamMissing('cache_code');
+        }
+        if (strpos($cache_code, "|") !== false) {
+            throw new InvalidParam('cache_code');
+        }
         $langpref = $request->get_parameter('langpref');
-        if (!$langpref) $langpref = "en";
+        if (!$langpref) {
+            $langpref = "en";
+        }
         $langpref .= "|".Settings::get('SITELANG');
         $fields = $request->get_parameter('fields');
-        if (!$fields) $fields = "code|name|location|type|status";
+        if (!$fields) {
+            $fields = "code|name|location|type|status";
+        }
         $log_fields = $request->get_parameter('log_fields');
-        if (!$log_fields) $log_fields = "uuid|date|user|type|comment";
+        if (!$log_fields) {
+            $log_fields = "uuid|date|user|type|comment";
+        }
         $lpc = $request->get_parameter('lpc');
-        if (!$lpc) $lpc = 10;
+        if (!$lpc) {
+            $lpc = 10;
+        }
         $attribution_append = $request->get_parameter('attribution_append');
-        if (!$attribution_append) $attribution_append = 'full';
+        if (!$attribution_append) {
+            $attribution_append = 'full';
+        }
         $params = array(
             'cache_codes' => $cache_code,
             'langpref' => $langpref,
@@ -46,11 +60,13 @@ 
             'log_fields' => $log_fields
         );
         $my_location = $request->get_parameter('my_location');
-        if ($my_location)
-            $params['my_location'] = $my_location;
+        if ($my_location) {
+                    $params['my_location'] = $my_location;
+        }
         $user_uuid = $request->get_parameter('user_uuid');
-        if ($user_uuid)
-            $params['user_uuid'] = $user_uuid;
+        if ($user_uuid) {
+                    $params['user_uuid'] = $user_uuid;
+        }
 
         # There's no need to validate the fields/lpc parameters as the 'geocaches'
         # method does this (it will raise a proper exception on invalid values).
@@ -58,8 +74,7 @@ 
         $results = OkapiServiceRunner::call('services/caches/geocaches', new OkapiInternalRequest(
             $request->consumer, $request->token, $params));
         $result = $results[$cache_code];
-        if ($result === null)
-        {
+        if ($result === null) {
             # Two errors messages (for OCDE). Makeshift solution for issue #350.
 
             $exists = Db::select_value("

htdocs/okapi/services/caches/shortcuts/search_and_retrieve.php

Braces +58 added lines, -52 removed lines

@@ -24,98 +24,104 @@
     {
         # Check search method
         $search_method = $request->get_parameter('search_method');
-        if (!$search_method)
-            throw new ParamMissing('search_method');
-        if (strpos($search_method, "services/caches/search/") !== 0)
-            throw new InvalidParam('search_method', "Should begin with 'services/caches/search/'.");
-        if (!OkapiServiceRunner::exists($search_method))
-            throw new InvalidParam('search_method', "Method does not exist: '$search_method'");
+        if (!$search_method) {
+                    throw new ParamMissing('search_method');
+        }
+        if (strpos($search_method, "services/caches/search/") !== 0) {
+                    throw new InvalidParam('search_method', "Should begin with 'services/caches/search/'.");
+        }
+        if (!OkapiServiceRunner::exists($search_method)) {
+                    throw new InvalidParam('search_method', "Method does not exist: '$search_method'");
+        }
         $search_params = $request->get_parameter('search_params');
-        if (!$search_params)
-            throw new ParamMissing('search_params');
+        if (!$search_params) {
+                    throw new ParamMissing('search_params');
+        }
         $search_params = json_decode($search_params, true);
-        if (!is_array($search_params))
-            throw new InvalidParam('search_params', "Should be a JSON-encoded dictionary");
+        if (!is_array($search_params)) {
+                    throw new InvalidParam('search_params', "Should be a JSON-encoded dictionary");
+        }
 
         # Check retrieval method
         $retr_method = $request->get_parameter('retr_method');
-        if (!$retr_method)
-            throw new ParamMissing('retr_method');
-        if (!OkapiServiceRunner::exists($retr_method))
-            throw new InvalidParam('retr_method', "Method does not exist: '$retr_method'");
+        if (!$retr_method) {
+                    throw new ParamMissing('retr_method');
+        }
+        if (!OkapiServiceRunner::exists($retr_method)) {
+                    throw new InvalidParam('retr_method', "Method does not exist: '$retr_method'");
+        }
         $retr_params = $request->get_parameter('retr_params');
-        if (!$retr_params)
-            throw new ParamMissing('retr_params');
+        if (!$retr_params) {
+                    throw new ParamMissing('retr_params');
+        }
         $retr_params = json_decode($retr_params, true);
-        if (!is_array($retr_params))
-            throw new InvalidParam('retr_params', "Should be a JSON-encoded dictionary");
+        if (!is_array($retr_params)) {
+                    throw new InvalidParam('retr_params', "Should be a JSON-encoded dictionary");
+        }
 
         self::map_values_to_strings($search_params);
         self::map_values_to_strings($retr_params);
 
         # Wrapped?
         $wrap = $request->get_parameter('wrap');
-        if ($wrap == null) throw new ParamMissing('wrap');
-        if (!in_array($wrap, array('true', 'false')))
-            throw new InvalidParam('wrap');
+        if ($wrap == null) {
+            throw new ParamMissing('wrap');
+        }
+        if (!in_array($wrap, array('true', 'false'))) {
+                    throw new InvalidParam('wrap');
+        }
         $wrap = ($wrap == 'true');
 
         # Run search method
-        try
-        {
+        try {
             $search_result = OkapiServiceRunner::call($search_method, new OkapiInternalRequest(
                 $request->consumer, $request->token, $search_params));
-        }
-        catch (BadRequest $e)
-        {
+        } catch (BadRequest $e) {
             throw new InvalidParam('search_params', "Search method responded with the ".
                 "following error message: ".$e->getMessage());
         }
 
         # Run retrieval method
-        try
-        {
+        try {
             $retr_result = OkapiServiceRunner::call($retr_method, new OkapiInternalRequest(
                 $request->consumer, $request->token, array_merge($retr_params,
                 array('cache_codes' => implode("|", $search_result['results'])))));
-        }
-        catch (BadRequest $e)
-        {
+        } catch (BadRequest $e) {
             throw new InvalidParam('retr_params', "Retrieval method responded with the ".
                 "following error message: ".$e->getMessage());
         }
 
-        if ($wrap)
-        {
+        if ($wrap) {
             # $retr_result might be a PHP object, but also might be a binary response
             # (e.g. a GPX file).
-            if ($retr_result instanceof OkapiHttpResponse)
-                $result = array('results' => $retr_result->get_body());
-            else
-                $result = array('results' => $retr_result);
-            foreach ($search_result as $key => &$value_ref)
-                if ($key != 'results')
+            if ($retr_result instanceof OkapiHttpResponse) {
+                            $result = array('results' => $retr_result->get_body());
+            } else {
+                            $result = array('results' => $retr_result);
+            }
+            foreach ($search_result as $key => &$value_ref) {
+                            if ($key != 'results')
                     $result[$key] = $value_ref;
+            }
             return Okapi::formatted_response($request, $result);
-        }
-        else
-        {
-            if ($retr_result instanceof OkapiHttpResponse)
-                return $retr_result;
-            else
-                return Okapi::formatted_response($request, $retr_result);
+        } else {
+            if ($retr_result instanceof OkapiHttpResponse) {
+                            return $retr_result;
+            } else {
+                            return Okapi::formatted_response($request, $retr_result);
+            }
         }
     }
 
     private static function map_values_to_strings(&$dict)
     {
-        foreach (array_keys($dict) as $key)
-        {
+        foreach (array_keys($dict) as $key) {
             $val = $dict[$key];
-            if (is_numeric($val) || is_string($val))
-                $dict[$key] = (string)$val;
-            else
-                throw new BadRequest("Invalid value format for key: ".$key);
+            if (is_numeric($val) || is_string($val)) {
+                            $dict[$key] = (string)$val;
+            } else {
+                            throw new BadRequest("Invalid value format for key: ".$key);
+            }
         }
     }
 }

htdocs/okapi/services/caches/search/bbox.php

Braces +24 added lines, -21 removed lines

@@ -30,26 +30,32 @@ 
         # It's much easier to grasp their meaning this way.
 
         $tmp = $request->get_parameter('bbox');
-        if (!$tmp)
-            throw new ParamMissing('bbox');
+        if (!$tmp) {
+                    throw new ParamMissing('bbox');
+        }
         $parts = explode('|', $tmp);
-        if (count($parts) != 4)
-            throw new InvalidParam('bbox', "Expecting 4 pipe-separated parts, got ".count($parts).".");
-        foreach ($parts as &$part_ref)
-        {
-            if (!preg_match("/^-?[0-9]+(\.?[0-9]*)$/", $part_ref))
-                throw new InvalidParam('bbox', "'$part_ref' is not a valid float number.");
+        if (count($parts) != 4) {
+                    throw new InvalidParam('bbox', "Expecting 4 pipe-separated parts, got ".count($parts).".");
+        }
+        foreach ($parts as &$part_ref) {
+            if (!preg_match("/^-?[0-9]+(\.?[0-9]*)$/", $part_ref)) {
+                            throw new InvalidParam('bbox', "'$part_ref' is not a valid float number.");
+            }
             $part_ref = floatval($part_ref);
         }
         list($bbsouth, $bbwest, $bbnorth, $bbeast) = $parts;
-        if ($bbnorth <= $bbsouth)
-            throw new InvalidParam('bbox', "Northern edge must be situated to the north of the southern edge.");
-        if ($bbeast == $bbwest)
-            throw new InvalidParam('bbox', "Eastern edge longitude is the same as the western one.");
-        if ($bbnorth > 90 || $bbnorth < -90 || $bbsouth > 90 || $bbsouth < -90)
-            throw new InvalidParam('bbox', "Latitudes have to be within -90..90 range.");
-        if ($bbeast > 180 || $bbeast < -180 || $bbwest > 180 || $bbwest < -180)
-            throw new InvalidParam('bbox', "Longitudes have to be within -180..180 range.");
+        if ($bbnorth <= $bbsouth) {
+                    throw new InvalidParam('bbox', "Northern edge must be situated to the north of the southern edge.");
+        }
+        if ($bbeast == $bbwest) {
+                    throw new InvalidParam('bbox', "Eastern edge longitude is the same as the western one.");
+        }
+        if ($bbnorth > 90 || $bbnorth < -90 || $bbsouth > 90 || $bbsouth < -90) {
+                    throw new InvalidParam('bbox', "Latitudes have to be within -90..90 range.");
+        }
+        if ($bbeast > 180 || $bbeast < -180 || $bbwest > 180 || $bbwest < -180) {
+                    throw new InvalidParam('bbox', "Longitudes have to be within -180..180 range.");
+        }
 
         # Construct SQL conditions for the specified bounding box.
 
@@ -64,16 +70,13 @@ 
             $lat >= '".Db::escape_string($bbsouth)."'
             and $lat < '".Db::escape_string($bbnorth)."'
         )";
-        if ($bbeast > $bbwest)
-        {
+        if ($bbeast > $bbwest) {
             # Easy one.
             $where_conds[] = "(
                 $lon >= '".Db::escape_string($bbwest)."'
                 and $lon < '".Db::escape_string($bbeast)."'
             )";
-        }
-        else
-        {
+        } else {
             # We'll have to assume that this bbox goes through the 180-degree meridian.
             # For example, $bbwest = 179 and $bbeast = -179.
             $where_conds[] = "(

htdocs/okapi/services/caches/search/nearest.php

Spacing +3 added lines, -3 removed lines

@@ -70,14 +70,14 @@ 
         {
             if (!preg_match("/^-?[0-9]+(\.?[0-9]*)$/", $tmp))
                 throw new InvalidParam('radius', "'$tmp' is not a valid float number.");
-            $radius = floatval($tmp);  # is given in kilometers
+            $radius = floatval($tmp); # is given in kilometers
             if ($radius <= 0)
                 throw new InvalidParam('radius', "Has to be a positive number.");
 
             # Apply a latitude-range prefilter if it looks promising.
             # See https://github.com/opencaching/okapi/issues/363 for more info.
 
-            $optimization_radius = 100;  # in kilometers, optimized for Opencaching.de
+            $optimization_radius = 100; # in kilometers, optimized for Opencaching.de
             $km2degrees_upper_estimate_factor = 0.01;
 
             if ($radius <= $optimization_radius)
@@ -89,7 +89,7 @@ 
                     ";
             }
 
-            $radius *= 1000;  # convert from kilometers to meters
+            $radius *= 1000; # convert from kilometers to meters
             $where_conds[] = "$distance_formula <= '".Db::escape_string($radius)."'";
         }
 

Braces +25 added lines, -22 removed lines

@@ -30,22 +30,26 @@ 
         # It's much easier to grasp their meaning this way.
 
         $tmp = $request->get_parameter('center');
-        if (!$tmp)
-            throw new ParamMissing('center');
+        if (!$tmp) {
+                    throw new ParamMissing('center');
+        }
         $parts = explode('|', $tmp);
-        if (count($parts) != 2)
-            throw new InvalidParam('center', "Expecting 2 pipe-separated parts, got ".count($parts).".");
-        foreach ($parts as &$part_ref)
-        {
-            if (!preg_match("/^-?[0-9]+(\.?[0-9]*)$/", $part_ref))
-                throw new InvalidParam('center', "'$part_ref' is not a valid float number.");
+        if (count($parts) != 2) {
+                    throw new InvalidParam('center', "Expecting 2 pipe-separated parts, got ".count($parts).".");
+        }
+        foreach ($parts as &$part_ref) {
+            if (!preg_match("/^-?[0-9]+(\.?[0-9]*)$/", $part_ref)) {
+                            throw new InvalidParam('center', "'$part_ref' is not a valid float number.");
+            }
             $part_ref = floatval($part_ref);
         }
         list($center_lat, $center_lon) = $parts;
-        if ($center_lat > 90 || $center_lat < -90)
-            throw new InvalidParam('center', "Latitudes have to be within -90..90 range.");
-        if ($center_lon > 180 || $center_lon < -180)
-            throw new InvalidParam('center', "Longitudes have to be within -180..180 range.");
+        if ($center_lat > 90 || $center_lat < -90) {
+                    throw new InvalidParam('center', "Latitudes have to be within -90..90 range.");
+        }
+        if ($center_lon > 180 || $center_lon < -180) {
+                    throw new InvalidParam('center', "Longitudes have to be within -180..180 range.");
+        }
 
         #
         # In the method description, we promised to return caches ordered by the *rough*
@@ -66,13 +70,14 @@ 
 
         $where_conds = array();
         $radius = null;
-        if ($tmp = $request->get_parameter('radius'))
-        {
-            if (!preg_match("/^-?[0-9]+(\.?[0-9]*)$/", $tmp))
-                throw new InvalidParam('radius', "'$tmp' is not a valid float number.");
+        if ($tmp = $request->get_parameter('radius')) {
+            if (!preg_match("/^-?[0-9]+(\.?[0-9]*)$/", $tmp)) {
+                            throw new InvalidParam('radius', "'$tmp' is not a valid float number.");
+            }
             $radius = floatval($tmp);  # is given in kilometers
-            if ($radius <= 0)
-                throw new InvalidParam('radius', "Has to be a positive number.");
+            if ($radius <= 0) {
+                            throw new InvalidParam('radius', "Has to be a positive number.");
+            }
 
             # Apply a latitude-range prefilter if it looks promising.
             # See https://github.com/opencaching/okapi/issues/363 for more info.
@@ -80,8 +85,7 @@ 
             $optimization_radius = 100;  # in kilometers, optimized for Opencaching.de
             $km2degrees_upper_estimate_factor = 0.01;
 
-            if ($radius <= $optimization_radius)
-            {
+            if ($radius <= $optimization_radius) {
                 $radius_degrees = $radius * $km2degrees_upper_estimate_factor;
                 $where_conds[] = "
                     caches.latitude >= '".Db::escape_string($center_lat - $radius_degrees)."'
@@ -100,8 +104,7 @@ 
         $search_assistant->set_search_params($search_params);
 
         $result = $search_assistant->get_common_search_result();
-        if ($radius == null)
-        {
+        if ($radius == null) {
             # 'more' is meaningless in this case, we'll remove it.
             unset($result['more']);
         }