vanderlee / PHPSwaggerGen

SwaggerGen/Parser/Text/Preprocessor.php

Indentation +25 added lines, -25 removed lines

@@ -18,30 +18,30 @@
 class Preprocessor extends AbstractPreprocessor
 {
 
-    protected function parseContent($content)
-    {
-        $pattern = '/\\s*([a-z]+)\\s*(.*)\\s*/';
-
-        $output = '';
-
-        foreach (preg_split('/(\\R)/m', $content, null, PREG_SPLIT_DELIM_CAPTURE) as $index => $line) {
-            if ($index % 2) {
-                $output .= $line;
-            } else {
-                $match = array();
-                if (preg_match($pattern, $line, $match) === 1) {
-                    if (!$this->handle($match[1], $match[2]) && $this->getState()) {
-                        $output .= $line;
-                    } else {
-                        $output .= '';
-                    }
-                } else {
-                    $output .= $line;
-                }
-            }
-        }
-
-        return $output;
-    }
+	protected function parseContent($content)
+	{
+		$pattern = '/\\s*([a-z]+)\\s*(.*)\\s*/';
+
+		$output = '';
+
+		foreach (preg_split('/(\\R)/m', $content, null, PREG_SPLIT_DELIM_CAPTURE) as $index => $line) {
+			if ($index % 2) {
+				$output .= $line;
+			} else {
+				$match = array();
+				if (preg_match($pattern, $line, $match) === 1) {
+					if (!$this->handle($match[1], $match[2]) && $this->getState()) {
+						$output .= $line;
+					} else {
+						$output .= '';
+					}
+				} else {
+					$output .= $line;
+				}
+			}
+		}
+
+		return $output;
+	}
 
 }

SwaggerGen/Parser/Text/Parser.php

Indentation +84 added lines, -84 removed lines

@@ -17,99 +17,99 @@
 class Parser implements IParser
 {
 
-    /**
-     * List of directories to scan for class files referenced in the parsed
-     * command statements.
-     *
-     * @var string[]
-     */
-    protected $common_dirs = [];
+	/**
+	 * List of directories to scan for class files referenced in the parsed
+	 * command statements.
+	 *
+	 * @var string[]
+	 */
+	protected $common_dirs = [];
 
-    /**
-     * @var AbstractPreprocessor
-     */
-    private $Preprocessor;
+	/**
+	 * @var AbstractPreprocessor
+	 */
+	private $Preprocessor;
 
-    /**
-     * Create a new text parser and set directories to scan for referenced
-     * class files.
-     *
-     * @param string[] $dirs
-     */
-    public function __construct(array $dirs = [])
-    {
-        foreach ($dirs as $dir) {
-            $this->common_dirs[] = realpath($dir);
-        }
+	/**
+	 * Create a new text parser and set directories to scan for referenced
+	 * class files.
+	 *
+	 * @param string[] $dirs
+	 */
+	public function __construct(array $dirs = [])
+	{
+		foreach ($dirs as $dir) {
+			$this->common_dirs[] = realpath($dir);
+		}
 
-        $this->Preprocessor = new Preprocessor();
-    }
+		$this->Preprocessor = new Preprocessor();
+	}
 
-    /**
-     * Add additional directories to scan for referenced class files.
-     *
-     * @param string[] $dirs
-     */
-    public function addDirs(array $dirs)
-    {
-        foreach ($dirs as $dir) {
-            $this->common_dirs[] = realpath($dir);
-        }
-    }
+	/**
+	 * Add additional directories to scan for referenced class files.
+	 *
+	 * @param string[] $dirs
+	 */
+	public function addDirs(array $dirs)
+	{
+		foreach ($dirs as $dir) {
+			$this->common_dirs[] = realpath($dir);
+		}
+	}
 
-    /**
-     * Parse a text file
-     *
-     * @param string $file
-     * @param string[] $dirs
-     * @param string[] $defines
-     * @return Statement[]
-     */
-    public function parse($file, array $dirs = [], array $defines = [])
-    {
-        return $this->parseText(file_get_contents(realpath($file)), $dirs);
-    }
+	/**
+	 * Parse a text file
+	 *
+	 * @param string $file
+	 * @param string[] $dirs
+	 * @param string[] $defines
+	 * @return Statement[]
+	 */
+	public function parse($file, array $dirs = [], array $defines = [])
+	{
+		return $this->parseText(file_get_contents(realpath($file)), $dirs);
+	}
 
-    /**
-     * Parse plain text
-     *
-     * @param string $text
-     * @param string[] $dirs
-     * @param string[] $defines
-     * @return Statement[]
-     */
-    public function parseText($text, array $dirs = [], array $defines = []): array
-    {
-        $this->Preprocessor->resetDefines();
-        $this->Preprocessor->addDefines($defines);
-        $text = $this->Preprocessor->preprocess($text);
+	/**
+	 * Parse plain text
+	 *
+	 * @param string $text
+	 * @param string[] $dirs
+	 * @param string[] $defines
+	 * @return Statement[]
+	 */
+	public function parseText($text, array $dirs = [], array $defines = []): array
+	{
+		$this->Preprocessor->resetDefines();
+		$this->Preprocessor->addDefines($defines);
+		$text = $this->Preprocessor->preprocess($text);
 
-        $Statements = [];
+		$Statements = [];
 
-        foreach (preg_split('/\\R/m', $text) as $line => $data) {
-            $data = trim($data);
-            $command = self::wordShift($data);
-            if (!empty($command)) {
-                $Statements[] = new Statement($command, $data, null, $line);
-            }
-        }
+		foreach (preg_split('/\\R/m', $text) as $line => $data) {
+			$data = trim($data);
+			$command = self::wordShift($data);
+			if (!empty($command)) {
+				$Statements[] = new Statement($command, $data, null, $line);
+			}
+		}
 
-        return $Statements;
-    }
+		return $Statements;
+	}
 
-    /**
-     * Get the first word from a string and remove it from the string.
-     *
-     * @param string $data
-     * @return boolean|string
-     */
-    private static function wordShift(&$data)
-    {
-        if (preg_match('~^(\S+)\s*(.*)$~', $data, $matches) === 1) {
-            $data = $matches[2];
-            return $matches[1];
-        }
-        return false;
-    }
+	/**
+	 * Get the first word from a string and remove it from the string.
+	 *
+	 * @param string $data
+	 * @return boolean|string
+	 */
+	private static function wordShift(&$data)
+	{
+		if (preg_match('~^(\S+)\s*(.*)$~', $data, $matches) === 1) {
+			$data = $matches[2];
+			return $matches[1];
+		}
+		return false;
+	}
 
 }

SwaggerGen/TypeRegistry.php

Indentation +46 added lines, -46 removed lines

@@ -15,55 +15,55 @@
 class TypeRegistry

 {

 

-    /**

-     * Map of format-name => class-name

-     *

-     * @var array

-     */

-    private $formats = array();

+	/**

+	 * Map of format-name => class-name

+	 *

+	 * @var array

+	 */

+	private $formats = array();

 

-    /**

-     * Add a type name from classname

-     *

-     * @param string $classname

-     */

-    public function add(string $classname): void

-    {

-        if (is_subclass_of($classname, ICustomType::class)) {

-            foreach ($classname::getFormats() as $format) {

-                $this->formats[$format] = $classname;

-            }

-        }

-    }

+	/**

+	 * Add a type name from classname

+	 *

+	 * @param string $classname

+	 */

+	public function add(string $classname): void

+	{

+		if (is_subclass_of($classname, ICustomType::class)) {

+			foreach ($classname::getFormats() as $format) {

+				$this->formats[$format] = $classname;

+			}

+		}

+	}

 

-    /**

-     * Remove type format by explicitly nulling it (disables it)

-     *

-     * @param string $name

-     */

-    public function remove($name)

-    {

-        $this->formats[$name] = null;

-    }

+	/**

+	 * Remove type format by explicitly nulling it (disables it)

+	 *

+	 * @param string $name

+	 */

+	public function remove($name)

+	{

+		$this->formats[$name] = null;

+	}

 

-    /**

-     * Is a type format known?

-     *

-     * @return bool

-     */

-    public function has($name)

-    {

-        return !empty($this->formats[$name]);

-    }

+	/**

+	 * Is a type format known?

+	 *

+	 * @return bool

+	 */

+	public function has($name)

+	{

+		return !empty($this->formats[$name]);

+	}

 

-    /**

-     * Get the format class name

-     *

-     * @return null|string

-     */

-    public function get($name)

-    {

-        return !empty($this->formats[$name]) ? $this->formats[$name] : null;

-    }

+	/**

+	 * Get the format class name

+	 *

+	 * @return null|string

+	 */

+	public function get($name)

+	{

+		return !empty($this->formats[$name]) ? $this->formats[$name] : null;

+	}

 

 }

Braces +10 added lines, -10 removed lines

@@ -12,8 +12,8 @@ 
  * @copyright  2014-2017 Martijn van der Lee

  * @license    https://opensource.org/licenses/MIT MIT

  */

-class TypeRegistry

-{

+class TypeRegistry
+{
 

     /**

      * Map of format-name => class-name

@@ -29,8 +29,8 @@ 
      */

     public function add(string $classname): void

     {

-        if (is_subclass_of($classname, ICustomType::class)) {

-            foreach ($classname::getFormats() as $format) {

+        if (is_subclass_of($classname, ICustomType::class)) {
+            foreach ($classname::getFormats() as $format) {
                 $this->formats[$format] = $classname;

             }

         }

@@ -41,8 +41,8 @@ 
      *

      * @param string $name

      */

-    public function remove($name)

-    {

+    public function remove($name)
+    {
         $this->formats[$name] = null;

     }

 

@@ -51,8 +51,8 @@ 
      *

      * @return bool

      */

-    public function has($name)

-    {

+    public function has($name)
+    {
         return !empty($this->formats[$name]);

     }

 

@@ -61,8 +61,8 @@ 
      *

      * @return null|string

      */

-    public function get($name)

-    {

+    public function get($name)
+    {
         return !empty($this->formats[$name]) ? $this->formats[$name] : null;

     }

 

tests/Base/PHPUnit7.php

Indentation +7 added lines, -7 removed lines

@@ -2,11 +2,11 @@
 
 class Base_PHPUnit7 extends PHPUnit\Framework\TestCase
 {
-    public function expectException(string $exception, string $message = ''): void
-    {
-        parent::expectException($exception);
-        if ($message !== '') {
-            parent::expectExceptionMessage($message);
-        }
-    }
+	public function expectException(string $exception, string $message = ''): void
+	{
+		parent::expectException($exception);
+		if ($message !== '') {
+			parent::expectExceptionMessage($message);
+		}
+	}
 }

tests/Base/PHPUnit5.php

Indentation +9 added lines, -9 removed lines

@@ -2,13 +2,13 @@
 
 class Base_PHPUnit5 extends PHPUnit_Framework_TestCase
 {
-    /**
-     * @param string $exception
-     * @param string $message
-     * @return void
-     */
-    public function expectException($exception, $message = '')
-    {
-        parent::setExpectedException($exception, $message);
-    }
+	/**
+	 * @param string $exception
+	 * @param string $message
+	 * @return void
+	 */
+	public function expectException($exception, $message = '')
+	{
+		parent::setExpectedException($exception, $message);
+	}
 }

tests/Base/PHPUnit6.php

Indentation +12 added lines, -12 removed lines

@@ -2,16 +2,16 @@
 
 class Base_PHPUnit6 extends PHPUnit\Framework\TestCase
 {
-    /**
-     * @param string $exception
-     * @param string $message
-     * @return void
-     */
-    public function expectException($exception, $message = '')
-    {
-        parent::expectException($exception);
-        if ($message !== '') {
-            parent::expectExceptionMessage($message);
-        }
-    }
+	/**
+	 * @param string $exception
+	 * @param string $message
+	 * @return void
+	 */
+	public function expectException($exception, $message = '')
+	{
+		parent::expectException($exception);
+		if ($message !== '') {
+			parent::expectExceptionMessage($message);
+		}
+	}
 }

tests/StatementTest.php

Indentation +22 added lines, -22 removed lines

@@ -3,29 +3,29 @@
 class StatementTest extends SwaggerGen_TestCase
 {
 
-    /**
-     * @covers \SwaggerGen\Statement::__construct
-     */
-    public function testConstructor()
-    {
-        $object = new \SwaggerGen\Statement('command', 'some data');
+	/**
+	 * @covers \SwaggerGen\Statement::__construct
+	 */
+	public function testConstructor()
+	{
+		$object = new \SwaggerGen\Statement('command', 'some data');
 
-        $this->assertInstanceOf('\SwaggerGen\Statement', $object);
-        $this->assertSame('command', $object->getCommand());
-        $this->assertSame('some data', $object->getData());
-    }
+		$this->assertInstanceOf('\SwaggerGen\Statement', $object);
+		$this->assertSame('command', $object->getCommand());
+		$this->assertSame('some data', $object->getData());
+	}
 
-    /**
-     * @covers \SwaggerGen\Statement::__construct
-     */
-    public function testConstructor_File()
-    {
-        $object = new \SwaggerGen\Statement('command', 'some data', 'file', 123);
-        $this->assertInstanceOf('\SwaggerGen\Statement', $object);
+	/**
+	 * @covers \SwaggerGen\Statement::__construct
+	 */
+	public function testConstructor_File()
+	{
+		$object = new \SwaggerGen\Statement('command', 'some data', 'file', 123);
+		$this->assertInstanceOf('\SwaggerGen\Statement', $object);
 
-        $this->assertSame('command', $object->getCommand());
-        $this->assertSame('some data', $object->getData());
-        $this->assertSame('file', $object->getFile());
-        $this->assertSame(123, $object->getLine());
-    }
+		$this->assertSame('command', $object->getCommand());
+		$this->assertSame('some data', $object->getData());
+		$this->assertSame('file', $object->getFile());
+		$this->assertSame(123, $object->getLine());
+	}
 }

tests/bootstrap.php

Indentation +13 added lines, -13 removed lines

@@ -1,10 +1,10 @@ 
 <?php
 
 spl_autoload_register(function ($classname) {
-    $file = dirname(__DIR__) . DIRECTORY_SEPARATOR . str_replace('\\', DIRECTORY_SEPARATOR, $classname) . '.php';
-    if (is_file($file)) {
-        require_once $file;
-    }
+	$file = dirname(__DIR__) . DIRECTORY_SEPARATOR . str_replace('\\', DIRECTORY_SEPARATOR, $classname) . '.php';
+	if (is_file($file)) {
+		require_once $file;
+	}
 });
 
 
@@ -16,20 +16,20 @@ 
 // inbetween there was `expectException()` without type hint
 
 if (class_exists('PHPUnit\Runner\Version')) {
-    $version = PHPUnit\Runner\Version::id();
+	$version = PHPUnit\Runner\Version::id();
 } else if (class_exists('PHPUnit_Runner_Version')) {
-    $version = PHPUnit_Runner_Version::id();
+	$version = PHPUnit_Runner_Version::id();
 } else {
-    throw new Exception('Cannot detect PHPUnit version');
+	throw new Exception('Cannot detect PHPUnit version');
 }
 
 if (version_compare($version, '7.0.0', '>=')) {
-    require dirname(__FILE__) . '/Base/PHPUnit7.php';
-    class_alias('Base_PHPUnit7', 'SwaggerGen_TestCase');
+	require dirname(__FILE__) . '/Base/PHPUnit7.php';
+	class_alias('Base_PHPUnit7', 'SwaggerGen_TestCase');
 } else if (version_compare($version, '6.0.0', '>=')) {
-    require dirname(__FILE__) . '/Base/PHPUnit6.php';
-    class_alias('Base_PHPUnit6', 'SwaggerGen_TestCase');
+	require dirname(__FILE__) . '/Base/PHPUnit6.php';
+	class_alias('Base_PHPUnit6', 'SwaggerGen_TestCase');
 } else {
-    require dirname(__FILE__) . '/Base/PHPUnit5.php';
-    class_alias('Base_PHPUnit5', 'SwaggerGen_TestCase');
+	require dirname(__FILE__) . '/Base/PHPUnit5.php';
+	class_alias('Base_PHPUnit5', 'SwaggerGen_TestCase');
 }

Spacing +1 added lines, -1 removed lines

@@ -1,6 +1,6 @@
 <?php
 
-spl_autoload_register(function ($classname) {
+spl_autoload_register(function($classname) {
     $file = dirname(__DIR__) . DIRECTORY_SEPARATOR . str_replace('\\', DIRECTORY_SEPARATOR, $classname) . '.php';
     if (is_file($file)) {
         require_once $file;

tests/output/OutputTest.php

Indentation +52 added lines, -52 removed lines

@@ -3,57 +3,57 @@
 class OutputTest extends SwaggerGen_TestCase
 {
 
-    /**
-     * @covers       \SwaggerGen\Exception::__construct
-     * @dataProvider provideAllCases
-     */
-    public function testAllCases($name, $files, $expected)
-    {
-        $SwaggerGen = new \SwaggerGen\SwaggerGen('example.com', '/base');
-        $actual = $SwaggerGen->getSwagger($files, array(), \SwaggerGen\SwaggerGen::FORMAT_JSON);
-        $this->assertJsonStringEqualsJsonString($expected, $this->normalizeJson($actual), $name);
-    }
-
-    /**
-     * Normalizes and pretty-prints json (whitespace mostly)
-     *
-     * This is useful to get better diff results when assertions fail.
-     *
-     * @param string $json
-     * @return string
-     */
-    private function normalizeJson($json)
-    {
-        return json_encode(
-            json_decode($json),
-            PHP_VERSION_ID >= 50400 ? constant('JSON_PRETTY_PRINT') : 0
-        );
-    }
-
-    public function provideAllCases()
-    {
-        $cases = array();
-
-        foreach (glob(__DIR__ . '/*', GLOB_ONLYDIR) as $dir) {
-            $path = realpath($dir);
-            $json = $this->normalizeJson(file_get_contents($path . '/expected.json'));
-
-            $files = array();
-            if (file_exists($path . '/source.php')) {
-                $files[] = $path . '/source.php';
-            }
-            if (file_exists($path . '/source.txt')) {
-                $files[] = $path . '/source.txt';
-            }
-
-            $cases[] = array(
-                basename($dir),
-                $files,
-                $json
-            );
-        }
-
-        return $cases;
-    }
+	/**
+	 * @covers       \SwaggerGen\Exception::__construct
+	 * @dataProvider provideAllCases
+	 */
+	public function testAllCases($name, $files, $expected)
+	{
+		$SwaggerGen = new \SwaggerGen\SwaggerGen('example.com', '/base');
+		$actual = $SwaggerGen->getSwagger($files, array(), \SwaggerGen\SwaggerGen::FORMAT_JSON);
+		$this->assertJsonStringEqualsJsonString($expected, $this->normalizeJson($actual), $name);
+	}
+
+	/**
+	 * Normalizes and pretty-prints json (whitespace mostly)
+	 *
+	 * This is useful to get better diff results when assertions fail.
+	 *
+	 * @param string $json
+	 * @return string
+	 */
+	private function normalizeJson($json)
+	{
+		return json_encode(
+			json_decode($json),
+			PHP_VERSION_ID >= 50400 ? constant('JSON_PRETTY_PRINT') : 0
+		);
+	}
+
+	public function provideAllCases()
+	{
+		$cases = array();
+
+		foreach (glob(__DIR__ . '/*', GLOB_ONLYDIR) as $dir) {
+			$path = realpath($dir);
+			$json = $this->normalizeJson(file_get_contents($path . '/expected.json'));
+
+			$files = array();
+			if (file_exists($path . '/source.php')) {
+				$files[] = $path . '/source.php';
+			}
+			if (file_exists($path . '/source.txt')) {
+				$files[] = $path . '/source.txt';
+			}
+
+			$cases[] = array(
+				basename($dir),
+				$files,
+				$json
+			);
+		}
+
+		return $cases;
+	}
 
 }