Apply fixes from StyleCI (#128)

dbu · web-flow · commit c8450998a982 · 2017-12-12T16:36:48.000+01:00
[ci skip] [skip ci]
diff --git a/src/Admin/Block/Extension/BlockCacheExtension.php b/src/Admin/Block/Extension/BlockCacheExtension.php
@@ -23,6 +23,7 @@
 class BlockCacheExtension extends AbstractAdminExtension
 {
     private $formGroup;
+
     private $formTab;
 
     public function __construct($formGroup = 'form.group_metadata', $formTab = 'form.tab_general')
diff --git a/src/Admin/Core/Extension/PublishTimePeriodExtension.php b/src/Admin/Core/Extension/PublishTimePeriodExtension.php
@@ -27,6 +27,7 @@ class PublishTimePeriodExtension extends AbstractAdminExtension
      * @var string
      */
     protected $formGroup;
+
     protected $formTab;
 
     /**
diff --git a/src/Admin/Core/Extension/PublishableExtension.php b/src/Admin/Core/Extension/PublishableExtension.php
@@ -27,6 +27,7 @@ class PublishableExtension extends AbstractAdminExtension
      * @var string
      */
     protected $formGroup;
+
     protected $formTab;
 
     /**
diff --git a/src/Admin/Menu/Extension/MenuNodeReferrersExtension.php b/src/Admin/Menu/Extension/MenuNodeReferrersExtension.php
@@ -23,6 +23,7 @@
 class MenuNodeReferrersExtension extends AbstractAdminExtension
 {
     private $formGroup;
+
     private $formTab;
 
     public function __construct($formGroup = 'form.group_menus', $formTab = 'form.tab_menu')
diff --git a/src/Admin/Menu/MenuAdmin.php b/src/Admin/Menu/MenuAdmin.php
@@ -25,7 +25,7 @@ protected function configureFormFields(FormMapper $formMapper)
         parent::configureFormFields($formMapper);
 
         $subject = $this->getSubject();
-        $isNew = $subject->getId() === null;
+        $isNew = null === $subject->getId();
 
         if (!$isNew) {
             $formMapper
diff --git a/src/Admin/Routing/Extension/RouteReferrersExtension.php b/src/Admin/Routing/Extension/RouteReferrersExtension.php
@@ -24,6 +24,7 @@
 class RouteReferrersExtension extends AbstractAdminExtension
 {
     private $formGroup;
+
     private $formTab;
 
     public function __construct($formGroup = 'form.group_routes', $formTab = 'form.tab_routes')
diff --git a/src/Admin/Seo/Extension/SeoContentAdminExtension.php b/src/Admin/Seo/Extension/SeoContentAdminExtension.php
@@ -30,6 +30,7 @@ class SeoContentAdminExtension extends AbstractAdminExtension
      * @var string
      */
     protected $formGroup;
+
     protected $formTab;
 
     /**
diff --git a/tests/Resources/Admin/CoreExtensionsAdmin.php b/tests/Resources/Admin/CoreExtensionsAdmin.php
@@ -14,5 +14,6 @@
 class CoreExtensionsAdmin extends BaseAdmin
 {
     protected $baseRouteName = 'cmf_core_extensions';
+
     protected $baseRoutePattern = 'cmf/core/extensions';
 }
diff --git a/tests/Resources/Admin/MenuContentAdmin.php b/tests/Resources/Admin/MenuContentAdmin.php
@@ -21,6 +21,7 @@
 class MenuContentAdmin extends Admin
 {
     protected $baseRouteName = 'cmf_menu_test_content';
+
     protected $baseRoutePattern = '/cmf/menu-test/content';
 
     protected function configureListFields(ListMapper $listMapper)
diff --git a/tests/Resources/Admin/SeoAwareContentAdmin.php b/tests/Resources/Admin/SeoAwareContentAdmin.php
@@ -14,5 +14,6 @@
 class SeoAwareContentAdmin extends BaseAdmin
 {
     protected $baseRouteName = 'cmf_seo_aware_content';
+
     protected $baseRoutePattern = 'cmf/seo/seoawarecontent';
 }
diff --git a/tests/Resources/DataFixtures/Phpcr/LoadMenuData.php b/tests/Resources/DataFixtures/Phpcr/LoadMenuData.php
@@ -23,6 +23,7 @@
 class LoadMenuData implements FixtureInterface
 {
     protected $menuRoot;
+
     protected $routeRoot;
 
     public function load(ObjectManager $manager)
diff --git a/tests/Resources/Document/MenuContent.php b/tests/Resources/Document/MenuContent.php
@@ -59,6 +59,7 @@ class MenuContent implements MenuNodeReferrersInterface, RouteReferrersReadInter
      * )
      */
     protected $routes;
+
     private $published;
 
     public function __construct()
diff --git a/tests/Unit/DependencyInjection/Factory/ContentAdminFactoryTest.php b/tests/Unit/DependencyInjection/Factory/ContentAdminFactoryTest.php
@@ -24,7 +24,9 @@
 class ContentAdminFactoryTest extends \PHPUnit_Framework_TestCase
 {
     private $factory;
+
     private $container;
+
     private $fileLoader;
 
     protected function setUp()
diff --git a/tests/Unit/Description/SonataEnhancerTest.php b/tests/Unit/Description/SonataEnhancerTest.php
@@ -27,6 +27,7 @@
 class SonataEnhancerTest extends \PHPUnit_Framework_TestCAse
 {
     private $admin;
+
     private $pool;
 
     public function setUp()
@@ -91,6 +92,7 @@ public function provideDescriptionData()
 class TestAdmin extends AbstractAdmin
 {
     protected $baseRouteName = 'std_class';
+
     protected $baseRoutePattern = '_';
 
     public function __toString()

Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,7 @@`
`23`	`23`	`class BlockCacheExtension extends AbstractAdminExtension`
`24`	`24`	`{`
`25`	`25`	`private $formGroup;`
	`26`	`+`
`26`	`27`	`private $formTab;`
`27`	`28`
`28`	`29`	`public function __construct($formGroup = 'form.group_metadata', $formTab = 'form.tab_general')`
Original file line number	Diff line number	Diff line change
`@@ -24,6 +24,7 @@`
`24`	`24`	`class RouteReferrersExtension extends AbstractAdminExtension`
`25`	`25`	`{`
`26`	`26`	`private $formGroup;`
	`27`	`+`
`27`	`28`	`private $formTab;`
`28`	`29`
`29`	`30`	`public function __construct($formGroup = 'form.group_routes', $formTab = 'form.tab_routes')`
Original file line number	Diff line number	Diff line change
`@@ -14,5 +14,6 @@`
`14`	`14`	`class CoreExtensionsAdmin extends BaseAdmin`
`15`	`15`	`{`
`16`	`16`	`protected $baseRouteName = 'cmf_core_extensions';`
	`17`	`+`
`17`	`18`	`protected $baseRoutePattern = 'cmf/core/extensions';`
`18`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,7 @@`
`21`	`21`	`class MenuContentAdmin extends Admin`
`22`	`22`	`{`
`23`	`23`	`protected $baseRouteName = 'cmf_menu_test_content';`
	`24`	`+`
`24`	`25`	`protected $baseRoutePattern = '/cmf/menu-test/content';`
`25`	`26`
`26`	`27`	`protected function configureListFields(ListMapper $listMapper)`