Merge pull request #71 from larriereguichet/release/v0.4.3

Release/v0.4.3

.scrutinizer.yml

@@ -1,5 +1,6 @@
 tools:
-    external_code_coverage: true
+    external_code_coverage:
+        timeout: 300
 
 filter:
     excluded_paths:

.travis.yml

@@ -1,7 +1,8 @@
 language: php
 
 php:
-    - 5.6
+    # temporary remove php 5.6 from Travis build as composer consumes too much memory and fails the build
+    #- 5.6
     - 7.0
     - hhvm
 
@@ -10,7 +11,7 @@ before_install:
     - cp Tests/Fixtures/app/config/parameters.yml.dist Tests/Fixtures/app/config/parameters.yml
 
 install:
-    - composer install
+    - composer install --profile --optimize-autoloader
 
 script:
   - bin/phpunit --coverage-text --coverage-clover=coverage.clover

Action/Action.php

@@ -161,6 +161,20 @@ public function getConfiguration()
      */
     public function isPaginationRequired()
     {
-        return $this->configuration->getParameter('load_strategy') === AdminInterface::LOAD_STRATEGY_MULTIPLE;
+        return $this
+            ->configuration
+            ->getParameter('load_strategy') === AdminInterface::LOAD_STRATEGY_MULTIPLE;
+    }
+
+    /**
+     * Return true if one or more entities should be loaded ini this action.
+     *
+     * @return bool
+     */
+    public function isLoadingRequired()
+    {
+        return $this
+            ->configuration
+            ->getParameter('load_strategy') !== AdminInterface::LOAD_STRATEGY_NONE;
     }
 }

Action/ActionInterface.php

@@ -72,4 +72,11 @@ public function addField(Field $field);
      * @return bool
      */
     public function isPaginationRequired();
+
+    /**
+     * Return true if the Action requires the loading of one or more entities.
+     *
+     * @return bool
+     */
+    public function isLoadingRequired();
 }