thirdparty docgen normalizer: fix return type for telephone

This commit is contained in:
Julien Fastré 2022-01-12 09:39:15 +01:00
parent 6b1310148f
commit 9fe744f09f
3 changed files with 15 additions and 6 deletions

View File

@ -11,6 +11,16 @@ and this project adheres to
## Unreleased ## Unreleased
<!-- write down unreleased development here --> <!-- write down unreleased development here -->
## Test releases
### test release 2022-01-12
* fix thirdparty normalizer on telephone field: https://gitlab.com/champs-libres/departement-de-la-vendee/accent-suivi-developpement/-/issues/322
### test release 2022-01-11
* vuejs: translate in French all multiselect widgets * vuejs: translate in French all multiselect widgets
* [address] define address lines according postal standards for France and Belgium (default) and change AddressRender, chill_entity_render_box and AddressRenderBox.vue * [address] define address lines according postal standards for France and Belgium (default) and change AddressRender, chill_entity_render_box and AddressRenderBox.vue
* [household] change translations (champs-libres/departement-de-la-vendee/accent-suivi-developpement#109) * [household] change translations (champs-libres/departement-de-la-vendee/accent-suivi-developpement#109)
@ -44,8 +54,6 @@ and this project adheres to
* List notifications, show, and comment in User section * List notifications, show, and comment in User section
* Notify button and contextual notification box on associated objects pages * Notify button and contextual notification box on associated objects pages
## Test releases
### test release 2021-12-14 ### test release 2021-12-14
* [asideactivity] creation of aside activity category fixed (https://gitlab.com/champs-libres/departement-de-la-vendee/accent-suivi-developpement/-/issues/262) * [asideactivity] creation of aside activity category fixed (https://gitlab.com/champs-libres/departement-de-la-vendee/accent-suivi-developpement/-/issues/262)

View File

@ -221,7 +221,7 @@ class ThirdParty implements TrackCreationInterface, TrackUpdateInterface
* message="Invalid phone number: it should begin with the international prefix starting with ""+"", hold only digits and be smaller than 20 characters. Ex: +33123456789" * message="Invalid phone number: it should begin with the international prefix starting with ""+"", hold only digits and be smaller than 20 characters. Ex: +33123456789"
* ) * )
* @PhonenumberConstraint(type="any") * @PhonenumberConstraint(type="any")
* @Groups({"read", "write", "dogen:read", "docgen:read:3party:parent"}) * @Groups({"read", "write", "docgen:read", "docgen:read:3party:parent"})
*/ */
private ?string $telephone = null; private ?string $telephone = null;
@ -464,10 +464,8 @@ class ThirdParty implements TrackCreationInterface, TrackUpdateInterface
/** /**
* Get telephone. * Get telephone.
*
* @return string|null
*/ */
public function getTelephone() public function getTelephone(): ?string
{ {
return $this->telephone; return $this->telephone;
} }

View File

@ -91,11 +91,14 @@ final class ThirdPartyDocGenNormalizerTest extends KernelTestCase
->setName('test') ->setName('test')
->setCivility((new Civility())->setName(['fr' => 'Monsieur'])->setAbbreviation(['fr' => 'M.'])) ->setCivility((new Civility())->setName(['fr' => 'Monsieur'])->setAbbreviation(['fr' => 'M.']))
->setEmail('info@cl.coop') ->setEmail('info@cl.coop')
->setTelephone('+32486123456')
->addTypesAndCategories('kind') ->addTypesAndCategories('kind')
->addTypesAndCategories((new ThirdPartyCategory())->setName(['fr' => 'category'])); ->addTypesAndCategories((new ThirdPartyCategory())->setName(['fr' => 'category']));
$actual = $this->normalizer->normalize($thirdparty, 'docgen', ['groups' => ['docgen:read']]); $actual = $this->normalizer->normalize($thirdparty, 'docgen', ['groups' => ['docgen:read']]);
$this->assertIsArray($actual); $this->assertIsArray($actual);
$this->assertArrayHasKey('telephone', $actual);
$this->assertEquals('+32486123456', $actual['telephone']);
} }
} }