Passed
Push — main ( a07285...1b25e2 )
by Osma
18:03 queued 13:47
created
src/controller/WebController.php 1 patch
Spacing   +22 added lines, -22 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__ . '/../../custom-templates', __DIR__ . '/../view']);
40
+        $loader = new \Twig\Loader\FilesystemLoader([__DIR__.'/../../custom-templates', __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
 
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
         $this->twig->addExtension(new LinkUrlExtension($model));
57 57
 
58 58
         // register a Twig filter for generating strings from language codes with CLDR
59
-        $langFilter = new \Twig\TwigFilter('lang_name', function ($langcode, $lang) {
59
+        $langFilter = new \Twig\TwigFilter('lang_name', function($langcode, $lang) {
60 60
             return Language::getName($langcode, $lang);
61 61
         });
62 62
         $this->twig->addFilter($langFilter);
@@ -83,16 +83,16 @@  discard block
 block discarded – undo
83 83
      */
84 84
     public function findCustomTemplates($dir)
85 85
     {
86
-        $customTemplateSubDirs = glob($dir . '/*', GLOB_ONLYDIR);
86
+        $customTemplateSubDirs = glob($dir.'/*', GLOB_ONLYDIR);
87 87
         $customTemplates = [];
88 88
 
89 89
         foreach ($customTemplateSubDirs as $slotDir) {
90 90
             $slotName = basename($slotDir);
91
-            $files = glob($slotDir . '/*.twig');
91
+            $files = glob($slotDir.'/*.twig');
92 92
             // Strip the directory part to make the file paths relative to the directory.
93 93
             // The "custom-templates" directory is registered to the Twig FilesystemLoader.
94
-            $customTemplates[$slotName] = array_map(function ($file) use ($dir) {
95
-                return str_replace($dir . '/', '', $file);
94
+            $customTemplates[$slotName] = array_map(function($file) use ($dir) {
95
+                return str_replace($dir.'/', '', $file);
96 96
             }, $files);
97 97
         }
98 98
 
@@ -269,16 +269,16 @@  discard block
 block discarded – undo
269 269
 
270 270
     private function createFeedbackHeaders($fromName, $fromEmail, $toMail, $sender)
271 271
     {
272
-        $headers = "MIME-Version: 1.0" . "\r\n";
273
-        $headers .= "Content-type: text/html; charset=UTF-8" . "\r\n";
272
+        $headers = "MIME-Version: 1.0"."\r\n";
273
+        $headers .= "Content-type: text/html; charset=UTF-8"."\r\n";
274 274
         if (!empty($toMail)) {
275
-            $headers .= "Cc: " . $this->model->getConfig()->getFeedbackAddress() . "\r\n";
275
+            $headers .= "Cc: ".$this->model->getConfig()->getFeedbackAddress()."\r\n";
276 276
         }
277 277
         if (!empty($fromEmail)) {
278 278
             $headers .= "Reply-To: $fromName <$fromEmail>\r\n";
279 279
         }
280 280
         $service = $this->model->getConfig()->getServiceName();
281
-        return $headers . "From: $fromName via $service <$sender>";
281
+        return $headers."From: $fromName via $service <$sender>";
282 282
     }
283 283
 
284 284
     /**
@@ -292,9 +292,9 @@  discard block
 block discarded – undo
292 292
     public function sendFeedback($request, $message, $messageSubject, $fromName = null, $fromEmail = null, $fromVocab = null, $toMail = null)
293 293
     {
294 294
         $toAddress = ($toMail) ? $toMail : $this->model->getConfig()->getFeedbackAddress();
295
-        $messageSubject = "[" . $this->model->getConfig()->getServiceName() . "] " . $messageSubject;
295
+        $messageSubject = "[".$this->model->getConfig()->getServiceName()."] ".$messageSubject;
296 296
         if ($fromVocab !== null && $fromVocab !== '') {
297
-            $message = 'Feedback from vocab: ' . strtoupper($fromVocab) . "<br />" . $message;
297
+            $message = 'Feedback from vocab: '.strtoupper($fromVocab)."<br />".$message;
298 298
         }
299 299
         $envelopeSender = $this->model->getConfig()->getFeedbackEnvelopeSender();
300 300
         // determine the sender address of the message
@@ -313,10 +313,10 @@  discard block
 block discarded – undo
313 313
         $headers = $this->createFeedbackHeaders($fromName, $fromEmail, $toMail, $sender);
314 314
         $params = empty($envelopeSender) ? '' : "-f $envelopeSender";
315 315
         // adding some information about the user for debugging purposes.
316
-        $message = $message . "<br /><br /> Debugging information:"
317
-            . "<br />Timestamp: " . date(DATE_RFC2822)
318
-            . "<br />User agent: " . $request->getServerConstant('HTTP_USER_AGENT')
319
-            . "<br />Referer: " . $request->getServerConstant('HTTP_REFERER');
316
+        $message = $message."<br /><br /> Debugging information:"
317
+            . "<br />Timestamp: ".date(DATE_RFC2822)
318
+            . "<br />User agent: ".$request->getServerConstant('HTTP_USER_AGENT')
319
+            . "<br />Referer: ".$request->getServerConstant('HTTP_REFERER');
320 320
 
321 321
         try {
322 322
             mail($toAddress, $messageSubject, $message, $headers, $params);
@@ -324,7 +324,7 @@  discard block
 block discarded – undo
324 324
             header("HTTP/1.0 404 Not Found");
325 325
             $template = $this->twig->load('error.twig');
326 326
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
327
-                error_log('Caught exception: ' . $e->getMessage());
327
+                error_log('Caught exception: '.$e->getMessage());
328 328
             }
329 329
 
330 330
             echo $template->render(
@@ -377,7 +377,7 @@  discard block
 block discarded – undo
377 377
                 } catch (ValueError $e) {
378 378
                     // fail fast with an error page if the vocabulary cannot be found
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
                     header("HTTP/1.0 400 Bad Request");
383 383
                     $this->invokeGenericErrorPage($request, $e->getMessage());
@@ -399,7 +399,7 @@  discard block
 block discarded – undo
399 399
             $errored = true;
400 400
             header("HTTP/1.0 500 Internal Server Error");
401 401
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
402
-                error_log('Caught exception: ' . $e->getMessage());
402
+                error_log('Caught exception: '.$e->getMessage());
403 403
             }
404 404
         }
405 405
         $vocabList = $this->model->getVocabularyList();
@@ -439,7 +439,7 @@  discard block
 block discarded – undo
439 439
         } catch (Exception $e) {
440 440
             header("HTTP/1.0 500 Internal Server Error");
441 441
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
442
-                error_log('Caught exception: ' . $e->getMessage());
442
+                error_log('Caught exception: '.$e->getMessage());
443 443
             }
444 444
 
445 445
             echo $template->render(
@@ -464,7 +464,7 @@  discard block
 block discarded – undo
464 464
         } catch (Exception $e) {
465 465
             header("HTTP/1.0 404 Not Found");
466 466
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
467
-                error_log('Caught exception: ' . $e->getMessage());
467
+                error_log('Caught exception: '.$e->getMessage());
468 468
             }
469 469
 
470 470
             echo $template->render(
Please login to merge, or discard this patch.