venta / framework

src/Container/src/ArgumentResolver.php

Spacing +2 added lines, -2 removed lines

@@ -50,13 +50,13 @@
     {
         $parameters = $function->getParameters();
 
-        return function (array $arguments = []) use ($function, $parameters) {
+        return function(array $arguments = []) use ($function, $parameters) {
             // Use passed arguments in place of reflected parameters.
             $provided = array_intersect_key($arguments, $parameters);
 
             // Remaining parameters will be resolved by container.
             $remaining = array_diff_key($parameters, $arguments);
-            $resolved = array_map(function (ReflectionParameter $parameter) use ($function) {
+            $resolved = array_map(function(ReflectionParameter $parameter) use ($function) {
 
                 // Recursively resolve function arguments.
                 $class = $parameter->getClass();

src/Container/src/ObjectInflector.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
                     $arguments = $callback($inflection);
 
                     // Wrap calling method with closure to avoid reflecting / resolving each time inflection applied.
-                    $this->inflections[$type][$method] = $inflection = function () use ($method, $arguments) {
+                    $this->inflections[$type][$method] = $inflection = function() use ($method, $arguments) {
                         $this->$method(...$arguments);
                     };
                 }

src/Container/tests/ObjectInflectorTest.php

Indentation +6 added lines, -6 removed lines

@@ -27,13 +27,13 @@
 
         // Defining expectations.
         $resolver->shouldReceive('reflectCallable')
-                 ->with([TestClass::class, 'setValue'])
-                 ->andReturn($reflection)
-                 ->once();
+                    ->with([TestClass::class, 'setValue'])
+                    ->andReturn($reflection)
+                    ->once();
         $resolver->shouldReceive('resolveArguments')
-                 ->with($reflection)
-                 ->andReturn($callback)
-                 ->once();
+                    ->with($reflection)
+                    ->andReturn($callback)
+                    ->once();
 
         // Creating inflector, setting resolver, adding inflection.
         $inflector = new ObjectInflector($resolver);

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
         // Callback to be returned by ArgumentResolver::resolveArguments() method.
         // Expects $arguments array to contain 'value' key to merge.
         // Will return array of arguments for TestClass::setValue() method call.
-        $callback = function (array $arguments) {
+        $callback = function(array $arguments) {
             return array_values(array_merge(['value' => 'to be replaced'], $arguments));
         };
 

src/Container/tests/ArgumentResolverTest.php

Spacing +7 added lines, -7 removed lines

@@ -19,7 +19,7 @@ 
     public function canReflectFunction()
     {
         $resolver = new ArgumentResolver(Mockery::mock(Container::class));
-        $reflection = $resolver->reflectCallable(function () {
+        $reflection = $resolver->reflectCallable(function() {
         });
 
         $this->assertInstanceOf(ReflectionFunction::class, $reflection);
@@ -42,7 +42,7 @@ 
     public function canResolveOptionalArguments()
     {
         $resolver = new ArgumentResolver(Mockery::mock(Container::class));
-        $function = function (string $scalar = 'reso') {
+        $function = function(string $scalar = 'reso') {
             return $scalar . 'lved';
         };
         $closure = $resolver->resolveArguments(new ReflectionFunction($function));
@@ -62,7 +62,7 @@ 
         $container->shouldReceive('has')->with(TestClassContract::class)->andReturn(true)->once();
         $container->shouldReceive('get')->with(TestClassContract::class)->andReturn($mock)->once();
 
-        $function = function (TestClassContract $test) {
+        $function = function(TestClassContract $test) {
             return $test;
         };
 
@@ -81,7 +81,7 @@ 
     public function canResolveWithPassedArguments()
     {
         $resolver = new ArgumentResolver(Mockery::mock(Container::class));
-        $function = function (string $scalar) {
+        $function = function(string $scalar) {
             return $scalar . 'd';
         };
         $closure = $resolver->resolveArguments(new ReflectionFunction($function));
@@ -97,7 +97,7 @@ 
     public function canResolveWithoutArguments()
     {
         $resolver = new ArgumentResolver(Mockery::mock(Container::class));
-        $function = function () {
+        $function = function() {
             return 'resolved';
         };
         $closure = $resolver->resolveArguments(new ReflectionFunction($function));
@@ -118,7 +118,7 @@ 
         $container->shouldReceive('has')->with(TestClassContract::class)->andReturn(false)->once();
         $container->shouldNotReceive('get');
 
-        $function = function (TestClassContract $test) {
+        $function = function(TestClassContract $test) {
             return $test;
         };
 
@@ -137,7 +137,7 @@ 
     public function failsToResolveMandatoryScalarArguments()
     {
         $resolver = new ArgumentResolver(Mockery::mock(Container::class));
-        $function = function (string $scalar) {
+        $function = function(string $scalar) {
             return $scalar . 'd';
         };
         $closure = $resolver->resolveArguments(new ReflectionFunction($function));

src/Container/tests/ContainerTest.php

Indentation +18 added lines, -18 removed lines

@@ -353,11 +353,11 @@ 
     public function canResolveFromInvokableObject()
     {
         $factory = Mockery::mock(TestClassFactory::class)
-                          ->shouldReceive('__invoke')
-                          ->withNoArgs()
-                          ->andReturn(Mockery::mock(TestClassContract::class))
-                          ->once()
-                          ->getMock();
+                            ->shouldReceive('__invoke')
+                            ->withNoArgs()
+                            ->andReturn(Mockery::mock(TestClassContract::class))
+                            ->once()
+                            ->getMock();
 
         $container = new Venta\Container\Container;
         $container->bindFactory(TestClassContract::class, $factory);
@@ -370,11 +370,11 @@ 
     public function canResolveFromObjectMethodArray()
     {
         $factory = Mockery::mock(TestClassFactory::class)
-                          ->shouldReceive('create')
-                          ->withNoArgs()
-                          ->andReturn(Mockery::mock(TestClassContract::class))
-                          ->once()
-                          ->getMock();
+                            ->shouldReceive('create')
+                            ->withNoArgs()
+                            ->andReturn(Mockery::mock(TestClassContract::class))
+                            ->once()
+                            ->getMock();
 
         $container = new Venta\Container\Container;
         $container->bindFactory(TestClassContract::class, [$factory, 'create']);
@@ -462,15 +462,15 @@ 
         $reflection = new ReflectionFunction($closure);
         $resolver = Mockery::mock(\Venta\Contracts\Container\ArgumentResolver::class);
         $resolver->shouldReceive('reflectCallable')
-                 ->with($closure)
-                 ->andReturn($reflection)
-                 ->once();
+                    ->with($closure)
+                    ->andReturn($reflection)
+                    ->once();
         $resolver->shouldReceive('resolveArguments')
-                 ->with($reflection)
-                 ->andReturn(function () {
-                     return [new stdClass()];
-                 })
-                 ->once();
+                    ->with($reflection)
+                    ->andReturn(function () {
+                        return [new stdClass()];
+                    })
+                    ->once();
 
         $inflector = Mockery::mock(ObjectInflector::class);
         $inflector->shouldReceive('applyInflections');

Spacing +11 added lines, -11 removed lines

@@ -95,7 +95,7 @@ 
         $object = new stdClass();
         $object->key = 'value';
         $container->bindInstance(stdClass::class, $object);
-        $result = $container->call(function (stdClass $dependency) {
+        $result = $container->call(function(stdClass $dependency) {
             return $dependency->key;
         });
 
@@ -166,7 +166,7 @@ 
     {
         $container = new Venta\Container\Container;
         $container->bindInstance(TestClassContract::class, new TestClass(new stdClass));
-        $container->decorate(TestClassContract::class, function ($previous) {
+        $container->decorate(TestClassContract::class, function($previous) {
             return new class($previous) implements TestClassContract
             {
             };
@@ -184,10 +184,10 @@ 
     public function canDecorateFactoryDefinedBinding()
     {
         $container = new Venta\Container\Container;
-        $container->bindFactory(TestClassContract::class, function () {
+        $container->bindFactory(TestClassContract::class, function() {
             return new TestClass(new stdClass);
         });
-        $container->decorate(TestClassContract::class, function ($previous) {
+        $container->decorate(TestClassContract::class, function($previous) {
             return new class($previous) implements TestClassContract
             {
             };
@@ -206,7 +206,7 @@ 
     {
         $container = new Venta\Container\Container;
         $container->bindClass(TestClassContract::class, TestClass::class);
-        $container->decorate(TestClassContract::class, function ($previous) {
+        $container->decorate(TestClassContract::class, function($previous) {
             return new class($previous) implements TestClassContract
             {
             };
@@ -305,7 +305,7 @@ 
     public function canResolveFromClosure()
     {
         $container = new Venta\Container\Container;
-        $container->bindFactory(stdClass::class, function () {
+        $container->bindFactory(stdClass::class, function() {
             return new stdClass;
         });
 
@@ -318,7 +318,7 @@ 
     public function canResolveFromClosureWithArguments()
     {
         $container = new Venta\Container\Container;
-        $container->bindFactory(TestClassContract::class, function (TestClass $class) {
+        $container->bindFactory(TestClassContract::class, function(TestClass $class) {
             return $class;
         });
 
@@ -402,7 +402,7 @@ 
     public function canResolveSharedFromClosure()
     {
         $container = new Venta\Container\Container;
-        $container->bindFactory(stdClass::class, function () {
+        $container->bindFactory(stdClass::class, function() {
             return new stdClass;
         }, true);
 
@@ -456,7 +456,7 @@ 
      */
     public function savesReflectionResolveResult()
     {
-        $closure = function (stdClass $dep) {
+        $closure = function(stdClass $dep) {
             return new TestClass($dep);
         };
         $reflection = new ReflectionFunction($closure);
@@ -467,7 +467,7 @@ 
                  ->once();
         $resolver->shouldReceive('resolveArguments')
                  ->with($reflection)
-                 ->andReturn(function () {
+                 ->andReturn(function() {
                      return [new stdClass()];
                  })
                  ->once();
@@ -501,7 +501,7 @@ 
     public function throwsContainerExceptionIfCantResolve()
     {
         $container = new Venta\Container\Container;
-        $container->bindFactory(TestClassContract::class, function ($someUnresolvableDependency) {
+        $container->bindFactory(TestClassContract::class, function($someUnresolvableDependency) {
         });
         $container->get(TestClassContract::class);
     }

src/Framework/ServiceProvider/RoutingServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
         $this->container->bindClass(RoutePathParserContract::class, RoutePathParser::class, true);
         $this->container->bindClass(FastRouteRouteParser::class, RoutePathParserContract::class);
 
-        $this->container->bindFactory(RouteCollector::class, function (FastRouteRouteParser $routePathParser) {
+        $this->container->bindFactory(RouteCollector::class, function(FastRouteRouteParser $routePathParser) {
             return new RouteCollector($routePathParser, new GroupCountBased);
         }, true);
     }

src/Framework/ServiceProvider/ConsoleServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -25,11 +25,11 @@
     public function boot()
     {
         // todo: refactor along with console package.
-        $this->container->bindFactory(InputInterface::class, function () {
+        $this->container->bindFactory(InputInterface::class, function() {
             return new ArgvInput;
         }, true);
 
-        $this->container->bindFactory(OutputInterface::class, function () {
+        $this->container->bindFactory(OutputInterface::class, function() {
             return new ConsoleOutput;
         }, true);
 

src/Framework/Kernel/Bootstrap/Logging.php

Spacing +2 added lines, -2 removed lines

@@ -23,7 +23,7 @@ 
     public function boot()
     {
         // todo: implement multi-channel configuration.
-        $this->container->bindFactory(LoggerInterface::class, function (Config $config) {
+        $this->container->bindFactory(LoggerInterface::class, function(Config $config) {
 
             $handler = new StreamHandler(
                 $this->kernel->getRootPath() . '/storage/logs/venta.log',
@@ -33,7 +33,7 @@ 
             $handler->pushProcessor(new PsrLogMessageProcessor);
 
             if (!$this->kernel->isCli()) {
-                $handler->pushProcessor(new WebProcessor(null, [ // todo: make list configurable?
+                $handler->pushProcessor(new WebProcessor(null, [// todo: make list configurable?
                     'url' => 'REQUEST_URI',
                     'ip' => 'REMOTE_ADDR',
                     'http_method' => 'REQUEST_METHOD',

src/Framework/tests/HttpApplicationTest.php

Indentation +2 added lines, -2 removed lines

@@ -70,8 +70,8 @@
         $middlewareCollector->shouldReceive('pushMiddleware')->with('name', 'middleware')->once();
         $strategy->shouldReceive('dispatch')->with($route)->andReturn($response)->once();
         $pipeline->shouldReceive('handle')
-                 ->with($request, Mockery::type(Closure::class)
-                 )->andReturnUsing(function ($request, $last) {
+                    ->with($request, Mockery::type(Closure::class)
+                    )->andReturnUsing(function ($request, $last) {
                 return $last($request);
             })->once();
         $emitter->shouldReceive('emit')->with($response)->once();

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
         $strategy->shouldReceive('dispatch')->with($route)->andReturn($response)->once();
         $pipeline->shouldReceive('handle')
                  ->with($request, Mockery::type(Closure::class)
-                 )->andReturnUsing(function ($request, $last) {
+                 )->andReturnUsing(function($request, $last) {
                 return $last($request);
             })->once();
         $emitter->shouldReceive('emit')->with($response)->once();