Merge pull request #221 from symfony-cmf/php-5.6

Php 5.6

Author: ElectricMaxxx

.travis.yml

@@ -1,8 +1,6 @@
 language: php
 
 php:
-  - 5.5
-  - 5.6
   - 7.0
   - hhvm
 
@@ -18,10 +16,11 @@ env:
   global:
     - SYMFONY_DEPRECATIONS_HELPER=42
     - SYMFONY_PHPUNIT_DIR=.phpunit SYMFONY_PHPUNIT_REMOVE="symfony/yaml"
+    - SYMFONY_PHPUNIT_VERSION=5.7
 
 matrix:
   include:
-    - php: 5.5
+    - php: 5.6
       env: COMPOSER_FLAGS="--prefer-lowest" SYMFONY_VERSION=2.8.* SYMFONY_DEPRECATIONS_HELPER=weak
     - php: 7.1
       env: DEPS=dev SYMFONY_VERSION=3.3.*

CHANGELOG.md

@@ -1,6 +1,7 @@
 Changelog
 =========
 
+ * **2017-01-17**: [BC BREAK] Removed DoctrineOrmMappingsPass - all active Doctrine versions contain the mapping pass. Use `Doctrine\Bundle\DoctrineBundle\DependencyInjection\Compiler\DoctrineOrmMappingsPass` instead. 
  * **2017-01-13**: [BC BREAK] Removed the `Slugifier` classes and the
    dependency on `symfony-cmf/slugifier-api`.
  * **2016-12-03**: [BC BREAK] Moved sonata admin related classes and

composer.json

@@ -12,7 +12,7 @@
         }
     ],
     "require": {
-        "php": "^5.5.9|^7.0",
+        "php": "^5.6.0|^7.0",
         "symfony/framework-bundle": "^2.8|^3.0"
     },
     "require-dev": {

src/DependencyInjection/CmfCoreExtension.php

@@ -28,9 +28,9 @@ class NonTranslatableMetadataListener implements EventSubscriber
      */
     public function getSubscribedEvents()
     {
-        return array(
+        return [
             'loadClassMetadata',
-        );
+        ];
     }
 
     /**
@@ -51,7 +51,7 @@ public function loadClassMetadata(LoadClassMetadataEventArgs $eventArgs)
         foreach ($meta->translatableFields as $field) {
             unset($meta->mappings[$field]['translated']);
         }
-        $meta->translatableFields = array();
+        $meta->translatableFields = [];
         if (null !== $meta->localeMapping) {
             unset($meta->mappings[$meta->localeMapping]);
             $meta->localeMapping = null;

src/DependencyInjection/Compiler/DoctrineOrmMappingsPass.php

@@ -14,6 +14,7 @@
 use Doctrine\Common\EventSubscriber;
 use Doctrine\Common\Persistence\Event\LoadClassMetadataEventArgs;
 use Doctrine\ODM\PHPCR\Mapping\ClassMetadata;
+use Symfony\Cmf\Bundle\CoreBundle\Translatable\TranslatableInterface;
 
 /**
  * Metadata listener for when the translations are globally defined.
@@ -40,9 +41,9 @@ public function __construct($translationStrategy)
      */
     public function getSubscribedEvents()
     {
-        return array(
+        return [
             'loadClassMetadata',
-        );
+        ];
     }
 
     /**
@@ -55,7 +56,7 @@ public function loadClassMetadata(LoadClassMetadataEventArgs $eventArgs)
         /** @var $meta ClassMetadata */
         $meta = $eventArgs->getClassMetadata();
 
-        if ($meta->getReflectionClass()->implementsInterface('Symfony\Cmf\Bundle\CoreBundle\Translatable\TranslatableInterface')) {
+        if ($meta->getReflectionClass()->implementsInterface(TranslatableInterface::class)) {
             $meta->setTranslator($this->translationStrategy);
         }
     }

src/Doctrine/Phpcr/NonTranslatableMetadataListener.php

@@ -91,8 +91,8 @@ public function onKernelRequest(GetResponseEvent $event)
      */
     public static function getSubscribedEvents()
     {
-        return array(
-            KernelEvents::REQUEST => array(array('onKernelRequest', 1)),
-        );
+        return [
+            KernelEvents::REQUEST => [['onKernelRequest', 1]],
+        ];
     }
 }

src/Doctrine/Phpcr/TranslatableMetadataListener.php

@@ -54,10 +54,10 @@ public function __construct(RouterInterface $router)
     public function buildView(FormView $view, FormInterface $form, array $options)
     {
         $routes = $options['routes'];
-        $paths = array();
+        $paths = [];
         foreach ($routes as $key => $route) {
             $name = isset($route['name']) ? $route['name'] : null;
-            $parameters = isset($route['parameters']) ? $route['parameters'] : array();
+            $parameters = isset($route['parameters']) ? $route['parameters'] : [];
             $referenceType = isset($route['referenceType']) ? $route['referenceType'] : UrlGeneratorInterface::ABSOLUTE_PATH;
             $paths[$key] = $this->router->generate($name, $parameters, $referenceType);
         }
@@ -75,9 +75,9 @@ public function setDefaultOptions(OptionsResolverInterface $resolver)
 
     public function configureOptions(OptionsResolver $resolver)
     {
-        $resolver->setDefaults(array(
-            'routes' => array(),
-        ));
+        $resolver->setDefaults([
+            'routes' => [],
+        ]);
     }
 
     /**

src/EventListener/PublishWorkflowListener.php

@@ -112,7 +112,7 @@ public function supportsClass($class)
     public function isGranted($attributes, $object = null)
     {
         if (!is_array($attributes)) {
-            $attributes = array($attributes);
+            $attributes = [$attributes];
         }
 
         if (1 === count($attributes)