minor #18664 [Form] [DoctrineBridge] updated changelogs after #18359 …

…(HeahDude) This PR was merged into the 3.1-dev branch. Discussion ---------- [Form] [DoctrineBridge] updated changelogs after #18359 | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | ~ | License | MIT | Doc PR | ~ follows #18359. Commits ------- 1c8dc9d [Form] [DoctrineBridge] updated changelogs after #18359
symfony · Apr 29, 2016 · 352997e · 352997e
2 parents 731d7a6 + 1c8dc9d
commit 352997e
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 7 deletions.
diff --git a/UPGRADE-3.1.md b/UPGRADE-3.1.md
@@ -39,7 +39,10 @@ Form
    'choice_label' => function ($choice) {
        return strtoupper($choice);
    },
-   ```   
+   ```
+
+ * Caching of the loaded `ChoiceListInterface` in the `LazyChoiceList` has been deprecated,
+   it must be cached in the `ChoiceLoaderInterface` implementation instead.
 
 FrameworkBundle
 ---------------

diff --git a/UPGRADE-4.0.md b/UPGRADE-4.0.md
@@ -36,6 +36,8 @@ Form
    },
    ```
 
+ * Caching of the loaded `ChoiceListInterface` in the `LazyChoiceList` has been removed,
+   it must be cached in the `ChoiceLoaderInterface` implementation instead.
 
 FrameworkBundle
 ---------------

diff --git a/src/Symfony/Bridge/Doctrine/CHANGELOG.md b/src/Symfony/Bridge/Doctrine/CHANGELOG.md
@@ -7,6 +7,8 @@ CHANGELOG
  * added "{{ value }}" message placeholder to UniqueEntityValidator
  * deprecated `MergeDoctrineCollectionListener::onBind` in favor of
    `MergeDoctrineCollectionListener::onSubmit`
+ * deprecated passing `ChoiceListFactoryInterface` as first argument of
+   `DoctrineChoiceLoader`'s constructor
 
 3.0.0
 -----

diff --git a/src/Symfony/Bridge/Doctrine/Form/ChoiceList/DoctrineChoiceLoader.php b/src/Symfony/Bridge/Doctrine/Form/ChoiceList/DoctrineChoiceLoader.php
@@ -24,11 +24,6 @@
  */
 class DoctrineChoiceLoader implements ChoiceLoaderInterface
 {
-    /**
-     * @var ChoiceListFactoryInterface
-     */
-    private $factory;
-
     /**
      * @var ObjectManager
      */
@@ -78,7 +73,6 @@ public function __construct($manager, $class, $idReader = null, $objectLoader =
 
             // Provide a BC layer since $factory has changed
             // form first to last argument as of 3.1
-            $this->factory = $manager;
             $manager = $class;
             $class = $idReader;
             $objectLoader = $factory;

diff --git a/src/Symfony/Component/Form/CHANGELOG.md b/src/Symfony/Component/Form/CHANGELOG.md
@@ -10,6 +10,7 @@ CHANGELOG
  * Using callable strings as choice options in `ChoiceType` has been deprecated
    and will be used as `PropertyPath` instead of callable in Symfony 4.0.
  * implemented `DataTransformerInterface` in `TextType`
+ * deprecated caching loaded choice list in `LazyChoiceList::$loadedList`
 
 3.0.0
 -----