open-orchestra / open-orchestra

src/OpenOrchestra/Tests/Functional/BackofficeBundle/EventSubscriber/BlockFormTypeSubscriberTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -165,7 +165,7 @@ 
     }
 
     /**
-     * @param $data
+     * @param BlockInterface $data
      */
     protected function assertBlock($data)
     {
@@ -177,7 +177,7 @@ 
     /**
      * @param string $condition
      *
-     * @return array
+     * @return string
      */
     protected function replaceKeywordLabelById($condition)
     {

Indentation +17 added lines, -17 removed lines

@@ -48,20 +48,20 @@ 
         $formType =  static::$kernel->getContainer()->get('open_orchestra_backoffice.generate_form_manager')->getFormType($block);
         $form = $this->formFactory->create($formType, $block, array('csrf_protection' => false));
 
-         $form->submit(array(
-             'style' => 'default',
-             'videoType' => 'youtube',
-             'youtubeVideoId' => 'videoId',
-             'youtubeAutoplay' => true,
-         ));
-
-         $this->assertTrue($form->isSynchronized());
-         /** @var BlockInterface $data */
-         $data = $form->getConfig()->getData();
-         $this->assertBlock($data);
-         $this->assertSame('videoId', $data->getAttribute('youtubeVideoId'));
-         $this->assertTrue($data->getAttribute('youtubeAutoplay'));
-         $this->assertFalse($data->getAttribute('youtubeFs'));
+            $form->submit(array(
+                'style' => 'default',
+                'videoType' => 'youtube',
+                'youtubeVideoId' => 'videoId',
+                'youtubeAutoplay' => true,
+            ));
+
+            $this->assertTrue($form->isSynchronized());
+            /** @var BlockInterface $data */
+            $data = $form->getConfig()->getData();
+            $this->assertBlock($data);
+            $this->assertSame('videoId', $data->getAttribute('youtubeVideoId'));
+            $this->assertTrue($data->getAttribute('youtubeAutoplay'));
+            $this->assertFalse($data->getAttribute('youtubeFs'));
     }
 
     /**
@@ -99,9 +99,9 @@ 
             array(ContentListStrategy::NAME, array(
                 'contentNodeId' => 'root',
                 'contentSearch' => array(
-                  'contentType' => 'news',
-                  'choiceType' => 'choice_and',
-                  'keywords' => null,
+                    'contentType' => 'news',
+                    'choiceType' => 'choice_and',
+                    'keywords' => null,
                 ),
                 'characterNumber' => 150,
                 'contentTemplateEnabled' => true,

Spacing +3 added lines, -3 removed lines

@@ -45,7 +45,7 @@ 
         $block->setComponent(VideoStrategy::NAME);
         $block->addAttribute('videoType', 'youtube');
         $block->addAttribute('youtubeFs', true);
-        $formType =  static::$kernel->getContainer()->get('open_orchestra_backoffice.generate_form_manager')->getFormType($block);
+        $formType = static::$kernel->getContainer()->get('open_orchestra_backoffice.generate_form_manager')->getFormType($block);
         $form = $this->formFactory->create($formType, $block, array('csrf_protection' => false));
 
          $form->submit(array(
@@ -74,7 +74,7 @@ 
     {
         $block = new Block();
         $block->setComponent($component);
-        $formType =  static::$kernel->getContainer()->get('open_orchestra_backoffice.generate_form_manager')->getFormType($block);
+        $formType = static::$kernel->getContainer()->get('open_orchestra_backoffice.generate_form_manager')->getFormType($block);
 
         $form = $this->formFactory->create($formType, $block, array('csrf_protection' => false));
 
@@ -128,7 +128,7 @@ 
     {
         $block = new Block();
         $block->setComponent($component);
-        $formType =  static::$kernel->getContainer()->get('open_orchestra_backoffice.generate_form_manager')->getFormType($block);
+        $formType = static::$kernel->getContainer()->get('open_orchestra_backoffice.generate_form_manager')->getFormType($block);
 
         $form = $this->formFactory->create($formType, $block, array('csrf_protection' => false));
         $submittedValue = array_merge(array('style' => 'default'), $value);

src/OpenOrchestra/Tests/Functional/ModelBundle/Repository/ContentRepositoryTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -548,7 +548,7 @@
     /**
      * @param string $condition
      *
-     * @return array
+     * @return string
      */
     protected function replaceKeywordLabelById($condition)
     {

Spacing +26 added lines, -26 removed lines

@@ -196,7 +196,7 @@ 
     {
         return array(
             array('car', ContentRepositoryInterface::CHOICE_AND, 'lorem', 3),
-            array('car',ContentRepositoryInterface::CHOICE_AND, 'sit', 1),
+            array('car', ContentRepositoryInterface::CHOICE_AND, 'sit', 1),
             array('car', ContentRepositoryInterface::CHOICE_AND, 'dolor', 0),
             array('car', ContentRepositoryInterface::CHOICE_AND, 'sit AND lorem', 1),
             array('news', ContentRepositoryInterface::CHOICE_AND, 'lorem', 1),
@@ -337,7 +337,7 @@ 
             'fields.tinimce_test.string_value' => 'attributes.tinimce_test.stringValue',
         );
 
-        $searchTypes = array (
+        $searchTypes = array(
             'attributes.car_name' => 'text',
             'attributes.description' => 'text',
             'attributes.on_market' => 'date',
@@ -349,7 +349,7 @@ 
         $contents = $this->repository->findForPaginateFilterByContentTypeSiteAndLanguage($configuration, $contentType, $siteId, $language, $searchTypes);
         $repositoryCount = $this->repository->countWithFilterAndContentTypeSiteAndLanguage($configuration, $contentType, $siteId, $language, $searchTypes);
 
-        if(!is_null($name)) {
+        if (!is_null($name)) {
             $this->assertEquals($name, $contents[0]->getName());
         }
         $this->assertCount($count, $contents);
@@ -362,16 +362,16 @@ 
     public function provideContentTypeAndPaginateAndSearchAndsiteId()
     {
         return array(
-            1  => array('car', null, array("name" => "name", "dir" => "asc"), '2', 0 ,5 , 'en', 3, 3, '206 3 portes en'),
-            2  => array('car', null, array("name" => "name", "dir" => "desc"), '2', 0 ,5 , 'en', 3, 3, 'R5 3 portes en'),
-            3  => array('car', null, array("name" => "fields.car_name.string_value", "dir" => "asc"), '2', 0 ,5 , 'en', 3, 3, '206 3 portes en'),
-            4  => array('car', null, array("name" => "fields.car_name.string_value", "dir" => "desc"),'2', 0 ,5 , 'en', 3, 3, 'R5 3 portes en'),
-            5  => array('car', null, null, null, 0 ,1 , 'en', 1, 2),
-            6  => array('car', array('attributes.car_name' => '206'), null, '2', 0 , 2 , 'en', 1, 1),
-            7  => array('news', null, null, '2', 0 , 100, 'fr', 4, 4),
-            8  => array('news', null, null, '2', 50 , 100, 'en', 0, 0),
-            9 => array('news', array('name' => 'news'), null, '2', 0 , null, 'fr', 0, 0),
-            10 => array('car', null, null, '2', 0 ,5 , 'en', 3, 3),
+            1  => array('car', null, array("name" => "name", "dir" => "asc"), '2', 0, 5, 'en', 3, 3, '206 3 portes en'),
+            2  => array('car', null, array("name" => "name", "dir" => "desc"), '2', 0, 5, 'en', 3, 3, 'R5 3 portes en'),
+            3  => array('car', null, array("name" => "fields.car_name.string_value", "dir" => "asc"), '2', 0, 5, 'en', 3, 3, '206 3 portes en'),
+            4  => array('car', null, array("name" => "fields.car_name.string_value", "dir" => "desc"), '2', 0, 5, 'en', 3, 3, 'R5 3 portes en'),
+            5  => array('car', null, null, null, 0, 1, 'en', 1, 2),
+            6  => array('car', array('attributes.car_name' => '206'), null, '2', 0, 2, 'en', 1, 1),
+            7  => array('news', null, null, '2', 0, 100, 'fr', 4, 4),
+            8  => array('news', null, null, '2', 50, 100, 'en', 0, 0),
+            9 => array('news', array('name' => 'news'), null, '2', 0, null, 'fr', 0, 0),
+            10 => array('car', null, null, '2', 0, 5, 'en', 3, 3),
         );
     }
 
@@ -523,63 +523,63 @@ 
      */
     protected function getDescriptionColumnEntity()
     {
-        return array (
+        return array(
             'name' =>
-            array (
+            array(
                 'key' => 'name',
                 'field' => 'name',
                 'type' => 'string',
             ),
             'language' =>
-            array (
+            array(
                 'key' => 'language',
                 'field' => 'language',
                 'type' => 'string',
             ),
             'status_label' =>
-            array (
+            array(
                 'key' => 'status_label',
                 'field' => 'status',
                 'type' => 'multiLanguages',
             ),
             'version' =>
-            array (
+            array(
                 'key' => 'version',
                 'field' => 'version',
                 'type' => 'integer',
             ),
             'linked_to_site' =>
-            array (
+            array(
                 'key' => 'linked_to_site',
                 'field' => 'linkedTosite',
                 'type' => 'boolean',
             ),
             'created_by' =>
-            array (
+            array(
                 'key' => 'created_by',
                 'field' => 'createdBy',
                 'type' => 'string',
             ),
             'updated_by' =>
-            array (
+            array(
                 'key' => 'updated_by',
                 'field' => 'updatedBy',
                 'type' => 'string',
             ),
             'created_at' =>
-            array (
+            array(
                 'key' => 'created_at',
                 'field' => 'createdAt',
                 'type' => 'date',
             ),
             'updated_at' =>
-            array (
+            array(
                 'key' => 'updated_at',
                 'field' => 'updatedAt',
                 'type' => 'date',
             ),
             'deleted' =>
-            array (
+            array(
                 'key' => 'deleted',
                 'field' => 'deleted',
                 'type' => 'boolean',
@@ -707,7 +707,7 @@ 
 
         $this->repository->softDeleteContent($contentId);
         $contents = $this->repository->findByContentId($contentId);
-        foreach ($contents as $content ) {
+        foreach ($contents as $content) {
             $this->assertTrue($content->isDeleted());
         }
         $this->repository->restoreDeletedContent($contentId);
@@ -717,7 +717,7 @@ 
         $documentManager->close();
 
         $contents = $this->repository->findByContentId($contentId);
-        foreach ($contents as $content ) {
+        foreach ($contents as $content) {
             $this->assertFalse($content->isDeleted());
         }
     }

src/OpenOrchestra/Tests/Functional/BaseApiBundle/Controller/AuthorizationControllersTest.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
         $tokenReponse = json_decode($this->client->getResponse()->getContent(), true);
         $refreshToken = $tokenReponse['refresh_token'];
 
-        $this->client->request('GET', '/oauth/access_token?grant_type=refresh_token&refresh_token=' . $refreshToken, array(), array(), array('PHP_AUTH_USER' => 'test_key', 'PHP_AUTH_PW' => 'test_secret'));
+        $this->client->request('GET', '/oauth/access_token?grant_type=refresh_token&refresh_token='.$refreshToken, array(), array(), array('PHP_AUTH_USER' => 'test_key', 'PHP_AUTH_PW' => 'test_secret'));
         $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
         $this->assertSame('application/json', $this->client->getResponse()->headers->get('content-type'));
     }

src/OpenOrchestra/Tests/Functional/ApiBundle/Controller/ContentControllerTest.php

Spacing +1 added lines, -1 removed lines

@@ -53,7 +53,7 @@
 
         $this->client->request(
             'POST',
-            '/api/content/' . $content->getId() . '/update',
+            '/api/content/'.$content->getId().'/update',
             array(),
             array(),
             array(),

src/OpenOrchestra/Tests/Functional/ApiBundle/Controller/AuthorizationControllersTest.php

Spacing +3 added lines, -3 removed lines

@@ -47,18 +47,18 @@
         $accessToken = $tokenReponse['access_token'];
         $refreshToken = $tokenReponse['refresh_token'];
 
-        $this->client->request('GET', '/api/node/root?access_token=' . $accessToken);
+        $this->client->request('GET', '/api/node/root?access_token='.$accessToken);
         $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
         $this->assertSame('application/json', $this->client->getResponse()->headers->get('content-type'));
 
-        $this->client->request('GET', '/oauth/access_token?grant_type=refresh_token&refresh_token=' . $refreshToken, array(), array(), array('PHP_AUTH_USER' => 'test_key', 'PHP_AUTH_PW' => 'test_secret'));
+        $this->client->request('GET', '/oauth/access_token?grant_type=refresh_token&refresh_token='.$refreshToken, array(), array(), array('PHP_AUTH_USER' => 'test_key', 'PHP_AUTH_PW' => 'test_secret'));
         $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
         $this->assertSame('application/json', $this->client->getResponse()->headers->get('content-type'));
         $newTokenReponse = json_decode($this->client->getResponse()->getContent(), true);
         $newAccessToken = $newTokenReponse['access_token'];
         $this->assertNotSame($accessToken, $newAccessToken);
 
-        $this->client->request('GET', '/api/node/root?access_token=' . $newAccessToken);
+        $this->client->request('GET', '/api/node/root?access_token='.$newAccessToken);
         $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
         $this->assertSame('application/json', $this->client->getResponse()->headers->get('content-type'));
     }

src/OpenOrchestra/Tests/Functional/BackofficeBundle/DependencyInjection/ContextManagerClassTest.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
      */
     public function testDifferentEnv($classExpected, $env, $debug)
     {
-        $kernel = static::createKernel(array('environment' => $env ,'debug' => $debug));
+        $kernel = static::createKernel(array('environment' => $env, 'debug' => $debug));
         $kernel->boot();
         $this->assertInstanceOf(
             $classExpected,

src/OpenOrchestra/Tests/Functional/BackofficeBundle/Controller/SiteControllerTest.php

Spacing +6 added lines, -6 removed lines

@@ -51,9 +51,9 @@ 
         $this->assertNodeCount(1, 'fr');
         $this->assertNodeCount(0, 'en');
 
-        $crawler = $this->client->request('GET', '/admin/site/form/' . $this->siteId);
+        $crawler = $this->client->request('GET', '/admin/site/form/'.$this->siteId);
         $form = $crawler->selectButton('Save')->form();
-        foreach($form->all() as $key => $value){
+        foreach ($form->all() as $key => $value) {
             if (preg_match('/^oo_site\[aliases\]\[.*\]\[language\]$/', $key)) {
                 $form[$key] = 'en';
             }
@@ -85,14 +85,14 @@ 
      */
     protected function createSite()
     {
-        $crawler =  $this->client->request('GET', '/admin/site/new');
+        $crawler = $this->client->request('GET', '/admin/site/new');
 
         $form = $crawler->selectButton('Save')->form();
         $form['oo_site[siteId]'] = $this->siteId;
-        $form['oo_site[name]'] = $this->siteId . 'domain';
-        foreach($form->all() as $key => $value){
+        $form['oo_site[name]'] = $this->siteId.'domain';
+        foreach ($form->all() as $key => $value) {
             if (preg_match('/^oo_site\[aliases\]\[.*\]\[domain\]$/', $key)) {
-                $form[$key] = $this->siteId . 'name';
+                $form[$key] = $this->siteId.'name';
             }
             if (preg_match('/^oo_site\[aliases\]\[.*\]\[language\]$/', $key)) {
                 $form[$key] = 'fr';

Indentation +2 added lines, -2 removed lines

@@ -114,9 +114,9 @@
      */
     protected function assertNodeCount($count, $language)
     {
-         $nodes = $this->nodeRepository->findNotDeletedSortByUpdatedAt(NodeInterface::ROOT_NODE_ID, $language, $this->siteId);
+            $nodes = $this->nodeRepository->findNotDeletedSortByUpdatedAt(NodeInterface::ROOT_NODE_ID, $language, $this->siteId);
 
-         $this->assertCount($count, $nodes);
+            $this->assertCount($count, $nodes);
     }
 
     /**

src/OpenOrchestra/Tests/Functional/MediaAdminBundle/EventSubscriber/UpdateMediaReferenceSubscriberTest.php

Doc Comments -1 removed lines

@@ -129,7 +129,6 @@
 
     /**
      * @param string $blockType
-     * @param string $id
      *
      * @return Block
      */

Spacing +2 added lines, -2 removed lines

@@ -66,8 +66,8 @@
         $block = $this->generateBlock($blockType['component']);
         $attributes = $block->getAttributes();
         $attributes['pictures'] = array(array('id' => $media->getId(), 'format' => ''));
-        $attributes['id'] = $blockType["component"] . self::ATTRIBUTE_ID_SUFFIX;
-        $method = $blockType["component"] . self::METHOD_SUFFIX;
+        $attributes['id'] = $blockType["component"].self::ATTRIBUTE_ID_SUFFIX;
+        $method = $blockType["component"].self::METHOD_SUFFIX;
         $attributes = $this->$method($attributes);
         $block->setAttributes($attributes);
 

src/OpenOrchestra/Tests/Functional/BackofficeBundle/Controller/EditNodeControllerTest.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
     {
         $nodeDocument = $this->nodeRepository->findInLastVersion($nodeId, $this->language, $this->siteId);
 
-        $url = '/admin/node/form/' . $this->siteId . '/' . $nodeDocument->getNodeId() . '/' . $this->language . '/' . $nodeDocument->getVersion();
+        $url = '/admin/node/form/'.$this->siteId.'/'.$nodeDocument->getNodeId().'/'.$this->language.'/'.$nodeDocument->getVersion();
 
         $crawler = $this->client->request('GET', $url);
         $formNode = $crawler->selectButton('Save')->form();