@@ -14,7 +14,7 @@ |
||
| 14 | 14 | */ |
| 15 | 15 | public static function create($data) : Module |
| 16 | 16 | { |
| 17 | - if(!array_key_exists('namespace', $data) || !array_key_exists('name', $data)){ |
|
| 17 | + if (!array_key_exists('namespace', $data) || !array_key_exists('name', $data)) { |
|
| 18 | 18 | throw new InvalidArgumentException('can not create Module. Not all nessesary data provided'); |
| 19 | 19 | } |
| 20 | 20 | |
@@ -13,8 +13,8 @@ |
||
| 13 | 13 | public function getData(string $json) |
| 14 | 14 | { |
| 15 | 15 | $data = json_decode($json, true); |
| 16 | - if($error = json_last_error() !== JSON_ERROR_NONE){ |
|
| 17 | - throw new RuntimeException( json_last_error_msg() ); |
|
| 16 | + if ($error = json_last_error() !== JSON_ERROR_NONE) { |
|
| 17 | + throw new RuntimeException(json_last_error_msg()); |
|
| 18 | 18 | } |
| 19 | 19 | |
| 20 | 20 | return $data; |
@@ -69,12 +69,12 @@ |
||
| 69 | 69 | private function validate(string $attr, string $attrName) |
| 70 | 70 | { |
| 71 | 71 | // empty |
| 72 | - if(strlen($attr) === 0){ |
|
| 72 | + if (strlen($attr) === 0) { |
|
| 73 | 73 | throw new InvalidArgumentException(sprintf('%s can not be empty', $attrName)); |
| 74 | 74 | } |
| 75 | 75 | |
| 76 | 76 | // whitespaces |
| 77 | - if(preg_match('/\s/', $attr)){ |
|
| 77 | + if (preg_match('/\s/', $attr)) { |
|
| 78 | 78 | throw new InvalidArgumentException(sprintf('illigal %s', $attrName)); |
| 79 | 79 | } |
| 80 | 80 | } |
@@ -42,13 +42,13 @@ |
||
| 42 | 42 | { |
| 43 | 43 | $modules = array(); |
| 44 | 44 | |
| 45 | - try{ |
|
| 45 | + try { |
|
| 46 | 46 | $modulesData = $this->storage->read(); |
| 47 | - }catch(FileNotFoundException $e){ |
|
| 47 | + } catch (FileNotFoundException $e) { |
|
| 48 | 48 | return []; |
| 49 | 49 | } |
| 50 | 50 | |
| 51 | - foreach($modulesData as $moduleData) |
|
| 51 | + foreach ($modulesData as $moduleData) |
|
| 52 | 52 | { |
| 53 | 53 | $modules[] = $this->mapper->getModule($moduleData); |
| 54 | 54 | } |
@@ -44,7 +44,7 @@ |
||
| 44 | 44 | |
| 45 | 45 | try{ |
| 46 | 46 | $modulesData = $this->storage->read(); |
| 47 | - }catch(FileNotFoundException $e){ |
|
| 47 | + } catch(FileNotFoundException $e){ |
|
| 48 | 48 | return []; |
| 49 | 49 | } |
| 50 | 50 | |
@@ -60,20 +60,20 @@ |
||
| 60 | 60 | public function testConstructorValidation($namespace, $name, $expectedExceptionMsg) |
| 61 | 61 | { |
| 62 | 62 | $this->expectExceptionMessage($expectedExceptionMsg); |
| 63 | - new Module($namespace,$name); |
|
| 63 | + new Module($namespace, $name); |
|
| 64 | 64 | } |
| 65 | 65 | |
| 66 | 66 | public function constructorDataProvider() |
| 67 | 67 | { |
| 68 | 68 | return [ |
| 69 | - ['', 'MyClass', 'namespace can not be empty'], // empty namespace |
|
| 70 | - ['My Namespace', 'MyClass', 'illigal namespace'], // space in namespace |
|
| 71 | - ["My\tNamespace", 'MyClass', 'illigal namespace'], // tab in namespace |
|
| 69 | + ['', 'MyClass', 'namespace can not be empty'], // empty namespace |
|
| 70 | + ['My Namespace', 'MyClass', 'illigal namespace'], // space in namespace |
|
| 71 | + ["My\tNamespace", 'MyClass', 'illigal namespace'], // tab in namespace |
|
| 72 | 72 | |
| 73 | - ['My\Namespace', '', 'name can not be empty'], // empty name |
|
| 74 | - ['My\Namespace', 'My Class', 'illigal name'], // space in name |
|
| 75 | - ['My\Namespace', "My\tClass", 'illigal name'], // tab in name |
|
| 76 | - ['My\Namespace', 'MyClass ', 'illigal name'], // space at the end of name |
|
| 73 | + ['My\Namespace', '', 'name can not be empty'], // empty name |
|
| 74 | + ['My\Namespace', 'My Class', 'illigal name'], // space in name |
|
| 75 | + ['My\Namespace', "My\tClass", 'illigal name'], // tab in name |
|
| 76 | + ['My\Namespace', 'MyClass ', 'illigal name'], // space at the end of name |
|
| 77 | 77 | ]; |
| 78 | 78 | } |
| 79 | 79 | } |
@@ -14,9 +14,9 @@ |
||
| 14 | 14 | { |
| 15 | 15 | $container = new Container(); |
| 16 | 16 | |
| 17 | - $container['filesystem_adapter'] = function($c){ return new MemoryAdapter(); }; |
|
| 18 | - $container['filesystem'] = function($c){ return new Filesystem($c['filesystem_adapter']); }; |
|
| 19 | - $container['config'] = function($c){ return new Config($c['filesystem']); }; |
|
| 17 | + $container['filesystem_adapter'] = function($c) { return new MemoryAdapter(); }; |
|
| 18 | + $container['filesystem'] = function($c) { return new Filesystem($c['filesystem_adapter']); }; |
|
| 19 | + $container['config'] = function($c) { return new Config($c['filesystem']); }; |
|
| 20 | 20 | |
| 21 | 21 | $app = new App($container); |
| 22 | 22 | |
@@ -29,8 +29,8 @@ |
||
| 29 | 29 | $repo = $this->getContainer()->get('moduleRepo'); |
| 30 | 30 | |
| 31 | 31 | /** @var Module $module */ |
| 32 | - foreach($repo->getAll() as $module){ |
|
| 33 | - $moduleClass = $module->getNamespace() . '\\' . $module->getName(); |
|
| 32 | + foreach ($repo->getAll() as $module) { |
|
| 33 | + $moduleClass = $module->getNamespace().'\\'.$module->getName(); |
|
| 34 | 34 | new $moduleClass($this); |
| 35 | 35 | } |
| 36 | 36 | } |
@@ -22,10 +22,10 @@ |
||
| 22 | 22 | |
| 23 | 23 | public function read() : array |
| 24 | 24 | { |
| 25 | - try{ |
|
| 25 | + try { |
|
| 26 | 26 | $json = $this->filesystem->read($this->filename); |
| 27 | 27 | } |
| 28 | - catch(FileNotFoundException $e) |
|
| 28 | + catch (FileNotFoundException $e) |
|
| 29 | 29 | { |
| 30 | 30 | $json = "{}"; |
| 31 | 31 | } |
@@ -24,8 +24,7 @@ |
||
| 24 | 24 | { |
| 25 | 25 | try{ |
| 26 | 26 | $json = $this->filesystem->read($this->filename); |
| 27 | - } |
|
| 28 | - catch(FileNotFoundException $e) |
|
| 27 | + } catch(FileNotFoundException $e) |
|
| 29 | 28 | { |
| 30 | 29 | $json = "{}"; |
| 31 | 30 | } |