comodojo / comodojo-installer

src/Comodojo/Installer/Plugin.php

Indentation +4 added lines, -4 removed lines

@@ -34,15 +34,15 @@
     protected $comodojo_configuration_persistence;
     
     /**
-    * {@inheritDoc}
-    */
+     * {@inheritDoc}
+     */
     public function deactivate(Composer $composer, IOInterface $io){
     
     }
     
     /**
-    * {@inheritDoc}
-    */
+     * {@inheritDoc}
+     */
     public function uninstall(Composer $composer, IOInterface $io){
     
     }

src/Comodojo/Installer/Persistence/JsonPersistence.php

Braces +12 added lines, -4 removed lines

@@ -29,9 +29,13 @@ 
 
         $name = $this->getParameters()->get('config-file');
 
-        if ( $name === null ) throw new InstallerException("No config-file name specified");
+        if ( $name === null ) {
+            throw new InstallerException("No config-file name specified");
+        }
 
-        if ( file_exists($name) && is_readable($name) ) return json_decode(file_get_contents($name), true);
+        if ( file_exists($name) && is_readable($name) ) {
+            return json_decode(file_get_contents($name), true);
+        }
 
         return [];
 
@@ -44,14 +48,18 @@ 
 
         $name = $this->getParameters()->get('config-file');
 
-        if ( $name === null ) throw new InstallerException("No config-file name specified");
+        if ( $name === null ) {
+            throw new InstallerException("No config-file name specified");
+        }
 
         $filedata = json_encode($data);
 
         if (
             ( file_exists($name) && is_writable($name) ) ||
             is_writeable(pathinfo($name, PATHINFO_DIRNAME))
-        ) return file_put_contents($name, $filedata, LOCK_EX);
+        ) {
+            return file_put_contents($name, $filedata, LOCK_EX);
+        }
 
         throw new InstallerException("Cannot write to configuration file $name");
 

src/Comodojo/Installer/Persistence/YamlPersistence.php

Braces +12 added lines, -4 removed lines

@@ -39,9 +39,13 @@ 
 
         $name = $this->getParameters()->get('config-file');
 
-        if ( $name === null ) throw new InstallerException("No config-file name specified");
+        if ( $name === null ) {
+            throw new InstallerException("No config-file name specified");
+        }
 
-        if ( file_exists($name) && is_readable($name) ) return Yaml::parse(file_get_contents($name));
+        if ( file_exists($name) && is_readable($name) ) {
+            return Yaml::parse(file_get_contents($name));
+        }
 
         return [];
 
@@ -54,7 +58,9 @@ 
 
         $name = $this->getParameters()->get('config-file');
 
-        if ( $name === null ) throw new InstallerException("No config-file name specified");
+        if ( $name === null ) {
+            throw new InstallerException("No config-file name specified");
+        }
 
         $config_depth = $this->getParameters()->get('depth');
 
@@ -65,7 +71,9 @@ 
         if (
             ( file_exists($name) && is_writable($name) ) ||
             is_writeable(pathinfo($name, PATHINFO_DIRNAME))
-        ) return file_put_contents($name, $filedata, LOCK_EX);
+        ) {
+            return file_put_contents($name, $filedata, LOCK_EX);
+        }
 
         throw new InstallerException("Cannot write to configuration file $name");
 

src/Comodojo/Installer/Components/InstallerConfigurationExtraParser.php

Braces +12 added lines, -4 removed lines

@@ -46,8 +46,12 @@ 
 
     public function __construct($name, array $content) {
 
-        if ( empty($content['driver']) ) throw new InstallerException("Missing driver for field $name");
-        if ( empty($content['persistence']) ) throw new InstallerException("Missing persistence for field $name");
+        if ( empty($content['driver']) ) {
+            throw new InstallerException("Missing driver for field $name");
+        }
+        if ( empty($content['persistence']) ) {
+            throw new InstallerException("Missing persistence for field $name");
+        }
 
         $this->properties = [
             "driver" => $content['driver'],
@@ -56,8 +60,12 @@ 
             "persistence-source" => null
         ];
 
-        if ( !empty($content['driver-source']) ) $this->properties['driver-source'] = $content['driver-source'];
-        if ( !empty($content['persistence-source']) ) $this->properties['persistence-source'] = $content['persistence-source'];
+        if ( !empty($content['driver-source']) ) {
+            $this->properties['driver-source'] = $content['driver-source'];
+        }
+        if ( !empty($content['persistence-source']) ) {
+            $this->properties['persistence-source'] = $content['persistence-source'];
+        }
 
         $parameters = isset($content['params']) && is_array($content['params']) ? $content['params'] : [] ;
 

src/Comodojo/Installer/Components/InstallerConfiguration.php

Braces +6 added lines, -2 removed lines

@@ -57,7 +57,9 @@ 
 
     public function __construct(array $parameters = []) {
 
-        if ( isset($parameters['package-types']) ) $this->package_types = $parameters['package-types'];
+        if ( isset($parameters['package-types']) ) {
+            $this->package_types = $parameters['package-types'];
+        }
 
         $this->global_config = new InstallerConfigurationGlobalParser($parameters);
 
@@ -67,7 +69,9 @@ 
             }
         }
 
-        if ( isset($parameters['post-installer-script']) ) $this->post_install_script = $parameters['post-installer-script'];
+        if ( isset($parameters['post-installer-script']) ) {
+            $this->post_install_script = $parameters['post-installer-script'];
+        }
 
     }