Completed
Pull Request — master (#369)
by Dan Michael O.
03:24
created
controller/WebController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 
74 74
         // register a Twig filter for generating URLs for vocabulary resources (concepts and groups)
75 75
         $controller = $this; // for use by anonymous function below
76
-        $urlFilter = new Twig_SimpleFilter('link_url', function ($uri, $vocab, $lang, $type = 'page', $clang = null, $term = null) use ($controller) {
76
+        $urlFilter = new Twig_SimpleFilter('link_url', function($uri, $vocab, $lang, $type = 'page', $clang = null, $term = null) use ($controller) {
77 77
             // $vocab can either be null, a vocabulary id (string) or a Vocabulary object
78 78
             if ($vocab === null) {
79 79
                 // target vocabulary is unknown, best bet is to link to the plain URI
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
         $this->twig->addFilter($urlFilter);
115 115
 
116 116
         // register a Twig filter for generating strings from language codes with CLDR
117
-        $langFilter = new Twig_SimpleFilter('lang_name', function ($langcode, $lang) use ($controller) {
117
+        $langFilter = new Twig_SimpleFilter('lang_name', function($langcode, $lang) use ($controller) {
118 118
             return Language::getName($langcode, $lang);
119 119
         });
120 120
         $this->twig->addFilter($langFilter);
@@ -611,7 +611,7 @@  discard block
 block discarded – undo
611 611
      * Loads and renders the view containing a list of recent changes in the vocabulary.
612 612
      * @param Request $request
613 613
      */
614
-    public function invokeChangeList($request, $prop='dc:created')
614
+    public function invokeChangeList($request, $prop = 'dc:created')
615 615
     {
616 616
         // set language parameters for gettext
617 617
         $this->setLanguageProperties($request->getLang());
Please login to merge, or discard this patch.
model/Model.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -381,10 +381,12 @@
 block discarded – undo
381 381
                 // register vocabulary ids as RDF namespace prefixes
382 382
                 $prefix = preg_replace('/\W+/', '', $voc->getId()); // strip non-word characters
383 383
                 try {
384
-                    if ($prefix != '' && EasyRdf_Namespace::get($prefix) === null) // if not already defined
384
+                    if ($prefix != '' && EasyRdf_Namespace::get($prefix) === null) {
385
+                      // if not already defined
385 386
                     {
386 387
                         EasyRdf_Namespace::set($prefix, $voc->getUriSpace());
387 388
                     }
389
+                    }
388 390
 
389 391
                 } catch (Exception $e) {
390 392
                     // not valid as namespace identifier, ignore
Please login to merge, or discard this patch.
model/Request.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -167,9 +167,11 @@
 block discarded – undo
167 167
      */
168 168
     public function setVocab($vocabid)
169 169
     {
170
-        if (strpos($vocabid, ' ') !== false) // if there are multiple vocabularies just storing the string
170
+        if (strpos($vocabid, ' ') !== false) {
171
+          // if there are multiple vocabularies just storing the string
171 172
         {
172 173
             $this->setVocabids($vocabid);
174
+        }
173 175
         } else {
174 176
             $this->vocab = $this->model->getVocabulary($vocabid);
175 177
         }
Please login to merge, or discard this patch.
model/Concept.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -461,7 +461,7 @@
 block discarded – undo
461 461
             }
462 462
         } catch (Exception $e) {
463 463
             trigger_error($e->getMessage(), E_USER_WARNING);
464
-            return gettext('skosmos:modified') . ' ' . (string) $this->resource->get('dc:modified') . ' ' . gettext('skosmos:created') . ' ' . (string) $this->resource->get('dc:created');
464
+            return gettext('skosmos:modified') . ' ' . (string)$this->resource->get('dc:modified') . ' ' . gettext('skosmos:created') . ' ' . (string)$this->resource->get('dc:created');
465 465
         }
466 466
         return $ret;
467 467
     }
Please login to merge, or discard this patch.
model/sparql/GenericSparql.php 1 patch
Braces   +9 added lines, -3 removed lines patch added patch discarded remove patch
@@ -56,12 +56,16 @@  discard block
 block discarded – undo
56 56
         $this->model = $model;
57 57
 
58 58
         // set graphClause so that it can be used by all queries
59
-        if ($this->isDefaultEndpoint()) // default endpoint; query any graph (and catch it in a variable)
59
+        if ($this->isDefaultEndpoint()) {
60
+          // default endpoint; query any graph (and catch it in a variable)
60 61
         {
61 62
             $this->graphClause = "GRAPH $graph";
62
-        } elseif ($graph) // query a specific graph
63
+        }
64
+        } elseif ($graph) {
65
+          // query a specific graph
63 66
         {
64 67
             $this->graphClause = "GRAPH <$graph>";
68
+        }
65 69
         } else // query the default graph
66 70
         {
67 71
             $this->graphClause = "";
@@ -616,9 +620,11 @@  discard block
 block discarded – undo
616 620
         // extra types to query, if using thesaurus arrays and no additional type restrictions have been applied
617 621
         $extratypes = ($arrayClass && $types === array('skos:Concept')) ? "UNION { ?s a <$arrayClass> }" : "";
618 622
 
619
-        if (sizeof($unprefixed_types) === 1) // if only one type limitation set no UNION needed
623
+        if (sizeof($unprefixed_types) === 1) {
624
+          // if only one type limitation set no UNION needed
620 625
         {
621 626
             $type = '<' . $unprefixed_types[0] . '>';
627
+        }
622 628
         } else { // multiple type limitations require setting a UNION for each of those
623 629
             $type = '[]';
624 630
             foreach ($unprefixed_types as $utype) {
Please login to merge, or discard this patch.
model/sparql/JenaTextSparql.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@
 block discarded – undo
37 37
    * @return string SPARQL text search clause
38 38
    */
39 39
 
40
-  protected function createTextQueryCondition($term, $property='')
40
+  protected function createTextQueryCondition($term, $property = '')
41 41
   {
42 42
     // construct the lucene search term for jena-text
43 43
     
Please login to merge, or discard this patch.
model/ConceptPropertyValueLiteral.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@
 block discarded – undo
44 44
                 return Punic\Calendar::formatDate($val, 'short');
45 45
             } catch (Exception $e) {
46 46
                 trigger_error($e->getMessage(), E_USER_WARNING);
47
-                return (string) $this->literal;
47
+                return (string)$this->literal;
48 48
             }
49 49
         }
50 50
         return $this->literal->getValue();
Please login to merge, or discard this patch.
model/DataObject.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -72,10 +72,12 @@
 block discarded – undo
72 72
 
73 73
         uksort($sortable, array($this, 'mycompare'));
74 74
         foreach ($sortable as $prop => $vals) {
75
-            if (is_array($prop)) // the ConceptProperty objects have their own sorting methods
75
+            if (is_array($prop)) {
76
+              // the ConceptProperty objects have their own sorting methods
76 77
             {
77 78
                 ksort($sortable[$prop]);
78 79
             }
80
+            }
79 81
 
80 82
         }
81 83
 
Please login to merge, or discard this patch.
model/Vocabulary.php 2 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   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -278,7 +278,7 @@  discard block
 block discarded – undo
278 278
      * Counts the statistics of the vocabulary.
279 279
      * @return array of the concept/group counts
280 280
      */
281
-    public function getStatistics($lang = '', $array=null, $group=null)
281
+    public function getStatistics($lang = '', $array = null, $group = null)
282 282
     {
283 283
         $sparql = $this->getSparql();
284 284
         // find the number of concepts
@@ -594,14 +594,14 @@  discard block
 block discarded – undo
594 594
     {
595 595
       $changelist = $this->getSparql()->queryChangeList($clang, $offset, $prop);
596 596
       $bydate = array();
597
-      foreach($changelist as $concept) {
597
+      foreach ($changelist as $concept) {
598 598
         $concept['datestring'] = Punic\Calendar::formatDate($concept['date'], 'medium', $lang);
599
-        $bydate[Punic\Calendar::getMonthName($concept['date'], 'wide', $lang, true) . Punic\Calendar::format($concept['date'], ' y', $lang) ][strtolower($concept['prefLabel'])] = $concept;
599
+        $bydate[Punic\Calendar::getMonthName($concept['date'], 'wide', $lang, true) . Punic\Calendar::format($concept['date'], ' y', $lang)][strtolower($concept['prefLabel'])] = $concept;
600 600
       }
601 601
       return $bydate;
602 602
     }
603 603
 
604
-    public function getTitle($lang=null) {
604
+    public function getTitle($lang = null) {
605 605
       return $this->config->getTitle($lang);
606 606
     }
607 607
     
Please login to merge, or discard this patch.