Move app/logs to var/logs

see https://symfony.com/doc/3.3/quick_tour/the_architecture.html
forkcms · Jul 27, 2017 · d439ac2d24be7be735f5ebea7b60ca4d60838b09 · d439ac2
1 parent 517fbd8
commit d439ac2d24be7be735f5ebea7b60ca4d60838b09
diff --git a/.gitignore b/.gitignore
@@ -14,7 +14,7 @@
 .*.swp
 .DS_Store
 /app/cache/*
-  
-/app/logs/*
+  
+/var/logs/*
 /app/sessions/*
 /bin/*
 !bin/console
diff --git a/.htaccess b/.htaccess
@@ -61,7 +61,7 @@ deny from env=stayout
 	RewriteRule composer\.lock - [F]
 	RewriteRule phpunit\.xml\.dist - [F]
 	RewriteRule .*\.md - [F]
-  
-	RewriteRule app/logs - [F]
+  
+	RewriteRule var/logs - [F]
 	RewriteRule app/config - [F]
 	RewriteRule src/Frontend/Cache/CompiledTemplates - [F]
 	RewriteRule src/Frontend/Cache/Search - [F]
diff --git a/.travis.yml b/.travis.yml
@@ -21,6 +21,6 @@ after_success:
  - php ocular.phar code-coverage:upload --format=php-clover coverage.clover
 after_failure:
-  
-  - cat app/logs/test.log
+  
+  - cat var/logs/test.log
 sudo: false
diff --git a/UPGRADE_3.8.md b/UPGRADE_3.8.md
@@ -81,8 +81,8 @@ For Nginx
            server_name fork-cms.com;
            root /var/www/fork;
            index index.php;
-  
-            error_log /var/www/fork/app/logs/error-nginx.log;
-  
-            access_log /var/www/fork/app/logs/access-nginx.log;
+  
+            error_log /var/www/fork/var/logs/error-nginx.log;
+  
+            access_log /var/www/fork/var/logs/access-nginx.log;
        #site root is redirected to the app boot script
        location = / {
diff --git a/UPGRADE_5.0.md b/UPGRADE_5.0.md
@@ -565,3 +565,7 @@ If you want to add a global anyway you should use assignGlobal as it uses a cust
 ### Console
 Moved `app/console` to `bin/console`
+  
+
+  
+### Logs
+  
+
+  
+Moved `app/logs` to `var/logs`
diff --git a/app/Kernel.php b/app/Kernel.php
@@ -221,4 +221,9 @@ protected function initializeContainer(): void
        parent::initializeContainer();
    }
+  
+
+  
+    public function getLogDir(): string
+  
+    {
+  
+        return dirname(__DIR__).'/var/logs/' . $this->environment;
+  
+    }
 }
diff --git a/app/config/config.yml b/app/config/config.yml
@@ -42,7 +42,7 @@ monolog:
    handlers:
        main:
            type:  stream
-  
-            path:  "%site.path_www%/app/logs/%kernel.environment%.log"
+  
+            path:  "%site.path_www%/var/logs/%kernel.environment%.log"
            level: error
        # swift:
        #     type:       swift_mailer
diff --git a/app/config/config_dev.yml b/app/config/config_dev.yml
@@ -11,7 +11,7 @@ monolog:
    handlers:
        main:
            type:  stream
-  
-            path:  "%site.path_www%/app/logs/%kernel.environment%.log"
+  
+            path:  "%site.path_www%/var/logs/%kernel.environment%.log"
            level: debug
 services:
diff --git a/app/config/config_install.yml b/app/config/config_install.yml
@@ -29,7 +29,7 @@ monolog:
    handlers:
        main:
            type:  stream
-  
-            path:  %site.path_www%/app/logs/logs.log
+  
+            path:  %site.path_www%/var/logs/logs.log
            level: debug
 parameters:
diff --git a/src/ForkCMS/Bundle/InstallerBundle/Resources/views/Installer/step1.html.twig b/src/ForkCMS/Bundle/InstallerBundle/Resources/views/Installer/step1.html.twig
@@ -261,7 +261,7 @@
                  </p>
                </div>
                <div class="list-group-item list-group-item-{{ checker.errors.fileSystemAppLogs }}">
-  
-                  <h4 class="list-group-item-heading">{{ rootDir }}/app/logs/</h4>
+  
+                  <h4 class="list-group-item-heading">{{ rootDir }}/var/logs/</h4>
                  <p class="list-group-item-text">
                    In this location the global logs will be stored.
                  </p>
diff --git a/src/ForkCMS/Bundle/InstallerBundle/Service/RequirementsChecker.php b/src/ForkCMS/Bundle/InstallerBundle/Service/RequirementsChecker.php
@@ -243,7 +243,7 @@ protected function checkFilePermissions(): void
        );
        $this->checkRequirement(
            'fileSystemAppLogs',
-  
-            $this->isRecursivelyWritable($this->rootDir . 'app/logs/'),
+  
+            $this->isRecursivelyWritable($this->rootDir . 'var/logs/'),
            self::STATUS_ERROR
        );
        $this->checkRequirement(