Simperfit
diff --git a/‎src/Symfony/Component/Form/ChoiceList/Factory/PropertyAccessDecorator.php
Lines changed: 14 additions & 0 deletions b/‎src/Symfony/Component/Form/ChoiceList/Factory/PropertyAccessDecorator.php
Lines changed: 14 additions & 0 deletions
@@ -86,6 +86,8 @@ public function createListFromChoices($choices, $value = null)
     {
         if (is_string($value) && !is_callable($value)) {
             $value = new PropertyPath($value);
+        } elseif (is_string($value) && is_callable($value)) {
+            @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);
         }
 
         if ($value instanceof PropertyPath) {
@@ -117,6 +119,8 @@ public function createListFromLoader(ChoiceLoaderInterface $loader, $value = nul
     {
         if (is_string($value) && !is_callable($value)) {
             $value = new PropertyPath($value);
+        } elseif (is_string($value) && is_callable($value)) {
+            @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);
         }
 
         if ($value instanceof PropertyPath) {
@@ -153,6 +157,8 @@ public function createView(ChoiceListInterface $list, $preferredChoices = null,
 
         if (is_string($label) && !is_callable($label)) {
             $label = new PropertyPath($label);
+        }  elseif (is_string($label) && is_callable($label)) {
+            @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);
         }
 
         if ($label instanceof PropertyPath) {
@@ -163,6 +169,8 @@ public function createView(ChoiceListInterface $list, $preferredChoices = null,
 
         if (is_string($preferredChoices) && !is_callable($preferredChoices)) {
             $preferredChoices = new PropertyPath($preferredChoices);
+        } elseif (is_string($preferredChoices) && is_callable($preferredChoices)) {
+            @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);
         }
 
         if ($preferredChoices instanceof PropertyPath) {
@@ -178,6 +186,8 @@ public function createView(ChoiceListInterface $list, $preferredChoices = null,
 
         if (is_string($index) && !is_callable($index)) {
             $index = new PropertyPath($index);
+        } elseif (is_string($index) && is_callable($index)) {
+            @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);
         }
 
         if ($index instanceof PropertyPath) {
@@ -188,6 +198,8 @@ public function createView(ChoiceListInterface $list, $preferredChoices = null,
 
         if (is_string($groupBy) && !is_callable($groupBy)) {
             $groupBy = new PropertyPath($groupBy);
+        } elseif (is_string($groupBy) && is_callable($groupBy)) {
+            @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);
         }
 
         if ($groupBy instanceof PropertyPath) {
@@ -202,6 +214,8 @@ public function createView(ChoiceListInterface $list, $preferredChoices = null,
 
         if (is_string($attr) && !is_callable($attr)) {
             $attr = new PropertyPath($attr);
+        } elseif (is_string($attr) && is_callable($attr)) {
+            @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);
         }
 
         if ($attr instanceof PropertyPath) {
Original file line number	Diff line number	Diff line change
`@@ -86,6 +86,8 @@ public function createListFromChoices($choices, $value = null)`
`86`	`86`	`{`
`87`	`87`	`if (is_string($value) && !is_callable($value)) {`
`88`	`88`	`$value = new PropertyPath($value);`
	`89`	`+ } elseif (is_string($value) && is_callable($value)) {`
	`90`	`+ @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);`
`89`	`91`	`}`
`90`	`92`
`91`	`93`	`if ($value instanceof PropertyPath) {`
`@@ -117,6 +119,8 @@ public function createListFromLoader(ChoiceLoaderInterface $loader, $value = nul`
`117`	`119`	`{`
`118`	`120`	`if (is_string($value) && !is_callable($value)) {`
`119`	`121`	`$value = new PropertyPath($value);`
	`122`	`+ } elseif (is_string($value) && is_callable($value)) {`
	`123`	`+ @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);`
`120`	`124`	`}`
`121`	`125`
`122`	`126`	`if ($value instanceof PropertyPath) {`
`@@ -153,6 +157,8 @@ public function createView(ChoiceListInterface $list, $preferredChoices = null,`
`153`	`157`
`154`	`158`	`if (is_string($label) && !is_callable($label)) {`
`155`	`159`	`$label = new PropertyPath($label);`
	`160`	`+ } elseif (is_string($label) && is_callable($label)) {`
	`161`	`+ @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);`
`156`	`162`	`}`
`157`	`163`
`158`	`164`	`if ($label instanceof PropertyPath) {`
`@@ -163,6 +169,8 @@ public function createView(ChoiceListInterface $list, $preferredChoices = null,`
`163`	`169`
`164`	`170`	`if (is_string($preferredChoices) && !is_callable($preferredChoices)) {`
`165`	`171`	`$preferredChoices = new PropertyPath($preferredChoices);`
	`172`	`+ } elseif (is_string($preferredChoices) && is_callable($preferredChoices)) {`
	`173`	`+ @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);`
`166`	`174`	`}`
`167`	`175`
`168`	`176`	`if ($preferredChoices instanceof PropertyPath) {`
`@@ -178,6 +186,8 @@ public function createView(ChoiceListInterface $list, $preferredChoices = null,`
`178`	`186`
`179`	`187`	`if (is_string($index) && !is_callable($index)) {`
`180`	`188`	`$index = new PropertyPath($index);`
	`189`	`+ } elseif (is_string($index) && is_callable($index)) {`
	`190`	`+ @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);`
`181`	`191`	`}`
`182`	`192`
`183`	`193`	`if ($index instanceof PropertyPath) {`
`@@ -188,6 +198,8 @@ public function createView(ChoiceListInterface $list, $preferredChoices = null,`
`188`	`198`
`189`	`199`	`if (is_string($groupBy) && !is_callable($groupBy)) {`
`190`	`200`	`$groupBy = new PropertyPath($groupBy);`
	`201`	`+ } elseif (is_string($groupBy) && is_callable($groupBy)) {`
	`202`	`+ @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);`
`191`	`203`	`}`
`192`	`204`
`193`	`205`	`if ($groupBy instanceof PropertyPath) {`
`@@ -202,6 +214,8 @@ public function createView(ChoiceListInterface $list, $preferredChoices = null,`
`202`	`214`
`203`	`215`	`if (is_string($attr) && !is_callable($attr)) {`
`204`	`216`	`$attr = new PropertyPath($attr);`
	`217`	`+ } elseif (is_string($attr) && is_callable($attr)) {`
	`218`	`+ @trigger_error('Treating strings as callable is deprecated since version 3.1 and will throw an error in 4.0. You should use a "\Closure" instead.', E_USER_DEPRECATED);`
`205`	`219`	`}`
`206`	`220`
`207`	`221`	`if ($attr instanceof PropertyPath) {`