Completed
Pull Request — master (#899)
by Dan Michael O.
01:48
created
model/Vocabulary.php 3 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -76,10 +76,12 @@
 block discarded – undo
76 76
      */
77 77
     public function getUriSpace()
78 78
     {
79
-        if ($this->urispace === null) // initialize cache
79
+        if ($this->urispace === null) {
80
+          // initialize cache
80 81
         {
81 82
             $this->urispace = $this->resource->getLiteral('void:uriSpace')->getValue();
82 83
         }
84
+        }
83 85
 
84 86
         return $this->urispace;
85 87
     }
Please login to merge, or discard this patch.
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
             return $lname;
93 93
         }
94 94
         // already a full URI
95
-        return $this->getUriSpace() . $lname;
95
+        return $this->getUriSpace().$lname;
96 96
     }
97 97
 
98 98
     /**
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
                 foreach ($conceptscheme->allLiterals($prop, null) as $val) {
154 154
                     $prop = (substr($prop, 0, 5) == 'dc11:') ? str_replace('dc11:', 'dc:', $prop) : $prop;
155 155
                     if ($val->getValue() instanceof DateTime) {
156
-                        $val = Punic\Calendar::formatDate($val->getValue(), 'full', $lang) . ' ' . Punic\Calendar::format($val->getValue(), 'HH:mm:ss', $lang);
156
+                        $val = Punic\Calendar::formatDate($val->getValue(), 'full', $lang).' '.Punic\Calendar::format($val->getValue(), 'HH:mm:ss', $lang);
157 157
                     }
158 158
                     $ret[$prop][] = $val;
159 159
                 }
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
         }
265 265
         // don't know how to parse
266 266
         $rev = $parts[2];
267
-        $datestr = $parts[3] . ' ' . $parts[4];
267
+        $datestr = $parts[3].' '.$parts[4];
268 268
 
269 269
         return "$datestr (r$rev)";
270 270
     }
@@ -273,7 +273,7 @@  discard block
 block discarded – undo
273 273
      * Counts the statistics of the vocabulary.
274 274
      * @return array of the concept/group counts
275 275
      */
276
-    public function getStatistics($lang = '', $array=null, $group=null)
276
+    public function getStatistics($lang = '', $array = null, $group = null)
277 277
     {
278 278
         $sparql = $this->getSparql();
279 279
         // find the number of concepts
@@ -482,7 +482,7 @@  discard block
 block discarded – undo
482 482
      */
483 483
     public function searchConceptsAlphabetical($letter, $limit = null, $offset = null, $clang = null)
484 484
     {
485
-        return $this->getSparql()->queryConceptsAlphabetical($letter, $clang, $limit, $offset, $this->config->getIndexClasses(),$this->config->getShowDeprecated());
485
+        return $this->getSparql()->queryConceptsAlphabetical($letter, $clang, $limit, $offset, $this->config->getIndexClasses(), $this->config->getShowDeprecated());
486 486
     }
487 487
 
488 488
     /**
@@ -591,14 +591,14 @@  discard block
 block discarded – undo
591 591
     {
592 592
       $changelist = $this->getSparql()->queryChangeList($clang, $offset, $prop);
593 593
       $bydate = array();
594
-      foreach($changelist as $concept) {
594
+      foreach ($changelist as $concept) {
595 595
         $concept['datestring'] = Punic\Calendar::formatDate($concept['date'], 'medium', $lang);
596
-        $bydate[Punic\Calendar::getMonthName($concept['date'], 'wide', $lang, true) . Punic\Calendar::format($concept['date'], ' y', $lang) ][strtolower($concept['prefLabel'])] = $concept;
596
+        $bydate[Punic\Calendar::getMonthName($concept['date'], 'wide', $lang, true).Punic\Calendar::format($concept['date'], ' y', $lang)][strtolower($concept['prefLabel'])] = $concept;
597 597
       }
598 598
       return $bydate;
599 599
     }
600 600
 
601
-    public function getTitle($lang=null) {
601
+    public function getTitle($lang = null) {
602 602
       return $this->config->getTitle($lang);
603 603
     }
604 604
     
Please login to merge, or discard this patch.
Doc Comments   +10 added lines, -1 removed lines patch added patch discarded remove patch
@@ -109,7 +109,7 @@  discard block
 block discarded – undo
109 109
      * Get the localname of a concept in the vocabulary. If the URI is not
110 110
      * in the URI space of this vocabulary, return the full URI.
111 111
      *
112
-     * @param $uri string full URI of concept
112
+     * @param string $uri string full URI of concept
113 113
      * @return string local name of concept, or original full URI if the local name cannot be determined
114 114
      */
115 115
     public function getLocalName($uri)
@@ -281,6 +281,8 @@  discard block
 block discarded – undo
281 281
 
282 282
     /**
283 283
      * Counts the statistics of the vocabulary.
284
+     * @param string $array
285
+     * @param string $group
284 286
      * @return array of the concept/group counts
285 287
      */
286 288
     public function getStatistics($lang = '', $array=null, $group=null)
@@ -320,6 +322,7 @@  discard block
 block discarded – undo
320 322
     /**
321 323
      * Gets the child relations of a concept and whether these children have more children.
322 324
      * @param string $uri
325
+     * @param string $lang
323 326
      */
324 327
     public function getConceptChildren($uri, $lang)
325 328
     {
@@ -428,6 +431,8 @@  discard block
 block discarded – undo
428 431
     /**
429 432
      * Lists the concepts available in the concept group.
430 433
      * @param $clname
434
+     * @param string $glname
435
+     * @param string $clang
431 436
      * @return array
432 437
      */
433 438
     public function listConceptGroupContents($glname, $clang)
@@ -595,6 +600,7 @@  discard block
 block discarded – undo
595 600
      * Returns a list of recently changed or entirely new concepts.
596 601
      * @param string $clang content language for the labels
597 602
      * @param string $lang UI language for the dates
603
+     * @param string $prop
598 604
      * @return Array
599 605
      */
600 606
     public function getChangeList($prop, $clang, $lang, $offset)
@@ -608,6 +614,9 @@  discard block
 block discarded – undo
608 614
       return $bydate;
609 615
     }
610 616
 
617
+    /**
618
+     * @param string $lang
619
+     */
611 620
     public function getTitle($lang=null) {
612 621
       return $this->config->getTitle($lang);
613 622
     }
Please login to merge, or discard this patch.
model/VocabularyCategory.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -10,6 +10,9 @@
 block discarded – undo
10 10
  */
11 11
 class VocabularyCategory extends DataObject
12 12
 {
13
+    /**
14
+     * @param Model $model
15
+     */
13 16
     public function __construct($model, $resource)
14 17
     {
15 18
         if (!($model instanceof Model)) {
Please login to merge, or discard this patch.
rest.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
     } elseif ($parts[1] == 'data') {
38 38
         $controller->data($request);
39 39
     } elseif (sizeof($parts) == 2) {
40
-        header("Location: " . $parts[1] . "/");
40
+        header("Location: ".$parts[1]."/");
41 41
     } else {
42 42
         $vocab = $parts[1];
43 43
         try {
@@ -93,5 +93,5 @@  discard block
 block discarded – undo
93 93
     }
94 94
 } catch (Exception $e) {
95 95
     header("HTTP/1.0 500 Internal Server Error");
96
-    echo('ERROR: ' . $e->getMessage());
96
+    echo('ERROR: '.$e->getMessage());
97 97
 }
Please login to merge, or discard this patch.
model/Request.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -106,7 +106,7 @@
 block discarded – undo
106 106
 
107 107
     public function getRequestUri()
108 108
     {
109
-        return $this->getServerConstant('HTTP_HOST') . $this->getServerConstant('REQUEST_URI');
109
+        return $this->getServerConstant('HTTP_HOST').$this->getServerConstant('REQUEST_URI');
110 110
     }
111 111
     
112 112
     /**
Please login to merge, or discard this patch.
Braces   +12 added lines, -4 removed lines patch added patch discarded remove patch
@@ -74,7 +74,9 @@  discard block
 block discarded – undo
74 74
      */
75 75
     public function getQueryParam($paramName)
76 76
     {
77
-        if (!isset($this->queryParams[$paramName])) return null;
77
+        if (!isset($this->queryParams[$paramName])) {
78
+          return null;
79
+        }
78 80
         $val = filter_var($this->queryParams[$paramName], FILTER_SANITIZE_STRING);
79 81
         return ($val !== null ? str_replace('\\', '', $val) : null);
80 82
     }
@@ -91,7 +93,9 @@  discard block
 block discarded – undo
91 93
 
92 94
     public function getQueryParamPOST($paramName)
93 95
     {
94
-        if (!isset($this->queryParamsPOST[$paramName])) return null;
96
+        if (!isset($this->queryParamsPOST[$paramName])) {
97
+          return null;
98
+        }
95 99
         return filter_var($this->queryParamsPOST[$paramName], FILTER_SANITIZE_STRING);
96 100
     }
97 101
 
@@ -106,7 +110,9 @@  discard block
 block discarded – undo
106 110
 
107 111
     public function getServerConstant($paramName)
108 112
     {
109
-        if (!isset($this->serverConstants[$paramName])) return null;
113
+        if (!isset($this->serverConstants[$paramName])) {
114
+          return null;
115
+        }
110 116
         return filter_var($this->serverConstants[$paramName], FILTER_SANITIZE_STRING);
111 117
     }
112 118
 
@@ -239,9 +245,11 @@  discard block
 block discarded – undo
239 245
      */
240 246
     public function setVocab($vocabid)
241 247
     {
242
-        if (strpos($vocabid, ' ') !== false) // if there are multiple vocabularies just storing the string
248
+        if (strpos($vocabid, ' ') !== false) {
249
+          // if there are multiple vocabularies just storing the string
243 250
         {
244 251
             $this->setVocabids($vocabid);
252
+        }
245 253
         } else {
246 254
             $this->vocab = $this->model->getVocabulary($vocabid);
247 255
         }
Please login to merge, or discard this patch.
entity.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,5 +23,5 @@
 block discarded – undo
23 23
 
24 24
 } catch (Exception $e) {
25 25
     header("HTTP/1.0 500 Internal Server Error");
26
-    echo('ERROR: ' . $e->getMessage());
26
+    echo('ERROR: '.$e->getMessage());
27 27
 }
Please login to merge, or discard this patch.
index.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -14,7 +14,7 @@  discard block
 block discarded – undo
14 14
     }
15 15
     require_once 'vendor/autoload.php';
16 16
 } catch (Exception $e) {
17
-    echo "Error: " . $e->getMessage();
17
+    echo "Error: ".$e->getMessage();
18 18
     return;
19 19
 }
20 20
 
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
     // if language code missing, redirect to guessed language
33 33
     // in any case, redirect to <lang>/
34 34
     $lang = sizeof($parts) == 2 && $parts[1] !== '' ? $parts[1] : $controller->guessLanguage();
35
-    header("Location: " . $lang . "/");
35
+    header("Location: ".$lang."/");
36 36
 } else {
37 37
   if (array_key_exists($parts[1], $config->getLanguages())) { // global pages
38 38
         $request->setLang($parts[1]);
@@ -61,8 +61,8 @@  discard block
 block discarded – undo
61 61
         }
62 62
         if (sizeof($parts) == 3) { // language code missing
63 63
             $lang = $controller->guessLanguage();
64
-            $newurl = $controller->getBaseHref() . $vocab . "/" . $lang . "/";
65
-            header("Location: " . $newurl);
64
+            $newurl = $controller->getBaseHref().$vocab."/".$lang."/";
65
+            header("Location: ".$newurl);
66 66
         } else {
67 67
             if (array_key_exists($parts[2], $config->getLanguages())) {
68 68
                 $lang = $parts[2];
@@ -103,10 +103,10 @@  discard block
 block discarded – undo
103 103
                 }
104 104
             } else { // language code missing, redirect to some language version
105 105
                 $lang = $controller->guessLanguage($vocab);
106
-                $newurl = $controller->getBaseHref() . $vocab . "/" . $lang . "/" . implode('/', array_slice($parts, 2));
106
+                $newurl = $controller->getBaseHref().$vocab."/".$lang."/".implode('/', array_slice($parts, 2));
107 107
                 $qs = $request->getServerConstant('QUERY_STRING');
108 108
                 if ($qs) {
109
-                    $newurl .= "?" . $qs;
109
+                    $newurl .= "?".$qs;
110 110
                 }
111 111
                 header("Location: $newurl");
112 112
             }
Please login to merge, or discard this patch.
model/SkosmosTurtleParser.php 2 patches
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -55,6 +55,7 @@
 block discarded – undo
55 55
 
56 56
     /**
57 57
      * Steps back, restoring the previous character or statement read() to the input buffer.
58
+     * @param string $chars
58 59
      */
59 60
     protected function unread($chars)
60 61
     {
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@  discard block
 block discarded – undo
21 21
      */
22 22
     protected function peek()
23 23
     {
24
-        if(!$this->dataLength) { $this->dataLength = strlen($this->data); }
24
+        if (!$this->dataLength) { $this->dataLength = strlen($this->data); }
25 25
         if ($this->dataLength > $this->bytePos) {
26 26
             $slice = substr($this->data, $this->bytePos, 4);
27 27
             return mb_substr($slice, 0, 1, "UTF-8");
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
     {
79 79
         if ($this->bytePos - 4 > 0) {
80 80
             $slice = substr($this->data, $this->bytePos - 4, 4);
81
-            while($slice != '') {
81
+            while ($slice != '') {
82 82
                 if (!self::isWhitespace(mb_substr($slice, -1, 1, "UTF-8"))) {
83 83
                     return;
84 84
                 }
Please login to merge, or discard this patch.
model/DataObject.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -79,10 +79,12 @@
 block discarded – undo
79 79
         if ($sortable !== null) {
80 80
             uksort($sortable, array($this, 'mycompare'));
81 81
             foreach ($sortable as $prop => $vals) {
82
-                if (is_array($prop)) // the ConceptProperty objects have their own sorting methods
82
+                if (is_array($prop)) {
83
+                  // the ConceptProperty objects have their own sorting methods
83 84
                 {
84 85
                     ksort($sortable[$prop]);
85 86
                 }
87
+                }
86 88
             }
87 89
         }
88 90
         return $sortable;
Please login to merge, or discard this patch.
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -124,8 +124,8 @@
 block discarded – undo
124 124
      */
125 125
     public function getEnvLang()
126 126
     {
127
-       // get language from locale, same as used by gettext, set by Controller
128
-       return substr(getenv("LC_ALL"), 0, 2); // @codeCoverageIgnore
127
+        // get language from locale, same as used by gettext, set by Controller
128
+        return substr(getenv("LC_ALL"), 0, 2); // @codeCoverageIgnore
129 129
     }
130 130
 
131 131
     /**
Please login to merge, or discard this patch.
model/sparql/GenericSparql.php 4 patches
Braces   +10 added lines, -6 removed lines patch added patch discarded remove patch
@@ -46,12 +46,16 @@  discard block
 block discarded – undo
46 46
         $this->client = new EasyRdf\Sparql\Client($endpoint);
47 47
 
48 48
         // set graphClause so that it can be used by all queries
49
-        if ($this->isDefaultEndpoint()) // default endpoint; query any graph (and catch it in a variable)
49
+        if ($this->isDefaultEndpoint()) {
50
+          // default endpoint; query any graph (and catch it in a variable)
50 51
         {
51 52
             $this->graphClause = "GRAPH $graph";
52
-        } elseif ($graph) // query a specific graph
53
+        }
54
+        } elseif ($graph) {
55
+          // query a specific graph
53 56
         {
54 57
             $this->graphClause = "GRAPH <$graph>";
58
+        }
55 59
         } else // query the default graph
56 60
         {
57 61
             $this->graphClause = "";
@@ -100,8 +104,9 @@  discard block
 block discarded – undo
100 104
             }
101 105
         }
102 106
         foreach ($graphs as $graph) {
103
-            if($graph !== NULL)
104
-                $clause .= "FROM NAMED <$graph> "; 
107
+            if($graph !== NULL) {
108
+                            $clause .= "FROM NAMED <$graph> ";
109
+            }
105 110
         }
106 111
         return $clause;
107 112
     }
@@ -1847,8 +1852,7 @@  discard block
 block discarded – undo
1847 1852
         if (sizeof($ret) > 0) {
1848 1853
             // existing concept, with children
1849 1854
             return $ret;
1850
-        }
1851
-        else {
1855
+        } else {
1852 1856
             // nonexistent concept
1853 1857
             return null;
1854 1858
         }
Please login to merge, or discard this patch.
Doc Comments   +8 added lines, -4 removed lines patch added patch discarded remove patch
@@ -289,6 +289,7 @@  discard block
 block discarded – undo
289 289
     /**
290 290
      * Counts the number of concepts in a easyRDF graph with a specific language.
291 291
      * @param array $langs Languages to query for
292
+     * @param string[] $classes
292 293
      * @return Array containing count of concepts for each language and property.
293 294
      */
294 295
     public function countLangConcepts($langs, $classes = null) {
@@ -473,7 +474,7 @@  discard block
 block discarded – undo
473 474
      * Returns information (as a graph) for one or more concept URIs
474 475
      * @param mixed $uris concept URI (string) or array of URIs
475 476
      * @param string|null $arrayClass the URI for thesaurus array class, or null if not used
476
-     * @param \Vocabulary[]|null $vocabs vocabularies to target
477
+     * @param Vocabulary[] $vocabs vocabularies to target
477 478
      * @return \EasyRdf\Graph
478 479
      */
479 480
     public function queryConceptInfoGraph($uris, $arrayClass = null, $vocabs = array()) {
@@ -489,7 +490,7 @@  discard block
 block discarded – undo
489 490
 
490 491
     /**
491 492
      * Returns information (as an array of Concept objects) for one or more concept URIs
492
-     * @param mixed $uris concept URI (string) or array of URIs
493
+     * @param string $uris concept URI (string) or array of URIs
493 494
      * @param string|null $arrayClass the URI for thesaurus array class, or null if not used
494 495
      * @param \Vocabulary[] $vocabs vocabularies to target
495 496
      * @param string|null $clang content language
@@ -704,7 +705,7 @@  discard block
 block discarded – undo
704 705
 
705 706
     /**
706 707
      * Generate a VALUES clause for limiting the targeted graphs.
707
-     * @param Vocabulary[]|null $vocabs array of Vocabulary objects to target
708
+     * @param Vocabulary[] $vocabs array of Vocabulary objects to target
708 709
      * @return string VALUES clause, or "" if not necessary to limit
709 710
      */
710 711
     protected function formatValuesGraph($vocabs) {
@@ -884,6 +885,7 @@  discard block
 block discarded – undo
884 885
      * @param string $searchLang language code used for matching labels (null means any language)
885 886
      * @param string[] $props properties to target e.g. array('skos:prefLabel','skos:altLabel')
886 887
      * @param boolean $unique restrict results to unique concepts (default: false)
888
+     * @param string $filterGraph
887 889
      * @return string sparql query
888 890
      */
889 891
     protected function generateConceptSearchQueryInner($term, $lang, $searchLang, $props, $unique, $filterGraph)
@@ -1158,6 +1160,7 @@  discard block
 block discarded – undo
1158 1160
     /**
1159 1161
      * Generates sparql query clauses used for creating the alphabetical index.
1160 1162
      * @param string $letter the letter (or special class) to search for
1163
+     * @param string $lang
1161 1164
      * @return array of sparql query clause strings
1162 1165
      */
1163 1166
     private function formatFilterConditions($letter, $lang) {
@@ -1325,6 +1328,7 @@  discard block
 block discarded – undo
1325 1328
     /**
1326 1329
      * Query for the first characters (letter or otherwise) of the labels in the particular language.
1327 1330
      * @param string $lang language
1331
+     * @param string[] $classes
1328 1332
      * @return array array of characters
1329 1333
      */
1330 1334
     public function queryFirstCharacters($lang, $classes = null) {
@@ -1647,7 +1651,7 @@  discard block
 block discarded – undo
1647 1651
     /**
1648 1652
      * Query a single transitive property of a concept.
1649 1653
      * @param string $uri
1650
-     * @param array $props the property/properties.
1654
+     * @param string[] $props the property/properties.
1651 1655
      * @param string $lang
1652 1656
      * @param string $fallbacklang language to use if label is not available in the preferred language
1653 1657
      * @param integer $limit
Please login to merge, or discard this patch.
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
      *
65 65
      * @param string $query
66 66
      * @return string
67
-    */
67
+     */
68 68
     protected function generateQueryPrefixes($query)
69 69
     {
70 70
         // Check for undefined prefixes
@@ -239,9 +239,9 @@  discard block
 block discarded – undo
239 239
         $gcl = $this->graphClause;
240 240
         $classes = ($classes) ? $classes : array('http://www.w3.org/2004/02/skos/core#Concept');
241 241
 
242
-	$quote_string = function($val) { return "'$val'"; };
243
-	$quoted_values = array_map($quote_string, $langs);
244
-	$langFilter = "FILTER(?lang IN (" . implode(',', $quoted_values) . "))";
242
+  $quote_string = function($val) { return "'$val'"; };
243
+  $quoted_values = array_map($quote_string, $langs);
244
+  $langFilter = "FILTER(?lang IN (" . implode(',', $quoted_values) . "))";
245 245
 
246 246
         $values = $this->formatValues('?type', $classes, 'uri');
247 247
         $valuesProp = $this->formatValues('?prop', $props, null);
@@ -1812,7 +1812,7 @@  discard block
 block discarded – undo
1812 1812
 
1813 1813
     /**
1814 1814
      * Generates a sparql query for finding the hierarchy for a concept.
1815
-	 * A concept may be a top concept in multiple schemes, returned as a single whitespace-separated literal.
1815
+     * A concept may be a top concept in multiple schemes, returned as a single whitespace-separated literal.
1816 1816
      * @param string $uri concept uri.
1817 1817
      * @param string $lang
1818 1818
      * @param string $fallback language to use if label is not available in the preferred language
@@ -1889,10 +1889,10 @@  discard block
 block discarded – undo
1889 1889
                 $ret[$uri]['exact'] = $row->exact->getUri();
1890 1890
             }
1891 1891
             if (isset($row->tops)) {
1892
-               $topConceptsList=explode(" ", $row->tops->getValue());
1893
-               // sort to garantee an alphabetical ordering of the URI
1894
-               sort($topConceptsList);
1895
-               $ret[$uri]['tops'] = $topConceptsList;
1892
+                $topConceptsList=explode(" ", $row->tops->getValue());
1893
+                // sort to garantee an alphabetical ordering of the URI
1894
+                sort($topConceptsList);
1895
+                $ret[$uri]['tops'] = $topConceptsList;
1896 1896
             }
1897 1897
             if (isset($row->children)) {
1898 1898
                 if (!isset($ret[$uri]['narrower'])) {
Please login to merge, or discard this patch.
Spacing   +53 added lines, -53 removed lines patch added patch discarded remove patch
@@ -87,11 +87,11 @@  discard block
 block discarded – undo
87 87
     protected function query($query) {
88 88
         $queryId = sprintf("%05d", rand(0, 99999));
89 89
         $logger = $this->model->getLogger();
90
-        $logger->info("[qid $queryId] SPARQL query:\n" . $this->generateQueryPrefixes($query) . "\n$query\n");
90
+        $logger->info("[qid $queryId] SPARQL query:\n".$this->generateQueryPrefixes($query)."\n$query\n");
91 91
         $starttime = microtime(true);
92 92
         $result = $this->client->query($query);
93 93
         $elapsed = intval(round((microtime(true) - $starttime) * 1000));
94
-        if(method_exists($result, 'numRows')) {
94
+        if (method_exists($result, 'numRows')) {
95 95
             $numRows = $result->numRows();
96 96
             $logger->info("[qid $queryId] result: $numRows rows returned in $elapsed ms");
97 97
         } else { // graph result
@@ -107,20 +107,20 @@  discard block
 block discarded – undo
107 107
      * @param Vocabulary[]|null $vocabs
108 108
      * @return string
109 109
      */
110
-    protected function generateFromClause($vocabs=null) {
110
+    protected function generateFromClause($vocabs = null) {
111 111
         $graphs = array();
112 112
         $clause = '';
113 113
         if (!$vocabs) {
114 114
             return $this->graph !== '?graph' && $this->graph !== NULL ? "FROM <$this->graph>" : '';
115 115
         }
116
-        foreach($vocabs as $vocab) {
116
+        foreach ($vocabs as $vocab) {
117 117
             $graph = $vocab->getGraph();
118 118
             if (!in_array($graph, $graphs)) {
119 119
                 array_push($graphs, $graph);
120 120
             }
121 121
         }
122 122
         foreach ($graphs as $graph) {
123
-            if($graph !== NULL)
123
+            if ($graph !== NULL)
124 124
                 $clause .= "FROM NAMED <$graph> "; 
125 125
         }
126 126
         return $clause;
@@ -241,7 +241,7 @@  discard block
 block discarded – undo
241 241
 
242 242
 	$quote_string = function($val) { return "'$val'"; };
243 243
 	$quoted_values = array_map($quote_string, $langs);
244
-	$langFilter = "FILTER(?lang IN (" . implode(',', $quoted_values) . "))";
244
+	$langFilter = "FILTER(?lang IN (".implode(',', $quoted_values)."))";
245 245
 
246 246
         $values = $this->formatValues('?type', $classes, 'uri');
247 247
         $valuesProp = $this->formatValues('?prop', $props, null);
@@ -667,9 +667,9 @@  discard block
 block discarded – undo
667 667
                 $conceptscheme['title'] = $row->title->getValue();
668 668
             }
669 669
             // add dct:subject and their labels in the result
670
-            if(isset($row->domain) && isset($row->domainLabel)){
671
-                $conceptscheme['subject']['uri']=$row->domain->getURI();
672
-                $conceptscheme['subject']['prefLabel']=$row->domainLabel->getValue();
670
+            if (isset($row->domain) && isset($row->domainLabel)) {
671
+                $conceptscheme['subject']['uri'] = $row->domain->getURI();
672
+                $conceptscheme['subject']['prefLabel'] = $row->domainLabel->getValue();
673 673
             }
674 674
 
675 675
             $ret[$row->cs->getURI()] = $conceptscheme;
@@ -732,7 +732,7 @@  discard block
 block discarded – undo
732 732
         foreach ($graphs as $graph) {
733 733
           $values[] = "<$graph>";
734 734
         }
735
-        return "FILTER (?graph IN (" . implode(',', $values) . "))";
735
+        return "FILTER (?graph IN (".implode(',', $values)."))";
736 736
     }
737 737
 
738 738
     /**
@@ -742,16 +742,16 @@  discard block
 block discarded – undo
742 742
      * @return string sparql query clauses
743 743
      */
744 744
     protected function formatLimitAndOffset($limit, $offset) {
745
-        $limit = ($limit) ? 'LIMIT ' . $limit : '';
746
-        $offset = ($offset) ? 'OFFSET ' . $offset : '';
745
+        $limit = ($limit) ? 'LIMIT '.$limit : '';
746
+        $offset = ($offset) ? 'OFFSET '.$offset : '';
747 747
         // eliminating whitespace and line changes when the conditions aren't needed.
748 748
         $limitandoffset = '';
749 749
         if ($limit && $offset) {
750
-            $limitandoffset = "\n" . $limit . "\n" . $offset;
750
+            $limitandoffset = "\n".$limit."\n".$offset;
751 751
         } elseif ($limit) {
752
-            $limitandoffset = "\n" . $limit;
752
+            $limitandoffset = "\n".$limit;
753 753
         } elseif ($offset) {
754
-            $limitandoffset = "\n" . $offset;
754
+            $limitandoffset = "\n".$offset;
755 755
         }
756 756
 
757 757
         return $limitandoffset;
@@ -903,14 +903,14 @@  discard block
 block discarded – undo
903 903
         // if search language and UI/display language differ, must also consider case where there is no prefLabel in
904 904
         // the display language; in that case, should use the label with the same language as the matched label
905 905
         $labelcondFallback = ($searchLang != $lang) ?
906
-          "OPTIONAL { # in case previous OPTIONAL block gives no labels\n" .
906
+          "OPTIONAL { # in case previous OPTIONAL block gives no labels\n".
907 907
           "?s skos:prefLabel ?label . FILTER (LANGMATCHES(LANG(?label), LANG(?match))) }" : "";
908 908
           
909 909
         //  Including the labels if there is no query term given.
910 910
         if ($rawterm === '') {
911 911
           $labelClause = "?s skos:prefLabel ?label .";
912
-          $labelClause = ($lang) ? $labelClause . " FILTER (LANGMATCHES(LANG(?label), '$lang'))" : $labelClause . "";
913
-          return $labelClause . " BIND(?label AS ?match)";
912
+          $labelClause = ($lang) ? $labelClause." FILTER (LANGMATCHES(LANG(?label), '$lang'))" : $labelClause."";
913
+          return $labelClause." BIND(?label AS ?match)";
914 914
         }
915 915
 
916 916
         /*
@@ -974,20 +974,20 @@  discard block
 block discarded – undo
974 974
         $schemecond = '';
975 975
         if (!empty($schemes)) {
976 976
             $conditions = array();
977
-            foreach($schemes as $scheme) {
977
+            foreach ($schemes as $scheme) {
978 978
                 $conditions[] = "{?s skos:inScheme <$scheme>}";
979 979
             }
980
-            $schemecond = '{'.implode(" UNION ",$conditions).'}';
980
+            $schemecond = '{'.implode(" UNION ", $conditions).'}';
981 981
         }
982
-        $filterDeprecated="";
982
+        $filterDeprecated = "";
983 983
         //show or hide deprecated concepts
984
-        if(!$showDeprecated){
985
-            $filterDeprecated="FILTER NOT EXISTS { ?s owl:deprecated true }";
984
+        if (!$showDeprecated) {
985
+            $filterDeprecated = "FILTER NOT EXISTS { ?s owl:deprecated true }";
986 986
         }
987 987
         // extra conditions for parent and group, if specified
988
-        $parentcond = ($params->getParentLimit()) ? "?s skos:broader+ <" . $params->getParentLimit() . "> ." : "";
989
-        $groupcond = ($params->getGroupLimit()) ? "<" . $params->getGroupLimit() . "> skos:member ?s ." : "";
990
-        $pgcond = $parentcond . $groupcond;
988
+        $parentcond = ($params->getParentLimit()) ? "?s skos:broader+ <".$params->getParentLimit()."> ." : "";
989
+        $groupcond = ($params->getGroupLimit()) ? "<".$params->getGroupLimit()."> skos:member ?s ." : "";
990
+        $pgcond = $parentcond.$groupcond;
991 991
 
992 992
         $orderextra = $this->isDefaultEndpoint() ? $this->graph : '';
993 993
 
@@ -1060,9 +1060,9 @@  discard block
 block discarded – undo
1060 1060
             $hit['type'][] = $this->shortenUri($typeuri);
1061 1061
         }
1062 1062
 
1063
-        if(!empty($fields)) {
1063
+        if (!empty($fields)) {
1064 1064
             foreach ($fields as $prop) {
1065
-                $propname = $prop . 's';
1065
+                $propname = $prop.'s';
1066 1066
                 if (isset($row->$propname)) {
1067 1067
                     foreach (explode("\n", $row->$propname->getValue()) as $line) {
1068 1068
                         $rdata = str_getcsv($line, ',', '"', '"');
@@ -1077,7 +1077,7 @@  discard block
 block discarded – undo
1077 1077
                             $propvals = $rdata[2];
1078 1078
                         }
1079 1079
 
1080
-                        $hit['skos:' . $prop][] = $propvals;
1080
+                        $hit['skos:'.$prop][] = $propvals;
1081 1081
                     }
1082 1082
                 }
1083 1083
             }
@@ -1153,7 +1153,7 @@  discard block
 block discarded – undo
1153 1153
      * @return array query result object
1154 1154
      */
1155 1155
     public function queryConcepts($vocabs, $fields = null, $unique = false, $params, $showDeprecated = false) {
1156
-        $query = $this->generateConceptSearchQuery($fields, $unique, $params,$showDeprecated);
1156
+        $query = $this->generateConceptSearchQuery($fields, $unique, $params, $showDeprecated);
1157 1157
         $results = $this->query($query);
1158 1158
         return $this->transformConceptSearchResults($results, $vocabs, $fields);
1159 1159
     }
@@ -1207,9 +1207,9 @@  discard block
 block discarded – undo
1207 1207
         $conditions = $this->formatFilterConditions($letter, $lang);
1208 1208
         $filtercondLabel = $conditions['filterpref'];
1209 1209
         $filtercondALabel = $conditions['filteralt'];
1210
-        $filterDeprecated="";
1211
-        if(!$showDeprecated){
1212
-            $filterDeprecated="FILTER NOT EXISTS { ?s owl:deprecated true }";
1210
+        $filterDeprecated = "";
1211
+        if (!$showDeprecated) {
1212
+            $filterDeprecated = "FILTER NOT EXISTS { ?s owl:deprecated true }";
1213 1213
         }
1214 1214
         $query = <<<EOQ
1215 1215
 SELECT DISTINCT ?s ?label ?alabel $fcl
@@ -1285,8 +1285,8 @@  discard block
 block discarded – undo
1285 1285
      * @param array $classes
1286 1286
      * @param boolean $showDeprecated whether to include deprecated concepts in the result (default: false)
1287 1287
      */
1288
-    public function queryConceptsAlphabetical($letter, $lang, $limit = null, $offset = null, $classes = null,$showDeprecated = false) {
1289
-        $query = $this->generateAlphabeticalListQuery($letter, $lang, $limit, $offset, $classes,$showDeprecated);
1288
+    public function queryConceptsAlphabetical($letter, $lang, $limit = null, $offset = null, $classes = null, $showDeprecated = false) {
1289
+        $query = $this->generateAlphabeticalListQuery($letter, $lang, $limit, $offset, $classes, $showDeprecated);
1290 1290
         $results = $this->query($query);
1291 1291
         return $this->transformAlphabeticalListResults($results);
1292 1292
     }
@@ -1620,7 +1620,7 @@  discard block
 block discarded – undo
1620 1620
                 if (!isset($row->label) || $row->label->getLang() === $lang) {
1621 1621
                     $ret[$row->object->getUri()] = $val;
1622 1622
                 } elseif ($row->label->getLang() === $fallbacklang) {
1623
-                    $val['label'] .= ' (' . $row->label->getLang() . ')';
1623
+                    $val['label'] .= ' ('.$row->label->getLang().')';
1624 1624
                     $ret[$row->object->getUri()] = $val;
1625 1625
                 }
1626 1626
             }
@@ -1716,10 +1716,10 @@  discard block
 block discarded – undo
1716 1716
 
1717 1717
             $label = null;
1718 1718
             if (isset($row->label)) {
1719
-                if ($row->label->getLang() == $lang || strpos($row->label->getLang(), $lang . "-") == 0) {
1719
+                if ($row->label->getLang() == $lang || strpos($row->label->getLang(), $lang."-") == 0) {
1720 1720
                     $label = $row->label->getValue();
1721 1721
                 } else {
1722
-                    $label = $row->label->getValue() . " (" . $row->label->getLang() . ")";
1722
+                    $label = $row->label->getValue()." (".$row->label->getLang().")";
1723 1723
                 }
1724 1724
 
1725 1725
             }
@@ -1795,8 +1795,8 @@  discard block
 block discarded – undo
1795 1795
         foreach ($result as $row) {
1796 1796
             if (isset($row->top) && isset($row->label)) {
1797 1797
                 $label = $row->label->getValue();
1798
-                if ($row->label->getLang() && $row->label->getLang() !== $lang && strpos($row->label->getLang(), $lang . "-") !== 0) {
1799
-                    $label .= ' (' . $row->label->getLang() . ')';
1798
+                if ($row->label->getLang() && $row->label->getLang() !== $lang && strpos($row->label->getLang(), $lang."-") !== 0) {
1799
+                    $label .= ' ('.$row->label->getLang().')';
1800 1800
                 }
1801 1801
                 $top = array('uri' => $row->top->getUri(), 'topConceptOf' => $row->topuri->getUri(), 'label' => $label, 'hasChildren' => filter_var($row->children->getValue(), FILTER_VALIDATE_BOOLEAN));
1802 1802
                 if (isset($row->notation)) {
@@ -1889,7 +1889,7 @@  discard block
 block discarded – undo
1889 1889
                 $ret[$uri]['exact'] = $row->exact->getUri();
1890 1890
             }
1891 1891
             if (isset($row->tops)) {
1892
-               $topConceptsList=explode(" ", $row->tops->getValue());
1892
+               $topConceptsList = explode(" ", $row->tops->getValue());
1893 1893
                // sort to garantee an alphabetical ordering of the URI
1894 1894
                sort($topConceptsList);
1895 1895
                $ret[$uri]['tops'] = $topConceptsList;
@@ -1902,8 +1902,8 @@  discard block
 block discarded – undo
1902 1902
                 $label = null;
1903 1903
                 if (isset($row->childlabel)) {
1904 1904
                     $label = $row->childlabel->getValue();
1905
-                    if ($row->childlabel->getLang() !== $lang && strpos($row->childlabel->getLang(), $lang . "-") !== 0) {
1906
-                        $label .= " (" . $row->childlabel->getLang() . ")";
1905
+                    if ($row->childlabel->getLang() !== $lang && strpos($row->childlabel->getLang(), $lang."-") !== 0) {
1906
+                        $label .= " (".$row->childlabel->getLang().")";
1907 1907
                     }
1908 1908
 
1909 1909
                 }
@@ -1924,8 +1924,8 @@  discard block
 block discarded – undo
1924 1924
             }
1925 1925
             if (isset($row->label)) {
1926 1926
                 $preflabel = $row->label->getValue();
1927
-                if ($row->label->getLang() && $row->label->getLang() !== $lang && strpos($row->label->getLang(), $lang . "-") !== 0) {
1928
-                    $preflabel .= ' (' . $row->label->getLang() . ')';
1927
+                if ($row->label->getLang() && $row->label->getLang() !== $lang && strpos($row->label->getLang(), $lang."-") !== 0) {
1928
+                    $preflabel .= ' ('.$row->label->getLang().')';
1929 1929
                 }
1930 1930
 
1931 1931
                 $ret[$uri]['prefLabel'] = $preflabel;
@@ -2046,9 +2046,9 @@  discard block
 block discarded – undo
2046 2046
      */
2047 2047
     private function generateConceptGroupContentsQuery($groupClass, $group, $lang, $showDeprecated = false) {
2048 2048
         $fcl = $this->generateFromClause();
2049
-        $filterDeprecated="";
2050
-        if(!$showDeprecated){
2051
-            $filterDeprecated="  FILTER NOT EXISTS { ?conc owl:deprecated true }";
2049
+        $filterDeprecated = "";
2050
+        if (!$showDeprecated) {
2051
+            $filterDeprecated = "  FILTER NOT EXISTS { ?conc owl:deprecated true }";
2052 2052
         }
2053 2053
         $query = <<<EOQ
2054 2054
 SELECT ?conc ?super ?label ?members ?type ?notation $fcl
@@ -2087,10 +2087,10 @@  discard block
 block discarded – undo
2087 2087
                     'type' => array($row->type->shorten()),
2088 2088
                 );
2089 2089
                 if (isset($row->label)) {
2090
-                    if ($row->label->getLang() == $lang || strpos($row->label->getLang(), $lang . "-") == 0) {
2090
+                    if ($row->label->getLang() == $lang || strpos($row->label->getLang(), $lang."-") == 0) {
2091 2091
                         $values[$row->conc->getURI()]['prefLabel'] = $row->label->getValue();
2092 2092
                     } else {
2093
-                        $values[$row->conc->getURI()]['prefLabel'] = $row->label->getValue() . " (" . $row->label->getLang() . ")";
2093
+                        $values[$row->conc->getURI()]['prefLabel'] = $row->label->getValue()." (".$row->label->getLang().")";
2094 2094
                     }
2095 2095
 
2096 2096
                 }
@@ -2118,8 +2118,8 @@  discard block
 block discarded – undo
2118 2118
      * @param boolean $showDeprecated whether to include deprecated concepts in search results
2119 2119
      * @return array Result array with concept URI as key and concept label as value
2120 2120
      */
2121
-    public function listConceptGroupContents($groupClass, $group, $lang,$showDeprecated = false) {
2122
-        $query = $this->generateConceptGroupContentsQuery($groupClass, $group, $lang,$showDeprecated);
2121
+    public function listConceptGroupContents($groupClass, $group, $lang, $showDeprecated = false) {
2122
+        $query = $this->generateConceptGroupContentsQuery($groupClass, $group, $lang, $showDeprecated);
2123 2123
         $result = $this->query($query);
2124 2124
         return $this->transformConceptGroupContentsResults($result, $lang);
2125 2125
     }
@@ -2132,7 +2132,7 @@  discard block
 block discarded – undo
2132 2132
      */
2133 2133
     private function generateChangeListQuery($lang, $offset, $prop) {
2134 2134
         $fcl = $this->generateFromClause();
2135
-        $offset = ($offset) ? 'OFFSET ' . $offset : '';
2135
+        $offset = ($offset) ? 'OFFSET '.$offset : '';
2136 2136
 
2137 2137
         $query = <<<EOQ
2138 2138
 SELECT DISTINCT ?concept ?date ?label $fcl
Please login to merge, or discard this patch.