@@ -39,7 +39,7 @@ discard block |
||
| 39 | 39 | 'template' => '@LAGAdmin/Field/actionCollection.html.twig', |
| 40 | 40 | 'actions' => [], |
| 41 | 41 | ]) |
| 42 | - ->setNormalizer('actions', function (Options $options, $value) use ($actionConfiguration, $defaultActions) { |
|
| 42 | + ->setNormalizer('actions', function(Options $options, $value) use ($actionConfiguration, $defaultActions) { |
|
| 43 | 43 | if (!is_array($value) || 0 === count($value)) { |
| 44 | 44 | $value = [ |
| 45 | 45 | 'edit' => [], |
@@ -138,7 +138,7 @@ discard block |
||
| 138 | 138 | '_self', |
| 139 | 139 | '_blank', |
| 140 | 140 | ]) |
| 141 | - ->setNormalizer('route', function (Options $options, $value) use ($actionConfiguration) { |
|
| 141 | + ->setNormalizer('route', function(Options $options, $value) use ($actionConfiguration) { |
|
| 142 | 142 | // route or url should be defined |
| 143 | 143 | if (!$value && !$options->offsetGet('url') && !$options->offsetGet('admin')) { |
| 144 | 144 | throw new InvalidOptionsException( |
@@ -156,7 +156,7 @@ discard block |
||
| 156 | 156 | |
| 157 | 157 | return $value; |
| 158 | 158 | }) |
| 159 | - ->setNormalizer('admin', function (Options $options, $value) { |
|
| 159 | + ->setNormalizer('admin', function(Options $options, $value) { |
|
| 160 | 160 | // if a Admin is defined, an Action should be defined too |
| 161 | 161 | if ($value && !$options->offsetGet('action')) { |
| 162 | 162 | throw new InvalidOptionsException( |
@@ -166,7 +166,7 @@ discard block |
||
| 166 | 166 | |
| 167 | 167 | return $value; |
| 168 | 168 | }) |
| 169 | - ->setNormalizer('parameters', function (Options $options, $values) { |
|
| 169 | + ->setNormalizer('parameters', function(Options $options, $values) { |
|
| 170 | 170 | $cleanedValues = []; |
| 171 | 171 | |
| 172 | 172 | foreach ($values as $name => $method) { |
@@ -32,7 +32,7 @@ |
||
| 32 | 32 | public function __construct( |
| 33 | 33 | RequestStack $requestStack, |
| 34 | 34 | ApplicationConfigurationStorage $applicationConfigurationStorage |
| 35 | - ){ |
|
| 35 | + ) { |
|
| 36 | 36 | $this->requestStack = $requestStack; |
| 37 | 37 | $this->applicationConfiguration = $applicationConfigurationStorage->getConfiguration(); |
| 38 | 38 | } |
@@ -89,7 +89,7 @@ discard block |
||
| 89 | 89 | $eventDispatcher |
| 90 | 90 | ->expects($this->once()) |
| 91 | 91 | ->method('dispatch') |
| 92 | - ->willReturnCallback(function ($eventName, $event) use ($queryBuilder, $admin) { |
|
| 92 | + ->willReturnCallback(function($eventName, $event) use ($queryBuilder, $admin) { |
|
| 93 | 93 | $this->assertEquals(AdminEvents::DOCTRINE_ORM_FILTER, $eventName); |
| 94 | 94 | /** @var DoctrineOrmFilterEvent $event */ |
| 95 | 95 | $this->assertInstanceOf(DoctrineOrmFilterEvent::class, $event); |
@@ -217,7 +217,7 @@ discard block |
||
| 217 | 217 | $requestStack |
| 218 | 218 | ); |
| 219 | 219 | |
| 220 | - $this->assertExceptionRaised(Exception::class, function () use ($provider, $admin) { |
|
| 220 | + $this->assertExceptionRaised(Exception::class, function() use ($provider, $admin) { |
|
| 221 | 221 | $provider->get($admin, 42); |
| 222 | 222 | }); |
| 223 | 223 | } |
@@ -230,7 +230,7 @@ discard block |
||
| 230 | 230 | /** @var RequestStack|MockObject $requestStack */ |
| 231 | 231 | $requestStack = $this->createMock(RequestStack::class); |
| 232 | 232 | |
| 233 | - $entity =new EntityFixture(42); |
|
| 233 | + $entity = new EntityFixture(42); |
|
| 234 | 234 | |
| 235 | 235 | /** @var EntityManagerInterface|MockObject $entityManager */ |
| 236 | 236 | $entityManager = $this->createMock(EntityManagerInterface::class); |
@@ -96,7 +96,7 @@ discard block |
||
| 96 | 96 | $admin = $this->createMock(AdminInterface::class); |
| 97 | 97 | $event = new AdminEvent($admin, $request); |
| 98 | 98 | |
| 99 | - $this->assertExceptionRaised(Exception::class, function () use ($subscriber, $event) { |
|
| 99 | + $this->assertExceptionRaised(Exception::class, function() use ($subscriber, $event) { |
|
| 100 | 100 | $subscriber->handleRequest($event); |
| 101 | 101 | }); |
| 102 | 102 | } |
@@ -106,7 +106,7 @@ discard block |
||
| 106 | 106 | */ |
| 107 | 107 | public function testCreateView() |
| 108 | 108 | { |
| 109 | - list($subscriber, , $viewFactory) = $this->createAdminSubscriberMock(); |
|
| 109 | + list($subscriber,, $viewFactory) = $this->createAdminSubscriberMock(); |
|
| 110 | 110 | $view = $this->createMock(ViewInterface::class); |
| 111 | 111 | |
| 112 | 112 | $actionConfiguration = $this->createMock(ActionConfiguration::class); |
@@ -244,7 +244,7 @@ discard block |
||
| 244 | 244 | */ |
| 245 | 245 | public function testLoadEntitiesWithMultipleStrategy() |
| 246 | 246 | { |
| 247 | - list($subscriber, , , $dataProviderFactory, , , ,) = $this->createAdminSubscriberMock(); |
|
| 247 | + list($subscriber,,, $dataProviderFactory,,,,) = $this->createAdminSubscriberMock(); |
|
| 248 | 248 | |
| 249 | 249 | $actionConfiguration = $this->createMock(ActionConfiguration::class); |
| 250 | 250 | $actionConfiguration |
@@ -314,7 +314,7 @@ discard block |
||
| 314 | 314 | */ |
| 315 | 315 | public function testLoadEntitiesWithUniqueStrategy() |
| 316 | 316 | { |
| 317 | - list($subscriber, , , $dataProviderFactory) = $this->createAdminSubscriberMock(); |
|
| 317 | + list($subscriber,,, $dataProviderFactory) = $this->createAdminSubscriberMock(); |
|
| 318 | 318 | |
| 319 | 319 | $actionConfiguration = $this->createMock(ActionConfiguration::class); |
| 320 | 320 | $actionConfiguration |
@@ -387,7 +387,7 @@ discard block |
||
| 387 | 387 | */ |
| 388 | 388 | public function testLoadEntitiesWithoutIdentifier() |
| 389 | 389 | { |
| 390 | - list($subscriber, , , $dataProviderFactory) = $this->createAdminSubscriberMock(); |
|
| 390 | + list($subscriber,,, $dataProviderFactory) = $this->createAdminSubscriberMock(); |
|
| 391 | 391 | |
| 392 | 392 | $actionConfiguration = $this->createMock(ActionConfiguration::class); |
| 393 | 393 | $actionConfiguration |
@@ -444,7 +444,7 @@ discard block |
||
| 444 | 444 | $request = new Request(); |
| 445 | 445 | $event = new EntityEvent($admin, $request); |
| 446 | 446 | |
| 447 | - $this->assertExceptionRaised(Exception::class, function () use ($subscriber, $event) { |
|
| 447 | + $this->assertExceptionRaised(Exception::class, function() use ($subscriber, $event) { |
|
| 448 | 448 | $subscriber->loadEntities($event); |
| 449 | 449 | }); |
| 450 | 450 | } |
@@ -454,7 +454,7 @@ discard block |
||
| 454 | 454 | */ |
| 455 | 455 | public function testSaveEntity() |
| 456 | 456 | { |
| 457 | - list($subscriber, , , $dataProviderFactory, , $session, $translator) = $this->createAdminSubscriberMock(); |
|
| 457 | + list($subscriber,,, $dataProviderFactory,, $session, $translator) = $this->createAdminSubscriberMock(); |
|
| 458 | 458 | |
| 459 | 459 | $adminConfiguration = $this->createMock(AdminConfiguration::class); |
| 460 | 460 | $adminConfiguration |
@@ -65,12 +65,12 @@ |
||
| 65 | 65 | // initialise database |
| 66 | 66 | if (!self::$isDatabaseCreated) { |
| 67 | 67 | // TODO remove database at the end of the tests |
| 68 | - exec(__DIR__ . '/app/console doctrine:database:create --if-not-exists', $output); |
|
| 69 | - exec(__DIR__ . '/app/console doctrine:schema:update --force', $output); |
|
| 68 | + exec(__DIR__.'/app/console doctrine:database:create --if-not-exists', $output); |
|
| 69 | + exec(__DIR__.'/app/console doctrine:schema:update --force', $output); |
|
| 70 | 70 | |
| 71 | 71 | foreach ($output as $line) { |
| 72 | 72 | // TODO only in verbose mode |
| 73 | - fwrite(STDOUT, $line . "\n"); |
|
| 73 | + fwrite(STDOUT, $line."\n"); |
|
| 74 | 74 | } |
| 75 | 75 | fwrite(STDOUT, "\n"); |
| 76 | 76 | self::$isDatabaseCreated = true; |
@@ -104,7 +104,7 @@ |
||
| 104 | 104 | $logger = new Definition(Logger::class, [ |
| 105 | 105 | 'default' |
| 106 | 106 | ]); |
| 107 | - $session= new Definition(Session::class); |
|
| 107 | + $session = new Definition(Session::class); |
|
| 108 | 108 | |
| 109 | 109 | $entityManager = new Definition(); |
| 110 | 110 | $entityManager->setClass(FakeEntityManager::class); |
@@ -49,7 +49,7 @@ discard block |
||
| 49 | 49 | $this->assertEquals('setConfiguration', $fieldCalls[0][0]); |
| 50 | 50 | |
| 51 | 51 | $this->assertInstanceOf(Reference::class, $fieldCalls[0][1][0]); |
| 52 | - $this->assertEquals('lag.admin.application', (string)$fieldCalls[0][1][0]); |
|
| 52 | + $this->assertEquals('lag.admin.application', (string) $fieldCalls[0][1][0]); |
|
| 53 | 53 | |
| 54 | 54 | } |
| 55 | 55 | |
@@ -73,7 +73,7 @@ discard block |
||
| 73 | 73 | |
| 74 | 74 | $this->assertExceptionRaised( |
| 75 | 75 | InvalidConfigurationException::class, |
| 76 | - function () use ($compilerPass, $containerBuilder) { |
|
| 76 | + function() use ($compilerPass, $containerBuilder) { |
|
| 77 | 77 | // process the compiler pass |
| 78 | 78 | $compilerPass->process($containerBuilder); |
| 79 | 79 | } |
@@ -23,11 +23,11 @@ |
||
| 23 | 23 | $registry->add('my.action', $action); |
| 24 | 24 | |
| 25 | 25 | // an exception SHOULD be thrown if an Action with the same has already been registered |
| 26 | - $this->assertExceptionRaised(Exception::class, function () use ($registry, $action) { |
|
| 26 | + $this->assertExceptionRaised(Exception::class, function() use ($registry, $action) { |
|
| 27 | 27 | $registry->add('my.action', $action); |
| 28 | 28 | }); |
| 29 | 29 | |
| 30 | - $this->assertExceptionRaised(Exception::class, function () use ($registry, $action) { |
|
| 30 | + $this->assertExceptionRaised(Exception::class, function() use ($registry, $action) { |
|
| 31 | 31 | $registry->get('badName'); |
| 32 | 32 | }); |
| 33 | 33 | |
@@ -239,13 +239,13 @@ |
||
| 239 | 239 | $actionRegistry |
| 240 | 240 | ); |
| 241 | 241 | |
| 242 | - $this->assertExceptionRaised(Exception::class, function () use ($actionFactory) { |
|
| 242 | + $this->assertExceptionRaised(Exception::class, function() use ($actionFactory) { |
|
| 243 | 243 | $actionFactory->getActions('test', [ |
| 244 | 244 | 'lol', |
| 245 | 245 | ]); |
| 246 | 246 | }); |
| 247 | 247 | |
| 248 | - $this->assertExceptionRaised(LogicException::class, function () use ($actionFactory) { |
|
| 248 | + $this->assertExceptionRaised(LogicException::class, function() use ($actionFactory) { |
|
| 249 | 249 | $actionFactory->getActions('test', [ |
| 250 | 250 | 'actions' => [ |
| 251 | 251 | 'my-action' => [], |