Merge pull request #1427 from nelmio/issue

Fix alternative names issue
This commit is contained in:
Guilhem N 2018-11-04 12:28:46 +01:00 committed by GitHub
commit 8839085239
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 11 additions and 6 deletions

View File

@ -98,13 +98,14 @@ final class ModelRegistry
$this->api->getDefinitions()->set($name, $schema);
}
}
if (0 === count($this->unregistered)) {
foreach ($this->alternativeNames as $model) {
$this->register($model);
}
$this->alternativeNames = [];
if (empty($this->unregistered) && !empty($this->alternativeNames)) {
foreach ($this->alternativeNames as $model) {
$this->register($model);
}
$this->alternativeNames = [];
$this->registerDefinitions();
}
}

View File

@ -45,7 +45,7 @@ class ObjectModelDescriber implements ModelDescriberInterface, ModelRegistryAwar
$class = $model->getType()->getClassName();
$context = [];
if (null !== $model->getGroups()) {
$context = ['serializer_groups' => $model->getGroups()];
$context = ['serializer_groups' => array_filter($model->getGroups(), 'is_string')];
}
$annotationsReader = new AnnotationsReader($this->doctrineReader, $this->modelRegistry);

View File

@ -104,6 +104,10 @@ class TestKernel extends Kernel
'serializer' => ['enable_annotations' => true],
]);
$c->loadFromExtension('twig', [
'strict_variables' => '%kernel.debug%',
]);
$c->loadFromExtension('fos_rest', [
'format_listener' => [
'rules' => [