feature #27429 [PropertyInfo] Auto-enable PropertyInfo component (sroze)
This PR was squashed before being merged into the 4.2-dev branch (closes #27429).
Discussion
----------
[PropertyInfo] Auto-enable PropertyInfo component
| Q | A
| ------------- | ---
| Branch? | master
| Bug fix? | no
| New feature? | yes
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | ø
| License | MIT
| Doc PR | ø
The PropertyInfo component (used by the Serializer to provide more insights from PhpDocs, etc...) is disabled by default, not sure why. This enables it by default when the component is installed.
Commits
-------
06ea72e3b2
[PropertyInfo] Auto-enable PropertyInfo component
This commit is contained in:
commit
c8ce780b9c
@ -21,6 +21,7 @@ use Symfony\Component\Config\Definition\ConfigurationInterface;
|
|||||||
use Symfony\Component\Form\Form;
|
use Symfony\Component\Form\Form;
|
||||||
use Symfony\Component\Lock\Lock;
|
use Symfony\Component\Lock\Lock;
|
||||||
use Symfony\Component\Lock\Store\SemaphoreStore;
|
use Symfony\Component\Lock\Store\SemaphoreStore;
|
||||||
|
use Symfony\Component\PropertyInfo\PropertyInfoExtractorInterface;
|
||||||
use Symfony\Component\Security\Csrf\CsrfTokenManagerInterface;
|
use Symfony\Component\Security\Csrf\CsrfTokenManagerInterface;
|
||||||
use Symfony\Component\Messenger\MessageBusInterface;
|
use Symfony\Component\Messenger\MessageBusInterface;
|
||||||
use Symfony\Component\Serializer\Serializer;
|
use Symfony\Component\Serializer\Serializer;
|
||||||
@ -833,7 +834,7 @@ class Configuration implements ConfigurationInterface
|
|||||||
->children()
|
->children()
|
||||||
->arrayNode('property_info')
|
->arrayNode('property_info')
|
||||||
->info('Property info configuration')
|
->info('Property info configuration')
|
||||||
->canBeEnabled()
|
->{!class_exists(FullStack::class) && interface_exists(PropertyInfoExtractorInterface::class) ? 'canBeDisabled' : 'canBeEnabled'}()
|
||||||
->end()
|
->end()
|
||||||
->end()
|
->end()
|
||||||
;
|
;
|
||||||
|
@ -219,7 +219,7 @@ class ConfigurationTest extends TestCase
|
|||||||
'throw_exception_on_invalid_index' => false,
|
'throw_exception_on_invalid_index' => false,
|
||||||
),
|
),
|
||||||
'property_info' => array(
|
'property_info' => array(
|
||||||
'enabled' => false,
|
'enabled' => !class_exists(FullStack::class),
|
||||||
),
|
),
|
||||||
'router' => array(
|
'router' => array(
|
||||||
'enabled' => false,
|
'enabled' => false,
|
||||||
|
@ -1153,12 +1153,6 @@ abstract class FrameworkExtensionTest extends TestCase
|
|||||||
$this->assertFalse($container->hasDefinition('serializer'));
|
$this->assertFalse($container->hasDefinition('serializer'));
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testPropertyInfoDisabled()
|
|
||||||
{
|
|
||||||
$container = $this->createContainerFromFile('default_config');
|
|
||||||
$this->assertFalse($container->has('property_info'));
|
|
||||||
}
|
|
||||||
|
|
||||||
public function testPropertyInfoEnabled()
|
public function testPropertyInfoEnabled()
|
||||||
{
|
{
|
||||||
$container = $this->createContainerFromFile('property_info');
|
$container = $this->createContainerFromFile('property_info');
|
||||||
|
Reference in New Issue
Block a user