0c9b563cae
Auto Tag Develop / tag (push) Successful in 9s
**MR stackée sur ERP-54** — cible = `feature/ERP-54-creer-entites-client-m1` (PAS `develop`). Tristan validera le stack en fin de chaîne. Branche l'API REST du répertoire clients (M1) sur l'entité `Client` d'ERP-54. ## Périmètre - **ClientProvider** : liste paginée (Paginator ORM aligné ERP-72, `?pagination=false`), exclusion archives+soft-delete par défaut (RG-1.24), `?includeArchived=true` (RG-1.25), tri `companyName ASC` (RG-1.26), filtres `?search` (fuzzy) + `?categoryType`, détail 404 si soft-deleted + embarque contacts/adresses/ribs. - **ClientProcessor** : normalisation (RG-1.18→1.21), 409 doublon nom (RG-1.16) + 409 restauration (RG-1.23), gating par onglet `accounting.manage`/`archive` + mode strict 403 (RG-1.28), archivage exclusif + `archivedAt` (RG-1.22), RG-1.01 / RG-1.03 (mutex + type catégorie) / RG-1.12 / RG-1.13 / RG-1.04. - **ClientReadGroupContextBuilder** : ajout conditionnel du groupe `client:read:accounting` selon `commercial.clients.accounting.view`. - **CategoryReferenceDenormalizer** : résout les IRI catégorie vers `Category` (dénormalisation impossible sur l'interface sinon). - **Contrats Shared** : `CategoryInterface::getCategoryTypeCode()`, `BusinessRoleAwareInterface` + `BusinessRoles::COMMERCIALE`. ## Coordination stack - Permissions `commercial.clients.*` **référencées** ici, déclarées en **ERP-59** (tests RBAC en **ERP-60**). - Rôle métier `commerciale` seedé par **ERP-74** (RG-1.04 dormante d'ici là). - Config globale pagination (itemsPerPage client / max 50) portée par **ERP-72**. - Référentiels comptables (PaymentType/Bank/...) exposés en **ERP-56** → RG-1.12/1.13 testées en unitaire ici (pas d'IRI référentiel disponible avant ERP-56). ## Tests 31 tests Commercial (intégration admin sur les RG métier + unitaires sur le gating / RG-1.04 / RG-1.12 / RG-1.13 / context builder). Suite complète verte (343 tests). Règle n°1 respectée (aucun import inter-modules dans Commercial). --------- Co-authored-by: tristan <tristan@yuno.malio.fr> Co-authored-by: Matthieu <contact@malio.fr> Co-authored-by: Matthieu <mtholot19@gmail.com> Reviewed-on: #31 Co-authored-by: THOLOT DECHENE Matthieu <matthieu@yuno.malio.fr> Co-committed-by: THOLOT DECHENE Matthieu <matthieu@yuno.malio.fr>
66 lines
2.5 KiB
PHP
66 lines
2.5 KiB
PHP
<?php
|
|
|
|
declare(strict_types=1);
|
|
|
|
namespace App\Module\Commercial\Infrastructure\ApiPlatform\Serializer;
|
|
|
|
use ApiPlatform\State\SerializerContextBuilderInterface;
|
|
use App\Module\Commercial\Domain\Entity\Client;
|
|
use Symfony\Bundle\SecurityBundle\Security;
|
|
use Symfony\Component\DependencyInjection\Attribute\AsDecorator;
|
|
use Symfony\Component\DependencyInjection\Attribute\AutowireDecorated;
|
|
use Symfony\Component\HttpFoundation\Request;
|
|
|
|
/**
|
|
* Decore le context builder de serialisation d'API Platform pour ajouter
|
|
* DYNAMIQUEMENT le groupe de lecture `client:read:accounting` sur les ressources
|
|
* Client, uniquement si l'utilisateur courant a la permission
|
|
* `commercial.clients.accounting.view` (cf. spec-back M1 § 2.7 / § 4.1 / § 4.2).
|
|
*
|
|
* Pourquoi un context builder et pas le Provider : un Provider retourne des
|
|
* donnees mais ne peut pas influencer les groupes de serialisation. Le contexte
|
|
* de normalisation est construit ici, en amont du serializer — c'est le point
|
|
* d'extension idiomatique d'API Platform pour conditionner un groupe selon
|
|
* l'utilisateur. Realise l'intention « ajout conditionnel du groupe accounting »
|
|
* de la spec.
|
|
*
|
|
* S'applique aux operations de LECTURE (normalization) sur Client : liste ET
|
|
* detail. Sans la permission, les champs comptables (siren, accountNumber,
|
|
* tvaMode, nTva, paymentDelay, paymentType, bank) ne sont jamais serialises.
|
|
*/
|
|
#[AsDecorator('api_platform.serializer.context_builder')]
|
|
final readonly class ClientReadGroupContextBuilder implements SerializerContextBuilderInterface
|
|
{
|
|
public function __construct(
|
|
#[AutowireDecorated]
|
|
private SerializerContextBuilderInterface $decorated,
|
|
private Security $security,
|
|
) {}
|
|
|
|
public function createFromRequest(Request $request, bool $normalization, ?array $extractedAttributes = null): array
|
|
{
|
|
$context = $this->decorated->createFromRequest($request, $normalization, $extractedAttributes);
|
|
|
|
// Uniquement en lecture, sur la ressource Client, avec la permission.
|
|
if (!$normalization) {
|
|
return $context;
|
|
}
|
|
|
|
if (Client::class !== ($context['resource_class'] ?? null)) {
|
|
return $context;
|
|
}
|
|
|
|
if (!$this->security->isGranted('commercial.clients.accounting.view')) {
|
|
return $context;
|
|
}
|
|
|
|
$groups = $context['groups'] ?? [];
|
|
if (!in_array('client:read:accounting', $groups, true)) {
|
|
$groups[] = 'client:read:accounting';
|
|
}
|
|
$context['groups'] = $groups;
|
|
|
|
return $context;
|
|
}
|
|
}
|