View GitHub Repository superhive

Schedule Inspection
Subscribe
Passed
Push — master ( a26a47...1bbad3 )
by Florent
08:19
created
Category
Toggle Patches
public/install.php 3 patches
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -3,20 +3,20 @@
 
 block discarded – undo
3 3 
 define('EXTRACT_DIRECTORY', "../composer");
4 4
5 5 
 if(file_exists(ROOT.'vendor')) {
6 
-  return header("Refresh:0; url=/");
6 
+    return header("Refresh:0; url=/");
7 7 
 }
8 8
9 9 
 if (file_exists(EXTRACT_DIRECTORY.'/vendor/autoload.php') == true) {
10 
-  echo "Extracted autoload already exists. Skipping phar extraction as presumably it's already extracted.";
10 
+    echo "Extracted autoload already exists. Skipping phar extraction as presumably it's already extracted.";
11 11 
 }
12 12 
 else{
13 
-  $composerPhar = ROOT.'composer.phar';
14 
-  if (!file_exists($composerPhar)) {
13 
+    $composerPhar = ROOT.'composer.phar';
14 
+    if (!file_exists($composerPhar)) {
15 15 
     $composer = file_get_contents('https://getcomposer.org/download/latest-stable/composer.phar');
16 16 
     file_put_contents($composerPhar, $composer);
17 
-  }
18 
-  $composerExtract = new Phar($composerPhar);
19 
-  $composerExtract->extractTo(EXTRACT_DIRECTORY);
17 
+    }
18 
+    $composerExtract = new Phar($composerPhar);
19 
+    $composerExtract->extractTo(EXTRACT_DIRECTORY);
20 20 
 }
21 21
22 22 
 require_once (EXTRACT_DIRECTORY.'/vendor/autoload.php');
Please login to merge, or discard this patch.
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -2,15 +2,15 @@  discard block
 
 block discarded – undo
2 2 
 define('ROOT', "../");
3 3 
 define('EXTRACT_DIRECTORY', "../composer");
4 4
5 
-if(file_exists(ROOT.'vendor')) {
5 
+if (file_exists(ROOT . 'vendor')) {
6 6 
   return header("Refresh:0; url=/");
7 7 
 }
8 8
9 
-if (file_exists(EXTRACT_DIRECTORY.'/vendor/autoload.php') == true) {
9 
+if (file_exists(EXTRACT_DIRECTORY . '/vendor/autoload.php') == true) {
10 10 
   echo "Extracted autoload already exists. Skipping phar extraction as presumably it's already extracted.";
11 11 
 }
12 
-else{
13 
-  $composerPhar = ROOT.'composer.phar';
12 
+else {
13 
+  $composerPhar = ROOT . 'composer.phar';
14 14 
   if (!file_exists($composerPhar)) {
15 15 
     $composer = file_get_contents('https://getcomposer.org/download/latest-stable/composer.phar');
16 16 
     file_put_contents($composerPhar, $composer);
 
@@ -19,7 +19,7 @@  discard block
 
 block discarded – undo
19 19 
   $composerExtract->extractTo(EXTRACT_DIRECTORY);
20 20 
 }
21 21
22 
-require_once (EXTRACT_DIRECTORY.'/vendor/autoload.php');
22 
+require_once (EXTRACT_DIRECTORY . '/vendor/autoload.php');
23 23
24 24 
 //Use the Composer classes
25 25 
 use Composer\Console\Application;
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch 
@@ -8,8 +8,7 @@
 
 block discarded – undo
8 8
9 9 
 if (file_exists(EXTRACT_DIRECTORY.'/vendor/autoload.php') == true) {
10 10 
   echo "Extracted autoload already exists. Skipping phar extraction as presumably it's already extracted.";
11 
-}
12 
-else{
11 
+} else{
13 12 
   $composerPhar = ROOT.'composer.phar';
14 13 
   if (!file_exists($composerPhar)) {
15 14 
     $composer = file_get_contents('https://getcomposer.org/download/latest-stable/composer.phar');
Please login to merge, or discard this patch.
public/index.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -1,7 +1,7 @@
 
 block discarded – undo
1 1 
 <?php
2 2 
 // Check if vendor folder exist
3 3 
 if(file_exists(__DIR__ . '/../vendor')) {
4 
-  (require __DIR__ . '/../src/start.php')->run();
4 
+    (require __DIR__ . '/../src/start.php')->run();
5 5 
 } else {
6 6 
 ?>
7 7 
 <!DOCTYPE html>
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch 
@@ -1,6 +1,6 @@
 
 block discarded – undo
1 1 
 <?php
2 2 
 // Check if vendor folder exist
3 
-if(file_exists(__DIR__ . '/../vendor')) {
3 
+if (file_exists(__DIR__ . '/../vendor')) {
4 4 
   (require __DIR__ . '/../src/start.php')->run();
5 5 
 } else {
6 6 
 ?>
Please login to merge, or discard this patch.
src/start.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch 
@@ -37,7 +37,7 @@  discard block
 
 block discarded – undo
37 37 
 $container->set('themesdir', __DIR__ . '/../public/themes/');
38 38
39 39 
 // Set settings array in container for use in all routes
40 
-$container->set('settings', function () {
40 
+$container->set('settings', function() {
41 41 
     $config = file_get_contents(__DIR__ . '/../config/config.json');
42 42 
     $settings = json_decode($config, true);
43 43 
     return $settings;
 
@@ -72,7 +72,7 @@  discard block
 
 block discarded – undo
72 72 
     mkdir($container->get('cachedir'), 0755, true);
73 73 
 } else {
74 74 
     // Flush Cache folder if disabled
75 
-    if ((file_exists($container->get('cachedir'))) && ($settings["devMode"] == true )) {
75 
+    if ((file_exists($container->get('cachedir'))) && ($settings["devMode"] == true)) {
76 76 
         $path = $container->get('cachedir');
77 77 
         $files = glob($path . '/*');
78 78 
         foreach ($files as $file) {
 
@@ -86,7 +86,7 @@  discard block
 
 block discarded – undo
86 86 
 AppFactory::setContainer($container);
87 87
88 88 
 // Set Twig engine for templating
89 
-$container->set('view', function () {
89 
+$container->set('view', function() {
90 90 
     $settings = json_decode(file_get_contents(__DIR__ . '/../config/config.json'), true);
91 91 
     $tpls = [
92 92 
         __DIR__ . "/../resources/views/",
 
@@ -114,7 +114,7 @@  discard block
 
 block discarded – undo
114 114 
 });
115 115
116 116 
 //Set Session engine
117 
-$container->set('session', function () {
117 
+$container->set('session', function() {
118 118 
     return new \SlimSession\Helper();
119 119 
 });
120 120 
 
@@ -185,7 +185,7 @@  discard block
 
 block discarded – undo
185 185
186 186 
 foreach ($pages as $page) {
187 187 
     $route = substr($page, 0, strrpos($page, "."));
188 
-    $app->get('/' . $route, function ($request, $response) {
188 
+    $app->get('/' . $route, function($request, $response) {
189 189 
         $settings = $this->get('settings');
190 190 
         $uri = $request->getUri();
191 191 
         $route = substr(strrchr($uri, "/"), 1);
Please login to merge, or discard this patch.
src/Controllers/PagesController.php 1 patch
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch 
@@ -114,13 +114,13 @@
 
 block discarded – undo
114 114 
 /**
115 115 
      * Administration delete page function
116 116 
      *
117 
-     * called to delete fpage and return to administration Pages section
118 
-     *
119 
-     * @param object $request
120 
-     * @param object $response
121 
-     * @param array $args
122 
-     *
123 
-     * @return object $response
117 
+ * called to delete fpage and return to administration Pages section
118 
+ *
119 
+ * @param object $request
120 
+ * @param object $response
121 
+ * @param array $args
122 
+ *
123 
+ * @return object $response
124 124 
      */
125 125 
     public function adminDelPage(Request $request, Response $response, array $args): Response
126 126 
     {
Please login to merge, or discard this patch.