MontealegreLuis / phuml

src/interfaces/processor/externalCommand.php

Spacing +10 added lines, -10 removed lines

@@ -3,24 +3,24 @@
 abstract class plExternalCommandProcessor extends plProcessor 
 {
 
-    abstract public function execute( $infile, $outfile, $type );
+    abstract public function execute($infile, $outfile, $type);
 
-    public function process( $input, $type ) 
+    public function process($input, $type) 
     {
         // Create temporary datafiles
-        $infile  = tempnam( '/tmp', 'phuml' );
-        $outfile = tempnam( '/tmp', 'phuml' );
+        $infile  = tempnam('/tmp', 'phuml');
+        $outfile = tempnam('/tmp', 'phuml');
         
-        file_put_contents( $infile, $input );
+        file_put_contents($infile, $input);
 
-        unlink( $outfile );
+        unlink($outfile);
 
-        $this->execute( $infile, $outfile, $type );
+        $this->execute($infile, $outfile, $type);
         
-        $outdata = file_get_contents( $outfile );
+        $outdata = file_get_contents($outfile);
 
-        unlink( $infile );
-        unlink( $outfile );
+        unlink($infile);
+        unlink($outfile);
 
         return $outdata;
     }

src/classes/phuml.php

Braces +2 added lines, -4 removed lines

@@ -26,8 +26,7 @@ 
         if ( $recursive === false ) 
         {
             $iterator = new DirectoryIterator( $directory );
-        }
-        else
+        } else
         {
             $iterator = new RecursiveIteratorIterator(
                 new RecursiveDirectoryIterator( $directory )
@@ -59,8 +58,7 @@ 
             {
                 throw new plPhumlInvalidProcessorChainException( 'application/phuml-structure', $processor->getInputTypes() );
             }
-        }
-        else
+        } else
         {
             $this->checkProcessorCompatibility( end( $this->processors ), $processor );
 

Spacing +30 added lines, -30 removed lines

@@ -18,32 +18,32 @@ 
         $this->files = [];
     }
 
-    public function addFile( $file ) 
+    public function addFile($file) 
     {
         $this->files[] = $file;
     }
 
-    public function addDirectory( $directory, $extension = 'php', $recursive = true ) 
+    public function addDirectory($directory, $extension = 'php', $recursive = true) 
     {
-        if ( $recursive === false ) 
+        if ($recursive === false) 
         {
-            $iterator = new DirectoryIterator( $directory );
+            $iterator = new DirectoryIterator($directory);
         }
         else
         {
             $iterator = new RecursiveIteratorIterator(
-                new RecursiveDirectoryIterator( $directory )
+                new RecursiveDirectoryIterator($directory)
             );
         }
 
-        foreach( $iterator as $entry ) 
+        foreach ($iterator as $entry) 
         {
-            if ( $entry->isDir() === true ) 
+            if ($entry->isDir() === true) 
             {
                 continue;
             }
             
-            if ( $sub = strtolower( substr( $entry->getFilename(), -1 * strlen( $extension ) ) ) !== strtolower( $extension ) ) 
+            if ($sub = strtolower(substr($entry->getFilename(), -1 * strlen($extension))) !== strtolower($extension)) 
             {
                 continue;
             }
@@ -52,77 +52,77 @@ 
         }       
     }
 
-    public function addProcessor( $processor ) 
+    public function addProcessor($processor) 
     {
-        if ( count( $this->processors ) === 0 ) 
+        if (count($this->processors) === 0) 
         {
             // First processor must support application/phuml-structure
-            if ( !in_array( 'application/phuml-structure', $processor->getInputTypes() ) ) 
+            if (!in_array('application/phuml-structure', $processor->getInputTypes())) 
             {
-                throw new plPhumlInvalidProcessorChainException( 'application/phuml-structure', $processor->getInputTypes() );
+                throw new plPhumlInvalidProcessorChainException('application/phuml-structure', $processor->getInputTypes());
             }
         }
         else
         {
-            $this->checkProcessorCompatibility( end( $this->processors ), $processor );
+            $this->checkProcessorCompatibility(end($this->processors), $processor);
 
         }
         $this->processors[] = $processor;
     }
 
-    private function checkProcessorCompatibility( $first, $second ) 
+    private function checkProcessorCompatibility($first, $second) 
     {
-        if ( !( $first instanceof plProcessor ) || !( $second instanceof plProcessor ) ) 
+        if (!($first instanceof plProcessor) || !($second instanceof plProcessor)) 
         {
             throw new plPhumlInvalidProcessorException();
         }
 
-        if ( !in_array( $first->getOutputType(), $second->getInputTypes() ) ) 
+        if (!in_array($first->getOutputType(), $second->getInputTypes())) 
         {
-            throw new plPhumlInvalidProcessorChainException( $first->getOutputType(), $second->getInputTypes() );
+            throw new plPhumlInvalidProcessorChainException($first->getOutputType(), $second->getInputTypes());
         }
     }
 
-    public function generate( $outfile ) 
+    public function generate($outfile) 
     {
         echo "[|] Parsing class structure", "\n";
-        $structure = $this->generator->createStructure( $this->files );
+        $structure = $this->generator->createStructure($this->files);
         
-        $temporary = array( $structure, 'application/phuml-structure' );
-        foreach( $this->processors as $processor ) 
+        $temporary = array($structure, 'application/phuml-structure');
+        foreach ($this->processors as $processor) 
         {            
             preg_match( 
                 '@^pl([A-Z][a-z]*)Processor$@',
-                get_class( $processor ),
+                get_class($processor),
                 $matches
             );
 
             echo "[|] Running '" . $matches[1] . "' processor", "\n";
             $temporary = array( 
-                $processor->process( $temporary[0], $temporary[1] ),
+                $processor->process($temporary[0], $temporary[1]),
                 $processor->getOutputType(),
             );
         }
 
         echo "[|] Writing generated data to disk", "\n";
-        end( $this->processors )->writeToDisk( $temporary[0], $outfile );
+        end($this->processors)->writeToDisk($temporary[0], $outfile);
     }
 
 
-    public function __get( $key )
+    public function __get($key)
     {
-        if ( !array_key_exists( $key, $this->properties ) )
+        if (!array_key_exists($key, $this->properties))
         {
-            throw new plBasePropertyException( $key, plBasePropertyException::READ );
+            throw new plBasePropertyException($key, plBasePropertyException::READ);
         }
         return $this->properties[$key];
     }
 
-    public function __set( $key, $val )
+    public function __set($key, $val)
     {
-        if ( !array_key_exists( $key, $this->properties ) )
+        if (!array_key_exists($key, $this->properties))
         {
-            throw new plBasePropertyException( $key, plBasePropertyException::WRITE );
+            throw new plBasePropertyException($key, plBasePropertyException::WRITE);
         }
         $this->properties[$key] = $val;            
     }

src/classes/processor/dot.php

Spacing +4 added lines, -4 removed lines

@@ -21,17 +21,17 @@
         return 'image/png';
     }
 
-    public function execute( $infile, $outfile, $type ) 
+    public function execute($infile, $outfile, $type) 
     {
         exec(
-            'dot -Tpng -o ' . escapeshellarg( $outfile ) . ' ' . escapeshellarg( $infile ),
+            'dot -Tpng -o ' . escapeshellarg($outfile) . ' ' . escapeshellarg($infile),
             $output,
             $return
         );
 
-        if ( $return !== 0 ) 
+        if ($return !== 0) 
         {
-            throw new plProcessorExternalExecutionException( $output );
+            throw new plProcessorExternalExecutionException($output);
         }
     }
 }

src/classes/processor/options.php

Spacing +13 added lines, -13 removed lines

@@ -8,20 +8,20 @@ 
 
     protected $properties = array();    
 
-    public function __get( $key )
+    public function __get($key)
     {
-        if ( !array_key_exists( $key, $this->properties ) )
+        if (!array_key_exists($key, $this->properties))
         {
-            throw new plProcessorOptionException( $key, plProcessorOptionException::READ );
+            throw new plProcessorOptionException($key, plProcessorOptionException::READ);
         }
         return $this->properties[$key]['data'];
     }
 
-    public function __set( $key, $val )
+    public function __set($key, $val)
     {
-        if ( !array_key_exists( $key, $this->properties ) )
+        if (!array_key_exists($key, $this->properties))
         {
-            throw new plProcessorOptionException( $key, plProcessorOptionException::WRITE );
+            throw new plProcessorOptionException($key, plProcessorOptionException::WRITE);
         }
         $this->properties[$key]['data'] = $val;            
     }
@@ -29,27 +29,27 @@ 
     public function getOptions() 
     {
         $options = array();
-        foreach( $this->properties as $key => $property ) 
+        foreach ($this->properties as $key => $property) 
         {
             $options[] = $key;
         }
         return $options;
     }
 
-    public function getOptionDescription( $option ) 
+    public function getOptionDescription($option) 
     {
-        if ( !array_key_exists( $option, $this->properties ) ) 
+        if (!array_key_exists($option, $this->properties)) 
         {
-            throw new plProcessorOptionException( $option, plProcessorOptionException::UNKNOWN );
+            throw new plProcessorOptionException($option, plProcessorOptionException::UNKNOWN);
         }
         return $this->properties[$option]['description'];
     }
 
-    public function getOptionType( $option ) 
+    public function getOptionType($option) 
     {
-        if ( !array_key_exists( $option, $this->properties ) ) 
+        if (!array_key_exists($option, $this->properties)) 
         {
-            throw new plProcessorOptionException( $option, plProcessorOptionException::UNKNOWN );
+            throw new plProcessorOptionException($option, plProcessorOptionException::UNKNOWN);
         }
         return $this->properties[$option]['type'];
     }

src/classes/processor/neato.php

Spacing +4 added lines, -4 removed lines

@@ -21,17 +21,17 @@
         return 'image/png';
     }
 
-    public function execute( $infile, $outfile, $type ) 
+    public function execute($infile, $outfile, $type) 
     {
         exec(
-            'neato -Tpng -o ' . escapeshellarg( $outfile ) . ' ' . escapeshellarg( $infile ),
+            'neato -Tpng -o ' . escapeshellarg($outfile) . ' ' . escapeshellarg($infile),
             $output,
             $return
         );
 
-        if ( $return !== 0 ) 
+        if ($return !== 0) 
         {
-            throw new plProcessorExternalExecutionException( $output );
+            throw new plProcessorExternalExecutionException($output);
         }
     }
 }

src/exceptions/phuml/invalidProcessorChain.php

Indentation +5 added lines, -5 removed lines

@@ -6,11 +6,11 @@
     {
         parent::__construct( 
             'To processors in the chain are incompatible. The first processor\'s output is "' 
-          . $first 
-          . '". The next Processor in the queue does only support the following input types: ' 
-          . implode( ', ', $second ) 
-          . '.' 
-       );
+            . $first 
+            . '". The next Processor in the queue does only support the following input types: ' 
+            . implode( ', ', $second ) 
+            . '.' 
+        );
     }
 }
 

Spacing +2 added lines, -2 removed lines

@@ -2,13 +2,13 @@
 
 class plPhumlInvalidProcessorChainException extends Exception 
 {
-    public function __construct( $first, $second ) 
+    public function __construct($first, $second) 
     {
         parent::__construct( 
             'To processors in the chain are incompatible. The first processor\'s output is "' 
           . $first 
           . '". The next Processor in the queue does only support the following input types: ' 
-          . implode( ', ', $second ) 
+          . implode(', ', $second) 
           . '.' 
        );
     }

src/exceptions/phuml/invalidProcessor.php

Spacing +1 added lines, -1 removed lines

@@ -4,7 +4,7 @@
 {
     public function __construct() 
     {
-        parent::__construct( 'The supplied processor is invalid.' );
+        parent::__construct('The supplied processor is invalid.');
     }
 }
 

src/exceptions/base/property.php

Spacing +2 added lines, -2 removed lines

@@ -5,9 +5,9 @@
     const READ = 1;
     const WRITE = 2;
     
-    public function __construct( $key, $type ) 
+    public function __construct($key, $type) 
     {
-        parent::__construct( 'Invalid property access. The property "' . $key . '" is not '. ( $type === self::READ ? 'readable' : 'writable' ) . '.'  );
+        parent::__construct('Invalid property access. The property "' . $key . '" is not ' . ($type === self::READ ? 'readable' : 'writable') . '.');
     }
 }
 

src/exceptions/generator/notFound.php

Spacing +2 added lines, -2 removed lines

@@ -2,9 +2,9 @@
 
 class plStructureGeneratorNotFoundException extends Exception
 {
-    public function __construct( $generator ) 
+    public function __construct($generator) 
     {
-        parent::__construct( 'The needed generator class "' . $generator . '" could not be found.' );
+        parent::__construct('The needed generator class "' . $generator . '" could not be found.');
     }
 }