Completed
Push — master ( 52e365...bfa678 )
by Henri
22:45 queued 06:05
created
controller/WebController.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
         // specify where to look for templates and cache
38 38
         $loader = new Twig_Loader_Filesystem('view');
39 39
         // initialize Twig environment
40
-        $this->twig = new Twig_Environment($loader, array('cache' => $tmpDir,'auto_reload' => true));
40
+        $this->twig = new Twig_Environment($loader, array('cache' => $tmpDir, 'auto_reload' => true));
41 41
         $this->twig->addExtension(new Twig_Extensions_Extension_I18n());
42 42
         // used for setting the base href for the relative urls
43 43
         $this->twig->addGlobal("BaseHref", $this->getBaseHref());
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
         $this->twig->addFilter(new Twig_SimpleFilter('link_url', array($this, 'linkUrlFilter')));
65 65
 
66 66
         // register a Twig filter for generating strings from language codes with CLDR
67
-        $langFilter = new Twig_SimpleFilter('lang_name', function ($langcode, $lang) {
67
+        $langFilter = new Twig_SimpleFilter('lang_name', function($langcode, $lang) {
68 68
             return Language::getName($langcode, $lang);
69 69
         });
70 70
         $this->twig->addFilter($langFilter);
@@ -111,18 +111,18 @@  discard block
 block discarded – undo
111 111
         $localname = $vocab->getLocalName($uri);
112 112
         if ($localname !== $uri && $localname === urlencode($localname)) {
113 113
             // check that the prefix stripping worked, and there are no problematic chars in localname
114
-            $paramstr = sizeof($params) > 0 ? '?' . http_build_query($params) : '';
114
+            $paramstr = sizeof($params) > 0 ? '?'.http_build_query($params) : '';
115 115
             if ($type && $type !== '' && $type !== 'vocab' && !($localname === '' && $type === 'page')) {
116
-                return "$vocid/$lang/$type/$localname" . $paramstr;
116
+                return "$vocid/$lang/$type/$localname".$paramstr;
117 117
             }
118 118
 
119
-            return "$vocid/$lang/$localname" . $paramstr;
119
+            return "$vocid/$lang/$localname".$paramstr;
120 120
         }
121 121
 
122 122
         // case 2: URI outside vocabulary namespace, or has problematic chars
123 123
         // pass the full URI as parameter instead
124 124
         $params['uri'] = $uri;
125
-        return "$vocid/$lang/$type/?" . http_build_query($params);
125
+        return "$vocid/$lang/$type/?".http_build_query($params);
126 126
     }
127 127
 
128 128
     /**
@@ -279,12 +279,12 @@  discard block
 block discarded – undo
279 279
     private function createFeedbackHeaders($fromName, $fromEmail, $toMail)
280 280
     {
281 281
         $headers = "MIME-Version: 1.0″ . '\r\n";
282
-        $headers .= "Content-type: text/html; charset=UTF-8" . "\r\n";
282
+        $headers .= "Content-type: text/html; charset=UTF-8"."\r\n";
283 283
         if ($toMail) {
284
-            $headers .= "Cc: " . $this->model->getConfig()->getFeedbackAddress() . "\r\n";
284
+            $headers .= "Cc: ".$this->model->getConfig()->getFeedbackAddress()."\r\n";
285 285
         }
286 286
 
287
-        $headers .= "From: $fromName <$fromEmail>" . "\r\n" . 'X-Mailer: PHP/' . phpversion();
287
+        $headers .= "From: $fromName <$fromEmail>"."\r\n".'X-Mailer: PHP/'.phpversion();
288 288
         return $headers;
289 289
     }
290 290
 
@@ -299,20 +299,20 @@  discard block
 block discarded – undo
299 299
     {
300 300
         $toAddress = ($toMail) ? $toMail : $this->model->getConfig()->getFeedbackAddress();
301 301
         if ($fromVocab !== null && $fromVocab !== '') {
302
-            $message = 'Feedback from vocab: ' . strtoupper($fromVocab) . "<br />" . $message;
302
+            $message = 'Feedback from vocab: '.strtoupper($fromVocab)."<br />".$message;
303 303
         }
304 304
 
305
-        $subject = SERVICE_NAME . " feedback";
305
+        $subject = SERVICE_NAME." feedback";
306 306
         $headers = $this->createFeedbackHeaders($fromName, $fromEmail, $toMail);
307 307
         $envelopeSender = FEEDBACK_ENVELOPE_SENDER;
308 308
         $params = empty($envelopeSender) ? '' : "-f $envelopeSender";
309 309
 
310 310
         // adding some information about the user for debugging purposes.
311
-        $message = $message . "<br /><br /> Debugging information:"
312
-            . "<br />Timestamp: " . date(DATE_RFC2822)
313
-            . "<br />User agent: " . $request->getServerConstant('HTTP_USER_AGENT')
314
-            . "<br />IP address: " . $request->getServerConstant('REMOTE_ADDR')
315
-            . "<br />Referer: " . $request->getServerConstant('HTTP_REFERER');
311
+        $message = $message."<br /><br /> Debugging information:"
312
+            . "<br />Timestamp: ".date(DATE_RFC2822)
313
+            . "<br />User agent: ".$request->getServerConstant('HTTP_USER_AGENT')
314
+            . "<br />IP address: ".$request->getServerConstant('REMOTE_ADDR')
315
+            . "<br />Referer: ".$request->getServerConstant('HTTP_REFERER');
316 316
 
317 317
         try {
318 318
             mail($toAddress, $subject, $message, $headers, $params);
@@ -320,7 +320,7 @@  discard block
 block discarded – undo
320 320
             header("HTTP/1.0 404 Not Found");
321 321
             $template = $this->twig->loadTemplate('error-page.twig');
322 322
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
323
-                error_log('Caught exception: ' . $e->getMessage());
323
+                error_log('Caught exception: '.$e->getMessage());
324 324
             }
325 325
 
326 326
             echo $template->render(
@@ -367,7 +367,7 @@  discard block
 block discarded – undo
367 367
         $vocids = ($vocabs !== null && $vocabs !== '') ? explode(' ', $vocabs) : null;
368 368
         $vocabObjects = array();
369 369
         if ($vocids) {
370
-            foreach($vocids as $vocid) {
370
+            foreach ($vocids as $vocid) {
371 371
                 $vocabObjects[] = $this->model->getVocabulary($vocid);
372 372
             }
373 373
         }
@@ -378,7 +378,7 @@  discard block
 block discarded – undo
378 378
         } catch (Exception $e) {
379 379
             header("HTTP/1.0 404 Not Found");
380 380
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
381
-                error_log('Caught exception: ' . $e->getMessage());
381
+                error_log('Caught exception: '.$e->getMessage());
382 382
             }
383 383
             $this->invokeGenericErrorPage($request, $e->getMessage());
384 384
             return;
@@ -394,7 +394,7 @@  discard block
 block discarded – undo
394 394
                 'search_count' => $counts,
395 395
                 'languages' => $this->languages,
396 396
                 'search_results' => $searchResults,
397
-                'rest' => $parameters->getOffset()>0,
397
+                'rest' => $parameters->getOffset() > 0,
398 398
                 'global_search' => true,
399 399
                 'term' => $request->getQueryParam('q'),
400 400
                 'lang_list' => $langList,
@@ -419,7 +419,7 @@  discard block
 block discarded – undo
419 419
         } catch (Exception $e) {
420 420
             header("HTTP/1.0 404 Not Found");
421 421
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
422
-                error_log('Caught exception: ' . $e->getMessage());
422
+                error_log('Caught exception: '.$e->getMessage());
423 423
             }
424 424
 
425 425
             echo $template->render(
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
         } catch (Exception $e) {
441 441
             header("HTTP/1.0 404 Not Found");
442 442
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
443
-                error_log('Caught exception: ' . $e->getMessage());
443
+                error_log('Caught exception: '.$e->getMessage());
444 444
             }
445 445
 
446 446
             echo $template->render(
@@ -457,7 +457,7 @@  discard block
 block discarded – undo
457 457
                 'vocab' => $vocab,
458 458
                 'search_results' => $searchResults,
459 459
                 'search_count' => $counts,
460
-                'rest' => $parameters->getOffset()>0,
460
+                'rest' => $parameters->getOffset() > 0,
461 461
                 'limit_parent' => $parameters->getParentLimit(),
462 462
                 'limit_type' =>  $request->getQueryParam('type') ? explode('+', $request->getQueryParam('type')) : null,
463 463
                 'limit_group' => $parameters->getGroupLimit(),
@@ -582,7 +582,7 @@  discard block
 block discarded – undo
582 582
      * Loads and renders the view containing a list of recent changes in the vocabulary.
583 583
      * @param Request $request
584 584
      */
585
-    public function invokeChangeList($request, $prop='dc:created')
585
+    public function invokeChangeList($request, $prop = 'dc:created')
586 586
     {
587 587
         // set language parameters for gettext
588 588
         $this->setLanguageProperties($request->getLang());
Please login to merge, or discard this patch.