cross-solution / YAWIK

module/Core/test/CoreTest/Queue/Controller/MongoQueueListControllerTest.php

Indentation +8 added lines, -8 removed lines

@@ -111,7 +111,7 @@ 
     {
         $queueName = 'test';
         $queue = $this->getMockBuilder(MongoQueue::class)->disableOriginalConstructor()
-                      ->setMethods(['listing'])->getMock();
+                        ->setMethods(['listing'])->getMock();
         $this->queueManager->expects($this->once())->method('get')->with($queueName)->willReturn($queue);
         $params = $this->setupParamsMock([
             ['queue', null, $queueName],
@@ -135,13 +135,13 @@ 
         $date = new \DateTime();
         $date->setTimezone(new \DateTimeZone('UTC'));
         $jobs = [
-           [
-               'job' => $job,
-               'created' => new \MongoDb\BSON\UTCDateTime($date->getTimestamp() * 1000),
-               'executed' => new \MongoDb\BSON\UTCDateTime($date->getTimestamp() * 1000),
-               'scheduled' => new \MongoDb\BSON\UTCDateTime($date->getTimestamp() * 1000),
-               'tried' => 10,
-           ],
+            [
+                'job' => $job,
+                'created' => new \MongoDb\BSON\UTCDateTime($date->getTimestamp() * 1000),
+                'executed' => new \MongoDb\BSON\UTCDateTime($date->getTimestamp() * 1000),
+                'scheduled' => new \MongoDb\BSON\UTCDateTime($date->getTimestamp() * 1000),
+                'tried' => 10,
+            ],
         ];
 
         $queue->expects($this->once())->method('listing')->with(['limit' => 10, 'status' => 1])->willReturn($jobs);

Spacing +6 added lines, -6 removed lines

@@ -44,7 +44,7 @@ 
         ],
     ];
 
-    private $inheritance = [ AbstractConsoleController::class ];
+    private $inheritance = [AbstractConsoleController::class];
 
     private function injectConstructorDependencies()
     {
@@ -149,11 +149,11 @@ 
         $lineTmpl = '%-20s %s';
         $console = $this->target->getConsole();
         $console->expects($this->exactly(7))->method('writeLine')->withConsecutive(
-            [ get_class($job) . ' [ test ]' ],
-            [ sprintf($lineTmpl, 'Created', $date->format('Y-m-d H:i:s')) ],
-            [ sprintf($lineTmpl, 'Executed', $date->format('Y-m-d H:i:s')) ],
-            [ sprintf($lineTmpl, 'Scheduled', $date->format('Y-m-d H:i:s')) ],
-            [ sprintf($lineTmpl, 'Tries', $jobs[0]['tried']) ]
+            [get_class($job).' [ test ]'],
+            [sprintf($lineTmpl, 'Created', $date->format('Y-m-d H:i:s'))],
+            [sprintf($lineTmpl, 'Executed', $date->format('Y-m-d H:i:s'))],
+            [sprintf($lineTmpl, 'Scheduled', $date->format('Y-m-d H:i:s'))],
+            [sprintf($lineTmpl, 'Tries', $jobs[0]['tried'])]
         )->willReturn(null);
 
         $this->assertEmpty($this->target->listAction());

module/Core/test/CoreTest/Log/Processor/ProcessIdTest.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
 
     private $target = ProcessId::class;
 
-    private $inheritance = [ ProcessorInterface::class ];
+    private $inheritance = [ProcessorInterface::class];
 
     public function testProcessAddsProcessIdToEventArray()
     {

module/Core/config/module.config.php

Indentation +8 added lines, -8 removed lines

@@ -76,25 +76,25 @@
     'log' => array(
         'Core/Log' => array(
             'writers' => array(
-                 array(
-                     'name' => 'stream',
+                    array(
+                        'name' => 'stream',
                     'priority' => 1000,
                     'options' => array(
-                         'stream' => getcwd().'/var/log/yawik.log',
+                            'stream' => getcwd().'/var/log/yawik.log',
+                    ),
                     ),
-                 ),
             ),
         ),
         'Log/Core/Mail' => array(
             'writers' => array(
-                 array(
-                     'name' => 'stream',
+                    array(
+                        'name' => 'stream',
                     'priority' => 1000,
                     'options' => array(
-                         'stream' => getcwd().'/var/log/mails.log',
+                            'stream' => getcwd().'/var/log/mails.log',
                     ),
 
-                 ),
+                    ),
             ),
         ),
         'Log/Core/Queue' => array(

Spacing +38 added lines, -38 removed lines

@@ -24,7 +24,7 @@ 
 use Core\Service\Tracy;
 use Laminas\I18n\Translator\Resources;
 
-$doctrineConfig = include __DIR__ . '/doctrine.config.php';
+$doctrineConfig = include __DIR__.'/doctrine.config.php';
 
 
 return array(
@@ -76,7 +76,7 @@ 
     ],
 
     'options' => [
-        'Core/MailServiceOptions' => [ 'class' => '\Core\Options\MailServiceOptions' ],
+        'Core/MailServiceOptions' => ['class' => '\Core\Options\MailServiceOptions'],
     ],
 
     'Core' => array(
@@ -139,7 +139,7 @@ 
                     'name' => 'stream',
                     'priority' => 1000,
                     'options' => [
-                        'stream' => getcwd() . '/var/log/mailqueue.log',
+                        'stream' => getcwd().'/var/log/mailqueue.log',
                         'formatter'  => [
                             'name' => 'simple',
                             'options' => [
@@ -375,19 +375,19 @@ 
             'modules/Core/jsonEntityHydrator' => 'Core\Entity\Hydrator\JsonEntityHydratorFactory',
             'Core/EntityHydrator' => 'Core\Entity\Hydrator\EntityHydratorFactory',
             'Core/Options' => 'Core\Factory\ModuleOptionsFactory',
-            'Core/DoctrineMongoODM/RepositoryEvents' => [\Core\Repository\DoctrineMongoODM\Event\RepositoryEventsSubscriber::class,'factory'],
-            'DefaultListeners' => ['Core\Listener\DefaultListener','factory'],
-            'templateProviderStrategy'   => ['Core\Form\Hydrator\Strategy\TemplateProviderStrategy','factory'],
-            'Core/Listener/DeferredListenerAggregate' => [\Core\Listener\DeferredListenerAggregate::class,'factory'],
+            'Core/DoctrineMongoODM/RepositoryEvents' => [\Core\Repository\DoctrineMongoODM\Event\RepositoryEventsSubscriber::class, 'factory'],
+            'DefaultListeners' => ['Core\Listener\DefaultListener', 'factory'],
+            'templateProviderStrategy'   => ['Core\Form\Hydrator\Strategy\TemplateProviderStrategy', 'factory'],
+            'Core/Listener/DeferredListenerAggregate' => [\Core\Listener\DeferredListenerAggregate::class, 'factory'],
             'Core/Listener/CreatePaginator' => 'Core\Listener\CreatePaginatorListener::factory',
             'Core/Locale' => 'Core\I18n\LocaleFactory',
             \Core\Listener\AjaxRouteListener::class => \Core\Factory\Listener\AjaxRouteListenerFactory::class,
             \Core\Listener\DeleteImageSetListener::class => \Core\Factory\Listener\DeleteImageSetListenerFactory::class,
             'Imagine' => \Core\Factory\Service\ImagineFactory::class,
-            'Core/Listener/Notification' => [\Core\Listener\NotificationListener::class,'factory'],
-            'Tracy' => [Tracy::class,'factory'],
+            'Core/Listener/Notification' => [\Core\Listener\NotificationListener::class, 'factory'],
+            'Tracy' => [Tracy::class, 'factory'],
             Service\EntityEraser\DefaultEntityLoaderListener::class => Service\EntityEraser\DefaultEntityLoaderListenerFactory::class,
-            ClearCacheService::class => [ClearCacheService::class,'factory'],
+            ClearCacheService::class => [ClearCacheService::class, 'factory'],
             Listener\ModuleVersionAdminWidgetProvider::class => Listener\ModuleVersionAdminWidgetProviderFactory::class,
             Queue\Worker\MongoWorker::class => \SlmQueue\Factory\WorkerFactory::class,
             'Core/HtmlPurifier' => \Core\Factory\Service\HtmlPurifierFactory::class,
@@ -415,7 +415,7 @@ 
         'translation_file_patterns' => array(
             [
                 'type' => 'gettext',
-                'base_dir' => __DIR__ . '/../language',
+                'base_dir' => __DIR__.'/../language',
                 'pattern' => '%s.mo',
             ],
             [
@@ -460,8 +460,8 @@ 
             'Core/File'    => FileControllerFactory::class,
             'Core/Content' => LazyControllerFactory::class,
             Controller\Console\PurgeController::class => Controller\Console\PurgeControllerFactory::class,
-            AssetsInstallController::class => [AssetsInstallController::class,'factory'],
-            ClearCacheController::class => [ClearCacheController::class,'factory'],
+            AssetsInstallController::class => [AssetsInstallController::class, 'factory'],
+            ClearCacheController::class => [ClearCacheController::class, 'factory'],
             Queue\Controller\MongoQueueController::class => Queue\Controller\MongoQueueControllerFactory::class,
             Queue\Controller\MongoQueueListController::class => Queue\Controller\MongoQueueListControllerFactory::class,
 
@@ -476,9 +476,9 @@ 
             'Core/SearchForm' => 'Core\Factory\Controller\Plugin\SearchFormFactory',
             'listquery' => 'Core\Controller\Plugin\ListQuery::factory',
             'mail' => 'Core\Controller\Plugin\Mail::factory',
-            'Core/Mailer' => ['Core\Controller\Plugin\Mailer','factory'],
-            'Core/CreatePaginator' => [\Core\Controller\Plugin\CreatePaginator::class,'factory'],
-            'Core/PaginatorService' => [\Core\Controller\Plugin\CreatePaginatorService::class,'factory'],
+            'Core/Mailer' => ['Core\Controller\Plugin\Mailer', 'factory'],
+            'Core/CreatePaginator' => [\Core\Controller\Plugin\CreatePaginator::class, 'factory'],
+            'Core/PaginatorService' => [\Core\Controller\Plugin\CreatePaginatorService::class, 'factory'],
             Controller\Plugin\EntityEraser::class => Controller\Plugin\EntityEraserFactory::class,
         ),
         'invokables' => array(
@@ -509,32 +509,32 @@ 
         'exception_template' => 'error/index',
         // Map template to files. Speeds up the lookup through the template stack.
         'template_map' => array(
-            'noscript-notice' => __DIR__ . '/../view/layout/_noscript-notice.phtml',
-            'layout/layout' => __DIR__ . '/../view/layout/layout.phtml',
-            'error/404' => __DIR__ . '/../view/error/404.phtml',
-            'error/403' => __DIR__ . '/../view/error/403.phtml',
-            'error/index' => __DIR__ . '/../view/error/index.phtml',
-            'main-navigation' => __DIR__ . '/../view/partial/main-navigation.phtml',
-            'pagination-control' => __DIR__ . '/../view/partial/pagination-control.phtml',
-            'core/loading-popup' => __DIR__ . '/../view/partial/loading-popup.phtml',
-            'core/notifications' => __DIR__ . '/../view/partial/notifications.phtml',
-            'form/core/buttons' => __DIR__ . '/../view/form/buttons.phtml',
-            'core/social-buttons' => __DIR__ . '/../view/partial/social-buttons.phtml',
-            'form/core/privacy' => __DIR__ . '/../view/form/privacy.phtml',
-            'core/form/permissions-fieldset' => __DIR__ . '/../view/form/permissions-fieldset.phtml',
-            'core/form/permissions-collection' => __DIR__ . '/../view/form/permissions-collection.phtml',
-            'core/form/container-view' => __DIR__ . '/../view/form/container.view.phtml',
-            'core/form/tree-manage.view' => __DIR__ . '/../view/form/tree-manage.view.phtml',
-            'core/form/tree-manage.form' => __DIR__ . '/../view/form/tree-manage.form.phtml',
-            'core/form/tree-add-item' => __DIR__ . '/../view/form/tree-add-item.phtml',
-            'mail/header' =>  __DIR__ . '/../view/mail/header.phtml',
-            'mail/footer' =>  __DIR__ . '/../view/mail/footer.phtml',
-            'mail/footer.en' =>  __DIR__ . '/../view/mail/footer.en.phtml',
+            'noscript-notice' => __DIR__.'/../view/layout/_noscript-notice.phtml',
+            'layout/layout' => __DIR__.'/../view/layout/layout.phtml',
+            'error/404' => __DIR__.'/../view/error/404.phtml',
+            'error/403' => __DIR__.'/../view/error/403.phtml',
+            'error/index' => __DIR__.'/../view/error/index.phtml',
+            'main-navigation' => __DIR__.'/../view/partial/main-navigation.phtml',
+            'pagination-control' => __DIR__.'/../view/partial/pagination-control.phtml',
+            'core/loading-popup' => __DIR__.'/../view/partial/loading-popup.phtml',
+            'core/notifications' => __DIR__.'/../view/partial/notifications.phtml',
+            'form/core/buttons' => __DIR__.'/../view/form/buttons.phtml',
+            'core/social-buttons' => __DIR__.'/../view/partial/social-buttons.phtml',
+            'form/core/privacy' => __DIR__.'/../view/form/privacy.phtml',
+            'core/form/permissions-fieldset' => __DIR__.'/../view/form/permissions-fieldset.phtml',
+            'core/form/permissions-collection' => __DIR__.'/../view/form/permissions-collection.phtml',
+            'core/form/container-view' => __DIR__.'/../view/form/container.view.phtml',
+            'core/form/tree-manage.view' => __DIR__.'/../view/form/tree-manage.view.phtml',
+            'core/form/tree-manage.form' => __DIR__.'/../view/form/tree-manage.form.phtml',
+            'core/form/tree-add-item' => __DIR__.'/../view/form/tree-add-item.phtml',
+            'mail/header' =>  __DIR__.'/../view/mail/header.phtml',
+            'mail/footer' =>  __DIR__.'/../view/mail/footer.phtml',
+            'mail/footer.en' =>  __DIR__.'/../view/mail/footer.en.phtml',
             //'startpage' => __DIR__ . '/../view/layout/startpage.phtml',
         ),
         // Where to look for view templates not mapped above
         'template_path_stack' => array(
-            __DIR__ . '/../view',
+            __DIR__.'/../view',
         ),
     ),
     'view_helpers' => array(

module/Core/src/Queue/Strategy/LogStrategy.php

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
         }
 
         if (!is_array($options)) {
-            throw new \InvalidArgumentException('Options must be of type array or ' . LoggerInterface::class);
+            throw new \InvalidArgumentException('Options must be of type array or '.LoggerInterface::class);
         }
 
         parent::__construct($options);
@@ -282,7 +282,7 @@ 
      */
     private function formatJob(JobInterface $job) : string
     {
-        return get_class($job) . ' [ ' . $job->getId() . ' ] ';
+        return get_class($job).' [ '.$job->getId().' ] ';
     }
 
 }

module/Core/src/Queue/Controller/MongoQueueListController.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
 
         $lineTmpl = '%-20s %s';
         foreach ($jobs as $job) {
-            $console->writeLine(get_class($job['job']) . ' [ ' . $job['job']->getId() . ' ]');
+            $console->writeLine(get_class($job['job']).' [ '.$job['job']->getId().' ]');
 
             foreach (['created', 'executed', 'scheduled'] as $key) {
                 $console->writeLine(sprintf($lineTmpl, ucFirst($key), $job[$key]->toDateTime()->format('Y-m-d H:i:s')));

module/Core/src/Queue/LazyJob.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
             $job = $options ? new $service(...$options) : new $service;
 
         } else {
-            throw new FatalJobException('A job with name "' . $service . '" could not be created.');
+            throw new FatalJobException('A job with name "'.$service.'" could not be created.');
         }
 
         $job->setContent($this->getContent());

module/Jobs/src/Queue/FetchExternalImageJob.php

Spacing +9 added lines, -9 removed lines

@@ -58,40 +58,40 @@
         $jobEntity = $this->repository->find($payload['jobId']);
 
         if (!$jobEntity) {
-            return $this->failure('No job entity with the id ' . $payload['jobId'] . ' was found.');
+            return $this->failure('No job entity with the id '.$payload['jobId'].' was found.');
         }
 
         $uri = $jobEntity->getLogoRef();
 
         if (0 !== strpos($uri, 'http')) {
-            $logger->notice('logoRef seems not to be external: ' . $uri);
+            $logger->notice('logoRef seems not to be external: '.$uri);
             return $this->success('Skip fetching for this job.');
         }
 
-        $logger->debug('Trying to fetch image from ' . $uri);
+        $logger->debug('Trying to fetch image from '.$uri);
 
         $client = new Client($uri);
         $response = $client->send();
 
         if (200 != $response->getStatusCode()) {
-            $logger->err('Received status code ' . $response->getStatusCode() . ' when trying to fetch ' . $uri);
+            $logger->err('Received status code '.$response->getStatusCode().' when trying to fetch '.$uri);
 
-            return $this->recoverable('Status code ' . $response->getStatusCode() . ' received.', ['delay' => '+5 minutes']);
+            return $this->recoverable('Status code '.$response->getStatusCode().' received.', ['delay' => '+5 minutes']);
         }
 
         $content = $response->getBody();
         $type = $response->getHeaders()->get('Content-Type')->getFieldValue();
         list(,$ext) = explode('/', $type, 2);
-        $imageName = '/static/Jobs/logos/' . $jobEntity->getId() . '.' . $ext;
+        $imageName = '/static/Jobs/logos/'.$jobEntity->getId().'.'.$ext;
 
         if (false === file_put_contents("public$imageName", $content, FILE_BINARY)) {
-            return $this->failure('Writing image to "public' . $imageName . '" failed.');
+            return $this->failure('Writing image to "public'.$imageName.'" failed.');
         }
 
-        $logger->info('Saved job logo as ' . basename($imageName));
+        $logger->info('Saved job logo as '.basename($imageName));
         $jobEntity->setLogoRef($imageName);
         $this->repository->store($jobEntity);
-        $logger->info('Saved job logo as ' . basename($imageName));
+        $logger->info('Saved job logo as '.basename($imageName));
 
         return $this->success();
     }

module/Jobs/src/Queue/FindJobsWithExternalImageJob.php

Spacing +3 added lines, -3 removed lines

@@ -74,8 +74,8 @@ 
         foreach ($cursor->toArray() as $job) {
             if (0 === strpos($job->getLogoRef(), 'http')) {
                 $queue->push(FetchExternalImageJob::create($job));
-                $logger->debug('Found external image uri: ' . $job->getLogoRef());
-                $logger->info('Pushed fetch image job for Job: '  . $job->getId());
+                $logger->debug('Found external image uri: '.$job->getLogoRef());
+                $logger->info('Pushed fetch image job for Job: '.$job->getId());
             } else {
                 $invalidJobs += 1;
             }
@@ -83,7 +83,7 @@ 
 
         $delay = 0 >= ($cursor->count() - $invalidJobs) ? '+2 hours' : '+5 minutes';
         $queue->push(self::create(), ['delay' => $delay]);
-        $logger->info('Reinserted to fetch more jobs with delay: ' . $delay);
+        $logger->info('Reinserted to fetch more jobs with delay: '.$delay);
 
         return $this->success();
     }

module/Core/src/Queue/MongoQueue.php

Spacing +4 added lines, -4 removed lines

@@ -211,7 +211,7 @@ 
         $time      = microtime(true);
         $micro     = sprintf("%06d", ($time - floor($time)) * 1000000);
         $this->now = new \DateTime(
-            date('Y-m-d H:i:s.' . $micro, $time),
+            date('Y-m-d H:i:s.'.$micro, $time),
             new \DateTimeZone(date_default_timezone_get())
         );
         $now = $this->dateTimeToUTCDateTime($this->now);
@@ -250,12 +250,12 @@ 
      */
     public function listing(array $options = [])
     {
-        $filter = [ 'queue' => $this->getName() ];
+        $filter = ['queue' => $this->getName()];
         if (isset($options['status'])) {
             $filter['status'] = $options['status'];
         }
 
-        $opt = [ 'sort' => [ 'scheduled' => 1, 'priority' => 1] ];
+        $opt = ['sort' => ['scheduled' => 1, 'priority' => 1]];
         if (isset($options['limit'])) {
             $opt['limit'] = $options['limit'];
         }
@@ -336,7 +336,7 @@ 
     {
         $time      = microtime(true);
         $micro     = sprintf("%06d", ($time - floor($time)) * 1000000);
-        $this->now = new \DateTime(date('Y-m-d H:i:s.' . $micro, $time), new \DateTimeZone(date_default_timezone_get()));
+        $this->now = new \DateTime(date('Y-m-d H:i:s.'.$micro, $time), new \DateTimeZone(date_default_timezone_get()));
         $scheduled = isset($options['scheduled']) ? Utils::createDateTime($options['scheduled']) : clone ($this->now);
 
         if (isset($options['delay'])) {