Passed
Push — issue1585-upgrade-twig-3 ( 5b4b41 )
by Osma
05:39
created
src/controller/WebController.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -37,9 +37,9 @@  discard block
 block discarded – undo
37 37
         }
38 38
 
39 39
         // specify where to look for templates and cache
40
-        $loader = new \Twig\Loader\FilesystemLoader(__DIR__ . '/../view');
40
+        $loader = new \Twig\Loader\FilesystemLoader(__DIR__.'/../view');
41 41
         // initialize Twig environment
42
-        $this->twig = new \Twig\Environment($loader, array('cache' => $tmpDir,'auto_reload' => true));
42
+        $this->twig = new \Twig\Environment($loader, array('cache' => $tmpDir, 'auto_reload' => true));
43 43
         // used for setting the base href for the relative urls
44 44
         $this->twig->addGlobal("BaseHref", $this->getBaseHref());
45 45
         // setting the service name string from the config.ttl
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
         $this->twig->addFilter(new \Twig\TwigFilter('link_url', array($this, 'linkUrlFilter')));
62 62
 
63 63
         // register a Twig filter for generating strings from language codes with CLDR
64
-        $langFilter = new \Twig\TwigFilter('lang_name', function ($langcode, $lang) {
64
+        $langFilter = new \Twig\TwigFilter('lang_name', function($langcode, $lang) {
65 65
             return Language::getName($langcode, $lang);
66 66
         });
67 67
         $this->twig->addFilter($langFilter);
@@ -247,16 +247,16 @@  discard block
 block discarded – undo
247 247
 
248 248
     private function createFeedbackHeaders($fromName, $fromEmail, $toMail, $sender)
249 249
     {
250
-        $headers = "MIME-Version: 1.0" . "\r\n";
251
-        $headers .= "Content-type: text/html; charset=UTF-8" . "\r\n";
250
+        $headers = "MIME-Version: 1.0"."\r\n";
251
+        $headers .= "Content-type: text/html; charset=UTF-8"."\r\n";
252 252
         if (!empty($toMail)) {
253
-            $headers .= "Cc: " . $this->model->getConfig()->getFeedbackAddress() . "\r\n";
253
+            $headers .= "Cc: ".$this->model->getConfig()->getFeedbackAddress()."\r\n";
254 254
         }
255 255
         if (!empty($fromEmail)) {
256 256
             $headers .= "Reply-To: $fromName <$fromEmail>\r\n";
257 257
         }
258 258
         $service = $this->model->getConfig()->getServiceName();
259
-        return $headers . "From: $fromName via $service <$sender>";
259
+        return $headers."From: $fromName via $service <$sender>";
260 260
     }
261 261
 
262 262
     /**
@@ -270,9 +270,9 @@  discard block
 block discarded – undo
270 270
     public function sendFeedback($request, $message, $messageSubject, $fromName = null, $fromEmail = null, $fromVocab = null, $toMail = null)
271 271
     {
272 272
         $toAddress = ($toMail) ? $toMail : $this->model->getConfig()->getFeedbackAddress();
273
-        $messageSubject = "[" . $this->model->getConfig()->getServiceName() . "] " . $messageSubject;
273
+        $messageSubject = "[".$this->model->getConfig()->getServiceName()."] ".$messageSubject;
274 274
         if ($fromVocab !== null && $fromVocab !== '') {
275
-            $message = 'Feedback from vocab: ' . strtoupper($fromVocab) . "<br />" . $message;
275
+            $message = 'Feedback from vocab: '.strtoupper($fromVocab)."<br />".$message;
276 276
         }
277 277
         $envelopeSender = $this->model->getConfig()->getFeedbackEnvelopeSender();
278 278
         // determine the sender address of the message
@@ -291,10 +291,10 @@  discard block
 block discarded – undo
291 291
         $headers = $this->createFeedbackHeaders($fromName, $fromEmail, $toMail, $sender);
292 292
         $params = empty($envelopeSender) ? '' : "-f $envelopeSender";
293 293
         // adding some information about the user for debugging purposes.
294
-        $message = $message . "<br /><br /> Debugging information:"
295
-            . "<br />Timestamp: " . date(DATE_RFC2822)
296
-            . "<br />User agent: " . $request->getServerConstant('HTTP_USER_AGENT')
297
-            . "<br />Referer: " . $request->getServerConstant('HTTP_REFERER');
294
+        $message = $message."<br /><br /> Debugging information:"
295
+            . "<br />Timestamp: ".date(DATE_RFC2822)
296
+            . "<br />User agent: ".$request->getServerConstant('HTTP_USER_AGENT')
297
+            . "<br />Referer: ".$request->getServerConstant('HTTP_REFERER');
298 298
 
299 299
         try {
300 300
             mail($toAddress, $messageSubject, $message, $headers, $params);
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
             header("HTTP/1.0 404 Not Found");
303 303
             $template = $this->twig->load('error.twig');
304 304
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
305
-                error_log('Caught exception: ' . $e->getMessage());
305
+                error_log('Caught exception: '.$e->getMessage());
306 306
             }
307 307
 
308 308
             echo $template->render(
@@ -349,13 +349,13 @@  discard block
 block discarded – undo
349 349
         $vocids = ($vocabs !== null && $vocabs !== '') ? explode(' ', $vocabs) : null;
350 350
         $vocabObjects = array();
351 351
         if ($vocids) {
352
-            foreach($vocids as $vocid) {
352
+            foreach ($vocids as $vocid) {
353 353
                 try {
354 354
                     $vocabObjects[] = $this->model->getVocabulary($vocid);
355 355
                 } catch (ValueError $e) {
356 356
                     // fail fast with an error page if the vocabulary cannot be found
357 357
                     if ($this->model->getConfig()->getLogCaughtExceptions()) {
358
-                        error_log('Caught exception: ' . $e->getMessage());
358
+                        error_log('Caught exception: '.$e->getMessage());
359 359
                     }
360 360
                     header("HTTP/1.0 400 Bad Request");
361 361
                     $this->invokeGenericErrorPage($request, $e->getMessage());
@@ -377,7 +377,7 @@  discard block
 block discarded – undo
377 377
             $errored = true;
378 378
             header("HTTP/1.0 500 Internal Server Error");
379 379
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
380
-                error_log('Caught exception: ' . $e->getMessage());
380
+                error_log('Caught exception: '.$e->getMessage());
381 381
             }
382 382
         }
383 383
         $vocabList = $this->model->getVocabularyList();
@@ -389,7 +389,7 @@  discard block
 block discarded – undo
389 389
                 'search_count' => $counts,
390 390
                 'languages' => $this->languages,
391 391
                 'search_results' => $searchResults,
392
-                'rest' => $parameters->getOffset()>0,
392
+                'rest' => $parameters->getOffset() > 0,
393 393
                 'global_search' => true,
394 394
                 'search_failed' => $errored,
395 395
                 'term' => $request->getQueryParamRaw('q'),
@@ -417,7 +417,7 @@  discard block
 block discarded – undo
417 417
         } catch (Exception $e) {
418 418
             header("HTTP/1.0 500 Internal Server Error");
419 419
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
420
-                error_log('Caught exception: ' . $e->getMessage());
420
+                error_log('Caught exception: '.$e->getMessage());
421 421
             }
422 422
 
423 423
             echo $template->render(
@@ -442,7 +442,7 @@  discard block
 block discarded – undo
442 442
         } catch (Exception $e) {
443 443
             header("HTTP/1.0 404 Not Found");
444 444
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
445
-                error_log('Caught exception: ' . $e->getMessage());
445
+                error_log('Caught exception: '.$e->getMessage());
446 446
             }
447 447
 
448 448
             echo $template->render(
@@ -461,7 +461,7 @@  discard block
 block discarded – undo
461 461
                 'vocab' => $vocab,
462 462
                 'search_results' => $searchResults,
463 463
                 'search_count' => $counts,
464
-                'rest' => $parameters->getOffset()>0,
464
+                'rest' => $parameters->getOffset() > 0,
465 465
                 'limit_parent' => $parameters->getParentLimit(),
466 466
                 'limit_type' =>  $request->getQueryParam('type') ? explode('+', $request->getQueryParam('type')) : null,
467 467
                 'limit_group' => $parameters->getGroupLimit(),
Please login to merge, or discard this patch.