open-orchestra / open-orchestra

tests/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);

tests/OpenOrchestra/Tests/Functional/MediaAdminBundle/Controller/MediaControllerTest.php

Doc Comments -1 removed lines

@@ -25,7 +25,6 @@
     }
 
     /**
-     * @param string $form
      */
     public function testMediaForms()
     {

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
      */
     public function testMediaForms()
     {
-        $url = '/admin/media/' . $this->media->getId();
+        $url = '/admin/media/'.$this->media->getId();
 
         $this->client->request('GET', $url);
 

tests/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);
 

tests/OpenOrchestra/Tests/Functional/MediaAdminBundle/Repository/MediaRepositoryTest.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 +4 added lines, -4 removed lines

@@ -137,10 +137,10 @@
      */
     public function providePaginateConfiguration()
     {
-        $mapping =  array();
-        $conf1 = PaginateFinderConfiguration::generateFromVariable(null , null, null, $mapping, null);
-        $conf2 = PaginateFinderConfiguration::generateFromVariable(null , null, null, $mapping, array('label' => 'dolor', 'language' => 'en'));
-        $conf3 = PaginateFinderConfiguration::generateFromVariable(null , null, null, $mapping, array('type' => 'pdf'));
+        $mapping = array();
+        $conf1 = PaginateFinderConfiguration::generateFromVariable(null, null, null, $mapping, null);
+        $conf2 = PaginateFinderConfiguration::generateFromVariable(null, null, null, $mapping, array('label' => 'dolor', 'language' => 'en'));
+        $conf3 = PaginateFinderConfiguration::generateFromVariable(null, null, null, $mapping, array('type' => 'pdf'));
 
         return array(
             'No criteria'       => array('2', $conf1, 6, 6),

tests/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());
         }
     }

tests/OpenOrchestra/Tests/Functional/GroupBundle/Repository/GroupRepositoryTest.php

Indentation +1 added lines, -1 removed lines

@@ -138,7 +138,7 @@
         $this->assertNull($this->repository->findOneByName('test'));
     }
 
-   /**
+    /**
      * Test remove users
      */
     public function testSoftDeleteGroupBySite()

Spacing +1 added lines, -1 removed lines

@@ -138,7 +138,7 @@
     /**
      * @return array
      */
-    public function provideColumnsAndSearchAndCount(){
+    public function provideColumnsAndSearchAndCount() {
 
         $configuration = new PaginateFinderConfiguration();
         $configuration->setPaginateConfiguration(null, 0, 100, array('label' => 'labels'));

tests/OpenOrchestra/Tests/Functional/UserAdminBundle/Controller/ApiControllersTest.php

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
      */
     public function testApi($url, $method = 'GET')
     {
-        $this->client->request($method, $url . '?access_token=' . $this->getAccessToken());
+        $this->client->request($method, $url.'?access_token='.$this->getAccessToken());
 
         $this->assertEquals(200, $this->client->getResponse()->getStatusCode());
         $this->assertSame('application/json', $this->client->getResponse()->headers->get('content-type'));

tests/OpenOrchestra/Tests/Functional/UserAdminBundle/Controller/FormControllersSecurityTest.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     public function testEditForm()
     {
         $user = $this->client->getContainer()->get('open_orchestra_user.repository.user')->findOneByUsername('p-admin');
-        $this->client->request('GET', '/admin/user/form/' . $user->getId());
+        $this->client->request('GET', '/admin/user/form/'.$user->getId());
         $this->assertEquals(403, $this->client->getResponse()->getStatusCode());
     }
 }

tests/OpenOrchestra/Tests/Functional/UserAdminBundle/Controller/ApiControllersSecurityTest.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
      */
     public function testApi($url, $method = 'GET')
     {
-        $this->client->request($method, $url . '?access_token=' . $this->getAccessToken());
+        $this->client->request($method, $url.'?access_token='.$this->getAccessToken());
 
         $this->assertEquals(403, $this->client->getResponse()->getStatusCode());
     }