complete installcli process

core/lib/Thelia/Command/Install.php

@@ -28,6 +28,7 @@ use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Filesystem\Filesystem;
 use Thelia\Command\ContainerAwareCommand;
+use Thelia\Install\CheckPermission;
 use Thelia\Install\Database;
 
 /**
@@ -82,6 +83,9 @@ class Install extends ContainerAwareCommand
             '',
             'Welcome to Thelia install process',
             'You need information about your database configuration (host, username, password, database name, etc)',
+            '',
+            '<info>Caution : You are installing Thelia in cli mode, we verify some information, but this information are only available for the cli php sapi</info>',
+            '<info>This informations can be different in your apache or cgi php.ini files</info>',
             ''
         ));
 
@@ -136,40 +140,35 @@ class Install extends ContainerAwareCommand
             "Checking some permissions"
         ));
 
-        $confDir = THELIA_ROOT . "local/config";
+        $permissions = new CheckPermission(false, $this->getContainer()->get('thelia.translator'));
-        $cacheDir = THELIA_ROOT . "cache";
+        $isValid = $permissions->exec();
-        $logDir = THELIA_ROOT . "log";
 
-        $conf   = is_writable($confDir);
+        foreach($permissions->getValidationMessages() as $item => $data) {
-        $cache  = is_writable($cacheDir);
+            if($data['status']) {
-        $log    = is_writable($logDir);
+                $output->writeln(array(
+                    sprintf("<info>%s ...</info> %s",
+                        $data['text'],
+                        "<info>Ok</info>")
+                    )
+                );
+            } else {
+                $output->writeln(array(
+                    sprintf("<error>%s </error>%s",
+                        $data['text'],
+                        sprintf("<error>%s</error>", $data["hint"])
+                    )
+                ));
+            }
 
-        $output->writeln(array(
-           sprintf(
-               "<info>config directory(%s)...</info> %s",
-               $confDir,
-               $conf ? "<info>Ok</info>" : "<error>Fail</error>"
-           ),
-           sprintf(
-               "<info>cache directory(%s)...</info> %s"
-               ,$cacheDir,
-               $cache ? "<info>Ok</info>" : "<error>Fail</error>"
-           ),
-           sprintf(
-               "<info>log directory(%s)...</info> %s",
-               $logDir,
-               $log ? "<info>Ok</info>" : "<error>Fail</error>"
-           ),
-        ));
-
-        if ($conf === false || $cache === false || $log === false) {
-           $output->writeln(array(
-              "",
-              "<error>Please put correct permission and reload install process</error>"
-           ));
-            exit;
         }
 
+        if(false === $isValid) {
+            $output->writeln(array(
+                "",
+                "<error>Please put correct permissions and reload install process</error>"
+            ));
+            exit;
+        }
     }
 
     /**

core/lib/Thelia/Install/CheckPermission.php

@@ -56,7 +56,7 @@ class CheckPermission extends BaseInstall
 
     /** @var array Minimum server configuration necessary */
     protected $minServerConfigurationNecessary = array(
-        'memory_limit' => 134217728,
+        'memory_limit' => 157286400,
         'post_max_size' => 20971520,
         'upload_max_filesize' => 2097152
     );
@@ -216,7 +216,7 @@ class CheckPermission extends BaseInstall
     protected function getI18nDirectoryHint($directory)
     {
         if ($this->translator !== null) {
-            $sentence = '<span class="label label-primary">chmod 777 %directory%</span> on your server with admin rights could help';
+            $sentence = 'chmod 777 %directory% on your server with admin rights could help';
             $translatedText = $this->translator->trans(
                 $sentence,
                 array(
@@ -246,9 +246,9 @@ class CheckPermission extends BaseInstall
     protected function getI18nConfigText($key, $expectedValue, $currentValue, $isValid)
     {
         if ($isValid) {
-            $sentence = 'Your <span class="label label-primary">%key%</span> server configuration (currently %currentValue%) is well enough to run Thelia2 (%expectedValue% needed)';
+            $sentence = 'Your %key% server configuration (currently %currentValue%) is well enough to run Thelia2 (%expectedValue% needed)';
         } else {
-            $sentence = 'Your <span class="label label-primary">%key%</span> server configuration (currently %currentValue%) is not sufficient enough in order to run Thelia2 (%expectedValue% needed)';
+            $sentence = 'Your %key% server configuration (currently %currentValue%) is not sufficient enough in order to run Thelia2 (%expectedValue% needed)';
         }
 
         $translatedText = $this->translator->trans(
@@ -271,7 +271,7 @@ class CheckPermission extends BaseInstall
      */
     protected function getI18nConfigHint()
     {
-        $sentence = 'Modifying this value on your server <span class="label label-primary">php.ini</span> file with admin rights could help';
+        $sentence = 'Modifying this value on your server php.ini file with admin rights could help';
         $translatedText = $this->translator->trans(
             $sentence,
             array(),
@@ -294,9 +294,9 @@ class CheckPermission extends BaseInstall
     {
         if ($this->translator !== null) {
             if ($isValid) {
-                $sentence = 'Your PHP version <span class="label label-primary">%currentValue%</span> is well enough to run Thelia2 (%expectedValue% needed)';
+                $sentence = 'Your PHP version %currentValue% is well enough to run Thelia2 (%expectedValue% needed)';
             } else {
-                $sentence = 'Your PHP version <span class="label label-primary">%currentValue%</span> is not sufficient enough to run Thelia2 (%expectedValue% needed)';
+                $sentence = 'Your PHP version %currentValue% is not sufficient enough to run Thelia2 (%expectedValue% needed)';
             }
 
             $translatedText = $this->translator->trans(
@@ -343,6 +343,10 @@ class CheckPermission extends BaseInstall
     {
         $serverValueInBytes = $this->returnBytes(ini_get($key));
 
+        if($serverValueInBytes == -1) {
+            return true;
+        }
+
         return ($serverValueInBytes >= $necessaryValueInBytes);
     }