Fixed CI

Author: dantleech

Admin/Extension/FrontendLinkExtension.php

@@ -41,7 +41,7 @@ class FrontendLinkExtension extends AdminExtension
     private $translator;
 
     /**
-     * @param RouterInterface $router
+     * @param RouterInterface     $router
      * @param TranslatorInterface $translator
      */
     public function __construct(RouterInterface $router, TranslatorInterface $translator)
@@ -99,16 +99,15 @@ public function configureTabMenu(
                 'uri' => $uri,
                 'attributes' => array(
                     'class' => 'sonata-admin-menu-item',
-                    'role' => 'menuitem'
+                    'role' => 'menuitem',
                 ),
                 'linkAttributes' => array(
                     'class' => 'sonata-admin-frontend-link',
                     'role' => 'button',
                     'target' => '_blank',
-                    'title' => $this->translator->trans('admin.menu_frontend_link_title', array(), 'CmfRoutingBundle')
-                )
+                    'title' => $this->translator->trans('admin.menu_frontend_link_title', array(), 'CmfRoutingBundle'),
+                ),
             )
         );
     }
-
 }

Admin/RedirectRouteAdmin.php

@@ -21,8 +21,9 @@ class RedirectRouteAdmin extends Admin
 {
     protected $translationDomain = 'CmfRoutingBundle';
 
-     /**
-     * Root path for the route parent selection
+    /**
+     * Root path for the route parent selection.
+     *
      * @var string
      */
     protected $routeRoot;
@@ -54,7 +55,7 @@ protected function configureFormFields(FormMapper $formMapper)
     protected function configureDatagridFilters(DatagridMapper $datagridMapper)
     {
         $datagridMapper
-            ->add('name',  'doctrine_phpcr_nodename')
+            ->add('name', 'doctrine_phpcr_nodename')
         ;
     }
 

Admin/RouteAdmin.php

@@ -24,19 +24,22 @@ class RouteAdmin extends Admin
     protected $translationDomain = 'CmfRoutingBundle';
 
     /**
-     * Root path for the route parent selection
+     * Root path for the route parent selection.
+     *
      * @var string
      */
     protected $routeRoot;
 
     /**
-     * Root path for the route content selection
+     * Root path for the route content selection.
+     *
      * @var string
      */
     protected $contentRoot;
 
     /**
-     * Full class name for content that can be referenced by a route
+     * Full class name for content that can be referenced by a route.
+     *
      * @var string
      */
     protected $contentClass;
@@ -163,7 +166,7 @@ protected function configureFieldsForDefaults($dynamicDefaults)
 
         //parse variable pattern and add defaults for tokens - taken from routecompiler
         /** @var $route Route */
-        $route =  $this->subject;
+        $route = $this->subject;
         if ($route && $route->getVariablePattern()) {
             preg_match_all('#\{\w+\}#', $route->getVariablePattern(), $matches, PREG_OFFSET_CAPTURE | PREG_SET_ORDER);
             foreach ($matches as $match) {

CmfRoutingBundle.php

@@ -21,7 +21,7 @@
 use Symfony\Cmf\Bundle\RoutingBundle\DependencyInjection\Compiler\SetRouterPass;
 
 /**
- * Bundle class
+ * Bundle class.
  */
 class CmfRoutingBundle extends Bundle
 {
@@ -92,8 +92,8 @@ private function buildOrmCompilerPass(ContainerBuilder $container)
         $container->addCompilerPass(
             $doctrineOrmCompiler::createXmlMappingDriver(
                 array(
-                    realpath(__DIR__ . '/Resources/config/doctrine-model') => 'Symfony\Cmf\Bundle\RoutingBundle\Model',
-                    realpath(__DIR__ . '/Resources/config/doctrine-orm') => 'Symfony\Cmf\Bundle\RoutingBundle\Doctrine\Orm',
+                    realpath(__DIR__.'/Resources/config/doctrine-model') => 'Symfony\Cmf\Bundle\RoutingBundle\Model',
+                    realpath(__DIR__.'/Resources/config/doctrine-orm') => 'Symfony\Cmf\Bundle\RoutingBundle\Doctrine\Orm',
                 ),
                 array('cmf_routing.dynamic.persistence.orm.manager_name'),
                 'cmf_routing.backend_type_orm',
@@ -107,8 +107,8 @@ private function buildOrmCompilerPass(ContainerBuilder $container)
      * DoctrineBundle (available only since DoctrineBundle 2.4 and Symfony 2.3)
      * Otherwise use the standalone one from CmfCoreBundle.
      *
-     * @return boolean|string the compiler pass to use or false if no suitable
-     *                        one was found
+     * @return bool|string the compiler pass to use or false if no suitable
+     *                     one was found
      */
     private function findDoctrineOrmCompiler()
     {
@@ -138,7 +138,7 @@ private function findDoctrineOrmCompiler()
      */
     private function buildBaseCompilerPass($compilerClass, $driverClass, $type)
     {
-        $arguments = array(array(realpath(__DIR__ . '/Resources/config/doctrine-base')), sprintf('.%s.xml', $type));
+        $arguments = array(array(realpath(__DIR__.'/Resources/config/doctrine-base')), sprintf('.%s.xml', $type));
         $locator = new Definition('Doctrine\Common\Persistence\Mapping\Driver\DefaultFileLocator', $arguments);
         $driver = new Definition($driverClass, array($locator));
 

Controller/RedirectController.php

@@ -13,7 +13,6 @@
 
 use Symfony\Component\Routing\RouterInterface;
 use Symfony\Component\HttpFoundation\RedirectResponse;
-
 use Symfony\Cmf\Component\Routing\RedirectRouteInterface;
 
 /**
@@ -31,6 +30,7 @@ class RedirectController
      * @var RouterInterface
      */
     protected $router;
+
     /**
      * @param RouterInterface $router the router to use to build urls
      */
@@ -40,7 +40,7 @@ public function __construct(RouterInterface $router)
     }
 
     /**
-     * Action to redirect based on a RedirectRouteInterface route
+     * Action to redirect based on a RedirectRouteInterface route.
      *
      * @param RedirectRouteInterface $contentDocument
      *