Passed
Pull Request — main (#1817)
by Osma
09:35 queued 05:44
created
src/controller/WebController.php 1 patch
Spacing   +19 added lines, -19 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', __DIR__ . '/../../slots']);
40
+        $loader = new \Twig\Loader\FilesystemLoader([__DIR__.'/../view', __DIR__.'/../../slots']);
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);
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 
78 78
         foreach ($slotSubDirs as $slotDir) {
79 79
             $slotName = basename($slotDir);
80
-            $files = glob($slotDir . '/*.twig');
80
+            $files = glob($slotDir.'/*.twig');
81 81
             // Strip the "../slots" prefix, it's not needed.
82 82
             // The "slots" directory is registered to the Twig FilesystemLoader.
83 83
             $slots[$slotName] = preg_filter('|../slots/|', '', $files);
@@ -255,16 +255,16 @@  discard block
 block discarded – undo
255 255
 
256 256
     private function createFeedbackHeaders($fromName, $fromEmail, $toMail, $sender)
257 257
     {
258
-        $headers = "MIME-Version: 1.0" . "\r\n";
259
-        $headers .= "Content-type: text/html; charset=UTF-8" . "\r\n";
258
+        $headers = "MIME-Version: 1.0"."\r\n";
259
+        $headers .= "Content-type: text/html; charset=UTF-8"."\r\n";
260 260
         if (!empty($toMail)) {
261
-            $headers .= "Cc: " . $this->model->getConfig()->getFeedbackAddress() . "\r\n";
261
+            $headers .= "Cc: ".$this->model->getConfig()->getFeedbackAddress()."\r\n";
262 262
         }
263 263
         if (!empty($fromEmail)) {
264 264
             $headers .= "Reply-To: $fromName <$fromEmail>\r\n";
265 265
         }
266 266
         $service = $this->model->getConfig()->getServiceName();
267
-        return $headers . "From: $fromName via $service <$sender>";
267
+        return $headers."From: $fromName via $service <$sender>";
268 268
     }
269 269
 
270 270
     /**
@@ -278,9 +278,9 @@  discard block
 block discarded – undo
278 278
     public function sendFeedback($request, $message, $messageSubject, $fromName = null, $fromEmail = null, $fromVocab = null, $toMail = null)
279 279
     {
280 280
         $toAddress = ($toMail) ? $toMail : $this->model->getConfig()->getFeedbackAddress();
281
-        $messageSubject = "[" . $this->model->getConfig()->getServiceName() . "] " . $messageSubject;
281
+        $messageSubject = "[".$this->model->getConfig()->getServiceName()."] ".$messageSubject;
282 282
         if ($fromVocab !== null && $fromVocab !== '') {
283
-            $message = 'Feedback from vocab: ' . strtoupper($fromVocab) . "<br />" . $message;
283
+            $message = 'Feedback from vocab: '.strtoupper($fromVocab)."<br />".$message;
284 284
         }
285 285
         $envelopeSender = $this->model->getConfig()->getFeedbackEnvelopeSender();
286 286
         // determine the sender address of the message
@@ -299,10 +299,10 @@  discard block
 block discarded – undo
299 299
         $headers = $this->createFeedbackHeaders($fromName, $fromEmail, $toMail, $sender);
300 300
         $params = empty($envelopeSender) ? '' : "-f $envelopeSender";
301 301
         // adding some information about the user for debugging purposes.
302
-        $message = $message . "<br /><br /> Debugging information:"
303
-            . "<br />Timestamp: " . date(DATE_RFC2822)
304
-            . "<br />User agent: " . $request->getServerConstant('HTTP_USER_AGENT')
305
-            . "<br />Referer: " . $request->getServerConstant('HTTP_REFERER');
302
+        $message = $message."<br /><br /> Debugging information:"
303
+            . "<br />Timestamp: ".date(DATE_RFC2822)
304
+            . "<br />User agent: ".$request->getServerConstant('HTTP_USER_AGENT')
305
+            . "<br />Referer: ".$request->getServerConstant('HTTP_REFERER');
306 306
 
307 307
         try {
308 308
             mail($toAddress, $messageSubject, $message, $headers, $params);
@@ -310,7 +310,7 @@  discard block
 block discarded – undo
310 310
             header("HTTP/1.0 404 Not Found");
311 311
             $template = $this->twig->load('error.twig');
312 312
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
313
-                error_log('Caught exception: ' . $e->getMessage());
313
+                error_log('Caught exception: '.$e->getMessage());
314 314
             }
315 315
 
316 316
             echo $template->render(
@@ -363,7 +363,7 @@  discard block
 block discarded – undo
363 363
                 } catch (ValueError $e) {
364 364
                     // fail fast with an error page if the vocabulary cannot be found
365 365
                     if ($this->model->getConfig()->getLogCaughtExceptions()) {
366
-                        error_log('Caught exception: ' . $e->getMessage());
366
+                        error_log('Caught exception: '.$e->getMessage());
367 367
                     }
368 368
                     header("HTTP/1.0 400 Bad Request");
369 369
                     $this->invokeGenericErrorPage($request, $e->getMessage());
@@ -385,7 +385,7 @@  discard block
 block discarded – undo
385 385
             $errored = true;
386 386
             header("HTTP/1.0 500 Internal Server Error");
387 387
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
388
-                error_log('Caught exception: ' . $e->getMessage());
388
+                error_log('Caught exception: '.$e->getMessage());
389 389
             }
390 390
         }
391 391
         $vocabList = $this->model->getVocabularyList();
@@ -425,7 +425,7 @@  discard block
 block discarded – undo
425 425
         } catch (Exception $e) {
426 426
             header("HTTP/1.0 500 Internal Server Error");
427 427
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
428
-                error_log('Caught exception: ' . $e->getMessage());
428
+                error_log('Caught exception: '.$e->getMessage());
429 429
             }
430 430
 
431 431
             echo $template->render(
@@ -450,7 +450,7 @@  discard block
 block discarded – undo
450 450
         } catch (Exception $e) {
451 451
             header("HTTP/1.0 404 Not Found");
452 452
             if ($this->model->getConfig()->getLogCaughtExceptions()) {
453
-                error_log('Caught exception: ' . $e->getMessage());
453
+                error_log('Caught exception: '.$e->getMessage());
454 454
             }
455 455
 
456 456
             echo $template->render(
Please login to merge, or discard this patch.