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/interfaces/generator.php

Spacing +5 added lines, -5 removed lines

@@ -2,17 +2,17 @@
 
 abstract class plStructureGenerator 
 {
-    public static function factory( $generator ) 
+    public static function factory($generator) 
     {
-        $classname = 'plStructure' . ucfirst( $generator ) . 'Generator';
-        if ( class_exists( $classname ) === false ) 
+        $classname = 'plStructure' . ucfirst($generator) . 'Generator';
+        if (class_exists($classname) === false) 
         {
-            throw new plStructureGeneratorNotFoundException( $generator );
+            throw new plStructureGeneratorNotFoundException($generator);
         }
         return new $classname();
     }
 
-    public abstract function createStructure( array $files );    
+    public abstract function createStructure(array $files);    
 }
 
 ?>

src/classes/phuml.php

Spacing +31 added lines, -31 removed lines

@@ -10,38 +10,38 @@ 
     public function __construct() 
     {
         $this->properties = array( 
-            'generator'     => plStructureGenerator::factory( 'tokenparser' ),
+            'generator'     => plStructureGenerator::factory('tokenparser'),
         );
 
         $this->files = array();
     }
 
-    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;
             }
@@ -50,77 +50,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;            
     }

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 );
 

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/classes/base.php

Spacing +9 added lines, -9 removed lines

@@ -5,24 +5,24 @@ 
     private static $autoload = array();
     private static $autoloadDirectory = array();
 
-    public static function autoload( $classname )
+    public static function autoload($classname)
     {
-        if ( isset( self::$autoload[$classname] ) ) 
+        if (isset(self::$autoload[$classname])) 
         {
-            include_once( self::$autoload[$classname] );
+            include_once(self::$autoload[$classname]);
         }
     }
 
-    public static function addAutoloadDirectory( $directory ) 
+    public static function addAutoloadDirectory($directory) 
     {
-        if ( !in_array( $directory, self::$autoloadDirectory ) && is_dir( $directory ) && is_readable( $directory ) )
+        if (!in_array($directory, self::$autoloadDirectory) && is_dir($directory) && is_readable($directory))
         {
             self::$autoloadDirectory[] = $directory;
-            foreach( $glob = glob( $directory."/*.php" ) as $file )
+            foreach ($glob = glob($directory . "/*.php") as $file)
             {
-                if ( is_array( $autoload = include( $file ) ) ) 
+                if (is_array($autoload = include($file))) 
                 {
-                    self::$autoload = array_merge( $autoload, self::$autoload );
+                    self::$autoload = array_merge($autoload, self::$autoload);
                 }
             }
         }
@@ -30,7 +30,7 @@ 
 
     public static function getAutoloadClasses() 
     {
-        return array_keys( self::$autoload );
+        return array_keys(self::$autoload);
     }
 }
 

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.');
     }
 }