MALIO-DEV/Inventory
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 2 Wiki Activity

WIP

Browse Source
This commit is contained in:
Matthieu
2026-03-31 17:57:59 +02:00
parent 1b1dab65b6
commit 476060cf7d
45 changed files with 8547 additions and 648 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
.claude/settings.json Normal file
View File

@@ -0,0 +1,6 @@
{
"enabledPlugins": {
"security-guidance@claude-plugins-official": true,
"claude-md-management@claude-plugins-official": true
}
}

0
.codex Normal file
View File

8
.gitignore vendored
View File

@@ -1,4 +1,3 @@
###> symfony/framework-bundle ### ###> symfony/framework-bundle ###
/.env.local /.env.local
/.env.local.php /.env.local.php
@@ -23,19 +22,16 @@
###> docker ### ###> docker ###
docker/.env.docker.local docker/.env.docker.local
###< docker ### ###< docker ###
###> lexik/jwt-authentication-bundle ###
/config/jwt/*.pem
###< lexik/jwt-authentication-bundle ###
###> migration archives ### ###> migration archives ###
/_archives/ /_archives/
###< migration archives ### ###< migration archives ###
###> temp files ### ###> temp files ###
*.sql *.sql
*.sql.gz
*.har *.har
FEATURE_IDEAS.md FEATURE_IDEAS.md
bin/.phpunit.result.cache
###< temp files ### ###< temp files ###
###> frontend ### ###> frontend ###

5
CLAUDE.md
View File

@@ -26,6 +26,7 @@ Voir `docs/GLOSSAIRE_METIER.md` — glossaire complet du domaine métier (concep
``` ```
Inventory/ # Backend Symfony (repo principal) Inventory/ # Backend Symfony (repo principal)
├── src/Entity/ # Entités Doctrine (annotations PHP 8 attributes) ├── src/Entity/ # Entités Doctrine (annotations PHP 8 attributes)
│ └── Trait/ # CuidEntityTrait (génération d'ID CUID)
├── src/Controller/ # Controllers custom (session, comments, audit…) ├── src/Controller/ # Controllers custom (session, comments, audit…)
├── src/EventSubscriber/ # Audit subscribers (onFlush) ├── src/EventSubscriber/ # Audit subscribers (onFlush)
├── src/Service/ # Services métier (sync, conversion, storage…) ├── src/Service/ # Services métier (sync, conversion, storage…)
@@ -148,6 +149,7 @@ Remplacent les anciennes colonnes JSON `structure` et `productIds` par des table
- `DocumentServeController` — `/api/documents/{id}/file|download` (GET) : servir/télécharger fichiers. - `DocumentServeController` — `/api/documents/{id}/file|download` (GET) : servir/télécharger fichiers.
- `ModelTypeConversionController` — `/api/model_types/{id}/conversion-check|convert` : vérification et conversion de ModelType. - `ModelTypeConversionController` — `/api/model_types/{id}/conversion-check|convert` : vérification et conversion de ModelType.
- `ModelTypeSyncController` — `/api/model_types/{id}/sync-preview|sync-confirm` (POST) : prévisualisation et application de sync ModelType→Composants. - `ModelTypeSyncController` — `/api/model_types/{id}/sync-preview|sync-confirm` (POST) : prévisualisation et application de sync ModelType→Composants.
- `EntityVersionController` — `/api/{entity}/{id}/versions` (GET), `/api/{entity}/{id}/versions/{version}/restore` (POST) : historique de versions numérotées et restauration.
- `HealthCheckController` — `/api/health` (GET) : health check. - `HealthCheckController` — `/api/health` (GET) : health check.
### Custom Fields — Architecture ### Custom Fields — Architecture
@@ -165,6 +167,8 @@ Remplacent les anciennes colonnes JSON `structure` et `productIds` par des table
- `SkeletonStructureService` — gestion de la structure skeleton (requirements) - `SkeletonStructureService` — gestion de la structure skeleton (requirements)
- `DocumentStorageService` — stockage et gestion des fichiers documents - `DocumentStorageService` — stockage et gestion des fichiers documents
- `PdfCompressorService` — compression des PDFs uploadés - `PdfCompressorService` — compression des PDFs uploadés
- `EntityVersionService` — gestion des versions numérotées (snapshot, restore) pour machines, pièces, composants, produits
- `ReferenceAutoGenerator` — génération automatique de références pour pièces et composants à partir de formules ModelType
- `src/Service/Sync/` — stratégies de sync par type de slot (tagged `app.sync_strategy`) - `src/Service/Sync/` — stratégies de sync par type de slot (tagged `app.sync_strategy`)
### DTOs (`src/DTO/`) ### DTOs (`src/DTO/`)
@@ -176,6 +180,7 @@ Remplacent les anciennes colonnes JSON `structure` et `productIds` par des table
### EventSubscribers notables (non-audit) ### EventSubscribers notables (non-audit)
- `PieceProductSyncSubscriber` — sync automatique des PieceProductSlots - `PieceProductSyncSubscriber` — sync automatique des PieceProductSlots
- `UniqueConstraintSubscriber` — traduit les erreurs de contrainte unique PG en messages utilisateur lisibles - `UniqueConstraintSubscriber` — traduit les erreurs de contrainte unique PG en messages utilisateur lisibles
- `ReferenceAutoSubscriber` — recalcule les références auto des pièces/composants quand les CustomFieldValues changent (onFlush)
### Rôles (hiérarchie) ### Rôles (hiérarchie)
``` ```

2
Inventory_frontend

Submodule Inventory_frontend updated: c82c21c0cd...958a00c8fc

29
TODO.md Normal file
View File

@@ -0,0 +1,29 @@
# TODO — MCP Inventory
## Bugs / Améliorations prioritaires
### sync_model_type ne fonctionne pas via MCP
Le tool `sync_model_type` attend un paramètre `structure` de type `array` (objet JSON imbriqué), mais le SDK MCP PHP ne supporte pas les objets complexes en paramètres — il reçoit un string au lieu d'un array.
**Solutions possibles :**
1. Accepter `structure` comme `string` (JSON encodé) et le décoder manuellement dans le tool
2. Créer des tools séparés : `add_product_requirement`, `add_custom_field_requirement`, `remove_requirement` au lieu d'un seul sync
3. Passer par des sous-paramètres plats (productTypeIds, customFieldNames, etc.)
**Impact :** L'IA ne peut pas ajouter de produits ni de champs personnalisés à une catégorie (ModelType) via MCP. Contournement actuel : passer par l'API REST.
---
### Resources MCP en erreur
Les 3 Resources (`SchemaResource`, `RolesResource`, `StatsResource`) produisent `[error] Failed to process MCP attribute`. Elles ne bloquent pas les tools mais ne sont pas exposées aux clients.
**Cause probable :** Incompatibilité du format `#[McpResource]` avec le SDK v0.4 / bundle v0.6.
---
## Améliorations futures
- [ ] Documentation utilisateur `docs/mcp/README.md` — guide d'utilisation pour les différents clients (Claude Desktop, ChatGPT, Codex)
- [ ] Mettre à jour CLAUDE.md avec la section MCP
- [ ] Ajouter le tool `upload_document` (upload de fichiers via MCP)
- [ ] Tester la compatibilité avec ChatGPT Desktop et Claude Desktop via tunnel

4
composer.json
View File

@@ -12,7 +12,6 @@
"doctrine/doctrine-bundle": "^3.2", "doctrine/doctrine-bundle": "^3.2",
"doctrine/doctrine-migrations-bundle": "^4.0", "doctrine/doctrine-migrations-bundle": "^4.0",
"doctrine/orm": "^3.6", "doctrine/orm": "^3.6",
"lexik/jwt-authentication-bundle": "^3.2",
"nelmio/cors-bundle": "^2.6", "nelmio/cors-bundle": "^2.6",
"nyholm/psr7": "^1.8", "nyholm/psr7": "^1.8",
"phpdocumentor/reflection-docblock": "^5.6", "phpdocumentor/reflection-docblock": "^5.6",
@@ -33,8 +32,7 @@
"symfony/twig-bundle": "8.0.*", "symfony/twig-bundle": "8.0.*",
"symfony/uid": "8.0.*", "symfony/uid": "8.0.*",
"symfony/validator": "8.0.*", "symfony/validator": "8.0.*",
"symfony/yaml": "8.0.*", "symfony/yaml": "8.0.*"
"vich/uploader-bundle": "^2.9"
}, },
"config": { "config": {
"allow-plugins": { "allow-plugins": {

536
composer.lock generated
View File

@@ -4,7 +4,7 @@
"Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies", "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#installing-dependencies",
"This file is @generated automatically" "This file is @generated automatically"
], ],
"content-hash": "b15a7808211e724ca29dd78602df3aab", "content-hash": "2db01f705a09cf38007a2baa3b078e49",
"packages": [ "packages": [
{ {
"name": "api-platform/doctrine-common", "name": "api-platform/doctrine-common",
@@ -2361,259 +2361,6 @@
}, },
"time": "2025-10-26T09:35:14+00:00" "time": "2025-10-26T09:35:14+00:00"
}, },
{
"name": "jms/metadata",
"version": "2.9.0",
"source": {
"type": "git",
"url": "https://github.com/schmittjoh/metadata.git",
"reference": "554319d2e5f0c5d8ccaeffe755eac924e14da330"
},
"dist": {
"type": "zip",
"url": "https://api.github.com/repos/schmittjoh/metadata/zipball/554319d2e5f0c5d8ccaeffe755eac924e14da330",
"reference": "554319d2e5f0c5d8ccaeffe755eac924e14da330",
"shasum": ""
},
"require": {
"php": "^7.2|^8.0"
},
"require-dev": {
"doctrine/cache": "^1.0|^2.0",
"doctrine/coding-standard": "^8.0",
"mikey179/vfsstream": "^1.6.7",
"phpunit/phpunit": "^8.5.42|^9.6.23",
"psr/container": "^1.0|^2.0",
"symfony/cache": "^3.1|^4.0|^5.0|^6.0|^7.0|^8.0",
"symfony/dependency-injection": "^3.1|^4.0|^5.0|^6.0|^7.0|^8.0"
},
"type": "library",
"extra": {
"branch-alias": {
"dev-master": "2.x-dev"
}
},
"autoload": {
"psr-4": {
"Metadata\\": "src/"
}
},
"notification-url": "https://packagist.org/downloads/",
"license": [
"MIT"
],
"authors": [
{
"name": "Johannes M. Schmitt",
"email": "schmittjoh@gmail.com"
},
{
"name": "Asmir Mustafic",
"email": "goetas@gmail.com"
}
],
"description": "Class/method/property metadata management in PHP",
"keywords": [
"annotations",
"metadata",
"xml",
"yaml"
],
"support": {
"issues": "https://github.com/schmittjoh/metadata/issues",
"source": "https://github.com/schmittjoh/metadata/tree/2.9.0"
},
"time": "2025-11-30T20:12:26+00:00"
},
{
"name": "lcobucci/jwt",
"version": "5.6.0",
"source": {
"type": "git",
"url": "https://github.com/lcobucci/jwt.git",
"reference": "bb3e9f21e4196e8afc41def81ef649c164bca25e"
},
"dist": {
"type": "zip",
"url": "https://api.github.com/repos/lcobucci/jwt/zipball/bb3e9f21e4196e8afc41def81ef649c164bca25e",
"reference": "bb3e9f21e4196e8afc41def81ef649c164bca25e",
"shasum": ""
},
"require": {
"ext-openssl": "*",
"ext-sodium": "*",
"php": "~8.2.0 || ~8.3.0 || ~8.4.0 || ~8.5.0",
"psr/clock": "^1.0"
},
"require-dev": {
"infection/infection": "^0.29",
"lcobucci/clock": "^3.2",
"lcobucci/coding-standard": "^11.0",
"phpbench/phpbench": "^1.2",
"phpstan/extension-installer": "^1.2",
"phpstan/phpstan": "^1.10.7",
"phpstan/phpstan-deprecation-rules": "^1.1.3",
"phpstan/phpstan-phpunit": "^1.3.10",
"phpstan/phpstan-strict-rules": "^1.5.0",
"phpunit/phpunit": "^11.1"
},
"suggest": {
"lcobucci/clock": ">= 3.2"
},
"type": "library",
"autoload": {
"psr-4": {
"Lcobucci\\JWT\\": "src"
}
},
"notification-url": "https://packagist.org/downloads/",
"license": [
"BSD-3-Clause"
],
"authors": [
{
"name": "Luís Cobucci",
"email": "lcobucci@gmail.com",
"role": "Developer"
}
],
"description": "A simple library to work with JSON Web Token and JSON Web Signature",
"keywords": [
"JWS",
"jwt"
],
"support": {
"issues": "https://github.com/lcobucci/jwt/issues",
"source": "https://github.com/lcobucci/jwt/tree/5.6.0"
},
"funding": [
{
"url": "https://github.com/lcobucci",
"type": "github"
},
{
"url": "https://www.patreon.com/lcobucci",
"type": "patreon"
}
],
"time": "2025-10-17T11:30:53+00:00"
},
{
"name": "lexik/jwt-authentication-bundle",
"version": "v3.2.0",
"source": {
"type": "git",
"url": "https://github.com/lexik/LexikJWTAuthenticationBundle.git",
"reference": "60df75dc70ee6f597929cb2f0812adda591dfa4b"
},
"dist": {
"type": "zip",
"url": "https://api.github.com/repos/lexik/LexikJWTAuthenticationBundle/zipball/60df75dc70ee6f597929cb2f0812adda591dfa4b",
"reference": "60df75dc70ee6f597929cb2f0812adda591dfa4b",
"shasum": ""
},
"require": {
"ext-openssl": "*",
"lcobucci/jwt": "^5.0",
"php": ">=8.2",
"symfony/clock": "^6.4|^7.0|^8.0",
"symfony/config": "^6.4|^7.0|^8.0",
"symfony/dependency-injection": "^6.4|^7.0|^8.0",
"symfony/deprecation-contracts": "^2.4|^3.0",
"symfony/event-dispatcher": "^6.4|^7.0|^8.0",
"symfony/http-foundation": "^6.4|^7.0|^8.0",
"symfony/http-kernel": "^6.4|^7.0|^8.0",
"symfony/property-access": "^6.4|^7.0|^8.0",
"symfony/security-bundle": "^6.4|^7.0|^8.0",
"symfony/translation-contracts": "^1.0|^2.0|^3.0"
},
"require-dev": {
"api-platform/core": "^3.0|^4.0",
"rector/rector": "^1.2",
"symfony/browser-kit": "^6.4|^7.0|^8.0",
"symfony/console": "^6.4|^7.0|^8.0",
"symfony/dom-crawler": "^6.4|^7.0|^8.0",
"symfony/filesystem": "^6.4|^7.0|^8.0",
"symfony/framework-bundle": "^6.4|^7.0|^8.0",
"symfony/phpunit-bridge": "^6.4|^7.0|^8.0",
"symfony/var-dumper": "^6.4|^7.0|^8.0",
"symfony/yaml": "^6.4|^7.0|^8.0"
},
"suggest": {
"gesdinet/jwt-refresh-token-bundle": "Implements a refresh token system over Json Web Tokens in Symfony",
"spomky-labs/lexik-jose-bridge": "Provides a JWT Token encoder with encryption support"
},
"type": "symfony-bundle",
"autoload": {
"psr-4": {
"Lexik\\Bundle\\JWTAuthenticationBundle\\": ""
},
"exclude-from-classmap": [
"/Tests/"
]
},
"notification-url": "https://packagist.org/downloads/",
"license": [
"MIT"
],
"authors": [
{
"name": "Jeremy Barthe",
"email": "j.barthe@lexik.fr",
"homepage": "https://github.com/jeremyb"
},
{
"name": "Nicolas Cabot",
"email": "n.cabot@lexik.fr",
"homepage": "https://github.com/slashfan"
},
{
"name": "Cedric Girard",
"email": "c.girard@lexik.fr",
"homepage": "https://github.com/cedric-g"
},
{
"name": "Dev Lexik",
"email": "dev@lexik.fr",
"homepage": "https://github.com/lexik"
},
{
"name": "Robin Chalas",
"email": "robin.chalas@gmail.com",
"homepage": "https://github.com/chalasr"
},
{
"name": "Lexik Community",
"homepage": "https://github.com/lexik/LexikJWTAuthenticationBundle/graphs/contributors"
}
],
"description": "This bundle provides JWT authentication for your Symfony REST API",
"homepage": "https://github.com/lexik/LexikJWTAuthenticationBundle",
"keywords": [
"Authentication",
"JWS",
"api",
"bundle",
"jwt",
"rest",
"symfony"
],
"support": {
"issues": "https://github.com/lexik/LexikJWTAuthenticationBundle/issues",
"source": "https://github.com/lexik/LexikJWTAuthenticationBundle/tree/v3.2.0"
},
"funding": [
{
"url": "https://github.com/chalasr",
"type": "github"
},
{
"url": "https://tidelift.com/funding/github/packagist/lexik/jwt-authentication-bundle",
"type": "tidelift"
}
],
"time": "2025-12-20T17:47:00+00:00"
},
{ {
"name": "mcp/sdk", "name": "mcp/sdk",
"version": "v0.4.0", "version": "v0.4.0",
@@ -5594,92 +5341,6 @@
], ],
"time": "2026-03-04T16:39:24+00:00" "time": "2026-03-04T16:39:24+00:00"
}, },
{
"name": "symfony/mime",
"version": "v8.0.0",
"source": {
"type": "git",
"url": "https://github.com/symfony/mime.git",
"reference": "7576ce3b2b4d3a2a7fe7020a07a392065d6ffd40"
},
"dist": {
"type": "zip",
"url": "https://api.github.com/repos/symfony/mime/zipball/7576ce3b2b4d3a2a7fe7020a07a392065d6ffd40",
"reference": "7576ce3b2b4d3a2a7fe7020a07a392065d6ffd40",
"shasum": ""
},
"require": {
"php": ">=8.4",
"symfony/polyfill-intl-idn": "^1.10",
"symfony/polyfill-mbstring": "^1.0"
},
"conflict": {
"egulias/email-validator": "~3.0.0",
"phpdocumentor/reflection-docblock": "<3.2.2",
"phpdocumentor/type-resolver": "<1.4.0"
},
"require-dev": {
"egulias/email-validator": "^2.1.10|^3.1|^4",
"league/html-to-markdown": "^5.0",
"phpdocumentor/reflection-docblock": "^3.0|^4.0|^5.0",
"symfony/dependency-injection": "^7.4|^8.0",
"symfony/process": "^7.4|^8.0",
"symfony/property-access": "^7.4|^8.0",
"symfony/property-info": "^7.4|^8.0",
"symfony/serializer": "^7.4|^8.0"
},
"type": "library",
"autoload": {
"psr-4": {
"Symfony\\Component\\Mime\\": ""
},
"exclude-from-classmap": [
"/Tests/"
]
},
"notification-url": "https://packagist.org/downloads/",
"license": [
"MIT"
],
"authors": [
{
"name": "Fabien Potencier",
"email": "fabien@symfony.com"
},
{
"name": "Symfony Community",
"homepage": "https://symfony.com/contributors"
}
],
"description": "Allows manipulating MIME messages",
"homepage": "https://symfony.com",
"keywords": [
"mime",
"mime-type"
],
"support": {
"source": "https://github.com/symfony/mime/tree/v8.0.0"
},
"funding": [
{
"url": "https://symfony.com/sponsor",
"type": "custom"
},
{
"url": "https://github.com/fabpot",
"type": "github"
},
{
"url": "https://github.com/nicolas-grekas",
"type": "github"
},
{
"url": "https://tidelift.com/funding/github/packagist/symfony/symfony",
"type": "tidelift"
}
],
"time": "2025-11-16T10:17:21+00:00"
},
{ {
"name": "symfony/options-resolver", "name": "symfony/options-resolver",
"version": "v8.0.0", "version": "v8.0.0",
@@ -5906,93 +5567,6 @@
], ],
"time": "2025-06-27T09:58:17+00:00" "time": "2025-06-27T09:58:17+00:00"
}, },
{
"name": "symfony/polyfill-intl-idn",
"version": "v1.33.0",
"source": {
"type": "git",
"url": "https://github.com/symfony/polyfill-intl-idn.git",
"reference": "9614ac4d8061dc257ecc64cba1b140873dce8ad3"
},
"dist": {
"type": "zip",
"url": "https://api.github.com/repos/symfony/polyfill-intl-idn/zipball/9614ac4d8061dc257ecc64cba1b140873dce8ad3",
"reference": "9614ac4d8061dc257ecc64cba1b140873dce8ad3",
"shasum": ""
},
"require": {
"php": ">=7.2",
"symfony/polyfill-intl-normalizer": "^1.10"
},
"suggest": {
"ext-intl": "For best performance"
},
"type": "library",
"extra": {
"thanks": {
"url": "https://github.com/symfony/polyfill",
"name": "symfony/polyfill"
}
},
"autoload": {
"files": [
"bootstrap.php"
],
"psr-4": {
"Symfony\\Polyfill\\Intl\\Idn\\": ""
}
},
"notification-url": "https://packagist.org/downloads/",
"license": [
"MIT"
],
"authors": [
{
"name": "Laurent Bassin",
"email": "laurent@bassin.info"
},
{
"name": "Trevor Rowbotham",
"email": "trevor.rowbotham@pm.me"
},
{
"name": "Symfony Community",
"homepage": "https://symfony.com/contributors"
}
],
"description": "Symfony polyfill for intl's idn_to_ascii and idn_to_utf8 functions",
"homepage": "https://symfony.com",
"keywords": [
"compatibility",
"idn",
"intl",
"polyfill",
"portable",
"shim"
],
"support": {
"source": "https://github.com/symfony/polyfill-intl-idn/tree/v1.33.0"
},
"funding": [
{
"url": "https://symfony.com/sponsor",
"type": "custom"
},
{
"url": "https://github.com/fabpot",
"type": "github"
},
{
"url": "https://github.com/nicolas-grekas",
"type": "github"
},
{
"url": "https://tidelift.com/funding/github/packagist/symfony/symfony",
"type": "tidelift"
}
],
"time": "2024-09-10T14:38:51+00:00"
},
{ {
"name": "symfony/polyfill-intl-normalizer", "name": "symfony/polyfill-intl-normalizer",
"version": "v1.33.0", "version": "v1.33.0",
@@ -8426,114 +8000,6 @@
], ],
"time": "2025-12-14T11:28:47+00:00" "time": "2025-12-14T11:28:47+00:00"
}, },
{
"name": "vich/uploader-bundle",
"version": "v2.9.1",
"source": {
"type": "git",
"url": "https://github.com/dustin10/VichUploaderBundle.git",
"reference": "945939a04a33c0b78c5fbb7ead31533d85112df5"
},
"dist": {
"type": "zip",
"url": "https://api.github.com/repos/dustin10/VichUploaderBundle/zipball/945939a04a33c0b78c5fbb7ead31533d85112df5",
"reference": "945939a04a33c0b78c5fbb7ead31533d85112df5",
"shasum": ""
},
"require": {
"doctrine/persistence": "^3.0 || ^4.0",
"ext-simplexml": "*",
"jms/metadata": "^2.4",
"php": "^8.1",
"symfony/config": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/console": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/dependency-injection": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/event-dispatcher-contracts": "^3.1",
"symfony/http-foundation": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/http-kernel": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/mime": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/property-access": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/string": "^5.4 || ^6.0 || ^7.0 || ^8.0"
},
"conflict": {
"doctrine/annotations": "<1.12",
"league/flysystem": "<2.0"
},
"require-dev": {
"dg/bypass-finals": "^1.9",
"doctrine/common": "^3.0",
"doctrine/doctrine-bundle": "^2.7 || ^3.0",
"doctrine/mongodb-odm": "^2.4",
"doctrine/orm": "^2.13 || ^3.0",
"ext-sqlite3": "*",
"knplabs/knp-gaufrette-bundle": "dev-master",
"league/flysystem-bundle": "^2.4 || ^3.0",
"league/flysystem-memory": "^2.0 || ^3.0",
"matthiasnoback/symfony-dependency-injection-test": "^5.1 || ^6.0",
"mikey179/vfsstream": "^1.6.11",
"phpunit/phpunit": "^10.5 || ^11.5 || ^12.2",
"symfony/asset": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/browser-kit": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/css-selector": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/doctrine-bridge": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/dom-crawler": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/form": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/framework-bundle": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/phpunit-bridge": "^7.3",
"symfony/security-csrf": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/translation": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/twig-bridge": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/twig-bundle": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/validator": "^5.4.22 || ^6.0 || ^7.0 || ^8.0",
"symfony/var-dumper": "^5.4 || ^6.0 || ^7.0 || ^8.0",
"symfony/yaml": "^5.4 || ^6.0 || ^7.0 || ^8.0"
},
"suggest": {
"doctrine/doctrine-bundle": "For integration with Doctrine",
"doctrine/mongodb-odm-bundle": "For integration with Doctrine ODM",
"doctrine/orm": "For integration with Doctrine ORM",
"doctrine/phpcr-odm": "For integration with Doctrine PHPCR",
"knplabs/knp-gaufrette-bundle": "For integration with Gaufrette",
"league/flysystem-bundle": "For integration with Flysystem",
"liip/imagine-bundle": "To generate image thumbnails",
"oneup/flysystem-bundle": "For integration with Flysystem",
"symfony/asset": "To generate better links",
"symfony/form": "To handle uploads in forms",
"symfony/yaml": "To use YAML mapping"
},
"type": "symfony-bundle",
"extra": {
"branch-alias": {
"dev-master": "2.x-dev"
}
},
"autoload": {
"psr-4": {
"Vich\\UploaderBundle\\": "src"
}
},
"notification-url": "https://packagist.org/downloads/",
"license": [
"MIT"
],
"authors": [
{
"name": "Dustin Dobervich",
"email": "ddobervich@gmail.com"
}
],
"description": "Ease file uploads attached to entities",
"homepage": "https://github.com/dustin10/VichUploaderBundle",
"keywords": [
"file uploads",
"upload"
],
"support": {
"issues": "https://github.com/dustin10/VichUploaderBundle/issues",
"source": "https://github.com/dustin10/VichUploaderBundle/tree/v2.9.1"
},
"time": "2025-12-10T08:23:38+00:00"
},
{ {
"name": "webmozart/assert", "name": "webmozart/assert",
"version": "2.0.0", "version": "2.0.0",

20
config/bundles.php
View File

@@ -6,7 +6,6 @@ use ApiPlatform\Symfony\Bundle\ApiPlatformBundle;
use DAMA\DoctrineTestBundle\DAMADoctrineTestBundle; use DAMA\DoctrineTestBundle\DAMADoctrineTestBundle;
use Doctrine\Bundle\DoctrineBundle\DoctrineBundle; use Doctrine\Bundle\DoctrineBundle\DoctrineBundle;
use Doctrine\Bundle\MigrationsBundle\DoctrineMigrationsBundle; use Doctrine\Bundle\MigrationsBundle\DoctrineMigrationsBundle;
use Lexik\Bundle\JWTAuthenticationBundle\LexikJWTAuthenticationBundle;
use Nelmio\CorsBundle\NelmioCorsBundle; use Nelmio\CorsBundle\NelmioCorsBundle;
use Symfony\AI\McpBundle\McpBundle; use Symfony\AI\McpBundle\McpBundle;
use Symfony\Bundle\FrameworkBundle\FrameworkBundle; use Symfony\Bundle\FrameworkBundle\FrameworkBundle;
@@ -14,14 +13,13 @@ use Symfony\Bundle\SecurityBundle\SecurityBundle;
use Symfony\Bundle\TwigBundle\TwigBundle; use Symfony\Bundle\TwigBundle\TwigBundle;
return [ return [
FrameworkBundle::class => ['all' => true], FrameworkBundle::class => ['all' => true],
TwigBundle::class => ['all' => true], TwigBundle::class => ['all' => true],
SecurityBundle::class => ['all' => true], SecurityBundle::class => ['all' => true],
DoctrineBundle::class => ['all' => true], DoctrineBundle::class => ['all' => true],
DoctrineMigrationsBundle::class => ['all' => true], DoctrineMigrationsBundle::class => ['all' => true],
NelmioCorsBundle::class => ['all' => true], NelmioCorsBundle::class => ['all' => true],
ApiPlatformBundle::class => ['all' => true], ApiPlatformBundle::class => ['all' => true],
LexikJWTAuthenticationBundle::class => ['all' => true], DAMADoctrineTestBundle::class => ['test' => true],
DAMADoctrineTestBundle::class => ['test' => true], McpBundle::class => ['all' => true],
McpBundle::class => ['all' => true],
]; ];

5
config/packages/framework.yaml
View File

@@ -3,7 +3,10 @@ framework:
secret: '%env(APP_SECRET)%' secret: '%env(APP_SECRET)%'
# Note that the session will be started ONLY if you read or write from it. # Note that the session will be started ONLY if you read or write from it.
session: true session:
cookie_secure: auto
cookie_samesite: lax
cookie_httponly: true
#esi: true #esi: true
#fragments: true #fragments: true

4
config/packages/lexik_jwt_authentication.yaml
View File

@@ -1,4 +0,0 @@
lexik_jwt_authentication:
secret_key: '%env(resolve:JWT_SECRET_KEY)%'
public_key: '%env(resolve:JWT_PUBLIC_KEY)%'
pass_phrase: '%env(JWT_PASSPHRASE)%'

4
config/packages/rate_limiter.yaml
View File

@@ -4,3 +4,7 @@ framework:
policy: sliding_window policy: sliding_window
limit: 5 limit: 5
interval: '1 minute' interval: '1 minute'
login:
policy: sliding_window
limit: 5
interval: '1 minute'

278
docs/CUSTOM_FIELDS_AUDIT_RECOVERY.md Normal file
View File

@@ -0,0 +1,278 @@
# Champs Personnalises - Diagnostic Et Recuperation
Date : 2026-03-23
---
## Contexte
Un bug sur la sauvegarde des categories (`ModelType`) pouvait recreer des definitions de champs personnalises avec de nouveaux IDs.
Effet de bord :
- les `CustomFieldValue` existants restaient lies aux anciens `CustomField`
- puis etaient supprimes en cascade
- resultat visible : apres modification d'une categorie, certaines valeurs de champs perso disparaissaient
Le correctif preventif a ete fait :
- conservation des `id/customFieldId` cote frontend pour `PIECE/PRODUCT`
- matching backend plus robuste sur `id`, puis `orderIndex`, puis nom
Ce document couvre uniquement :
- comment detecter ce qui manque
- comment lire le listing
- comment identifier ce qui est recuperable depuis l'audit
- comment restaurer proprement
---
## Commandes Disponibles
### 1. Lister tous les champs perso manquants ou vides
Dans le conteneur :
```bash
php bin/console app:check-missing-custom-field-values
```
Variantes utiles :
```bash
php bin/console app:check-missing-custom-field-values --entity=piece
php bin/console app:check-missing-custom-field-values --entity=composant
php bin/console app:check-missing-custom-field-values --max-rows=1000
php bin/console app:check-missing-custom-field-values --limit=500 --max-rows=1000
```
### 2. Afficher uniquement les cas recuperables depuis l'audit
```bash
php bin/console app:check-missing-custom-field-values --recoverable-only
```
Variantes :
```bash
php bin/console app:check-missing-custom-field-values --entity=piece --recoverable-only
php bin/console app:check-missing-custom-field-values --entity=composant --recoverable-only
php bin/console app:check-missing-custom-field-values --recoverable-only --max-rows=1000
```
### 3. Dry-run de restauration pour une piece
```bash
php bin/console app:restore-piece-custom-field-values <pieceId>
```
Exemple :
```bash
php bin/console app:restore-piece-custom-field-values cl731386df55fcb9e6a01e0a63
```
### 4. Appliquer la restauration pour une piece
```bash
php bin/console app:restore-piece-custom-field-values <pieceId> --apply
```
---
## Colonnes Du Listing
La commande `app:check-missing-custom-field-values` affiche :
- `Entity` : `piece` ou `composant`
- `ID` : identifiant de l'entite
- `Name` : nom de l'entite
- `Reference` : reference metier si presente
- `Category` : nom de la categorie (`ModelType`)
- `Field` : nom du champ personnalise attendu par la categorie
- `Issue` : `missing` ou `empty`
- `Recoverable` : `yes` ou `no`
- `Audit value` : derniere valeur non vide retrouvee dans l'audit si disponible
---
## Signification Des Statuts
### `missing`
Il n'existe actuellement **aucune** ligne `CustomFieldValue` pour ce champ sur l'entite.
Cela peut vouloir dire :
- la valeur n'a jamais ete saisie
- la valeur a ete perdue lors du bug
- le champ a ete ajoute plus tard sur la categorie sans initialisation des anciennes entites
### `empty`
La ligne `CustomFieldValue` existe, mais sa valeur est vide.
Cela est plus suspect qu'un `missing`, mais ne prouve pas a lui seul une perte.
### `Recoverable = yes`
L'audit contient au moins une ancienne valeur non vide pour ce champ.
En pratique :
- c'est le signal le plus utile
- ce sont les cas a traiter en priorite
- ces cas sont potentiellement restaurables automatiquement
### `Recoverable = no`
L'audit de cette entite ne contient pas de valeur non vide exploitable pour ce champ.
Cela ne veut **pas** forcement dire qu'il n'y a jamais eu de valeur.
Cela veut simplement dire :
- rien de recuperable n'a ete trouve dans les logs d'audit consultes
---
## Lecture Des Cas Typiques
### Cas 1
```text
piece ... Roulement ... Diametre ... missing ... no
```
Interpretation :
- le champ `Diametre` est attendu sur cette piece
- aucune valeur n'existe actuellement
- l'audit ne permet pas de retrouver une ancienne valeur
Conclusion :
- non recuperable automatiquement
- a verifier metierement si la valeur a deja existe ou non
### Cas 2
```text
piece ... Arbre ... Diametre ... empty ... yes ... 35 mm
```
Interpretation :
- une ligne de valeur existe mais elle est vide
- l'audit montre qu'une ancienne valeur `35 mm` existait
Conclusion :
- cas typique de restauration automatique possible
### Cas 3
```text
piece ... Joint ... Matiere ... missing ... yes ... NBR
```
Interpretation :
- la valeur n'existe plus du tout
- l'audit permet de retrouver `NBR`
Conclusion :
- forte probabilite de perte historique
- recuperable automatiquement
---
## Priorisation Recommandee
Ordre de traitement conseille :
1. `empty + yes`
2. `missing + yes`
3. `empty + no`
4. `missing + no`
Pourquoi :
- les `yes` sont les seuls cas recuperables automatiquement
- les `empty` indiquent souvent une valeur ecrasee
- les `missing no` sont nombreux mais souvent ambigus
---
## Procedure Recommandee
### Etape 1 - Scanner globalement
```bash
php bin/console app:check-missing-custom-field-values --recoverable-only --max-rows=1000
```
### Etape 2 - Identifier les pieces prioritaires
Chercher :
- les pieces les plus critiques metierement
- les categories fortement touchees (`Roulement`, `Joint`, `Arbre`, etc.)
- les cas avec valeur d'audit explicite
### Etape 3 - Faire un dry-run piece par piece
```bash
php bin/console app:restore-piece-custom-field-values <pieceId>
```
### Etape 4 - Appliquer uniquement apres verification
```bash
php bin/console app:restore-piece-custom-field-values <pieceId> --apply
```
---
## Limites Actuelles
### Ce qui est pris en charge
- diagnostic global sur les `pieces`
- diagnostic global sur les `composants`
- restauration automatique ciblee sur les `pieces`
### Ce qui n'est pas encore automatise
- restauration automatique en masse
- restauration automatique des `composants`
- reconstitution si l'audit ne contient aucune ancienne valeur exploitable
---
## Interpretation Metier
Le listing global ne doit pas etre lu comme :
> "866 valeurs ont ete perdues"
Il doit etre lu comme :
> "866 couples entite/champ sont actuellement manquants ou vides par rapport aux definitions de categories"
Parmi eux :
- certains n'ont jamais ete renseignes
- certains ont probablement ete perdus
- seuls les cas `Recoverable = yes` sont candidates a une recuperation automatique fiable
---
## Commandes Resumees
```bash
# Tout lister
php bin/console app:check-missing-custom-field-values
# Afficher uniquement les cas recuperables
php bin/console app:check-missing-custom-field-values --recoverable-only
# Scanner seulement les pieces
php bin/console app:check-missing-custom-field-values --entity=piece --recoverable-only
# Scanner seulement les composants
php bin/console app:check-missing-custom-field-values --entity=composant --recoverable-only
# Dry-run de restauration d'une piece
php bin/console app:restore-piece-custom-field-values <pieceId>
# Application reelle
php bin/console app:restore-piece-custom-field-values <pieceId> --apply
```

144
docs/CUSTOM_FIELDS_RECOVERABLE_RESULTS_2026-03-23.md Normal file
View File

@@ -0,0 +1,144 @@
# Resultats Recuperables - Champs Personnalises
Date : 2026-03-23
Source : `php bin/console app:check-missing-custom-field-values --recoverable-only`
---
## Resume
- Total : 40 cas recuperables
- Pieces : 40
- Composants : 0
- Type de probleme observe : uniquement `empty`
- Categorie dominante : `Arbre`
- Champ le plus frequent : `Diamètre`
Conclusion :
- il n'y a pas ici une grande dispersion de cas heterogenes
- la quasi-totalite du lot correspond a des valeurs historisees recuperables sur des pieces de categorie `Arbre`
- ces cas sont de bons candidats a une restauration automatique
---
## Tableau
| Entity | ID | Name | Reference | Category | Field | Issue | Recoverable | Audit value |
|---|---|---|---|---|---|---|---|---|
| piece | `clc08fbdcd334ed869772d98ee` | Arbre de la cage écureuil pied E4 | | Arbre | Diamètre | empty | yes | 45 mm |
| piece | `cl8570d729efd017c12a2d5c3d` | Arbre du tambour tête E7 | | Arbre | Diamètre | empty | yes | 40 mm |
| piece | `cle1db7051dbef91fc009073a6` | Arbre de la cage écureuil pied E6 | | Arbre | Diamètre | empty | yes | 45 mm |
| piece | `cl9282d473ff01b5d1df8bc945` | Arbre E1 | | Arbre | Diamètre | empty | yes | 50 |
| piece | `cl22e81a055f9c393d8d2c82fc` | Arbre du palier pied E3 | | Arbre | Diamètre | empty | yes | 50 mm |
| piece | `clca9379d4aa76de6772ebbe1a` | Arbre pignon | `0-5720-00` | Arbre | Type | empty | yes | 20 DTS |
| piece | `clc97804ec0bf8b6d9bb530717` | Arbre du palier tête E2 E2B | | Arbre | Diamètre | empty | yes | 40 |
| piece | `cl1597f1500c1052e9e7a95c51` | Arbre du palier pied E2 E2B | | Arbre | Diamètre | empty | yes | 35 mm |
| piece | `cleea7ff4b9b1a6396a0bb9ea8` | Arbre du tambour tête E1 | | Arbre | Diamètre | empty | yes | 70 mm |
| piece | `cl5c71e3777146de5508e07156` | Arbre de la cage écureuil pied E1 | | Arbre | Diamètre | empty | yes | 50 mm |
| piece | `cl731386df55fcb9e6a01e0a63` | Arbre de la cage écureuil pied E2 E2B | | Arbre | Diamètre | empty | yes | 35 mm |
| piece | `clfaf128312d5c253d928f47ac` | Arbre du palier pied E4 | | Arbre | Diamètre | empty | yes | 45 mm |
| piece | `clbf9f0070ebd464b3c309c646` | Arbre du palier pied E8 | | Arbre | Diamètre | empty | yes | 50 mm |
| piece | `clc7c00cad416477d4438cd61a` | Arbre du tambour tête E8 | | Arbre | Diamètre | empty | yes | 70 mm |
| piece | `cl3f01a1a514423359405a4825` | Arbre du palier tête E7 | | Arbre | Diamètre | empty | yes | 40 mm |
| piece | `clf16e543545eddd01b20077df` | Arbre du tambour tête E5 | | Arbre | Diamètre | empty | yes | 55 mm |
| piece | `clb6c61ebb8da2c4361265f766` | Arbre du palier tête E6 | | Arbre | Diamètre | empty | yes | 55 mm |
| piece | `cl8da1b875191c617e5852bf81` | Arbre du tambour tête E2 E2B | | Arbre | Diamètre | empty | yes | 40 mm |
| piece | `cl8da1b875191c617e5852bf81` | Arbre du tambour tête E2 E2B | | Arbre | Diamètre palier | empty | yes | 40 |
| piece | `cla82d44c52d7eb2a592f4120d` | Arbre du palier pied E7 | | Arbre | Diamètre | empty | yes | 35 mm |
| piece | `clf8562d27a542f86f8f4a5629` | Arbre du palier tête E8 | | Arbre | Diamètre | empty | yes | 70 mm |
| piece | `clde7ee756c2cf264c062b861d` | Arbre du palier pied E6 | | Arbre | Diamètre | empty | yes | 45 mm |
| piece | `cl6667d159f6d07ba77fa79b39` | Arbre de la cage écureuil pied E5 | | Arbre | Diamètre | empty | yes | 45 mm |
| piece | `cl455ad597bcee2a8e3c099420` | Arbre du palier pied E5 | | Arbre | Diamètre | empty | yes | 45 mm |
| piece | `cl22c13dbc4d38a1f846323ae6` | Arbre de la cage écureuil pied E3 | | Arbre | Diamètre | empty | yes | 50 mm |
| piece | `cl1406ef19de58fdd1adf40221` | Arbre de la cage écureuil pied E7 | | Arbre | Diamètre | empty | yes | 35 mm |
| piece | `clafaa71cbf49777fbb8415f19` | Arbre du tambour tête E3 | | Arbre | Diamètre | empty | yes | 70 mm |
| piece | `cle255aea44755dbbe7e466a99` | Arbre du palier tête E5 | | Arbre | Diamètre | empty | yes | 55 mm |
| piece | `cl3d978dd4b071daff8fb185f7` | Arbre du palier pied E1 | | Arbre | Diamètre | empty | yes | 50 mm |
| piece | `cl5e8aba1867089544d71fe2c5` | Arbre du palier tête E4 | | Arbre | Diamètre | empty | yes | 55 mm |
| piece | `cl04c79cd568894a5674b46a31` | Arbre du palier pied élévateur expédition | | Arbre | Diamètre | empty | yes | 50 mm |
| piece | `cl50fe870a07e42759b37b511f` | Arbre du tambour tête E6 | | Arbre | Diamètre | empty | yes | 55 mm |
| piece | `cl531dde45c3fc64c1a3b16ca0` | Arbre de la cage écureuil pied élévateur expédition | | Arbre | Diamètre | empty | yes | 50 mm |
| piece | `cleca9e4baa9e9205f1dd948e1` | Arbre du palier tête E3 | | Arbre | Diamètre | empty | yes | 70 mm |
| piece | `cl5ee293dc7b61feba510082a4` | Arbre du tambour tête élévateur expédition | | Arbre | Diamètre | empty | yes | 70 mm |
| piece | `cled68ff759b1f02f482990fb3` | Arbre du tambour du palier tête E11 | | Arbre | Diamètre | empty | yes | 70 mm |
| piece | `cmkr0qjw5004s1eq6pen63x7j` | Arbre du palier tête E1 | | Arbre | Diamètre | empty | yes | 70 mm |
| piece | `cl2c3570dd00372fed44cd5a43` | Arbre du palier tête élévateur expédition | `Décolleter a Ø40 pour réducteur` | Arbre | Diamètre | empty | yes | 70 mm |
| piece | `cl7b3702f04d24d87e47232a14` | Arbre du tambour tête E4 | | Arbre | Diamètre | empty | yes | 55 mm |
| piece | `cldd656c6092225f53a22badc0` | Arbre de la cage écureuil pied E8 | | Arbre | Diamètre | empty | yes | 50 mm |
---
## Observations
### 1. Lot tres homogene
Le resultat est tres concentre :
- uniquement des `pieces`
- uniquement des cas `empty`
- presque uniquement sur le champ `Diamètre`
- presque toute la liste est dans la categorie `Arbre`
Cela ressemble davantage a une vague de perte coherente qu'a du bruit metier aleatoire.
### 2. Valeurs d'audit tres exploitables
Les valeurs retrouvees sont directement reutilisables :
- `35 mm`
- `40 mm`
- `45 mm`
- `50 mm`
- `55 mm`
- `70 mm`
- `20 DTS`
### 3. Cas particulier multi-champs
L'entite `cl8da1b875191c617e5852bf81` a deux champs recuperables :
- `Diamètre`
- `Diamètre palier`
### 4. Piece initialement signalee
La piece `cl731386df55fcb9e6a01e0a63` est bien presente dans le resultat :
- nom : `Arbre de la cage écureuil pied E2 E2B`
- champ : `Diamètre`
- valeur recuperable : `35 mm`
---
## Priorite De Restauration
Priorite haute :
- restaurer tout ce lot `Arbre` en premier
- ce sont des cas homogènes et recuperables
Ordre recommande :
1. piece `cl731386df55fcb9e6a01e0a63`
2. piece avec plusieurs champs recuperables : `cl8da1b875191c617e5852bf81`
3. reste du lot `Arbre`
---
## Commandes Utiles
Dry-run pour une piece :
```bash
php bin/console app:restore-piece-custom-field-values <pieceId>
```
Application reelle :
```bash
php bin/console app:restore-piece-custom-field-values <pieceId> --apply
```
Exemple pour la piece initiale :
```bash
php bin/console app:restore-piece-custom-field-values cl731386df55fcb9e6a01e0a63
php bin/console app:restore-piece-custom-field-values cl731386df55fcb9e6a01e0a63 --apply
```

137
docs/DOUBLONS_REFERENCES_COMPOSANTS.md Normal file
View File

@@ -0,0 +1,137 @@
# Doublons de références — Composants
> Généré le 2026-03-26 à partir du dump de production `inventory (17).sql.gz`
**13 références en doublon** pour un total de **41 composants concernés**.
## Résumé
| Référence | Nb | Composants |
|---|---|---|
| Tambour lisse | 9 | Tambour tête E1, E2 E2B, E3, E4, E5, E6, E7, E8, élévateur expédition |
| FY50 FM | 5 | Opposé commande Vis 21, Palier Opposé Commande Vis 19, Palier Vis 18 (côté commande), Palier Vis 21 (côté commande), Palier côté commande Vis 20 |
| PB 2220 | 4 | Réducteur pendulaire E1, E3, E8, élévateur expédition |
| SNU 511 609 | 4 | Palier pied E1, E3, E8, élévateur expédition |
| SNU 516 613 | 4 | Palier tête E1, E3, E8, élévateur expédition |
| 512610 SNH SKF | 3 | Palier tête E4, E5, E6 |
| FY 50 FM | 2 | Palier V18 (opposé commande), Palier côté commande Vis 19 |
| FY60 | 2 | Palier Vis 17 (coté commande), Palier Vis 17 (opposé commande) |
| FY60 WF | 2 | Palier Opposé commande Vis 22, Palier côté commande Vis 22 |
| PB 2012 | 2 | Réducteur pendulaire E2-E2B, E7 |
| PB 2112 | 2 | Réducteur pendulaire E4, E6 |
| SNU 509 | 2 | Palier tête E2 et E2B, E7 |
| VCF 207 | 2 | Palier pied E2 et E2B, E7 |
## Détail par référence
### Tambour lisse (9 composants)
| Nom | ID |
|---|---|
| Tambour tête E1 | cl4660bae41d2af254e6c3b726 |
| Tambour tête E2 E2B | cl5e9c6b18bccd38517026dc1c |
| Tambour tête E3 | clba5633e840726188261145f9 |
| Tambour tête E4 | cl10c0924d10135c5f515378ac |
| Tambour tête E5 | cl7f254c23161d9c853c3e6d92 |
| Tambour tête E6 | cl3dbac5194bc192a0589465ba |
| Tambour tête E7 | cla833681664bb851ca61aca51 |
| Tambour tête E8 | cl36d84884cad86fbc92dba133 |
| Tambour tête élévateur expédition | cl5a8f9656aa7e14c012f30700 |
### FY50 FM (5 composants)
| Nom | ID |
|---|---|
| Opposé commande Vis 21 | cl055eff4115f9c75d850c9459 |
| Palier Opposé Commande Vis 19 | cl6831a23892243bbaa2f823b4 |
| Palier Vis 18 (côté commande) | cld1391112241147dc064b35da |
| Palier Vis 21 (côté commande) | cl9f8253f4537a657f7378a2e9 |
| Palier côté commande Vis 20 | cl203937da81135d8b34d7bb0f |
### PB 2220 (4 composants)
| Nom | ID |
|---|---|
| Réducteur pendulaire E1 | cla59f867feafbb0937862064a |
| Réducteur pendulaire E3 | cl33683086c4de13f80db59606 |
| Réducteur pendulaire E8 | cl94fb77cf922aa1462a8d13cc |
| Réducteur pendulaire élévateur expédition | cl3f02941228dfef4c91a75d1a |
### SNU 511 609 (4 composants)
| Nom | ID |
|---|---|
| Palier pied E1 | cl81e703e9f200163a4ea473df |
| Palier pied E3 | cl3d38928c11d70614bb09fe8e |
| Palier pied E8 | cl78b79a8f90f12842b5683403 |
| Palier pied élévateur expédition | clf35b4455617ae94f2a1add46 |
### SNU 516 613 (4 composants)
| Nom | ID |
|---|---|
| Palier tête E1 | cmkr0nq1a004e1eq6v6ubxlfl |
| Palier tête E3 | cl92b8908c71616c542d958007 |
| Palier tête E8 | clce6dde0609d90764da383d75 |
| Palier tête élévateur expédition | clb7322b05f9a4554fa5a75d5a |
### 512610 SNH SKF (3 composants)
| Nom | ID |
|---|---|
| Palier tête E4 | cl8e90ad1b633046f5f1344b93 |
| Palier tête E5 | clbbe4096490ff89b08644c793 |
| Palier tête E6 | cl51c9a1c3dce52856e3404a38 |
### FY 50 FM (2 composants)
| Nom | ID |
|---|---|
| Palier V18 (opposé commande) | cl2ff55d9fa9c52c18f2d88222 |
| Palier côté commande Vis 19 | clbddd1dca5efa881b23eaa1cd |
### FY60 (2 composants)
| Nom | ID |
|---|---|
| Palier Vis 17 (coté commande) | cl02b0a0a543cc699681b6ae8c |
| Palier Vis 17 (opposé commande) | clc0ba9245b63613307cc26a19 |
### FY60 WF (2 composants)
| Nom | ID |
|---|---|
| Palier Opposé commande Vis 22 | cl318b49462097fb2e1f793305 |
| Palier côté commande Vis 22 | cl6bc818a2d8661b5e0ce2d0c0 |
### PB 2012 (2 composants)
| Nom | ID |
|---|---|
| Réducteur pendulaire E2-E2B | cl9b746a66f583fc85b3d176c4 |
| Réducteur pendulaire E7 | clc0db3b431d75c6355608efd5 |
### PB 2112 (2 composants)
| Nom | ID |
|---|---|
| Réducteur pendulaire E4 | clf5a1c9e1f8202b632f173bd3 |
| Réducteur pendulaire E6 | cle1899c6522cb8b8abd366a24 |
### SNU 509 (2 composants)
| Nom | ID |
|---|---|
| Palier tête E2 et E2B | cl4e600dcadb34f817a888ffa3 |
| Palier tête E7 | cl84271e9ab5351cbd188b0d3a |
### VCF 207 (2 composants)
| Nom | ID |
|---|---|
| Palier pied E2 et E2B | cld516a118bb1c478722a1d39b |
| Palier pied E7 | cl908dbf171798f087b12d6f2a |
## Note
Ces doublons sont des composants **distincts** (noms différents, installés sur différents élévateurs) qui partagent la même référence fournisseur. Il ne s'agit pas nécessairement d'entrées à fusionner, mais de pièces identiques utilisées à plusieurs emplacements.

399
docs/FONCTIONNEMENT.md Normal file
View File

@@ -0,0 +1,399 @@
# Fonctionnement de l'application Inventory
## 1. A quoi sert cette application ?
Inventory est une application de **gestion d'inventaire industriel**. Elle permet de suivre et documenter l'ensemble du parc de machines d'une entreprise, avec tous les elements qui les composent : composants, pieces detachees et produits consommables.
L'objectif principal est d'avoir une **vue complete et structuree** de chaque machine : quels composants elle contient, quelles pieces sont montees dessus, quels produits sont utilises, qui les fabrique, combien ils coutent, et toute la documentation associee (manuels, fiches techniques, etc.).
---
## 2. Les entites principales
L'application s'articule autour de 7 entites fondamentales :
```
+-----------------------------------------------------------+
| SITE |
| (usine, atelier, entrepot...) |
| - nom, adresse, contact, telephone, ville, code postal |
| - couleur (pour identification visuelle) |
+-----------------------------------------------------------+
|
| contient des
v
+-----------------------------------------------------------+
| MACHINE |
| (machine industrielle sur un site) |
| - nom (unique), reference, prix |
| - rattachee a 1 site obligatoirement |
| - peut avoir plusieurs fournisseurs/constructeurs |
+-----------------------------------------------------------+
|
| est composee de
v
+-------------------+ +-------------------+ +-------------------+
| COMPOSANT | | PIECE | | PRODUIT |
| (element fonct.) | | (piece detachee) | | (consommable) |
| - nom, ref, desc | | - nom, ref, desc | | - nom, ref |
| - prix | | - prix | | - prix fournisseur |
| - categorie | | - categorie | | - categorie |
| - fournisseurs | | - fournisseurs | | - fournisseurs |
+-------------------+ +-------------------+ +-------------------+
```
### Site
Un **site** represente un lieu physique : une usine, un atelier, un entrepot. Chaque site possede un nom, une adresse complete et un contact. Toutes les machines sont obligatoirement rattachees a un site.
### Machine
Une **machine** est l'entite centrale. C'est un equipement industriel installe sur un site. Chaque machine a un nom unique, une reference optionnelle et un prix. Elle contient une structure hierarchique de composants, pieces et produits.
### Composant
Un **composant** represente un element fonctionnel d'une machine (ex : un moteur, un systeme hydraulique, un automate). Un composant peut lui-meme contenir des sous-composants, des pieces et des produits, formant une structure arborescente.
### Piece
Une **piece** est une piece detachee (ex : un roulement, un joint, un filtre). Les pieces peuvent etre rattachees directement a une machine ou a un composant au sein d'une machine.
### Produit
Un **produit** est un consommable ou article fournisseur (ex : huile, lubrifiant, boulon specifique). Comme les pieces, les produits peuvent etre associes a une machine, a un composant ou a une piece.
### Constructeur (Fournisseur)
Un **constructeur** est un fabricant ou fournisseur. C'est un referentiel partage : le meme fournisseur peut etre associe a des machines, des composants, des pieces et des produits. Chaque fournisseur a un nom, un email et un telephone.
### Categorie (ModelType)
Une **categorie** (appelee ModelType dans le systeme) permet de classifier les composants, les pieces et les produits. Le systeme de categories est explique en detail dans la section suivante.
---
## 3. Le systeme de categories (ModelType)
Les categories sont un element cle de l'application. Elles servent a **classifier ET a structurer** les elements de l'inventaire.
### Trois familles de categories
Il existe trois familles de categories, une par type d'element :
| Famille | S'applique aux | Exemples |
|-------------|----------------|-----------------------------------------|
| COMPONENT | Composants | "Moteur electrique", "Systeme hydraulique" |
| PIECE | Pieces | "Roulement", "Joint torique", "Filtre" |
| PRODUCT | Produits | "Huile moteur", "Graisse", "Boulon M8" |
### Le squelette (skeleton) : la structure imposee
La vraie puissance des categories de composants reside dans leur **squelette**. Quand on cree une categorie de composant, on definit un modele qui impose :
- **Quelles pieces** sont necessaires (par type de piece)
- **Quels produits** sont necessaires (par type de produit)
- **Quels sous-composants** sont necessaires (par type de composant)
- **Quels champs personnalises** doivent etre remplis
**Exemple concret :** La categorie "Moteur electrique" pourrait imposer :
- 1 piece de type "Roulement"
- 1 piece de type "Joint"
- 1 produit de type "Huile moteur"
- 1 sous-composant de type "Variateur"
- Des champs personnalises : "Puissance (kW)", "Vitesse (tr/min)", "Tension (V)"
```
Categorie "Moteur electrique" (squelette)
|
|-- Piece requise : type "Roulement" --> l'utilisateur choisira quel roulement precis
|-- Piece requise : type "Joint" --> l'utilisateur choisira quel joint precis
|-- Produit requis : type "Huile moteur" --> l'utilisateur choisira quelle huile precise
|-- Sous-composant : type "Variateur" --> l'utilisateur choisira quel variateur precis
|-- Champ personnalise : "Puissance (kW)" --> l'utilisateur saisira la valeur
|-- Champ personnalise : "Tension (V)" --> l'utilisateur saisira la valeur
```
Les categories de pieces peuvent elles aussi definir des produits requis et des champs personnalises. Les categories de produits peuvent definir des champs personnalises.
### Champs personnalises
Les champs personnalises permettent d'ajouter des informations specifiques selon la categorie. Chaque champ a :
- Un **nom** (ex : "Puissance")
- Un **type** (texte, nombre, date, etc.)
- Un caractere **obligatoire ou non**
- Des **options** possibles (pour les listes deroulantes)
- Une **valeur par defaut**
- Un **ordre d'affichage**
Les machines disposent aussi de champs personnalises, mais ceux-ci sont definis directement sur chaque machine (et non via une categorie).
---
## 4. Le cycle de vie d'un composant
Voici les etapes typiques de creation et utilisation d'un composant :
```
1. CREATION 2. SELECTION CATEGORIE 3. REMPLISSAGE SQUELETTE
+-------------------+ +------------------------+ +---------------------------+
| Saisir : | | Choisir la categorie : | | Le squelette apparait : |
| - Nom | ----> | "Moteur electrique" | --> | - Piece "Roulement" : [?] |
| - Reference | | | | - Piece "Joint" : [?] |
| - Description | | Le systeme charge le | | - Produit "Huile" : [?] |
| - Prix | | squelette associe | | |
| - Fournisseurs | +------------------------+ | Choisir dans le catalogue |
+-------------------+ | chaque element concret |
+---------------------------+
|
5. DOCUMENTS 4. CHAMPS PERSONNALISES
+---------------------+ +-----------------------------+
| Joindre des fichiers | <---- | Remplir les champs definis |
| - Manuels PDF | | par la categorie : |
| - Fiches techniques | | - Puissance : 15 kW |
| - Photos | | - Tension : 400 V |
| - Schemas | | - Vitesse : 1500 tr/min |
+---------------------+ +-----------------------------+
```
**Etape 1 - Creation :** L'utilisateur saisit les informations de base du composant (nom, reference, description, prix) et selectionne un ou plusieurs fournisseurs.
**Etape 2 - Selection de la categorie :** L'utilisateur choisit la categorie du composant (ex : "Moteur electrique"). Le systeme charge alors le squelette defini pour cette categorie.
**Etape 3 - Remplissage du squelette :** Des "emplacements" (slots) apparaissent pour chaque element requis par le squelette. L'utilisateur selectionne dans le catalogue existant les pieces, produits et sous-composants concrets qui correspondent a chaque emplacement.
**Etape 4 - Champs personnalises :** L'utilisateur remplit les champs personnalises definis par la categorie (puissance, tension, etc.).
**Etape 5 - Documents :** L'utilisateur peut joindre des fichiers au composant : manuels PDF, fiches techniques, photos, schemas...
Ce meme principe s'applique aux pieces (qui peuvent avoir des produits associes et des champs personnalises definis par leur categorie) et aux produits (qui peuvent avoir des champs personnalises).
---
## 5. Les roles utilisateurs
L'application utilise 4 niveaux de droits, organises en hierarchie. Chaque role herite automatiquement des droits du role inferieur :
```
+------------------------------------------------------------------+
| ROLE_ADMIN |
| Tout faire + gerer les utilisateurs (creer, modifier, supprimer |
| des comptes, attribuer des roles) |
+------------------------------------------------------------------+
| herite de
v
+------------------------------------------------------------------+
| ROLE_GESTIONNAIRE |
| Creer, modifier et supprimer les machines, composants, pieces, |
| produits, sites, fournisseurs, categories, documents, |
| commentaires. C'est le role d'edition principal. |
+------------------------------------------------------------------+
| herite de
v
+------------------------------------------------------------------+
| ROLE_VIEWER |
| Consulter tout l'inventaire en lecture seule : naviguer dans |
| les machines, voir les structures, les catalogues, l'historique |
| et les documents. |
+------------------------------------------------------------------+
| herite de
v
+------------------------------------------------------------------+
| ROLE_USER |
| Role de base attribue automatiquement a tout utilisateur |
| connecte. Acces minimal. |
+------------------------------------------------------------------+
```
En resume :
- **Admin** : fait tout, y compris gerer les comptes utilisateurs
- **Gestionnaire** : cree et modifie les donnees de l'inventaire
- **Viewer** : consulte l'inventaire sans pouvoir le modifier
- **User** : role de base, acces minimal
---
## 6. Les fonctionnalites cles
### Catalogues
L'application propose des **catalogues** pour chaque type d'element :
- **Catalogue des composants** : liste tous les composants avec recherche par nom, reference ou categorie
- **Catalogue des pieces** : liste toutes les pieces detachees
- **Catalogue des produits** : liste tous les produits fournisseurs
- **Liste des machines** : toutes les machines, organisees par site
- **Liste des sites** : tous les sites industriels
- **Liste des fournisseurs** : tous les constructeurs/fournisseurs
Chaque catalogue offre des fonctions de **recherche**, de **tri** et de **pagination**.
### Recherche
La recherche est disponible dans tous les catalogues et permet de filtrer par :
- Nom (recherche partielle, insensible a la casse)
- Reference (recherche partielle)
- Categorie (filtre exact ou par nom)
### Historique et audit
Chaque modification dans l'application est **tracee automatiquement**. Le systeme enregistre :
- **Qui** a fait la modification (quel utilisateur)
- **Quand** la modification a ete faite
- **Quoi** a ete modifie (les champs avant/apres)
- **Sur quel element** (machine, composant, piece, produit...)
On peut consulter :
- L'**historique d'une entite** : toutes les modifications apportees a une machine, un composant, etc.
- Le **journal d'activite global** : toutes les modifications recentes dans l'application
### Commentaires
Les utilisateurs peuvent **commenter** n'importe quel element de l'inventaire (machines, composants, pieces, produits, categories). Les commentaires ont un systeme de **resolution** : un commentaire ouvert peut etre marque comme "resolu" par un gestionnaire. Un compteur de commentaires non resolus est disponible.
### Documents
Des fichiers peuvent etre joints a toutes les entites principales :
- **Sites** : plans, reglements
- **Machines** : manuels, fiches techniques
- **Composants** : documentations constructeur
- **Pieces** : plans de pieces, specifications
- **Produits** : fiches de donnees de securite, catalogues
Les fichiers sont uploades via l'interface et peuvent etre consultes ou telecharges a tout moment. L'application gere differents formats : PDF, images, etc.
### Clonage de machines
Quand une nouvelle machine est identique ou similaire a une existante, il est possible de **cloner une machine**. Le clonage copie :
- Les champs personnalises et leurs valeurs
- Toute la structure : les liens vers les composants, pieces et produits
- La hierarchie (quel composant contient quelles pieces, etc.)
L'utilisateur choisit un nouveau nom et un site de destination. La machine clonee peut ensuite etre modifiee independamment de l'originale.
---
## 7. La structure des machines
### Vue d'ensemble
Chaque machine possede une **structure hierarchique** qui decrit de quoi elle est composee. Cette structure est une arborescence :
```
Machine "Presse hydraulique PH-200"
|
|-- Composant "Moteur principal M1"
| |-- Piece "Roulement SKF 6205" (quantite: 2)
| | |-- Produit "Graisse SKF LGMT2"
| |-- Piece "Joint Viton DN50"
| |-- Produit "Huile Total Azolla ZS 46"
| |-- Sous-composant "Variateur ABB ACS580"
| |-- Piece "Fusible 63A"
| |-- Produit "Pate thermique"
|
|-- Composant "Groupe hydraulique GH-01"
| |-- Piece "Filtre Parker 926169Q"
| |-- Piece "Verin Bosch CDT3" (quantite: 4)
| |-- Produit "Huile hydraulique HLP 46"
|
|-- Piece "Courroie Gates PowerGrip" (piece directement sur la machine)
|-- Produit "Boulon M12x50 Inox" (produit directement sur la machine)
```
### Les liens (links)
Les elements ne sont pas directement "dans" la machine. Ils y sont rattaches par des **liens** :
- **MachineComponentLink** : rattache un composant a une machine
- **MachinePieceLink** : rattache une piece a une machine
- **MachineProductLink** : rattache un produit a une machine
Ces liens permettent :
- De definir la **hierarchie** : un composant peut etre parent d'une piece ou d'un produit, un sous-composant peut etre enfant d'un autre composant
- De specifier une **quantite** (ex : 4 verins identiques)
- De faire des **surcharges** : modifier le nom, la reference ou le prix d'un element specifiquement dans le contexte de cette machine, sans modifier l'element du catalogue
### Hierarchie parent-enfant
```
MachineComponentLink (composant dans la machine)
|
|-- parentLink --> null (composant racine, directement dans la machine)
| ou
|-- parentLink --> autre MachineComponentLink (sous-composant)
|
|-- pieceLinks --> MachinePieceLink[] (pieces de ce composant)
|-- productLinks --> MachineProductLink[] (produits de ce composant)
MachinePieceLink (piece dans la machine)
|
|-- parentLink --> MachineComponentLink (piece rattachee a un composant)
| ou
|-- parentLink --> null (piece directement sur la machine)
|
|-- productLinks --> MachineProductLink[] (produits de cette piece)
```
### Catalogue vs. Structure machine
Un point important : les **composants, pieces et produits existent dans un catalogue global**. Quand on les ajoute a une machine, on cree un lien vers l'element du catalogue. Le meme composant du catalogue peut donc etre utilise dans plusieurs machines.
Les surcharges (nom, reference, prix) permettent d'adapter les informations au contexte d'une machine specifique sans modifier la fiche catalogue.
```
CATALOGUE (reference globale) MACHINE (utilisation specifique)
+-------------------------+ +--------------------------------+
| Composant "Moteur 15kW" | | Lien vers "Moteur 15kW" |
| Ref: MOT-15-01 | <-------- | Surcharge nom: "Moteur gauche" |
| Prix: 2500 EUR | | Surcharge prix: 2200 EUR |
+-------------------------+ +--------------------------------+
```
---
## Schemas recapitulatifs
### Relations entre entites
```
+--------+
| Site |
+--------+
|
contient (1..N)
|
+-----------+
| Machine |------------ Fournisseurs (N..N)
+-----------+
/ | \
/ | \
Composants Pieces Produits
(via liens) (via liens) (via liens)
+-----------+ +--------+ +---------+
| Composant | | Piece | | Produit |
+-----------+ +--------+ +---------+
| | |
|-- Categorie |-- Categorie |-- Categorie
|-- Fournisseurs -- Fournisseurs -- Fournisseurs
|-- Documents |-- Documents |-- Documents
|-- Champs perso -- Champs perso -- Champs perso
|
|-- Sous-composants (arborescence)
|-- Pieces (slots depuis le squelette)
|-- Produits (slots depuis le squelette)
```
### Flux de creation typique
```
1. Creer les SITES
|
2. Creer les CATEGORIES (avec leurs squelettes)
|
3. Creer les FOURNISSEURS
|
4. Creer les PRODUITS (en les categorisant)
|
5. Creer les PIECES (en les categorisant, en leur associant des produits)
|
6. Creer les COMPOSANTS (en choisissant une categorie,
| en remplissant le squelette avec des pieces/produits/sous-composants)
|
7. Creer les MACHINES (sur un site)
|
8. STRUCTURER les machines (ajouter composants, pieces, produits)
|
9. DOCUMENTER (joindre des fichiers a chaque element)
```

146
docs/GLOSSAIRE_METIER.md Normal file
View File

@@ -0,0 +1,146 @@
# Glossaire Métier — Inventory
## Contexte
**Inventory** est une application de gestion de parc machines industriel. Elle permet aux équipes de maintenance de cataloguer leurs machines, leurs sous-ensembles (composants), les pièces de rechange et les consommables associés. Chaque machine est rattachée à un site physique (usine, atelier). L'application gère la hiérarchie complète : Machine → Composants → Pièces/Produits, avec traçabilité (audit), documentation technique et champs personnalisables.
---
## Concepts Métier
### Hiérarchie principale
| Terme | Définition | Exemples concrets |
|-------|-----------|-------------------|
| **Site** | Lieu physique (usine, atelier, entrepôt). Regroupe les machines d'un même emplacement. | Usine de Lyon, Atelier Nord |
| **Machine** | Équipement industriel installé sur un site. C'est l'unité de base du parc. Contient des composants, pièces et produits. | Presse hydraulique, Tour CNC, Ligne d'embouteillage |
| **Composant** | Sous-ensemble fonctionnel d'une machine. Peut contenir des pièces, des produits, et d'autres sous-composants (imbrication). | Moteur, Pompe, Tableau électrique, Vérin |
| **Pièce** | Pièce mécanique/physique qu'on monte ou remplace. C'est l'unité de maintenance. | Joint, Écrou, Roulement, Capteur, Courroie |
| **Produit** | Consommable qu'on utilise sans monter. S'use et se renouvelle. | Huile, Dégraissant, Graisse, Liquide de refroidissement |
### Configuration et templates
| Terme | Définition |
|-------|-----------|
| **Modèle Type** (ModelType) | Template réutilisable qui définit la composition attendue d'un composant, d'une pièce ou d'un produit. Par exemple : "Pompe centrifuge XYZ nécessite 2 joints, 1 roulement et de l'huile hydraulique". |
| **Skeleton** (squelette) | La structure "vide" définie par un modèle type : la liste des emplacements requis (pièces, produits, sous-composants) avant qu'on y mette les éléments réels. |
| **Slot** (emplacement) | Emplacement concret dans un composant ou une pièce, créé à partir du skeleton. Chaque slot est à remplir avec une pièce, un produit ou un sous-composant réel. Un slot peut rester vide (pas encore sourcé). |
| **Sync** (synchronisation) | Propagation des modifications d'un modèle type vers tous les composants existants de ce type. Par exemple : ajouter un slot "filtre" au modèle type met à jour tous les composants de ce type. Surtout utilisé en phase de saisie initiale, quand on ajuste les modèles au fur et à mesure qu'on découvre la vraie composition des machines. |
| **Catégorie de modèle** | Un modèle type est classé en 3 catégories : Composant, Pièce ou Produit. Détermine quels skeletons il peut définir. |
### Transverse
| Terme | Définition |
|-------|-----------|
| **Constructeur** | Fournisseur ou fabricant. Peut être associé à une machine, un composant, une pièce ou un produit. Permet de tracer la chaîne d'approvisionnement. |
| **Champ personnalisé** (CustomField) | Attribut dynamique défini par l'utilisateur et attaché à une machine ou à un modèle type. Les composants/pièces/produits d'un même modèle type partagent les mêmes champs personnalisés. Exemples : "N° de série", "Date de garantie", "Intervalle de maintenance". |
| **Document** | Fichier attaché à n'importe quelle entité (machine, composant, pièce, produit, site, commentaire). Typé : Documentation, Devis, Facture, Plan, Photo, Autre. |
| **Commentaire** | Annotation utilisateur sur une entité, avec un statut ouvert ou résolu. Permet de signaler un problème, poser une question ou laisser une note. Peut contenir des pièces jointes. |
| **Journal d'audit** (AuditLog) | Historique automatique et immuable de toutes les créations, modifications et suppressions. Enregistre qui a fait quoi, quand, avec le détail des changements. |
### Utilisateurs et rôles
| Rôle | Droits |
|------|--------|
| **Admin** | Accès complet : gestion des utilisateurs, configuration, toutes les opérations |
| **Gestionnaire** | Créer, modifier, supprimer des machines/composants/pièces/produits |
| **Viewer** | Consultation seule, pas de modification |
| **User** | Rôle de base (accès minimal) |
---
## Workflows Utilisateur
### 1. Créer une machine
1. Choisir le **site** où la machine est installée
2. Renseigner nom, référence, prix, fournisseur(s)
3. Ajouter des **composants** à la machine (voir workflow 2)
4. Ajouter des **pièces** et **produits** directement sur la machine si nécessaire
5. Ajouter des **champs personnalisés** et des **documents**
### 2. Ajouter un composant à une machine
1. Choisir un **modèle type** pour le composant (ex: "Pompe centrifuge XYZ")
2. Les **slots** sont pré-créés automatiquement à partir du skeleton du modèle type
3. Remplir chaque slot en sélectionnant la pièce/produit/sous-composant réel
4. Les slots peuvent rester vides et être remplis plus tard
### 3. Créer ou modifier un modèle type
1. Nommer le modèle type et choisir sa catégorie (Composant, Pièce ou Produit)
2. Définir les emplacements requis : quelles pièces, quels produits, quels sous-composants
3. Définir les champs personnalisés (métadonnées) pour les entités de ce type
4. Si des composants existent déjà avec ce modèle type → utiliser le **sync** (workflow 4)
### 4. Synchroniser un modèle type
1. Modifier les emplacements du modèle type (ajout/suppression de slots)
2. Lancer un **sync preview** : visualiser l'impact sur les composants existants
3. Confirmer → les slots sont ajoutés/supprimés sur tous les composants du type
4. Surtout utile en phase de saisie initiale quand les données sont ajustées progressivement
### 5. Cloner une machine
1. Sélectionner une machine existante
2. Lancer le clonage → copie complète (composants, pièces, produits, liens, champs personnalisés)
3. Renommer la machine clonée et l'affecter à un site
### 6. Gérer les documents
1. Sélectionner une entité (machine, composant, pièce, produit, site)
2. Uploader un fichier (PDF, image, etc.)
3. Choisir le type : Documentation, Devis, Facture, Plan, Photo, Autre
4. Les documents sont consultables et téléchargeables depuis la fiche de l'entité
---
## Relations — Vue d'ensemble
```
Site
└── Machine
├── Composant (→ défini par un Modèle Type)
│ ├── Slot Pièce → Pièce (joint, écrou…)
│ ├── Slot Produit → Produit (huile, dégraissant…)
│ └── Slot Sous-composant → Composant (imbrication)
├── Pièce (directement sur la machine)
│ └── Slot Produit → Produit
└── Produit (directement sur la machine)
Modèle Type (template)
├── Skeleton Pièce Requirement → "il faut une pièce de type X"
├── Skeleton Produit Requirement → "il faut un produit de type Y"
└── Skeleton Sous-composant Requirement → "il faut un composant de type Z"
Transverse (attachable à toute entité) :
• Constructeur (fournisseur)
• Document (fichier)
• Commentaire (annotation)
• Champ personnalisé (métadonnée dynamique)
• Journal d'audit (historique automatique)
```
---
## Correspondance Métier ↔ Code
| Terme métier | Entité code | Table PG |
|-------------|-------------|----------|
| Site | `Site` | `site` |
| Machine | `Machine` | `machine` |
| Composant | `Composant` | `composant` |
| Pièce | `Piece` | `piece` |
| Produit | `Product` | `product` |
| Modèle Type | `ModelType` | `model_type` |
| Slot pièce (composant) | `ComposantPieceSlot` | `composant_piece_slot` |
| Slot produit (composant) | `ComposantProductSlot` | `composant_product_slot` |
| Slot sous-composant | `ComposantSubcomponentSlot` | `composant_subcomponent_slot` |
| Slot produit (pièce) | `PieceProductSlot` | `piece_product_slot` |
| Skeleton pièce | `SkeletonPieceRequirement` | `skeleton_piece_requirement` |
| Skeleton produit | `SkeletonProductRequirement` | `skeleton_product_requirement` |
| Skeleton sous-composant | `SkeletonSubcomponentRequirement` | `skeleton_subcomponent_requirement` |
| Constructeur | `Constructeur` | `constructeur` |
| Champ personnalisé | `CustomField` | `custom_field` |
| Valeur champ perso | `CustomFieldValue` | `custom_field_value` |
| Document | `Document` | `document` |
| Commentaire | `Comment` | `comment` |
| Journal d'audit | `AuditLog` | `audit_log` |
| Utilisateur | `Profile` | `profile` |
| Lien machine-composant | `MachineComponentLink` | `machine_component_link` |
| Lien machine-pièce | `MachinePieceLink` | `machine_piece_link` |
| Lien machine-produit | `MachineProductLink` | `machine_product_link` |

346
docs/REVIEW_ARCHITECTURE.md Normal file
View File

@@ -0,0 +1,346 @@
# Revue d'architecture - Sources de complexite et effets de bord
Date : 2026-03-23
Branche analysee : `develop`
---
## Diagnostic - Top 10 des sources de complexite
| # | Source | Impact | Effort |
|---|--------|--------|--------|
| 1 | Duplication massive du `smartMatch` dans les Sync Strategies | Bugs silencieux, maintenance triple | M |
| 2 | Custom Fields : 4 FK nullable sur une seule entite (polymorphisme pauvre) | Integrite fragile, code defensif partout | L |
| 3 | Composables frontend geants avec responsabilites multiples | Difficile a tester, refactoring risque | M |
| 4 | 3 fichiers utils de custom fields frontend avec logique qui se chevauche | Incoherences, bugs de merge/dedup | M |
| 5 | `pendingStructure` : canal de communication cache entre deserialisation et processor | Effet de bord invisible, timing fragile | S |
| 6 | `PieceProductSyncSubscriber` : legacy sync dans un subscriber Doctrine | Side effect cache, recompute du changeset | S |
| 7 | Double flush dans les processors (decorated + flush manuel) | Audit logs potentiellement incomplets | S |
| 8 | `MachineStructureController` : God controller avec normalisation JSON manuelle | Bypass API Platform, 300+ LOC de serialisation | L |
| 9 | Chaine de dependances circulaire dans `useMachineDetailData` | Proxy refs, ordre d'initialisation fragile | M |
| 10 | Frontend : typage `any` systematique sur les entites | Pas de filet de securite TypeScript | L |
---
## Analyse detaillee
### 1. Duplication du `smartMatch` dans les Sync Strategies
**Fichiers concernes :**
- `/src/Service/Sync/ComposantSyncStrategy.php` (lignes 380-446)
- `/src/Service/Sync/PieceSyncStrategy.php` (lignes 244-308)
**Probleme :** `smartMatch()`, `smartMatchPreview()` et toute la logique de sync des custom field values sont copiees-collees entre `ComposantSyncStrategy` et `PieceSyncStrategy`. Le `ProductSyncStrategy` a une version simplifiee (pas de slots). Si un bug est corrige dans l'un, il faut penser a le corriger dans l'autre.
**Effets de bord concrets :**
- Un correctif sur le matching des slots dans une strategie peut etre oublie dans l'autre
- Le compteur de preview custom fields utilise `orderIndex` comme cle de matching, ce qui est fragile (reindexation = faux positif)
**Solution proposee (effort M) :**
Extraire un trait ou une classe abstraite `AbstractSlotSyncStrategy` :
```php
// AVANT : smartMatch() duplique dans ComposantSyncStrategy et PieceSyncStrategy
// APRES : extraire dans un trait
trait SlotSyncTrait
{
protected function smartMatch(array $existingTypeIds, array $proposedTypeIds): array
{
// ... logique unique
}
protected function syncCustomFieldValues(
object $entity,
string $fkField,
array $customFields,
bool $confirmDeletions,
): array {
// ... logique unique pour add/remove CFValues
}
}
```
La methode `execute()` de chaque strategie ne garderait que la boucle specifique a son type de slot (piece slots, product slots, subcomponent slots), et deleguerait le matching et la gestion des CF values au trait.
---
### 2. Custom Fields : polymorphisme par FK nullable
**Fichiers concernes :**
- `/src/Entity/CustomField.php` - 4 FK nullable : `machine`, `typeComposant`, `typePiece`, `typeProduct`
- `/src/Entity/CustomFieldValue.php` - 4 FK nullable : `machine`, `composant`, `piece`, `product`
- `/src/Controller/CustomFieldValueController.php` - `resolveTarget()` fait un switch sur 4 types
**Probleme :** Un `CustomFieldValue` peut pointer vers machine OU composant OU piece OU produit via 4 colonnes nullable. Rien n'empeche au niveau DB qu'un CFV pointe vers deux entites en meme temps. Le frontend doit deviner le type cible. Chaque nouveau type d'entite necessite d'ajouter une colonne, un setter, et un cas dans tous les switches.
**Effets de bord concrets :**
- Le `CustomFieldValueController::resolveTarget()` tente 4 cles dans un ordre specifique -- si le payload a `machineId` ET `composantId`, seul `machine` est utilise (silent bug)
- Les audit subscribers (`getOwnerFromCustomFieldValue`) doivent tester chaque getter -- si `getComposant()` renvoie un objet alors que `getMachine()` aussi, le comportement est indetermine
- La serialisation API Platform expose les 4 FK meme quand 3 sont null
**Solution proposee (effort L) :**
Option pragmatique (pas de refactoring DB) : ajouter une colonne discriminante `entityType` (enum) + contrainte CHECK :
```sql
ALTER TABLE custom_field_values
ADD COLUMN entity_type VARCHAR(20) NOT NULL DEFAULT 'machine';
ALTER TABLE custom_field_values
ADD CONSTRAINT chk_single_fk CHECK (
(entity_type = 'machine' AND machineId IS NOT NULL AND composantId IS NULL AND pieceId IS NULL AND productId IS NULL) OR
(entity_type = 'composant' AND composantId IS NOT NULL AND machineId IS NULL AND pieceId IS NULL AND productId IS NULL) OR
(entity_type = 'piece' AND pieceId IS NOT NULL AND machineId IS NULL AND composantId IS NULL AND productId IS NULL) OR
(entity_type = 'product' AND productId IS NOT NULL AND machineId IS NULL AND composantId IS NULL AND pieceId IS NULL)
);
```
Cela securise l'integrite sans changer l'architecture. Le `resolveTarget` et les audit subscribers pourraient ensuite brancher sur `entityType` au lieu de tester 4 FK.
---
### 3. Composables frontend geants (400-550 LOC)
**Fichiers concernes :**
- `/Inventory_frontend/app/composables/useComponentEdit.ts` (550 LOC)
- `/Inventory_frontend/app/composables/usePieceEdit.ts` (472 LOC)
- `/Inventory_frontend/app/composables/useMachineDetailData.ts` (468 LOC)
- `/Inventory_frontend/app/composables/useComponentCreate.ts` (417 LOC)
**Probleme :** Ces composables orchestrent en un seul fichier : le chargement de donnees, la gestion de formulaire, la persistence des custom fields, la gestion des documents, l'historique, la resolution de labels, et la soumission. Chacun instancie 8-12 sous-composables.
**Effets de bord concrets :**
- `useComponentEdit` instancie `usePieces()`, `useProducts()`, `useComposants()` a chaque montage de page, meme si ces catalogues sont deja charges -- requetes API en double
- La logique de soumission (`submitEdition`, `submitCreation`) melange la construction du payload, la validation, l'appel API, et la persistence des custom fields -- si une etape echoue, l'etat local est partiellement modifie
- Les watchers sur `selectedType`/`selectedTypeStructure` dans `useComponentCreate` et `useComponentEdit` font des choses differentes pour le meme concept -- source de divergence
**Solution proposee (effort M) :**
Decouper chaque composable geant en sous-composables par responsabilite, comme deja fait pour `useMachineDetailData` (qui delegue a `useMachineDetailDocuments`, `useMachineDetailCustomFields`, etc.) :
```
useComponentEdit.ts (550 LOC)
-> useComponentEditForm.ts (~100 LOC : reactive form, validation)
-> useComponentEditDocuments.ts (~80 LOC : upload, preview, delete)
-> useComponentEditSlots.ts (~120 LOC : slot selection/save)
-> useComponentEditCustomFields.ts (~60 LOC : build inputs, save)
-> useComponentEdit.ts (~150 LOC : orchestrateur)
```
Appliquer le meme pattern a `usePieceEdit` et `useComponentCreate`. Les blocs communs (document handling, custom field save, price formatting) deviendraient des composables partages.
---
### 4. Triple duplication de la logique custom fields frontend
**Fichiers concernes :**
- `/Inventory_frontend/app/shared/utils/customFieldFormUtils.ts` (404 LOC) - pour les pages create/edit
- `/Inventory_frontend/app/shared/utils/customFieldUtils.ts` (440 LOC) - pour la page machine detail
- `/Inventory_frontend/app/shared/utils/entityCustomFieldLogic.ts` (335 LOC) - pour les composants item
**Probleme :** Ces 3 fichiers resolvent le meme probleme (normaliser des definitions de custom fields + merger avec des valeurs existantes) avec des implementations differentes :
- `customFieldFormUtils.ts` : `resolveFieldName()`, `resolveFieldType()`, `buildCustomFieldInputs()`
- `entityCustomFieldLogic.ts` : `resolveFieldName()` (differente!), `resolveFieldType()` (differente!), `mergeFieldDefinitionsWithValues()`
- `customFieldUtils.ts` : `extractDefinitionName()`, `normalizeExistingCustomFieldDefinitions()`, `mergeCustomFieldValuesWithDefinitions()`
**Effets de bord concrets :**
- Trois facons differentes de resoudre le nom d'un champ -- `resolveFieldName` dans `customFieldFormUtils` teste `name`, `key`, `label` ; dans `entityCustomFieldLogic` elle teste `name` seulement et retourne `'Champ'` par defaut
- Trois algorithmes de merge values/definitions -- un bug corrige dans l'un n'est pas corrige dans les autres
- La deduplication par `name+type` dans `entityCustomFieldLogic.ts` et par `orderIndex` dans `customFieldUtils.ts` produit des resultats differents pour les memes donnees
**Solution proposee (effort M) :**
Fusionner en un seul module `customFields.ts` avec :
1. Une seule fonction `resolveFieldName(field: any): string`
2. Une seule fonction `mergeDefinitionsWithValues(defs, values): MergedField[]`
3. Une seule fonction `deduplicateFields(fields): MergedField[]`
Les 3 fichiers actuels deviendraient des re-exports ou des wrappers fins. Commencer par aligner les signatures, puis remplacer les imports un par un.
---
### 5. `pendingStructure` : canal de communication cache
**Fichiers concernes :**
- `/src/Entity/ModelType.php` et `/src/Entity/Composant.php` -- propriete `#[ApiProperty]` non mappee en DB
- `/src/State/ModelTypeProcessor.php` (lignes 33-43)
- `/src/State/ComposantProcessor.php` (lignes 42-51)
**Probleme :** Le champ `structure` envoye par le frontend est intercepte par API Platform dans un champ `pendingStructure` (non mappe en DB), puis lu par le processor apres le `persist` du decorated processor. Ce mecanisme est invisible : rien dans l'entite n'indique qu'un setter a un effet de bord differe.
**Effets de bord concrets :**
- Si le `decorated->process()` leve une exception, le `pendingStructure` reste dans l'entite -- pas de cleanup
- Le `flush()` supplementaire dans le processor (ligne 43 de `ModelTypeProcessor`) declenche les audit subscribers une deuxieme fois pour le meme cycle de request -- les snapshots d'audit peuvent capturer un etat intermediaire
- Un developpeur qui modifie le `ModelType` via Doctrine directement (fixture, migration, CLI) ne beneficie pas de ce mecanisme -- les skeleton requirements ne sont pas mis a jour
**Solution proposee (effort S) :**
Documenter explicitement ce pattern dans l'entite avec un docblock. Ajouter un `try/finally` pour le cleanup :
```php
// ModelTypeProcessor::process()
try {
$result = $this->decorated->process($data, $operation, $uriVariables, $context);
if (null !== $pendingStructure) {
$this->skeletonStructureService->updateSkeletonRequirements($data, $pendingStructure);
$this->entityManager->flush();
}
return $result;
} finally {
$data->clearPendingStructure();
}
```
---
### 6. `PieceProductSyncSubscriber` : side effect cache
**Fichier concerne :**
- `/src/EventSubscriber/PieceProductSyncSubscriber.php`
**Probleme :** Ce subscriber Doctrine ecoute `prePersist` et `preUpdate` pour synchroniser la relation legacy `product` (ManyToOne) avec la collection `productIds` (JSON array). Sur `preUpdate`, il fait un `recomputeSingleEntityChangeSet` (ligne 50-51), ce qui modifie le changeset en cours de flush.
**Effets de bord concrets :**
- Le recompute du changeset peut interferer avec les audit subscribers qui lisent ce meme changeset -- l'audit log peut capturer le changement de `product` comme une modification manuelle alors qu'il est automatique
- L'ordre d'execution des subscribers n'est pas garanti -- si l'audit subscriber s'execute avant le sync, il ne voit pas le changement de `product`
- Si `productIds` est vide, le subscriber ne touche pas `product` -- mais si `product` avait deja une valeur, elle reste (pas de cleanup)
**Solution proposee (effort S) :**
Remplacer ce subscriber par une logique explicite dans le controller/processor qui traite les pieces. Le sync `productIds -> product` devrait etre fait AVANT le flush, pas dans un subscriber. Cela supprime l'ambiguite sur l'ordre d'execution et le recompute.
Alternativement, si la relation legacy `product` (ManyToOne) n'est plus utilisee par le frontend, la supprimer completement et ne garder que `productIds` / les product slots.
---
### 7. Double flush dans les processors
**Fichiers concernes :**
- `/src/State/ModelTypeProcessor.php` (ligne 36 via decorated, ligne 43 manuellement)
- `/src/State/ComposantProcessor.php` (ligne 45 via decorated, ligne 132 manuellement)
**Probleme :** Le decorated processor fait un `flush()` pour persister l'entite, puis un second `flush()` est appele pour persister les skeleton requirements ou slots. Chaque flush declenche `onFlush` dans tous les audit subscribers.
**Effets de bord concrets :**
- Le premier flush capture le `create` de l'entite dans l'audit log
- Le second flush peut generer un `update` de la meme entite si les slots ont modifie une relation qui declenche un dirty check (par ex. si `$composant->incrementVersion()` etait appele)
- En cas d'erreur entre les deux flush, l'entite est persistee mais ses slots ne le sont pas -- etat inconsistant
**Solution proposee (effort S) :**
Wrapper les deux operations dans une transaction explicite, et ne faire qu'un seul flush a la fin :
```php
public function process(mixed $data, Operation $operation, ...): mixed
{
return $this->entityManager->wrapInTransaction(function () use ($data, $operation, ...) {
// Ne pas flush dans le decorated -- utiliser le mode COMMIT_ON_CLOSE
$result = $this->decorated->process($data, $operation, $uriVariables, $context);
if (null !== $pendingStructure) {
$this->skeletonStructureService->updateSkeletonRequirements($data, $pendingStructure);
}
$data->clearPendingStructure();
// Un seul flush
$this->entityManager->flush();
return $result;
});
}
```
> Note : cela necessite de verifier que le decorated processor ne fait pas deja un flush interne non configurable. Si c'est le cas, il faudrait potentiellement ne pas utiliser le decorated et gerer le persist manuellement.
---
### 8. `MachineStructureController` : God controller
**Fichier concerne :**
- `/src/Controller/MachineStructureController.php` (300+ LOC)
**Probleme :** Ce controller gere GET structure, PATCH structure, et POST clone. Il contient toute la logique de normalisation JSON des links (component, piece, product), la resolution des entites, et la serialisation manuelle de la reponse -- tout ce qu'API Platform fait normalement automatiquement.
**Effets de bord concrets :**
- La normalisation JSON manuelle (`normalizeStructureResponse`) ne passe pas par les serialization groups d'API Platform -- si un champ est ajoute a une entite avec un group, il n'apparaitra pas dans la reponse structure
- Le PATCH structure fait `$this->entityManager->flush()` sans transaction -- si la creation d'un link echoue, les precedents sont deja persistes
- Le clone copie les custom fields mais pas les documents -- comportement potentiellement inattendu
- 8 repositories injectes dans le constructeur -- code smell
**Solution proposee (effort L) :**
1. Extraire la logique de normalisation dans un `MachineStructureSerializer` service
2. Extraire la logique de clone dans un `MachineCloneService`
3. Wrapper le PATCH et le clone dans des transactions
4. A terme, considerer un DTO + custom provider/processor API Platform pour le GET/PATCH structure
---
### 9. Dependance circulaire dans `useMachineDetailData`
**Fichier concerne :**
- `/Inventory_frontend/app/composables/useMachineDetailData.ts` (lignes 119-187)
**Probleme :** `useMachineDetailProducts` a besoin de `machineProductLinks` (venant de hierarchy), et `useMachineDetailHierarchy` a besoin de `findProductById` (venant de products). La solution actuelle utilise un `_machineProductLinksProxy` ref avec un watcher pour synchroniser.
**Effets de bord concrets :**
- Le proxy ref est mis a jour de facon asynchrone via un watcher -- pendant le premier tick de rendu, `_machineProductLinksProxy` est vide meme si les liens sont deja charges
- L'ordre d'initialisation des sous-composables est fragile -- deplacer une ligne peut casser la boucle
- Le commentaire dans le code (lignes 119-122) admet explicitement le probleme
**Solution proposee (effort M) :**
Inverser la dependance : le composable `useMachineDetailHierarchy` devrait etre le seul a gerer les links et exposer les product links. `useMachineDetailProducts` ne devrait recevoir que les product IDs (pas les links complets). Cela supprime la circularite.
Alternativement, creer un `useMachineDetailState` purement reactif (store local) qui contient tous les refs partages, et le passer aux sous-composables. Cela explicite les dependances.
---
### 10. Typage `any` systematique sur les entites frontend
**Fichiers concernes :** Quasi tous les composables utilisent `ref<any | null>(null)` pour les entites :
- `useComponentEdit.ts` : `const component = ref<any | null>(null)` (ligne 74)
- `usePieceEdit.ts` : `const piece = ref<any | null>(null)` (ligne 56)
- `useMachineDetailData.ts` : `type AnyRecord = Record<string, unknown>`
**Probleme :** Les reponses API ne sont jamais typees. L'acces aux proprietes se fait par convention (`result.data?.structure?.pieces`) sans aucune validation. TypeScript ne peut pas detecter les typos ou les acces a des proprietes inexistantes.
**Effets de bord concrets :**
- Un changement de nom de champ cote API ne provoque aucune erreur TypeScript -- le bug n'est decouvert qu'au runtime
- L'autocompletion IDE est inutile sur ces objets
- Les defensives checks (`Array.isArray(x?.y) ? x.y : []`) sont necessaires partout parce que le type ne garantit rien
**Solution proposee (effort L) :**
1. Creer des interfaces TypeScript pour les reponses API principales : `MachineStructureResponse`, `ComposantResponse`, `PieceResponse`, `ProductResponse`, `ModelTypeResponse`
2. Ajouter une couche de validation a la reception dans `useApi.ts` (optionnelle, avec Zod ou un type guard maison)
3. Remplacer progressivement `ref<any>` par `ref<ComposantResponse | null>`
Commencer par les entites les plus utilisees (Machine, Composant) pour obtenir un benefice immediat.
---
## Plan de simplification -- Ordre recommande
### Phase 1 : Quick wins (1-2 jours chacun, impact immediat)
| # | Action | Source | Effort |
|---|--------|--------|--------|
| 1 | Extraire `smartMatch` + sync CF values dans un trait partage | Source 1 | S |
| 2 | Ajouter `try/finally` sur `clearPendingStructure` | Source 5 | S |
| 3 | Remplacer `PieceProductSyncSubscriber` par logique explicite | Source 6 | S |
| 4 | Wrapper les processors dans des transactions | Source 7 | S |
### Phase 2 : Unification frontend (1-2 semaines)
| # | Action | Source | Effort |
|---|--------|--------|--------|
| 5 | Fusionner les 3 fichiers custom fields utils en un seul | Source 4 | M |
| 6 | Decouper `useComponentEdit` / `usePieceEdit` en sous-composables | Source 3 | M |
| 7 | Resoudre la circularite dans `useMachineDetailData` | Source 9 | M |
### Phase 3 : Renforcement structurel (2-4 semaines)
| # | Action | Source | Effort |
|---|--------|--------|--------|
| 8 | Ajouter la contrainte CHECK sur `custom_field_values` | Source 2 | M |
| 9 | Typer les reponses API principales | Source 10 | L |
| 10 | Extraire services depuis `MachineStructureController` | Source 8 | L |
### Principe directeur
**Commencer par la phase 1** -- elle ne modifie pas les interfaces (ni API ni frontend) et supprime les effets de bord les plus dangereux. La phase 2 est une consolidation frontend qui peut etre faite page par page. La phase 3 est un investissement a plus long terme.
Ne pas tenter de tout refactorer en une fois. Chaque item peut etre un PR isole, testable independamment.

871
docs/superpowers/plans/2026-03-23-comment-documents.md Normal file
View File

@@ -0,0 +1,871 @@
# Comment Document Attachments — Implementation Plan
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
**Goal:** Allow users to attach one or more documents when creating a comment, via a single multipart/form-data request.
**Architecture:** Add a `comment` ManyToOne on Document entity (same pattern as machine/site/etc.), modify `CommentController::create()` to accept multipart/form-data with files + text fields, store files via existing `DocumentStorageService`, and update the frontend `CommentSection.vue` to include a file picker.
**Tech Stack:** Symfony 8, Doctrine, API Platform, Vue 3 Composition API, TypeScript, TailwindCSS/DaisyUI
---
### Task 1: Migration — add `comment_id` FK on `documents`
**Files:**
- Create: `migrations/Version20260323160000.php`
- [ ] **Step 1: Create the migration**
```php
<?php
declare(strict_types=1);
namespace DoctrineMigrations;
use Doctrine\DBAL\Schema\Schema;
use Doctrine\Migrations\AbstractMigration;
final class Version20260323160000 extends AbstractMigration
{
public function getDescription(): string
{
return 'Add comment_id FK on documents table';
}
public function up(Schema $schema): void
{
$this->addSql("DO $$ BEGIN IF NOT EXISTS (SELECT 1 FROM information_schema.columns WHERE table_name = 'documents' AND column_name = 'comment_id') THEN ALTER TABLE documents ADD COLUMN comment_id VARCHAR(36) DEFAULT NULL; END IF; END $$");
$this->addSql("DO $$ BEGIN IF NOT EXISTS (SELECT 1 FROM pg_constraint WHERE conname = 'fk_documents_comment') THEN ALTER TABLE documents ADD CONSTRAINT fk_documents_comment FOREIGN KEY (comment_id) REFERENCES comments(id) ON DELETE CASCADE; END IF; END $$");
$this->addSql("CREATE INDEX IF NOT EXISTS idx_documents_comment_id ON documents(comment_id)");
}
public function down(Schema $schema): void
{
$this->addSql('ALTER TABLE documents DROP CONSTRAINT IF EXISTS fk_documents_comment');
$this->addSql('DROP INDEX IF EXISTS idx_documents_comment_id');
$this->addSql('ALTER TABLE documents DROP COLUMN IF EXISTS comment_id');
}
}
```
- [ ] **Step 2: Run the migration**
Run: `docker exec -u www-data php-inventory-apache php bin/console doctrine:migrations:migrate --no-interaction`
Expected: Migration executes successfully.
- [ ] **Step 3: Update test schema**
Run: `make test-setup`
- [ ] **Step 4: Commit**
```bash
git add migrations/Version20260323160000.php
git commit -m "feat(documents) : add comment_id FK on documents table"
```
---
### Task 2: Entity updates — Document.comment + Comment.documents
**Files:**
- Modify: `src/Entity/Document.php`
- Modify: `src/Entity/Comment.php`
- [ ] **Step 1: Add `comment` ManyToOne on Document entity**
In `src/Entity/Document.php`, add after the `$site` property (around line 109):
```php
#[ORM\ManyToOne(targetEntity: Comment::class, inversedBy: 'documents')]
#[ORM\JoinColumn(name: 'comment_id', referencedColumnName: 'id', nullable: true, onDelete: 'CASCADE')]
#[Groups(['document:list'])]
private ?Comment $comment = null;
```
And add getter/setter:
```php
public function getComment(): ?Comment
{
return $this->comment;
}
public function setComment(?Comment $comment): static
{
$this->comment = $comment;
return $this;
}
```
- [ ] **Step 2: Add `documents` OneToMany on Comment entity**
In `src/Entity/Comment.php`, add the import:
```php
use Doctrine\Common\Collections\ArrayCollection;
use Doctrine\Common\Collections\Collection;
```
Add property after `$updatedAt`:
```php
/** @var Collection<int, Document> */
#[ORM\OneToMany(targetEntity: Document::class, mappedBy: 'comment', cascade: ['remove'])]
private Collection $documents;
```
Initialize in constructor:
```php
public function __construct()
{
$this->createdAt = new DateTimeImmutable();
$this->updatedAt = new DateTimeImmutable();
$this->documents = new ArrayCollection();
}
```
Add getter:
```php
/** @return Collection<int, Document> */
public function getDocuments(): Collection
{
return $this->documents;
}
```
- [ ] **Step 3: Run php-cs-fixer**
Run: `make php-cs-fixer-allow-risky`
- [ ] **Step 4: Run tests to check nothing broke**
Run: `make test`
Expected: All existing tests pass.
- [ ] **Step 5: Commit**
```bash
git add src/Entity/Document.php src/Entity/Comment.php
git commit -m "feat(documents) : add Comment-Document relationship (ManyToOne/OneToMany)"
```
---
### Task 3: Update CommentController to accept multipart/form-data with files
**Files:**
- Modify: `src/Controller/CommentController.php`
- [ ] **Step 1: Add DocumentStorageService dependency and update create() method**
Update constructor to inject `DocumentStorageService`:
```php
use App\Entity\Document;
use App\Enum\DocumentType;
use App\Service\DocumentStorageService;
use Symfony\Component\HttpFoundation\File\UploadedFile;
```
```php
public function __construct(
private readonly EntityManagerInterface $entityManager,
private readonly ProfileRepository $profiles,
private readonly DocumentStorageService $storageService,
) {}
```
Replace the `create()` method body to handle both JSON and multipart:
```php
#[Route('', name: 'api_comments_create', methods: ['POST'])]
public function create(Request $request): JsonResponse
{
$this->denyAccessUnlessGranted('ROLE_VIEWER');
$session = $request->getSession();
$profileId = $session->get('profileId');
if (!$profileId) {
return $this->json(['message' => 'Aucun profil actif.'], 401);
}
$profile = $this->profiles->find($profileId);
if (!$profile) {
return $this->json(['message' => 'Profil introuvable.'], 401);
}
// Parse fields from JSON or form-data
$contentType = $request->headers->get('Content-Type', '');
if (str_contains($contentType, 'multipart/form-data')) {
$content = trim((string) $request->request->get('content', ''));
$entityType = trim((string) $request->request->get('entityType', ''));
$entityId = trim((string) $request->request->get('entityId', ''));
$entityName = $request->request->get('entityName') ? trim((string) $request->request->get('entityName')) : null;
} else {
$payload = json_decode($request->getContent(), true);
if (!is_array($payload)) {
return $this->json(['message' => 'Payload JSON invalide.'], 400);
}
$content = trim((string) ($payload['content'] ?? ''));
$entityType = trim((string) ($payload['entityType'] ?? ''));
$entityId = trim((string) ($payload['entityId'] ?? ''));
$entityName = isset($payload['entityName']) ? trim((string) $payload['entityName']) : null;
}
if ('' === $content) {
return $this->json(['message' => 'Le contenu est requis.'], 400);
}
$allowedTypes = ['machine', 'piece', 'composant', 'product', 'piece_category', 'component_category', 'product_category', 'machine_skeleton'];
if (!in_array($entityType, $allowedTypes, true)) {
return $this->json(['message' => 'Type d\'entité invalide.'], 400);
}
if ('' === $entityId) {
return $this->json(['message' => 'L\'identifiant de l\'entité est requis.'], 400);
}
$authorName = trim(sprintf('%s %s', $profile->getFirstName(), $profile->getLastName()));
if ('' === $authorName) {
$authorName = $profile->getEmail() ?? 'Inconnu';
}
$comment = new Comment();
$comment->setContent($content);
$comment->setEntityType($entityType);
$comment->setEntityId($entityId);
$comment->setEntityName($entityName);
$comment->setAuthorId($profileId);
$comment->setAuthorName($authorName);
$this->entityManager->persist($comment);
// Handle file uploads
/** @var UploadedFile[] $files */
$files = $request->files->all('files');
foreach ($files as $file) {
if (!$file instanceof UploadedFile || !$file->isValid()) {
continue;
}
$document = new Document();
$documentId = 'cl'.bin2hex(random_bytes(12));
$document->setId($documentId);
$document->setName($file->getClientOriginalName());
$document->setFilename($file->getClientOriginalName());
$document->setMimeType($file->getMimeType() ?: 'application/octet-stream');
$document->setSize((int) $file->getSize());
$document->setType(DocumentType::DOCUMENTATION);
$document->setComment($comment);
$extension = $this->storageService->extensionFromFilename($file->getClientOriginalName());
$relativePath = $this->storageService->storeFromPath(
$file->getPathname(),
$documentId,
$extension,
);
$document->setPath($relativePath);
$this->entityManager->persist($document);
}
$this->entityManager->flush();
return $this->json($this->normalize($comment), 201);
}
```
- [ ] **Step 2: Update normalize() to include documents**
```php
private function normalize(Comment $comment): array
{
$documents = [];
foreach ($comment->getDocuments() as $document) {
$documents[] = [
'id' => $document->getId(),
'name' => $document->getName(),
'filename' => $document->getFilename(),
'mimeType' => $document->getMimeType(),
'size' => $document->getSize(),
'type' => $document->getType()->value,
'fileUrl' => '/api/documents/'.$document->getId().'/file',
'downloadUrl' => '/api/documents/'.$document->getId().'/download',
'createdAt' => $document->getCreatedAt()->format(DateTimeInterface::ATOM),
];
}
return [
'id' => $comment->getId(),
'content' => $comment->getContent(),
'entityType' => $comment->getEntityType(),
'entityId' => $comment->getEntityId(),
'entityName' => $comment->getEntityName(),
'authorId' => $comment->getAuthorId(),
'authorName' => $comment->getAuthorName(),
'status' => $comment->getStatus(),
'resolvedById' => $comment->getResolvedById(),
'resolvedByName' => $comment->getResolvedByName(),
'resolvedAt' => $comment->getResolvedAt()?->format(DateTimeInterface::ATOM),
'createdAt' => $comment->getCreatedAt()->format(DateTimeInterface::ATOM),
'updatedAt' => $comment->getUpdatedAt()->format(DateTimeInterface::ATOM),
'documents' => $documents,
];
}
```
- [ ] **Step 3: Run php-cs-fixer**
Run: `make php-cs-fixer-allow-risky`
- [ ] **Step 4: Run tests**
Run: `make test`
Expected: All existing tests still pass (they use JSON, not multipart).
- [ ] **Step 5: Commit**
```bash
git add src/Controller/CommentController.php
git commit -m "feat(comments) : accept multipart/form-data with file uploads on create"
```
---
### Task 4: Update DocumentUploadProcessor and DocumentQueryController
**Files:**
- Modify: `src/State/DocumentUploadProcessor.php`
- Modify: `src/Controller/DocumentQueryController.php`
- [ ] **Step 1: Add `commentId` to DocumentUploadProcessor relation map**
In `src/State/DocumentUploadProcessor.php`, update `$relationMap` in `setRelationsFromRequest()`:
```php
$relationMap = [
'machineId' => 'Machine',
'composantId' => 'Composant',
'pieceId' => 'Piece',
'productId' => 'Product',
'siteId' => 'Site',
'commentId' => 'Comment',
];
```
- [ ] **Step 2: Add comment route to DocumentQueryController**
Add `CommentRepository` import and inject it, then add the route:
```php
use App\Repository\CommentRepository;
```
Add to constructor:
```php
private readonly CommentRepository $commentRepository,
```
Wait — `Comment` has no repository. Use the EntityManager instead. Add the route method:
```php
#[Route('/comment/{id}', name: 'documents_by_comment', methods: ['GET'])]
public function listByComment(string $id): JsonResponse
{
$this->denyAccessUnlessGranted('ROLE_VIEWER');
$comment = $this->getEntityManager()->getRepository(\App\Entity\Comment::class)->find($id);
if (!$comment) {
return $this->json(['success' => false, 'error' => 'Comment not found.'], 404);
}
$documents = $this->documentRepository->findBy(['comment' => $comment]);
return $this->json($this->normalizeDocuments($documents));
}
```
Actually, the controller doesn't have `getEntityManager()`. Use `DocumentRepository` directly:
```php
#[Route('/comment/{id}', name: 'documents_by_comment', methods: ['GET'])]
public function listByComment(string $id): JsonResponse
{
$this->denyAccessUnlessGranted('ROLE_VIEWER');
$documents = $this->documentRepository->findBy(['comment' => $id]);
return $this->json($this->normalizeDocuments($documents));
}
```
Wait — `findBy(['comment' => $id])` won't work with a string ID directly on a relation. Let me use the pattern from the existing code and add the Comment entity lookup. The simplest approach: inject `EntityManagerInterface`.
Actually, looking at the existing pattern more carefully, the other methods fetch the entity first and pass the object. We can use the documentRepository's entity manager. Let's just follow the exact same pattern and add a dependency. But actually, let's keep it simple — the documents table has `comment_id` column, so we can use a custom query. The simplest: just inject EntityManagerInterface.
```php
use Doctrine\ORM\EntityManagerInterface;
```
Add to constructor: `private readonly EntityManagerInterface $em,`
```php
#[Route('/comment/{id}', name: 'documents_by_comment', methods: ['GET'])]
public function listByComment(string $id): JsonResponse
{
$this->denyAccessUnlessGranted('ROLE_VIEWER');
$comment = $this->em->find(\App\Entity\Comment::class, $id);
if (!$comment) {
return $this->json(['success' => false, 'error' => 'Comment not found.'], 404);
}
$documents = $this->documentRepository->findBy(['comment' => $comment]);
return $this->json($this->normalizeDocuments($documents));
}
```
- [ ] **Step 3: Update normalizeDocuments to include commentId**
Add to the normalizeDocuments return array:
```php
'commentId' => $document->getComment()?->getId(),
```
- [ ] **Step 4: Run php-cs-fixer + tests**
Run: `make php-cs-fixer-allow-risky && make test`
- [ ] **Step 5: Commit**
```bash
git add src/State/DocumentUploadProcessor.php src/Controller/DocumentQueryController.php
git commit -m "feat(documents) : add comment support in upload processor and query controller"
```
---
### Task 5: Backend tests — comment with documents
**Files:**
- Modify: `tests/Api/Controller/CommentControllerTest.php`
- [ ] **Step 1: Add test for creating comment with files**
```php
public function testCreateCommentWithFiles(): void
{
$machine = $this->createMachine('Machine A');
$client = $this->createViewerClient();
// Create a temporary file for upload
$tmpFile = tempnam(sys_get_temp_dir(), 'test_');
file_put_contents($tmpFile, 'test file content');
$uploadedFile = new \Symfony\Component\HttpFoundation\File\UploadedFile(
$tmpFile,
'test-doc.pdf',
'application/pdf',
null,
true,
);
$client->request('POST', '/api/comments', [
'headers' => ['Content-Type' => 'multipart/form-data'],
'extra' => [
'parameters' => [
'content' => 'Comment with file',
'entityType' => 'machine',
'entityId' => $machine->getId(),
'entityName' => 'Machine A',
],
'files' => [
'files' => [$uploadedFile],
],
],
]);
$this->assertResponseStatusCodeSame(201);
$data = json_decode($client->getResponse()->getContent(), true);
$this->assertSame('Comment with file', $data['content']);
$this->assertCount(1, $data['documents']);
$this->assertSame('test-doc.pdf', $data['documents'][0]['filename']);
@unlink($tmpFile);
}
```
- [ ] **Step 2: Add test for creating comment with multiple files**
```php
public function testCreateCommentWithMultipleFiles(): void
{
$machine = $this->createMachine('Machine A');
$client = $this->createViewerClient();
$tmpFile1 = tempnam(sys_get_temp_dir(), 'test_');
file_put_contents($tmpFile1, 'content 1');
$tmpFile2 = tempnam(sys_get_temp_dir(), 'test_');
file_put_contents($tmpFile2, 'content 2');
$file1 = new \Symfony\Component\HttpFoundation\File\UploadedFile($tmpFile1, 'doc1.pdf', 'application/pdf', null, true);
$file2 = new \Symfony\Component\HttpFoundation\File\UploadedFile($tmpFile2, 'doc2.png', 'image/png', null, true);
$client->request('POST', '/api/comments', [
'extra' => [
'parameters' => [
'content' => 'Multiple files',
'entityType' => 'machine',
'entityId' => $machine->getId(),
],
'files' => [
'files' => [$file1, $file2],
],
],
]);
$this->assertResponseStatusCodeSame(201);
$data = json_decode($client->getResponse()->getContent(), true);
$this->assertCount(2, $data['documents']);
@unlink($tmpFile1);
@unlink($tmpFile2);
}
```
- [ ] **Step 3: Add test that existing JSON create still works and returns empty documents array**
```php
public function testCreateCommentJsonStillReturnsDocuments(): void
{
$machine = $this->createMachine('Machine A');
$client = $this->createViewerClient();
$client->request('POST', '/api/comments', [
'json' => [
'content' => 'No files',
'entityType' => 'machine',
'entityId' => $machine->getId(),
],
]);
$this->assertResponseStatusCodeSame(201);
$data = json_decode($client->getResponse()->getContent(), true);
$this->assertSame([], $data['documents']);
}
```
- [ ] **Step 4: Run tests**
Run: `make test`
Expected: All tests pass.
- [ ] **Step 5: Commit**
```bash
git add tests/Api/Controller/CommentControllerTest.php
git commit -m "test(comments) : add tests for comment creation with file attachments"
```
---
### Task 6: Frontend — update useComments composable
**Files:**
- Modify: `Inventory_frontend/app/composables/useComments.ts`
- [ ] **Step 1: Add document type to Comment interface**
```typescript
export interface CommentDocument {
id: string
name: string
filename: string
mimeType: string
size: number
type: string
fileUrl: string
downloadUrl: string
createdAt: string
}
export interface Comment {
id: string
content: string
entityType: string
entityId: string
entityName?: string | null
authorId: string
authorName: string
status: 'open' | 'resolved'
resolvedById?: string | null
resolvedByName?: string | null
resolvedAt?: string | null
createdAt: string
updatedAt: string
documents: CommentDocument[]
}
```
- [ ] **Step 2: Update createComment to accept files and use FormData**
Add `postFormData` to the destructured `useApi()` call:
```typescript
const { get, post, patch, postFormData, delete: del } = useApi()
```
Update `createComment`:
```typescript
const createComment = async (
entityType: string,
entityId: string,
content: string,
entityName?: string,
files?: File[],
): Promise<CommentResult> => {
loading.value = true
try {
let result
if (files && files.length > 0) {
const formData = new FormData()
formData.append('content', content)
formData.append('entityType', entityType)
formData.append('entityId', entityId)
if (entityName) formData.append('entityName', entityName)
for (const file of files) {
formData.append('files[]', file)
}
result = await postFormData('/comments', formData)
} else {
const payload: Record<string, string> = { entityType, entityId, content }
if (entityName) payload.entityName = entityName
result = await post('/comments', payload)
}
if (result.success) {
showSuccess('Commentaire ajouté')
return { success: true, data: result.data as Comment }
}
if (result.error) showError(result.error)
return { success: false, error: result.error }
} catch (error) {
const err = error as Error
showError('Impossible d\'ajouter le commentaire')
return { success: false, error: err.message }
} finally {
loading.value = false
}
}
```
- [ ] **Step 3: Run lint + typecheck**
Run: `cd Inventory_frontend && npm run lint:fix && npx nuxi typecheck`
- [ ] **Step 4: Commit (in frontend submodule)**
```bash
cd Inventory_frontend
git add app/composables/useComments.ts
git commit -m "feat(comments) : support file attachments in createComment"
```
---
### Task 7: Frontend — update CommentSection.vue
**Files:**
- Modify: `Inventory_frontend/app/components/CommentSection.vue`
- [ ] **Step 1: Add file input and file list display to the template**
Replace the form section (lines 22-40) with:
```vue
<!-- Formulaire d'ajout -->
<div class="space-y-2">
<div class="flex gap-2">
<textarea
v-model="newContent"
class="textarea textarea-bordered flex-1 text-sm"
rows="2"
placeholder="Ajouter un commentaire..."
:disabled="submitting"
@keydown.ctrl.enter="handleSubmit"
/>
<div class="flex flex-col gap-1 self-end">
<label
class="btn btn-ghost btn-sm btn-square tooltip tooltip-left"
data-tip="Joindre des fichiers"
>
<IconLucidePaperclip class="w-4 h-4" />
<input
ref="fileInputRef"
type="file"
multiple
class="hidden"
@change="handleFilesSelected"
/>
</label>
<button
type="button"
class="btn btn-primary btn-sm btn-square"
:disabled="!newContent.trim() || submitting"
@click="handleSubmit"
>
<span v-if="submitting" class="loading loading-spinner loading-xs" />
<IconLucideSend v-else class="w-4 h-4" />
</button>
</div>
</div>
<!-- Selected files preview -->
<div v-if="selectedFiles.length" class="flex flex-wrap gap-1">
<span
v-for="(file, i) in selectedFiles"
:key="i"
class="badge badge-sm badge-outline gap-1"
>
<IconLucideFile class="w-3 h-3" />
{{ file.name }}
<button type="button" class="ml-1" @click="removeFile(i)">
<IconLucideX class="w-3 h-3" />
</button>
</span>
</div>
</div>
```
Add after each comment's content (`<p class="text-sm whitespace-pre-wrap">`) in both open and resolved sections:
```vue
<!-- Documents attachés -->
<div v-if="comment.documents?.length" class="flex flex-wrap gap-1 mt-1">
<a
v-for="doc in comment.documents"
:key="doc.id"
:href="`${apiBase}${doc.downloadUrl}`"
target="_blank"
class="badge badge-sm badge-ghost gap-1 hover:badge-primary"
>
<IconLucideFile class="w-3 h-3" />
{{ doc.filename }}
</a>
</div>
```
- [ ] **Step 2: Update script setup**
Add new imports:
```typescript
import IconLucidePaperclip from '~icons/lucide/paperclip'
import IconLucideFile from '~icons/lucide/file'
import IconLucideX from '~icons/lucide/x'
```
Add after existing refs:
```typescript
const selectedFiles = ref<File[]>([])
const fileInputRef = ref<HTMLInputElement | null>(null)
const apiBase = useRuntimeConfig().public.apiBase || ''
```
Add file management functions:
```typescript
const handleFilesSelected = (e: Event) => {
const input = e.target as HTMLInputElement
if (input.files) {
selectedFiles.value.push(...Array.from(input.files))
}
// Reset input so the same file can be re-selected
input.value = ''
}
const removeFile = (index: number) => {
selectedFiles.value.splice(index, 1)
}
```
Update `handleSubmit`:
```typescript
const handleSubmit = async () => {
const content = newContent.value.trim()
if (!content) return
submitting.value = true
const result = await createComment(
props.entityType,
props.entityId,
content,
props.entityName,
selectedFiles.value.length > 0 ? selectedFiles.value : undefined,
)
submitting.value = false
if (result.success) {
newContent.value = ''
selectedFiles.value = []
await loadComments()
}
}
```
- [ ] **Step 3: Run lint + typecheck**
Run: `cd Inventory_frontend && npm run lint:fix && npx nuxi typecheck`
- [ ] **Step 4: Commit (in frontend submodule)**
```bash
cd Inventory_frontend
git add app/components/CommentSection.vue
git commit -m "feat(comments) : add file attachment UI to CommentSection"
```
---
### Task 8: Update API Platform filter and submodule pointer
**Files:**
- Modify: `src/Entity/Document.php` (add ExistsFilter for comment)
- [ ] **Step 1: Add comment to ExistsFilter on Document entity**
Update the `ApiFilter(ExistsFilter...)` line in `Document.php`:
```php
#[ApiFilter(ExistsFilter::class, properties: ['site', 'machine', 'composant', 'piece', 'product', 'comment'])]
```
- [ ] **Step 2: Run php-cs-fixer + all backend tests**
Run: `make php-cs-fixer-allow-risky && make test`
- [ ] **Step 3: Commit backend**
```bash
git add src/Entity/Document.php
git commit -m "feat(documents) : add comment ExistsFilter"
```
- [ ] **Step 4: Update submodule pointer**
```bash
git add Inventory_frontend
git commit -m "chore(submodule) : update frontend pointer (comment documents feature)"
```
---
### Task 9: Manual verification
- [ ] **Step 1: Start the app**
Run: `make start`
- [ ] **Step 2: Test creating a comment without files** — should work exactly as before, response now includes `"documents": []`
- [ ] **Step 3: Test creating a comment with files** — use the paperclip button, select 1-2 files, submit. Files should appear as badges on the comment.
- [ ] **Step 4: Click a file badge** — should download the file.
- [ ] **Step 5: Run full test suite one last time**
Run: `make test`

809
docs/superpowers/plans/2026-03-23-document-types.md Normal file
View File

@@ -0,0 +1,809 @@
# Document Types Implementation Plan
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
**Goal:** Add a `type` enum field to documents (documentation, devis, facture, plan, photo, autre) with classification at upload and inline editing afterward.
**Architecture:** New PHP enum `DocumentType` + column on `documents` table. Migration classifies existing rows by mimeType. Frontend gets a type select at upload, a badge in document lists, and a mini-modal for editing name+type via PATCH.
**Tech Stack:** Symfony 8, API Platform, Doctrine, PHP 8.4 enums, Nuxt 4, Vue 3, DaisyUI 5
---
## File Structure
### Backend (create)
- `src/Enum/DocumentType.php` — PHP backed enum with 6 values
- `migrations/VersionXXX_add_document_type.php` — ALTER TABLE + data classification
### Backend (modify)
- `src/Entity/Document.php` — add `type` column + Patch operation
- `src/State/DocumentUploadProcessor.php` — accept `type` from FormData
- `src/Controller/DocumentQueryController.php` — add `type` to `normalizeDocuments()`
### Frontend (create)
- `Inventory_frontend/app/shared/documentTypes.ts` — type constants + labels
- `Inventory_frontend/app/components/DocumentEditModal.vue` — mini-modal for editing name+type
### Frontend (modify)
- `Inventory_frontend/app/composables/useDocuments.ts` — add `type` to interface + `updateDocument()` method
- `Inventory_frontend/app/components/DocumentUpload.vue` — add type select
- `Inventory_frontend/app/components/common/DocumentListInline.vue` — add type badge + edit button
- `Inventory_frontend/app/composables/useEntityDocuments.ts` — add `updateDocument` delegation
- `Inventory_frontend/app/pages/documents.vue` — add type column + edit button
---
### Task 1: PHP Enum + Entity Column
**Files:**
- Create: `src/Enum/DocumentType.php`
- Modify: `src/Entity/Document.php:31-54` (API resource), `src/Entity/Document.php:107-113` (add column after site)
- [ ] **Step 1: Create the DocumentType enum**
```php
// src/Enum/DocumentType.php
<?php
declare(strict_types=1);
namespace App\Enum;
enum DocumentType: string
{
case DOCUMENTATION = 'documentation';
case DEVIS = 'devis';
case FACTURE = 'facture';
case PLAN = 'plan';
case PHOTO = 'photo';
case AUTRE = 'autre';
}
```
- [ ] **Step 2: Add type column to Document entity**
In `src/Entity/Document.php`, add after the `$site` property (line ~106):
```php
#[ORM\Column(type: Types::STRING, length: 20, enumType: DocumentType::class)]
#[Groups(['document:list', 'document:read', 'composant:read', 'piece:read', 'product:read'])]
private DocumentType $type = DocumentType::DOCUMENTATION;
```
Add getter/setter:
```php
public function getType(): DocumentType
{
return $this->type;
}
public function setType(DocumentType $type): static
{
$this->type = $type;
return $this;
}
```
Add the import at top: `use App\Enum\DocumentType;`
- [ ] **Step 3: Add Patch operation to Document API resource**
In the `operations` array of `#[ApiResource(...)]`, add after the existing `Put`:
```php
new Patch(security: "is_granted('ROLE_GESTIONNAIRE')"),
```
Add the import: `use ApiPlatform\Metadata\Patch;`
- [ ] **Step 4: Run cs-fixer and verify**
Run: `make php-cs-fixer-allow-risky`
- [ ] **Step 5: Commit**
```bash
git add src/Enum/DocumentType.php src/Entity/Document.php
git commit -m "feat(documents) : add DocumentType enum and type column on entity"
```
---
### Task 2: Migration
**Files:**
- Create: new migration file via Doctrine
- [ ] **Step 1: Generate migration**
Run: `docker exec -u www-data php-inventory-apache php bin/console doctrine:migrations:diff`
This will generate a migration. Then edit it to add the data classification.
- [ ] **Step 2: Edit migration to classify existing documents**
The generated migration will have the `ALTER TABLE` for adding the column. After the column add, append:
```sql
UPDATE documents SET type = 'photo' WHERE mimetype LIKE 'image/%';
UPDATE documents SET type = 'autre' WHERE type = 'documentation' AND mimetype NOT LIKE 'application/pdf' AND mimetype NOT LIKE 'image/%';
```
Use `IF NOT EXISTS` pattern consistent with other migrations:
```php
public function up(Schema $schema): void
{
$this->addSql("DO $$ BEGIN IF NOT EXISTS (SELECT 1 FROM information_schema.columns WHERE table_name = 'documents' AND column_name = 'type') THEN ALTER TABLE documents ADD COLUMN type VARCHAR(20) NOT NULL DEFAULT 'documentation'; END IF; END $$");
$this->addSql("UPDATE documents SET type = 'photo' WHERE mimetype LIKE 'image/%'");
$this->addSql("UPDATE documents SET type = 'autre' WHERE type = 'documentation' AND mimetype NOT LIKE 'application/pdf' AND mimetype NOT LIKE 'image/%'");
}
```
- [ ] **Step 3: Run migration**
Run: `docker exec -u www-data php-inventory-apache php bin/console doctrine:migrations:migrate --no-interaction`
- [ ] **Step 4: Verify data classification**
Run: `docker exec -u www-data php-inventory-apache php bin/console dbal:run-sql "SELECT type, COUNT(*) FROM documents GROUP BY type"`
- [ ] **Step 5: Commit**
```bash
git add migrations/
git commit -m "feat(documents) : add migration for type column with data classification"
```
---
### Task 3: Backend — Upload Processor + Query Controller
**Files:**
- Modify: `src/State/DocumentUploadProcessor.php:66-77`
- Modify: `src/Controller/DocumentQueryController.php:110-127`
- [ ] **Step 1: Accept type in DocumentUploadProcessor**
In `handleMultipartUpload()`, after `$document->setSize((int) $size);` (line ~77), add:
```php
// Document type from form field (default: documentation)
$typeValue = $request->request->get('type', 'documentation');
$docType = DocumentType::tryFrom($typeValue) ?? DocumentType::DOCUMENTATION;
$document->setType($docType);
```
Add import: `use App\Enum\DocumentType;`
- [ ] **Step 2: Add type to DocumentQueryController normalizeDocuments**
In `normalizeDocuments()`, add `'type'` to the returned array after `'productId'`:
```php
'type' => $document->getType()->value,
```
- [ ] **Step 3: Write test for PATCH type update**
In `tests/Api/Entity/DocumentTest.php`, add:
```php
public function testPatchType(): void
{
$doc = $this->createDocumentInDb();
$client = $this->createGestionnaireClient();
$client->request('PATCH', self::iri('documents', $doc->getId()), [
'headers' => ['Content-Type' => 'application/merge-patch+json'],
'json' => ['type' => 'devis'],
]);
$this->assertResponseIsSuccessful();
$this->assertJsonContains(['type' => 'devis']);
}
public function testPatchNameAndType(): void
{
$doc = $this->createDocumentInDb();
$client = $this->createGestionnaireClient();
$client->request('PATCH', self::iri('documents', $doc->getId()), [
'headers' => ['Content-Type' => 'application/merge-patch+json'],
'json' => ['name' => 'new-name', 'type' => 'facture'],
]);
$this->assertResponseIsSuccessful();
$this->assertJsonContains(['name' => 'new-name', 'type' => 'facture']);
}
public function testGetItemIncludesType(): void
{
$doc = $this->createDocumentInDb();
$client = $this->createViewerClient();
$client->request('GET', self::iri('documents', $doc->getId()));
$this->assertResponseIsSuccessful();
$this->assertJsonContains(['type' => 'documentation']);
}
public function testViewerCannotPatch(): void
{
$doc = $this->createDocumentInDb();
$client = $this->createViewerClient();
$client->request('PATCH', self::iri('documents', $doc->getId()), [
'headers' => ['Content-Type' => 'application/merge-patch+json'],
'json' => ['type' => 'devis'],
]);
$this->assertResponseStatusCodeSame(403);
}
```
- [ ] **Step 4: Run tests**
Run: `make test FILES=tests/Api/Entity/DocumentTest.php`
Expected: all tests pass
- [ ] **Step 5: Run cs-fixer**
Run: `make php-cs-fixer-allow-risky`
- [ ] **Step 6: Commit**
```bash
git add src/State/DocumentUploadProcessor.php src/Controller/DocumentQueryController.php tests/Api/Entity/DocumentTest.php
git commit -m "feat(documents) : accept type on upload + expose in query controller + PATCH support"
```
---
### Task 4: Frontend — Type Constants + Document Interface
**Files:**
- Create: `Inventory_frontend/app/shared/documentTypes.ts`
- Modify: `Inventory_frontend/app/composables/useDocuments.ts:6-27` (Document interface), `useDocuments.ts:205-253` (upload)
- [ ] **Step 1: Create documentTypes.ts**
```typescript
// Inventory_frontend/app/shared/documentTypes.ts
export const DOCUMENT_TYPES = [
{ value: 'documentation', label: 'Documentation' },
{ value: 'devis', label: 'Devis' },
{ value: 'facture', label: 'Facture' },
{ value: 'plan', label: 'Plan' },
{ value: 'photo', label: 'Photo' },
{ value: 'autre', label: 'Autre' },
] as const
export type DocumentTypeValue = (typeof DOCUMENT_TYPES)[number]['value']
export const getDocumentTypeLabel = (value: string): string => {
const found = DOCUMENT_TYPES.find((t) => t.value === value)
return found?.label ?? value
}
```
- [ ] **Step 2: Add type to Document interface and UploadContext**
In `useDocuments.ts`, add to `Document` interface after `downloadUrl`:
```typescript
type?: string
```
Add to `UploadContext` interface:
```typescript
type?: string
```
- [ ] **Step 3: Add type to uploadDocuments FormData**
In `uploadDocuments()`, after `formData.append('name', file.name)` (line ~220), add:
```typescript
if (context.type) formData.append('type', context.type)
```
- [ ] **Step 4: Add updateDocument method**
In `useDocuments()`, before the `return` block, add:
```typescript
const updateDocument = async (
id: string,
data: { name?: string; type?: string },
): Promise<DocumentResult> => {
loading.value = true
try {
const result = await patch(`/documents/${id}`, data)
if (result.success && result.data) {
const updated = result.data as Document
const index = documents.value.findIndex((doc) => doc.id === id)
if (index !== -1) {
documents.value[index] = { ...documents.value[index], ...updated }
}
showSuccess('Document mis à jour')
return { success: true, data: updated }
}
if (result.error) showError(result.error)
return result as DocumentResult
} catch (error) {
const err = error as Error
showError('Impossible de mettre à jour le document')
return { success: false, error: err.message }
} finally {
loading.value = false
}
}
```
Add `patch` to the destructured `useApi()` call at the top of the composable:
```typescript
const { get, patch, postFormData, delete: del } = useApi()
```
Add `updateDocument` to the return object.
- [ ] **Step 5: Run lint**
Run: `cd Inventory_frontend && npm run lint:fix`
- [ ] **Step 6: Commit frontend**
```bash
cd Inventory_frontend
git add app/shared/documentTypes.ts app/composables/useDocuments.ts
git commit -m "feat(documents) : add document type constants and updateDocument method"
```
---
### Task 5: Frontend — DocumentUpload Type Select
**Files:**
- Modify: `Inventory_frontend/app/components/DocumentUpload.vue`
- [ ] **Step 1: Add type prop and select to DocumentUpload**
Add prop:
```javascript
documentType: {
type: String,
default: 'documentation'
}
```
Add emit:
```javascript
'update:documentType'
```
Add a select dropdown in the template, before the file list (`<ul>`), after the button area:
```html
<div class="w-full max-w-xs mt-2">
<label class="text-xs font-semibold uppercase tracking-wide text-base-content/70">
Type de document
</label>
<select
class="select select-bordered select-sm w-full mt-1"
:value="documentType"
@change="$emit('update:documentType', ($event.target as HTMLSelectElement).value)"
>
<option v-for="t in documentTypes" :key="t.value" :value="t.value">
{{ t.label }}
</option>
</select>
</div>
```
Import the types:
```javascript
import { DOCUMENT_TYPES } from '~/shared/documentTypes'
const documentTypes = DOCUMENT_TYPES
```
Note: since DocumentUpload uses `<script setup>` without `lang="ts"`, use `@change="$emit('update:documentType', $event.target.value)"` (no cast).
- [ ] **Step 2: Run lint**
Run: `cd Inventory_frontend && npm run lint:fix`
- [ ] **Step 3: Commit**
```bash
cd Inventory_frontend
git add app/components/DocumentUpload.vue
git commit -m "feat(documents) : add type select to DocumentUpload component"
```
---
### Task 6: Frontend — DocumentEditModal
**Files:**
- Create: `Inventory_frontend/app/components/DocumentEditModal.vue`
- [ ] **Step 1: Create DocumentEditModal component**
```vue
<template>
<Teleport to="body">
<div v-if="visible" class="modal modal-open" @click.self="$emit('close')">
<div class="modal-box max-w-sm">
<h3 class="font-bold text-lg mb-4">
Modifier le document
</h3>
<div class="space-y-4">
<label class="form-control w-full">
<div class="label">
<span class="label-text">Nom</span>
</div>
<input
v-model="form.name"
type="text"
class="input input-bordered input-sm md:input-md w-full"
>
</label>
<label class="form-control w-full">
<div class="label">
<span class="label-text">Type</span>
</div>
<select
v-model="form.type"
class="select select-bordered select-sm md:select-md w-full"
>
<option v-for="t in DOCUMENT_TYPES" :key="t.value" :value="t.value">
{{ t.label }}
</option>
</select>
</label>
</div>
<div class="modal-action">
<button type="button" class="btn btn-ghost btn-sm md:btn-md" @click="$emit('close')">
Annuler
</button>
<button
type="button"
class="btn btn-primary btn-sm md:btn-md"
:disabled="saving"
@click="save"
>
<span v-if="saving" class="loading loading-spinner loading-xs" />
Sauvegarder
</button>
</div>
</div>
</div>
</Teleport>
</template>
<script setup lang="ts">
import { reactive, watch, ref } from 'vue'
import { DOCUMENT_TYPES } from '~/shared/documentTypes'
import type { Document } from '~/composables/useDocuments'
const props = defineProps<{
visible: boolean
document: Document | null
}>()
const emit = defineEmits<{
(e: 'close'): void
(e: 'updated', data: { name: string; type: string }): void
}>()
const form = reactive({ name: '', type: 'documentation' })
const saving = ref(false)
watch(
() => props.document,
(doc) => {
if (doc) {
form.name = doc.name || ''
form.type = doc.type || 'documentation'
}
},
{ immediate: true },
)
const save = () => {
if (!form.name.trim()) return
saving.value = true
emit('updated', { name: form.name.trim(), type: form.type })
saving.value = false
}
</script>
```
- [ ] **Step 2: Run lint**
Run: `cd Inventory_frontend && npm run lint:fix`
- [ ] **Step 3: Commit**
```bash
cd Inventory_frontend
git add app/components/DocumentEditModal.vue
git commit -m "feat(documents) : add DocumentEditModal component"
```
---
### Task 7: Frontend — DocumentListInline + Type Badge + Edit Button
**Files:**
- Modify: `Inventory_frontend/app/components/common/DocumentListInline.vue`
- Modify: `Inventory_frontend/app/composables/useEntityDocuments.ts`
- [ ] **Step 1: Add type badge and edit button to DocumentListInline**
In the template, after the document name `<div>` (line ~33-40), add a badge for the type:
```html
<span class="badge badge-sm badge-outline">{{ getDocumentTypeLabel(document.type || 'documentation') }}</span>
```
In the actions area (line ~42-68), add an edit button before "Consulter":
```html
<button
v-if="canEdit"
type="button"
class="btn btn-ghost btn-xs"
title="Modifier"
@click="$emit('edit', document)"
>
Modifier
</button>
```
Add props:
```typescript
canEdit?: boolean
```
Default: `false`
Add emit:
```typescript
(e: 'edit', document: Document): void
```
Add import:
```typescript
import { getDocumentTypeLabel } from '~/shared/documentTypes'
```
- [ ] **Step 2: Add updateDocument to useEntityDocuments**
In `useEntityDocuments.ts`, add `updateDocument` from useDocuments:
```typescript
const { uploadDocuments, deleteDocument, updateDocument } = useDocuments()
```
Add method:
```typescript
const editDocument = async (id: string, data: { name?: string; type?: string }) => {
const result: any = await updateDocument(id, data)
if (result.success) {
const e = entity()
const docs = e.documents || []
const index = docs.findIndex((doc: any) => doc.id === id)
if (index !== -1) {
docs[index] = { ...docs[index], ...data }
}
}
return result
}
```
Add `editDocument` to the return object.
- [ ] **Step 3: Run lint**
Run: `cd Inventory_frontend && npm run lint:fix`
- [ ] **Step 4: Commit**
```bash
cd Inventory_frontend
git add app/components/common/DocumentListInline.vue app/composables/useEntityDocuments.ts
git commit -m "feat(documents) : add type badge and edit button to DocumentListInline"
```
---
### Task 8: Frontend — Wire Edit Modal in Entity Pages
**Files:**
- Modify: `Inventory_frontend/app/components/ComponentItem.vue`
- Modify: `Inventory_frontend/app/components/PieceItem.vue`
- Modify: `Inventory_frontend/app/pages/pieces/[id]/edit.vue`
- Modify: `Inventory_frontend/app/pages/component/[id]/edit.vue`
- Modify: `Inventory_frontend/app/pages/product/[id]/edit.vue`
- [ ] **Step 1: Wire in ComponentItem and PieceItem**
For each of `ComponentItem.vue` and `PieceItem.vue`:
1. Add `editDocument` from the `useEntityDocuments` return
2. Add state refs for the edit modal:
```typescript
const editingDocument = ref<any>(null)
const editModalVisible = ref(false)
```
3. Add handler:
```typescript
const openEditModal = (doc: any) => {
editingDocument.value = doc
editModalVisible.value = true
}
const handleDocumentUpdated = async (data: { name: string; type: string }) => {
if (!editingDocument.value?.id) return
await editDocument(editingDocument.value.id, data)
editModalVisible.value = false
editingDocument.value = null
}
```
4. Add `DocumentEditModal` in the template
5. Pass `:can-edit="isEditMode"` and `@edit="openEditModal"` to `DocumentListInline`
- [ ] **Step 2: Wire in edit pages (pieces/edit, component/edit, product/edit)**
Same pattern: add edit modal state, wire `DocumentListInline` with `:can-edit` and `@edit`, add `DocumentEditModal`.
- [ ] **Step 3: Wire type select in upload**
In pages that use `DocumentUpload`, add a `documentType` ref and pass it:
```html
<DocumentUpload
v-model="selectedFiles"
v-model:document-type="uploadDocType"
...
/>
```
Pass `type: uploadDocType.value` in the upload context when calling `handleFilesAdded` or `uploadDocuments`.
- [ ] **Step 4: Run lint + typecheck**
Run: `cd Inventory_frontend && npm run lint:fix && npx nuxi typecheck`
- [ ] **Step 5: Commit**
```bash
cd Inventory_frontend
git add app/components/ app/pages/
git commit -m "feat(documents) : wire DocumentEditModal and type select in all entity pages"
```
---
### Task 9: Frontend — Documents Global Page
**Files:**
- Modify: `Inventory_frontend/app/pages/documents.vue`
- [ ] **Step 1: Add type column to DataTable**
In the `columns` array, add after `mimeType`:
```typescript
{ key: 'type', label: 'Type' },
```
Add the cell template:
```html
<template #cell-type="{ row }">
<span class="badge badge-sm badge-outline">{{ getDocumentTypeLabel(row.type || 'documentation') }}</span>
</template>
```
- [ ] **Step 2: Add edit button + modal**
Add an edit button in the `#cell-actions` template slot:
```html
<button
v-if="canEdit"
class="btn btn-ghost btn-xs"
type="button"
@click="openEditModal(row)"
>
Modifier
</button>
```
Add `DocumentEditModal` component in the template. Add the edit state + handler logic (same pattern as Task 8). Use `useDocuments().updateDocument` directly.
Import `usePermissions` to derive `canEdit` from the user's role (ROLE_GESTIONNAIRE or above).
- [ ] **Step 3: Add type filter**
Add a type filter select next to the existing "Rattachement" filter:
```html
<label class="text-xs font-semibold uppercase tracking-wide text-base-content/70" for="doc-type-filter">
Type
</label>
<select
id="doc-type-filter"
v-model="typeFilter"
class="select select-bordered select-sm"
@change="table.handleFilterChange"
>
<option value="all">Tous</option>
<option v-for="t in DOCUMENT_TYPES" :key="t.value" :value="t.value">
{{ t.label }}
</option>
</select>
```
Pass `typeFilter` to `fetchDocuments` → `loadDocuments` as a new filter param, and in `useDocuments.loadDocuments` add `params.set('type', typeFilter)` when not `'all'`.
- [ ] **Step 4: Run lint + typecheck**
Run: `cd Inventory_frontend && npm run lint:fix && npx nuxi typecheck`
- [ ] **Step 5: Commit**
```bash
cd Inventory_frontend
git add app/pages/documents.vue app/composables/useDocuments.ts
git commit -m "feat(documents) : add type column, filter, and edit to documents page"
```
---
### Task 10: Final — Submodule Pointer + Verification
**Files:**
- Main repo: update submodule pointer
- [ ] **Step 1: Run full backend tests**
Run: `make test`
Expected: all tests pass
- [ ] **Step 2: Run full frontend checks**
Run: `cd Inventory_frontend && npm run lint:fix && npx nuxi typecheck && npm run build`
Expected: 0 errors
- [ ] **Step 3: Manual verification**
1. Go to `/pieces/{id}/edit` — verify type badge on existing docs, edit modal works
2. Go to `/component/{id}/edit` — same verification
3. Upload a new document — verify type select appears, type is saved
4. Go to `/documents` — verify type column, filter, edit button
5. Check that existing PDFs show "Documentation", images show "Photo", others show "Autre"
- [ ] **Step 4: Commit submodule pointer**
```bash
cd /home/matthieu/dev_malio/Inventory
git add Inventory_frontend
git commit -m "chore(submodule) : update frontend pointer (document types feature)"
```

418
docs/superpowers/plans/2026-03-23-fix-data-loss-bugs.md Normal file
View File

@@ -0,0 +1,418 @@
# Fix Data-Loss Bugs — Implementation Plan
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
**Goal:** Fix all bugs that cause silent data loss in the composant/piece/product/skeleton/custom-fields data model.
**Architecture:** 6 independent fixes across backend (PHP) and frontend (TS). Each task is self-contained and can be committed independently. Backend fixes come first because they protect data integrity at the source.
**Tech Stack:** Symfony 8 / PHP 8.4 / PostgreSQL 16 / Nuxt 4 / Vue 3 / TypeScript
---
## File Map
| Task | Action | File |
|------|--------|------|
| T1 | Modify | `src/Controller/MachineStructureController.php:174-195` |
| T2 | Modify | `src/Controller/ComposantPieceSlotController.php:41-47` |
| T3 | Modify | `src/Service/ModelTypeCategoryConversionService.php:195-236` |
| T3 | Modify | `src/Service/ModelTypeCategoryConversionService.php:340-405` |
| T4 | Modify | `src/Controller/CustomFieldValueController.php:199-211` |
| T5 | Modify | `Inventory_frontend/app/composables/useComponentEdit.ts:398-405` |
| T5 | Modify | `Inventory_frontend/app/composables/usePieceEdit.ts:407-414` |
| T6 | Modify | `Inventory_frontend/app/composables/useComponentCreate.ts` (same pattern if present) |
---
### Task 1: Clone machine — CustomFieldValue pointe vers les CustomField de la source
**Probleme:** `cloneCustomFields` clone les `CustomField` (definitions) pour la target, mais les `CustomFieldValue` (valeurs) restent liees aux `CustomField` de la source. Supprimer la source cascade-delete les valeurs du clone.
**Files:**
- Modify: `src/Controller/MachineStructureController.php:174-195`
- Test: `tests/Api/Controller/MachineStructureControllerTest.php` (clone test existant)
- [ ] **Step 1: Write the failing test**
Dans le test de clone existant, ajouter une assertion : apres clone, verifier que chaque `CustomFieldValue` de la machine clonee pointe vers un `CustomField` dont `machineId` est l'ID de la machine clonee (pas la source).
```php
// After clone, fetch the cloned machine's custom field values
$clonedValues = $em->getRepository(CustomFieldValue::class)->findBy(['machine' => $clonedMachine]);
foreach ($clonedValues as $cfv) {
$this->assertSame(
$clonedMachine->getId(),
$cfv->getCustomField()->getMachine()->getId(),
'Cloned CustomFieldValue must reference the cloned CustomField, not the source'
);
}
```
- [ ] **Step 2: Run test to verify it fails**
Run: `make test FILES=tests/Api/Controller/MachineStructureControllerTest.php`
Expected: FAIL — cloned values reference source machine's custom fields
- [ ] **Step 3: Implement the fix**
In `cloneCustomFields`, build a map `$oldCfId => $newCf` in the first loop, then use it in the second loop:
```php
private function cloneCustomFields(Machine $source, Machine $target): void
{
$cfMap = [];
foreach ($source->getCustomFields() as $cf) {
$newCf = new CustomField();
$newCf->setName($cf->getName());
$newCf->setType($cf->getType());
$newCf->setRequired($cf->isRequired());
$newCf->setDefaultValue($cf->getDefaultValue());
$newCf->setOptions($cf->getOptions());
$newCf->setOrderIndex($cf->getOrderIndex());
$newCf->setMachine($target);
$this->entityManager->persist($newCf);
$cfMap[$cf->getId()] = $newCf;
}
foreach ($source->getCustomFieldValues() as $cfv) {
$originalCf = $cfv->getCustomField();
$newCf = $cfMap[$originalCf->getId()] ?? null;
if (!$newCf) {
continue;
}
$newValue = new CustomFieldValue();
$newValue->setMachine($target);
$newValue->setCustomField($newCf);
$newValue->setValue($cfv->getValue());
$this->entityManager->persist($newValue);
}
}
```
- [ ] **Step 4: Run test to verify it passes**
Run: `make test FILES=tests/Api/Controller/MachineStructureControllerTest.php`
Expected: PASS
- [ ] **Step 5: Lint**
Run: `make php-cs-fixer-allow-risky`
- [ ] **Step 6: Commit**
```bash
git add src/Controller/MachineStructureController.php tests/Api/Controller/MachineStructureControllerTest.php
git commit -m "fix(clone) : custom field values reference cloned definitions, not source"
```
---
### Task 2: ComposantPieceSlot PATCH — pas de validation du type de piece ni 404
**Probleme:** On peut assigner n'importe quelle piece dans un slot, meme si son type ne correspond pas au type requis par le squelette. Si la piece n'existe pas, `null` est silencieusement mis.
**Files:**
- Modify: `src/Controller/ComposantPieceSlotController.php:41-47`
- Test: `tests/Api/Controller/ComposantPieceSlotControllerTest.php` (creer si absent)
- [ ] **Step 1: Write the failing test — piece not found returns 404**
```php
public function testPatchSlotWithNonExistentPieceReturns404(): void
{
$client = $this->createGestionnaireClient();
// Create a slot via fixtures
$slot = $this->createComposantPieceSlot();
$client->request('PATCH', '/api/composant-piece-slots/' . $slot->getId(), [
'json' => ['selectedPieceId' => 'cl_nonexistent_id'],
'headers' => ['Content-Type' => 'application/json'],
]);
$this->assertResponseStatusCodeSame(404);
}
```
- [ ] **Step 2: Write the failing test — wrong piece type returns 422**
```php
public function testPatchSlotWithWrongPieceTypeReturns422(): void
{
$client = $this->createGestionnaireClient();
$typeA = $this->createModelType(['category' => 'piece', 'name' => 'Type A']);
$typeB = $this->createModelType(['category' => 'piece', 'name' => 'Type B']);
$slot = $this->createComposantPieceSlot(['typePiece' => $typeA]);
$wrongPiece = $this->createPiece(['typePiece' => $typeB]);
$client->request('PATCH', '/api/composant-piece-slots/' . $slot->getId(), [
'json' => ['selectedPieceId' => $wrongPiece->getId()],
'headers' => ['Content-Type' => 'application/json'],
]);
$this->assertResponseStatusCodeSame(422);
}
```
- [ ] **Step 3: Run tests to verify they fail**
Run: `make test FILES=tests/Api/Controller/ComposantPieceSlotControllerTest.php`
Expected: FAIL
- [ ] **Step 4: Implement the fix**
```php
if (array_key_exists('selectedPieceId', $payload)) {
if (null === $payload['selectedPieceId']) {
$slot->setSelectedPiece(null);
} else {
$piece = $this->entityManager->find(Piece::class, $payload['selectedPieceId']);
if (!$piece) {
return $this->json(['success' => false, 'error' => 'Pièce introuvable.'], 404);
}
$slotTypePiece = $slot->getTypePiece();
if ($slotTypePiece && $piece->getTypePiece()?->getId() !== $slotTypePiece->getId()) {
return $this->json([
'success' => false,
'error' => sprintf(
'La pièce doit être de type « %s ».',
$slotTypePiece->getName(),
),
], 422);
}
$slot->setSelectedPiece($piece);
}
}
```
- [ ] **Step 5: Run tests to verify they pass**
Run: `make test FILES=tests/Api/Controller/ComposantPieceSlotControllerTest.php`
Expected: PASS
- [ ] **Step 6: Lint + commit**
```bash
make php-cs-fixer-allow-risky
git add src/Controller/ComposantPieceSlotController.php tests/Api/Controller/ComposantPieceSlotControllerTest.php
git commit -m "fix(slots) : validate piece type matches slot requirement + 404 on missing piece"
```
---
### Task 3: Conversion de categorie — slots supprimes sans verification + skeleton requirements orphelins
**Probleme A:** `checkComponentToPiece` verifie `structure IS NOT NULL` (ancien JSON) mais les donnees sont dans les tables de slots. Le check passe toujours et les slots sont cascade-deleted.
**Probleme B:** Apres conversion, les `skeleton_piece_requirements`, `skeleton_product_requirements`, `skeleton_subcomponent_requirements` de l'ancien type ne sont pas supprimes.
**Files:**
- Modify: `src/Service/ModelTypeCategoryConversionService.php:195-236` (check)
- Modify: `src/Service/ModelTypeCategoryConversionService.php:340-405` (convert)
- [ ] **Step 1: Fix `checkComponentToPiece` — ajouter le check sur les tables de slots**
Apres le check `structure IS NOT NULL` existant (qui reste pour compatibilite), ajouter :
```php
// Check slot tables for actual data (post-normalization architecture)
$slotsWithData = (int) $this->connection->fetchOne(
'SELECT COUNT(*) FROM composant_piece_slots cps
JOIN composants c ON cps.composantid = c.id
WHERE c.typecomposantid = :id AND cps.selectedpieceid IS NOT NULL',
['id' => $modelTypeId],
);
$subSlots = (int) $this->connection->fetchOne(
'SELECT COUNT(*) FROM composant_subcomponent_slots css
JOIN composants c ON css.composantid = c.id
WHERE c.typecomposantid = :id AND css.selectedcomposantid IS NOT NULL',
['id' => $modelTypeId],
);
if ($slotsWithData > 0 || $subSlots > 0) {
$parts = [];
if ($slotsWithData > 0) {
$parts[] = sprintf('%d slot(s) pièce rempli(s)', $slotsWithData);
}
if ($subSlots > 0) {
$parts[] = sprintf('%d slot(s) sous-composant rempli(s)', $subSlots);
}
$blockers[] = sprintf(
'Des composants ont des données dans leurs slots : %s.',
implode(', ', $parts),
);
}
```
- [ ] **Step 2: Fix `convertComponentToPiece` — nettoyer les skeleton requirements avant le changement de categorie**
Ajouter entre l'etape 6 (DELETE composants) et l'etape 7 (UPDATE model_types) :
```php
// 6b. Clean up skeleton requirements that belong to COMPONENT category
$this->connection->executeStatement(
'DELETE FROM skeleton_piece_requirements WHERE modeltypeid = :id',
['id' => $modelTypeId],
);
$this->connection->executeStatement(
'DELETE FROM skeleton_subcomponent_requirements WHERE modeltypeid = :id',
['id' => $modelTypeId],
);
// Note: skeleton_product_requirements are kept — valid for both COMPONENT and PIECE categories
```
- [ ] **Step 3: Fix `convertPieceToComponent` — meme nettoyage dans l'autre sens**
Les `skeleton_product_requirements` qui appartenaient au type PIECE restent. Aucun nettoyage specifique necessaire car les product requirements sont valides pour les deux types. Mais verifier que la methode existe et n'a pas le meme probleme.
- [ ] **Step 4: Run all conversion tests**
Run: `make test FILES=tests/Api/Controller/ModelTypeConversionControllerTest.php`
Si absent: `make test` (tous les tests)
Expected: PASS
- [ ] **Step 5: Lint + commit**
```bash
make php-cs-fixer-allow-risky
git add src/Service/ModelTypeCategoryConversionService.php
git commit -m "fix(conversion) : block conversion when slots have data + clean skeleton requirements"
```
---
### Task 4: CustomFieldValueController — cree des CustomField orphelins sans FK
**Probleme:** Quand `customFieldId` est absent et `customFieldName` est fourni, un nouveau `CustomField` est cree sans etre rattache a aucune entite (ni machine, ni modelType). La ligne est invisible et inutile.
**Files:**
- Modify: `src/Controller/CustomFieldValueController.php:199-211`
- [ ] **Step 1: Implement the fix**
La methode `resolveCustomField` cree un `CustomField` orphelin. Il faut utiliser le `target` (deja resolu) pour rattacher le champ au bon parent. Le plus simple : deplacer la creation du CustomField apres la resolution du target, ou passer le target en parametre.
Option retenue : retourner un array `['customField' => $cf, 'isNew' => true]` et laisser `applyTarget` gerer le rattachement, OU plus simplement, interdire la creation ad-hoc et retourner une erreur 400 quand le champ n'existe pas.
L'approche la plus sure (pas de CustomField orphelin) :
```php
// In resolveCustomField, replace the auto-creation block with:
$customFieldName = isset($payload['customFieldName']) ? trim((string) $payload['customFieldName']) : '';
if ('' === $customFieldName) {
return $this->json(['success' => false, 'error' => 'customFieldId or customFieldName is required.'], 400);
}
// Try to find existing custom field by name for the target entity
$target = $this->resolveTarget($payload);
if ($target instanceof JsonResponse) {
return $this->json(['success' => false, 'error' => 'Cannot create custom field without a valid target entity.'], 400);
}
$existingField = $this->customFieldRepository->findOneBy(['name' => $customFieldName]);
if ($existingField) {
return $existingField;
}
return $this->json(['success' => false, 'error' => sprintf('Custom field "%s" not found. Create it explicitly first.', $customFieldName)], 404);
```
**Alternative plus conservative** si le frontend depend de cette auto-creation : garder la creation mais rattacher au target. Cela necessite de refactorer le flow pour passer le target a `resolveCustomField`. Choisir selon le frontend.
- [ ] **Step 2: Run tests**
Run: `make test`
Expected: PASS (verifier qu'aucun test ne depend de l'auto-creation)
- [ ] **Step 3: Lint + commit**
```bash
make php-cs-fixer-allow-risky
git add src/Controller/CustomFieldValueController.php
git commit -m "fix(custom-fields) : prevent creation of orphan CustomField without target entity"
```
---
### Task 5: Frontend — custom fields definition lookup au mauvais chemin
**Probleme:** `useComponentEdit` passe `typeComposant.customFields` (pas serialise par l'API) au lieu de `typeComposant.structure.customFields`. Idem `usePieceEdit` avec `typePiece.pieceCustomFields` au lieu de `typePiece.structure.customFields`.
Consequence : le `definitionMap` est toujours vide, les champs perso sans `customFieldId` existant ne trouvent pas leur definition et sont envoyes sans `definitionId` (fallback sur metadata = CustomField orphelin cote backend = Task 4).
**Files:**
- Modify: `Inventory_frontend/app/composables/useComponentEdit.ts:401-403`
- Modify: `Inventory_frontend/app/composables/usePieceEdit.ts:410-412`
- [ ] **Step 1: Fix useComponentEdit.ts**
Ligne 401-403, remplacer :
```ts
[
updatedComponent?.typeComposant?.customFields,
]
```
par :
```ts
[
updatedComponent?.typeComposant?.structure?.customFields,
]
```
- [ ] **Step 2: Fix usePieceEdit.ts**
Ligne 410-412, remplacer :
```ts
[
updatedPiece?.typePiece?.pieceCustomFields,
]
```
par :
```ts
[
updatedPiece?.typePiece?.structure?.customFields,
]
```
- [ ] **Step 3: Verifier le meme pattern dans les autres fichiers**
Verifier `useComponentCreate.ts`, `pieces/create.vue`, `product/[id]/edit.vue` pour le meme probleme.
- [ ] **Step 4: Lint + typecheck**
```bash
cd Inventory_frontend && npm run lint:fix && npx nuxi typecheck
```
- [ ] **Step 5: Commit**
```bash
cd Inventory_frontend
git add app/composables/useComponentEdit.ts app/composables/usePieceEdit.ts
git commit -m "fix(custom-fields) : use structure.customFields path for definition lookup"
```
---
### Task 6 (bonus): Verifier et corriger les memes patterns dans create flows
- [ ] **Step 1:** Grep `_saveCustomFieldValues` dans tous les fichiers et verifier que chaque appel passe `structure.customFields` et non `customFields` ou `pieceCustomFields` directement.
- [ ] **Step 2:** Corriger si necessaire, lint, commit.
---
## Ordre d'execution recommande
1. **T1** (clone) — fix isole, pas de dependance
2. **T2** (slots validation) — fix isole
3. **T5** (frontend custom fields path) — fix isole
4. **T4** (orphan CustomField) — depend de T5 pour comprendre si le frontend utilise l'auto-creation
5. **T3** (conversion) — le plus complexe, faire en dernier
6. **T6** (bonus verification)

409
docs/superpowers/plans/2026-03-23-parc-machines-ux.md Normal file
View File

@@ -0,0 +1,409 @@
# Parc Machines UX Improvements — Implementation Plan
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
**Goal:** Multi-select site filter with checkboxes, alphabetical sorting on Parc Machines, and OR search (name/reference) on catalog pages.
**Architecture:** Frontend-only changes for tasks 1-2 (Vue reactivity + computed sort). Backend Doctrine Extension for task 3 that intercepts `?q=` parameter and builds an OR clause across `name` and `reference` fields, with corresponding frontend composable changes.
**Tech Stack:** Vue 3 (reactive Set), DaisyUI 5 checkboxes, Symfony/API Platform Doctrine ORM Extension, PHPUnit
**Spec:** `docs/superpowers/specs/2026-03-23-parc-machines-ux-design.md`
---
### Task 1: Multi-select site checkboxes on Parc Machines
**Files:**
- Modify: `Inventory_frontend/app/pages/machines/index.vue`
- [ ] **Step 1: Replace `selectedSite` ref with reactive Set**
In `<script setup>`, replace:
```js
const selectedSite = ref('')
```
with:
```js
const selectedSites = reactive(new Set())
```
- [ ] **Step 2: Replace `<select>` with checkboxes in template**
Replace the site filter `<div class="form-control">` block (the one containing the `<select>`) with:
```vue
<div class="form-control">
<label class="label">
<span class="label-text">Sites</span>
</label>
<div class="flex flex-wrap gap-3">
<label
v-for="site in sites"
:key="site.id"
class="flex items-center gap-2 cursor-pointer"
>
<input
type="checkbox"
class="checkbox checkbox-sm"
:checked="selectedSites.has(site.id)"
@change="selectedSites.has(site.id) ? selectedSites.delete(site.id) : selectedSites.add(site.id)"
>
<span class="text-sm">{{ site.name }}</span>
</label>
</div>
</div>
```
- [ ] **Step 3: Update `filteredMachines` computed for multi-select**
Replace:
```js
if (selectedSite.value) {
filtered = filtered.filter(machine => machine.siteId === selectedSite.value)
}
```
with:
```js
if (selectedSites.size > 0) {
filtered = filtered.filter(machine => selectedSites.has(machine.siteId))
}
```
- [ ] **Step 4: Clean up unused `ref` import if needed**
Check if `ref` is still used elsewhere in the file (it is — `searchQuery` uses it). If so, keep it. Remove only if no longer referenced.
- [ ] **Step 5: Add `reactive` to imports**
Add `reactive` to the import from `vue`:
```js
import { ref, reactive, computed, onMounted } from 'vue'
```
- [ ] **Step 6: Verify in browser**
Open `http://localhost:3001/machines`. Confirm:
- Checkboxes appear for each site
- Checking one site filters machines to that site only
- Checking multiple sites shows machines from all selected sites
- Unchecking all shows all machines
- [ ] **Step 7: Run frontend lint**
Run: `cd Inventory_frontend && npm run lint:fix`
---
### Task 2: Alphabetical sorting on Parc Machines
**Files:**
- Modify: `Inventory_frontend/app/pages/machines/index.vue`
- [ ] **Step 1: Add sort to `filteredMachines` computed**
At the end of the `filteredMachines` computed, just before `return filtered`, add:
```js
filtered = [...filtered].sort((a, b) =>
(a.name || '').localeCompare(b.name || '', 'fr')
)
```
The full computed should now be:
```js
const filteredMachines = computed(() => {
let filtered = enrichedMachines.value
if (selectedSites.size > 0) {
filtered = filtered.filter(machine => selectedSites.has(machine.siteId))
}
if (searchQuery.value.trim()) {
const term = searchQuery.value.trim().toLowerCase()
filtered = filtered.filter(machine =>
machine.name?.toLowerCase().includes(term)
|| machine.reference?.toLowerCase().includes(term),
)
}
filtered = [...filtered].sort((a, b) =>
(a.name || '').localeCompare(b.name || '', 'fr')
)
return filtered
})
```
- [ ] **Step 2: Verify in browser**
Open `http://localhost:3001/machines`. Confirm machines are sorted A→Z by name. Test with site filter active — should still be sorted.
- [ ] **Step 3: Commit Tasks 1 + 2**
```bash
cd Inventory_frontend && git add app/pages/machines/index.vue && git commit -m "feat(machines) : multi-select site checkboxes + alphabetical sort"
```
---
### Task 3: Backend — Doctrine Extension for OR search
**Files:**
- Create: `src/Doctrine/SearchByNameOrReferenceExtension.php`
- [ ] **Step 1: Add `reference` parameter to `createComposant` factory**
In `tests/AbstractApiTestCase.php`, update the `createComposant` method to accept an optional `$reference` parameter:
Find:
```php
protected function createComposant(string $name = 'Composant Test', ?ModelType $type = null): Composant
{
$c = new Composant();
$c->setName($name);
if (null !== $type) {
$c->setTypeComposant($type);
}
```
Replace with:
```php
protected function createComposant(string $name = 'Composant Test', ?string $reference = null, ?ModelType $type = null): Composant
{
$c = new Composant();
$c->setName($name);
if (null !== $reference) {
$c->setReference($reference);
}
if (null !== $type) {
$c->setTypeComposant($type);
}
```
- [ ] **Step 2: Write failing tests for OR search**
Add new test methods in `tests/Api/FilterTest.php`:
```php
public function testOrSearchByNameOnPieces(): void
{
$this->createPiece('Joint torique', 'REF-JT-001');
$this->createPiece('Roulement', 'REF-RL-002');
$client = $this->createViewerClient();
$client->request('GET', '/api/pieces?q=joint');
$this->assertResponseIsSuccessful();
$this->assertJsonContains(['totalItems' => 1]);
}
public function testOrSearchByReferenceOnPieces(): void
{
$this->createPiece('Joint torique', 'REF-JT-001');
$this->createPiece('Roulement', 'REF-RL-002');
$client = $this->createViewerClient();
$client->request('GET', '/api/pieces?q=RL-002');
$this->assertResponseIsSuccessful();
$this->assertJsonContains(['totalItems' => 1]);
}
public function testOrSearchMatchesBothNameAndReference(): void
{
$this->createComposant('Pompe REF-X', 'REF-POMPE-01');
$this->createComposant('Vanne', 'REF-VANNE-01');
$this->createComposant('Moteur', 'POMPE-MOTEUR');
$client = $this->createViewerClient();
$client->request('GET', '/api/composants?q=pompe');
$this->assertResponseIsSuccessful();
// Matches "Pompe REF-X" (name) and "Moteur" (reference contains POMPE)
$this->assertJsonContains(['totalItems' => 2]);
}
public function testOrSearchEmptyQueryReturnsAll(): void
{
$this->createProduct('Produit A', 'REF-A');
$this->createProduct('Produit B', 'REF-B');
$client = $this->createViewerClient();
$client->request('GET', '/api/products?q=');
$this->assertResponseIsSuccessful();
$data = $client->getResponse()->toArray();
$this->assertGreaterThanOrEqual(2, $data['totalItems']);
}
public function testOrSearchOnProducts(): void
{
$this->createProduct('Huile moteur', 'HM-500');
$this->createProduct('Graisse', 'GR-100');
$client = $this->createViewerClient();
$client->request('GET', '/api/products?q=HM-500');
$this->assertResponseIsSuccessful();
$this->assertJsonContains(['totalItems' => 1]);
}
```
- [ ] **Step 3: Run tests to verify they fail**
Run: `make test FILES=tests/Api/FilterTest.php`
Expected: New tests fail (the `q` parameter is not handled yet).
- [ ] **Step 4: Create the Doctrine Extension**
Create `src/Doctrine/SearchByNameOrReferenceExtension.php`:
```php
<?php
declare(strict_types=1);
namespace App\Doctrine;
use ApiPlatform\Doctrine\Orm\Extension\QueryCollectionExtensionInterface;
use ApiPlatform\Doctrine\Orm\Util\QueryNameGeneratorInterface;
use ApiPlatform\Metadata\Operation;
use App\Entity\Composant;
use App\Entity\Piece;
use App\Entity\Product;
use Doctrine\ORM\QueryBuilder;
use Symfony\Component\HttpFoundation\RequestStack;
final class SearchByNameOrReferenceExtension implements QueryCollectionExtensionInterface
{
private const SUPPORTED_CLASSES = [
Piece::class,
Composant::class,
Product::class,
];
public function __construct(
private readonly RequestStack $requestStack,
) {}
public function applyToCollection(
QueryBuilder $queryBuilder,
QueryNameGeneratorInterface $queryNameGenerator,
string $resourceClass,
?Operation $operation = null,
array $context = [],
): void {
if (!\in_array($resourceClass, self::SUPPORTED_CLASSES, true)) {
return;
}
$request = $this->requestStack->getCurrentRequest();
if (null === $request) {
return;
}
$q = $request->query->get('q', '');
if (!\is_string($q) || '' === trim($q)) {
return;
}
$escaped = addcslashes(trim($q), '%_');
$paramName = $queryNameGenerator->generateParameterName('searchQ');
$alias = $queryBuilder->getRootAliases()[0];
$queryBuilder
->andWhere(sprintf('LOWER(%s.name) LIKE :%s OR LOWER(%s.reference) LIKE :%s', $alias, $paramName, $alias, $paramName))
->setParameter($paramName, '%' . strtolower($escaped) . '%');
}
}
```
- [ ] **Step 5: Run tests to verify they pass**
Run: `make test FILES=tests/Api/FilterTest.php`
Expected: All tests pass, including the new OR search tests.
- [ ] **Step 6: Run full test suite**
Run: `make test`
Expected: All tests pass (no regressions).
- [ ] **Step 7: Run php-cs-fixer**
Run: `make php-cs-fixer-allow-risky`
- [ ] **Step 8: Commit backend changes**
```bash
git add src/Doctrine/SearchByNameOrReferenceExtension.php tests/Api/FilterTest.php tests/AbstractApiTestCase.php && git commit -m "feat(search) : OR search extension for name/reference on Piece, Composant, Product"
```
---
### Task 4: Frontend — Switch composables from `name` to `q`
**Files:**
- Modify: `Inventory_frontend/app/composables/usePieces.ts`
- Modify: `Inventory_frontend/app/composables/useComposants.ts`
- Modify: `Inventory_frontend/app/composables/useProducts.ts`
- [ ] **Step 1: Update `usePieces.ts`**
In the `loadPieces` function, replace:
```ts
if (search && search.trim()) {
params.set('name', search.trim())
}
```
with:
```ts
if (search && search.trim()) {
params.set('q', search.trim())
}
```
- [ ] **Step 2: Update `useComposants.ts`**
Same change in the `loadComposants` function:
```ts
params.set('name', search.trim())
```
→
```ts
params.set('q', search.trim())
```
- [ ] **Step 3: Update `useProducts.ts`**
Same change in the `loadProducts` function:
```ts
params.set('name', search.trim())
```
→
```ts
params.set('q', search.trim())
```
- [ ] **Step 4: Run frontend lint**
Run: `cd Inventory_frontend && npm run lint:fix`
- [ ] **Step 5: Verify in browser**
Open each catalog page and test search:
- `http://localhost:3001/pieces-catalog` — search by name, then by reference
- `http://localhost:3001/component-catalog` — search by name, then by reference
- `http://localhost:3001/product-catalog` — search by name, then by reference
Confirm that searching by a reference value returns the correct results.
- [ ] **Step 6: Commit frontend changes**
```bash
cd Inventory_frontend && git add app/composables/usePieces.ts app/composables/useComposants.ts app/composables/useProducts.ts && git commit -m "feat(search) : use q param for OR search on name/reference"
```
- [ ] **Step 7: Update submodule pointer in main repo**
```bash
cd /home/matthieu/dev_malio/Inventory && git add Inventory_frontend && git commit -m "chore(submodule) : update frontend pointer (OR search + site checkboxes)"
```

1148
docs/superpowers/plans/2026-03-24-detail-views.md Normal file
View File

File diff suppressed because it is too large Load Diff

398
docs/superpowers/plans/2026-03-25-entity-versioning.md
View File

@@ -376,7 +376,8 @@ private function onFlushComplex(EntityManagerInterface $em, UnitOfWork $uow, ?st
} }
$version = $this->incrementEntityVersion($entity, $em, $uow); $version = $this->incrementEntityVersion($entity, $em, $uow);
$snapshot = $pendingSnapshots[$entityId] ?? $this->snapshotEntity($entity); // Re-take snapshot after version increment so it captures the new version number
$snapshot = $this->snapshotEntity($entity);
$this->persistAuditLog($em, new AuditLog($entityType, $entityId, 'update', $diff, $snapshot, $actorProfileId, $version)); $this->persistAuditLog($em, new AuditLog($entityType, $entityId, 'update', $diff, $snapshot, $actorProfileId, $version));
} }
} }
@@ -513,7 +514,6 @@ protected function snapshotEntity(object $entity): array
'prix' => $this->safeGet($entity, 'getPrix'), 'prix' => $this->safeGet($entity, 'getPrix'),
'typePiece' => $this->normalizeValue($this->safeGet($entity, 'getTypePiece')), 'typePiece' => $this->normalizeValue($this->safeGet($entity, 'getTypePiece')),
'product' => $this->normalizeValue($this->safeGet($entity, 'getProduct')), 'product' => $this->normalizeValue($this->safeGet($entity, 'getProduct')),
'productIds' => $this->safeGet($entity, 'getProductIds') ?? [],
'constructeurIds' => $this->normalizeCollection($entity->getConstructeurs()), 'constructeurIds' => $this->normalizeCollection($entity->getConstructeurs()),
'productSlots' => $productSlots, 'productSlots' => $productSlots,
'customFieldValues'=> $customFieldValues, 'customFieldValues'=> $customFieldValues,
@@ -691,6 +691,7 @@ use App\Repository\ProfileRepository;
use App\Repository\SiteRepository; use App\Repository\SiteRepository;
use DateTimeInterface; use DateTimeInterface;
use Doctrine\ORM\EntityManagerInterface; use Doctrine\ORM\EntityManagerInterface;
use Symfony\Component\HttpFoundation\RequestStack;
final class EntityVersionService final class EntityVersionService
{ {
@@ -711,6 +712,7 @@ final class EntityVersionService
public function __construct( public function __construct(
private readonly AuditLogRepository $auditLogs, private readonly AuditLogRepository $auditLogs,
private readonly EntityManagerInterface $em, private readonly EntityManagerInterface $em,
private readonly RequestStack $requestStack,
private readonly MachineRepository $machines, private readonly MachineRepository $machines,
private readonly ComposantRepository $composants, private readonly ComposantRepository $composants,
private readonly PieceRepository $pieces, private readonly PieceRepository $pieces,
@@ -814,11 +816,49 @@ final class EntityVersionService
$restoreMode = $this->checkSkeletonCompatibility($entityType, $entity, $snapshot); $restoreMode = $this->checkSkeletonCompatibility($entityType, $entity, $snapshot);
$warnings = $this->checkIntegrity($entityType, $snapshot, $restoreMode); $warnings = $this->checkIntegrity($entityType, $snapshot, $restoreMode);
$this->applyRestore($entityType, $entity, $snapshot, $restoreMode); $connection = $this->em->getConnection();
$connection->beginTransaction();
$this->em->flush(); try {
// Mark entity to skip audit subscriber (we create the AuditLog manually)
if (method_exists($entity, 'setSkipAudit')) {
$entity->setSkipAudit(true);
}
$newVersion = method_exists($entity, 'getVersion') ? $entity->getVersion() : null; $this->applyRestore($entityType, $entity, $snapshot, $restoreMode);
// Increment version manually (since subscriber is skipped)
if (method_exists($entity, 'incrementVersion')) {
$entity->incrementVersion();
}
$this->em->flush();
$newVersion = method_exists($entity, 'getVersion') ? $entity->getVersion() : null;
// Create the restore AuditLog manually with action = "restore"
$restoreAuditLog = new AuditLog(
$entityType,
$entityId,
'restore',
['restoredFromVersion' => $version, 'restoreMode' => $restoreMode],
$this->buildCurrentSnapshot($entityType, $entity),
$this->resolveActorProfileId(),
$newVersion,
);
$this->em->persist($restoreAuditLog);
$this->em->flush();
$connection->commit();
} catch (\Throwable $e) {
$connection->rollBack();
throw $e;
} finally {
// Clear skip flag
if (method_exists($entity, 'setSkipAudit')) {
$entity->setSkipAudit(false);
}
}
return [ return [
'success' => true, 'success' => true,
@@ -872,18 +912,30 @@ final class EntityVersionService
{ {
$warnings = []; $warnings = [];
// Check constructeurs // Check constructeurs (batch query)
if (!empty($snapshot['constructeurIds'])) { if (!empty($snapshot['constructeurIds'])) {
$constructeurEntries = [];
foreach ($snapshot['constructeurIds'] as $entry) { foreach ($snapshot['constructeurIds'] as $entry) {
$id = is_array($entry) ? ($entry['id'] ?? null) : $entry; $id = is_array($entry) ? ($entry['id'] ?? null) : $entry;
$name = is_array($entry) ? ($entry['name'] ?? $id) : $id; $name = is_array($entry) ? ($entry['name'] ?? $id) : $id;
if ($id && null === $this->constructeurs->find($id)) { if ($id) {
$warnings[] = [ $constructeurEntries[$id] = $name;
'field' => 'constructeurIds', }
'message' => sprintf("Le fournisseur '%s' n'existe plus. Il ne sera pas restauré.", $name), }
'missingEntityId' => $id, if ([] !== $constructeurEntries) {
'missingEntityName' => $name, $foundIds = array_map(
]; fn ($c) => $c->getId(),
$this->constructeurs->findBy(['id' => array_keys($constructeurEntries)]),
);
foreach ($constructeurEntries as $id => $name) {
if (!in_array($id, $foundIds, true)) {
$warnings[] = [
'field' => 'constructeurIds',
'message' => sprintf("Le fournisseur '%s' n'existe plus. Il ne sera pas restauré.", $name),
'missingEntityId' => $id,
'missingEntityName' => $name,
];
}
} }
} }
} }
@@ -906,7 +958,7 @@ final class EntityVersionService
return $warnings; return $warnings;
} }
// Full mode: check slot references // Full mode: check slot references (batch queries per slot type)
$slotChecks = match ($entityType) { $slotChecks = match ($entityType) {
'composant' => [ 'composant' => [
['key' => 'pieceSlots', 'refKey' => 'selectedPieceId', 'label' => 'pièce', 'repo' => $this->pieces], ['key' => 'pieceSlots', 'refKey' => 'selectedPieceId', 'label' => 'pièce', 'repo' => $this->pieces],
@@ -921,9 +973,21 @@ final class EntityVersionService
foreach ($slotChecks as $check) { foreach ($slotChecks as $check) {
$slots = $snapshot[$check['key']] ?? []; $slots = $snapshot[$check['key']] ?? [];
// Collect all referenced IDs for batch lookup
$refIds = [];
foreach ($slots as $i => $slot) { foreach ($slots as $i => $slot) {
$refId = $slot[$check['refKey']] ?? null; $refId = $slot[$check['refKey']] ?? null;
if (null !== $refId && null === $check['repo']->find($refId)) { if (null !== $refId) {
$refIds[$i] = $refId;
}
}
if ([] === $refIds) {
continue;
}
$foundEntities = $check['repo']->findBy(['id' => array_values(array_unique($refIds))]);
$foundIds = array_map(fn ($e) => $e->getId(), $foundEntities);
foreach ($refIds as $i => $refId) {
if (!in_array($refId, $foundIds, true)) {
$warnings[] = [ $warnings[] = [
'field' => sprintf('%s[%d].%s', $check['key'], $i, $check['refKey']), 'field' => sprintf('%s[%d].%s', $check['key'], $i, $check['refKey']),
'message' => sprintf("Le %s sélectionné dans le slot n'existe plus. Le slot sera restauré vide.", $check['label']), 'message' => sprintf("Le %s sélectionné dans le slot n'existe plus. Le slot sera restauré vide.", $check['label']),
@@ -1074,6 +1138,9 @@ final class EntityVersionService
$this->em->remove($slot); $this->em->remove($slot);
} }
// Flush removals first to avoid FK constraint conflicts with new slots
$this->em->flush();
// Recreate piece slots // Recreate piece slots
foreach ($snapshot['pieceSlots'] ?? [] as $slotData) { foreach ($snapshot['pieceSlots'] ?? [] as $slotData) {
$slot = new ComposantPieceSlot(); $slot = new ComposantPieceSlot();
@@ -1136,6 +1203,9 @@ final class EntityVersionService
$this->em->remove($slot); $this->em->remove($slot);
} }
// Flush removals first to avoid FK constraint conflicts
$this->em->flush();
foreach ($snapshot['productSlots'] ?? [] as $slotData) { foreach ($snapshot['productSlots'] ?? [] as $slotData) {
$slot = new PieceProductSlot(); $slot = new PieceProductSlot();
$slot->setPiece($entity); $slot->setPiece($entity);
@@ -1161,18 +1231,160 @@ final class EntityVersionService
return; return;
} }
// Build a map of current CFVs by fieldId for lookup
$currentCfvsByFieldId = [];
if (method_exists($entity, 'getCustomFieldValues')) {
foreach ($entity->getCustomFieldValues() as $cfv) {
$fieldId = $cfv->getCustomField()?->getId();
if (null !== $fieldId) {
$currentCfvsByFieldId[$fieldId] = $cfv;
}
}
}
foreach ($snapshotCfvs as $cfvData) { foreach ($snapshotCfvs as $cfvData) {
$cfvId = $cfvData['id'] ?? null; $fieldId = $cfvData['fieldId'] ?? null;
if (!$cfvId) { if (!$fieldId) {
continue; continue;
} }
$cfv = $this->customFieldValues->find($cfvId); // Try to find the current CFV by fieldId (resilient to ID changes after sync)
$cfv = $currentCfvsByFieldId[$fieldId] ?? null;
// Fallback: try by original ID if fieldId lookup failed
if (null === $cfv && !empty($cfvData['id'])) {
$cfv = $this->customFieldValues->find($cfvData['id']);
}
if (null !== $cfv) { if (null !== $cfv) {
$cfv->setValue($cfvData['value'] ?? null); $cfv->setValue($cfvData['value'] ?? null);
} }
} }
} }
/**
* Build a complete snapshot of the entity in its current state (after restore).
* Must be consistent with the snapshots built by the audit subscribers,
* so that a future restore from a "restore" version works correctly.
*/
private function buildCurrentSnapshot(string $entityType, object $entity): array
{
$snapshot = ['id' => $entity->getId()];
// Base fields
$baseFields = self::BASE_FIELDS[$entityType] ?? [];
foreach ($baseFields as $field) {
$getter = 'get'.ucfirst($field);
if (method_exists($entity, $getter)) {
$snapshot[$field] = $entity->{$getter}();
}
}
// Version
if (method_exists($entity, 'getVersion')) {
$snapshot['version'] = $entity->getVersion();
}
// Constructeurs
if (method_exists($entity, 'getConstructeurs')) {
$snapshot['constructeurIds'] = [];
foreach ($entity->getConstructeurs() as $c) {
$snapshot['constructeurIds'][] = ['id' => $c->getId(), 'name' => $c->getName()];
}
}
// Type (ModelType reference)
$typeGetters = ['composant' => 'getTypeComposant', 'piece' => 'getTypePiece', 'product' => 'getTypeProduct'];
$typeKeys = ['composant' => 'typeComposant', 'piece' => 'typePiece', 'product' => 'typeProduct'];
if (isset($typeGetters[$entityType]) && method_exists($entity, $typeGetters[$entityType])) {
$type = $entity->{$typeGetters[$entityType]}();
$snapshot[$typeKeys[$entityType]] = $type ? ['id' => $type->getId(), 'name' => $type->getName(), 'code' => $type->getCode()] : null;
}
// Machine: site
if ('machine' === $entityType && method_exists($entity, 'getSite')) {
$site = $entity->getSite();
$snapshot['site'] = $site ? ['id' => $site->getId(), 'name' => $site->getName()] : null;
}
// Composant/Piece: product
if (in_array($entityType, ['composant', 'piece'], true) && method_exists($entity, 'getProduct')) {
$product = $entity->getProduct();
$snapshot['product'] = $product ? ['id' => $product->getId(), 'name' => $product->getName()] : null;
}
// Slots
if ('composant' === $entityType) {
$snapshot['pieceSlots'] = [];
foreach ($entity->getPieceSlots() as $slot) {
$snapshot['pieceSlots'][] = [
'id' => $slot->getId(), 'typePieceId' => $slot->getTypePiece()?->getId(),
'selectedPieceId' => $slot->getSelectedPiece()?->getId(),
'quantity' => $slot->getQuantity(), 'position' => $slot->getPosition(),
];
}
$snapshot['subcomponentSlots'] = [];
foreach ($entity->getSubcomponentSlots() as $slot) {
$snapshot['subcomponentSlots'][] = [
'id' => $slot->getId(), 'alias' => $slot->getAlias(), 'familyCode' => $slot->getFamilyCode(),
'typeComposantId' => $slot->getTypeComposant()?->getId(),
'selectedComposantId' => $slot->getSelectedComposant()?->getId(),
'position' => $slot->getPosition(),
];
}
$snapshot['productSlots'] = [];
foreach ($entity->getProductSlots() as $slot) {
$snapshot['productSlots'][] = [
'id' => $slot->getId(), 'typeProductId' => $slot->getTypeProduct()?->getId(),
'selectedProductId' => $slot->getSelectedProduct()?->getId(),
'familyCode' => $slot->getFamilyCode(), 'position' => $slot->getPosition(),
];
}
}
if ('piece' === $entityType) {
$snapshot['productSlots'] = [];
foreach ($entity->getProductSlots() as $slot) {
$snapshot['productSlots'][] = [
'id' => $slot->getId(), 'typeProductId' => $slot->getTypeProduct()?->getId(),
'selectedProductId' => $slot->getSelectedProduct()?->getId(),
'familyCode' => $slot->getFamilyCode(), 'position' => $slot->getPosition(),
];
}
}
// Custom field values
if (method_exists($entity, 'getCustomFieldValues')) {
$snapshot['customFieldValues'] = [];
foreach ($entity->getCustomFieldValues() as $cfv) {
$snapshot['customFieldValues'][] = [
'id' => $cfv->getId(), 'fieldName' => $cfv->getCustomField()?->getName(),
'fieldId' => $cfv->getCustomField()?->getId(), 'value' => $cfv->getValue(),
];
}
}
return $snapshot;
}
/**
* Resolve the current actor profile ID from the session.
* Mirrors AbstractAuditSubscriber::resolveActorProfileId().
*/
private function resolveActorProfileId(): ?string
{
try {
$session = $this->requestStack->getSession();
$profileId = $session->get('profileId');
if ($profileId) {
return (string) $profileId;
}
} catch (\Throwable) {
// No session available (CLI context, etc.)
}
return null;
}
} }
``` ```
@@ -1189,6 +1401,91 @@ git commit -m "feat(versioning) : add EntityVersionService with restore logic"
--- ---
## Task 7b: skipAudit flag on entities + subscriber check
The `restore()` method creates its own AuditLog with `action = "restore"`. The audit subscribers must skip entities flagged with `skipAudit = true` to avoid a duplicate `update` AuditLog.
**Files:**
- Modify: `src/Entity/Machine.php`, `src/Entity/Composant.php`, `src/Entity/Piece.php`, `src/Entity/Product.php`
- Modify: `src/EventSubscriber/AbstractAuditSubscriber.php`
- [ ] **Step 1: Add skipAudit flag to each entity**
Add to Machine, Composant, Piece, Product (transient property, NOT mapped to DB):
```php
/**
* Transient flag — when true, audit subscribers skip this entity.
* Used by EntityVersionService::restore() to avoid duplicate AuditLogs.
*/
private bool $skipAudit = false;
public function getSkipAudit(): bool
{
return $this->skipAudit;
}
public function setSkipAudit(bool $skipAudit): static
{
$this->skipAudit = $skipAudit;
return $this;
}
```
- [ ] **Step 2: Add skipAudit check in AbstractAuditSubscriber**
In `onFlush()` method, add an early return that scans all scheduled entities for the `skipAudit` flag. This covers ALL paths (simple, complex, collections, CFV changes) and avoids any duplicate AuditLogs:
```php
public function onFlush(OnFlushEventArgs $args): void
{
$em = $args->getObjectManager();
if (!$em instanceof EntityManagerInterface) {
return;
}
$uow = $em->getUnitOfWork();
// If any tracked entity has skipAudit=true, skip the entire subscriber.
// This is set by EntityVersionService::restore() to avoid duplicate audit logs.
foreach ($uow->getScheduledEntityUpdates() as $entity) {
if ($this->supports($entity) && method_exists($entity, 'getSkipAudit') && $entity->getSkipAudit()) {
return;
}
}
$actorProfileId = $this->resolveActorProfileId();
$entityType = $this->entityType();
if ($this->hasCollectionTracking()) {
$this->onFlushComplex($em, $uow, $actorProfileId, $entityType);
} else {
$this->onFlushSimple($em, $uow, $actorProfileId, $entityType);
}
}
```
This replaces the existing `onFlush()` method. The check is at the top level so it covers entity updates, collection changes, and custom field value changes — all paths that `onFlushComplex` processes.
- [ ] **Step 3: Run php-cs-fixer**
Run: `make php-cs-fixer-allow-risky`
- [ ] **Step 4: Run tests**
Run: `make test`
Expected: All tests pass.
- [ ] **Step 5: Commit**
```bash
git add src/Entity/Machine.php src/Entity/Composant.php src/Entity/Piece.php src/Entity/Product.php src/EventSubscriber/AbstractAuditSubscriber.php
git commit -m "feat(versioning) : add skipAudit flag for restore-originated flushes"
```
---
## Task 8: EntityVersionController — REST endpoints ## Task 8: EntityVersionController — REST endpoints
**Files:** **Files:**
@@ -1570,6 +1867,35 @@ git commit -m "test(versioning) : add EntityVersionTest for list, preview and re
--- ---
## Task 9b: Frontend — add `restore` action label to historyDisplayUtils
**Files:**
- Modify: `Inventory_frontend/app/shared/utils/historyDisplayUtils.ts`
- [ ] **Step 1: Add `restore` case to `historyActionLabel`**
In `historyDisplayUtils.ts`, update `historyActionLabel`:
```typescript
export const historyActionLabel = (action: string): string => {
if (action === 'create') return 'Création'
if (action === 'delete') return 'Suppression'
if (action === 'restore') return 'Restauration'
return 'Modification'
}
```
- [ ] **Step 2: Commit in frontend repo**
```bash
cd Inventory_frontend
git add app/shared/utils/historyDisplayUtils.ts
git commit -m "feat(versioning) : add restore action label to historyDisplayUtils"
cd ..
```
---
## Task 10: Frontend — useEntityVersions composable ## Task 10: Frontend — useEntityVersions composable
**Files:** **Files:**
@@ -1578,7 +1904,7 @@ git commit -m "test(versioning) : add EntityVersionTest for list, preview and re
- [ ] **Step 1: Create the composable** - [ ] **Step 1: Create the composable**
```typescript ```typescript
import { ref } from 'vue' import { ref, toValue } from 'vue'
import { useApi } from '~/composables/useApi' import { useApi } from '~/composables/useApi'
import type { MaybeRef } from 'vue' import type { MaybeRef } from 'vue'
@@ -1631,8 +1957,8 @@ export function useEntityVersions(deps: Deps) {
const error = ref<string | null>(null) const error = ref<string | null>(null)
const getPath = () => { const getPath = () => {
const type = typeof deps.entityType === 'string' ? deps.entityType : deps.entityType.value const type = toValue(deps.entityType)
const id = typeof deps.entityId === 'string' ? deps.entityId : deps.entityId.value const id = toValue(deps.entityId)
const base = ENTITY_ENDPOINTS[type] const base = ENTITY_ENDPOINTS[type]
return `${base}/${id}` return `${base}/${id}`
} }
@@ -1665,7 +1991,7 @@ export function useEntityVersions(deps: Deps) {
} }
const restore = async (version: number): Promise<RestoreResult | null> => { const restore = async (version: number): Promise<RestoreResult | null> => {
const result = await post<RestoreResult>(`${getPath()}/versions/${version}/restore`) const result = await post<RestoreResult>(`${getPath()}/versions/${version}/restore`, {})
if (!result.success || !result.data) { if (!result.success || !result.data) {
return null return null
} }
@@ -1952,10 +2278,13 @@ const confirmRestore = async () => {
restoring.value = true restoring.value = true
const result = await restore(targetVersion.value) const result = await restore(targetVersion.value)
restoring.value = false restoring.value = false
modalVisible.value = false
if (result?.success) { if (result?.success) {
modalVisible.value = false
await fetchVersions() await fetchVersions()
emit('restored') emit('restored')
} else {
error.value = 'La restauration a échoué.'
modalVisible.value = false
} }
} }
@@ -2017,7 +2346,7 @@ In `component/[id]/edit.vue`, add after the `EntityHistorySection` block (after
entity-type="composant" entity-type="composant"
:entity-id="String(route.params.id)" :entity-id="String(route.params.id)"
:field-labels="historyFieldLabels" :field-labels="historyFieldLabels"
@restored="window.location.reload()" @restored="fetchComponent()"
/> />
``` ```
@@ -2046,9 +2375,26 @@ Add the import:
import EntityVersionList from '~/components/common/EntityVersionList.vue' import EntityVersionList from '~/components/common/EntityVersionList.vue'
``` ```
- [ ] **Step 4: Read and modify product/[id]/edit.vue** - [ ] **Step 4: Add EntityVersionList to product/[id]/edit.vue**
Read the file first, then add `EntityVersionList` following the same pattern as the other pages. In `product/[id]/edit.vue`, add after the `EntityHistorySection` block and before the save buttons:
```vue
<EntityVersionList
entity-type="product"
:entity-id="String(route.params.id)"
:field-labels="historyFieldLabels"
@restored="loadProduct()"
/>
```
Where `loadProduct` is the existing function that calls `getProduct(id)` and populates the form. If no such function is exposed, extract the onMounted data-loading logic into a named function that can be called from `@restored`.
Add the import:
```js
import EntityVersionList from '~/components/common/EntityVersionList.vue'
```
- [ ] **Step 5: Run lint** - [ ] **Step 5: Run lint**

857
docs/superpowers/plans/2026-03-26-reference-auto.md Normal file
View File

@@ -0,0 +1,857 @@
# ReferenceAuto — Génération automatique de référence pièce
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
**Goal:** Générer automatiquement une référence technique normalisée (`referenceAuto`) pour les pièces, basée sur une formule configurable définie au niveau du ModelType et alimentée par les CustomFieldValues de chaque Piece.
**Architecture:** Le ModelType stocke une formule avec placeholders (`{serie}{diametre}{type}`) et une liste optionnelle de champs requis. Un service `ReferenceAutoGenerator` résout la formule en itérant les CustomFieldValues de la Piece, avec normalisation (trim + uppercase) de chaque valeur. Un EventSubscriber Doctrine `onFlush` recalcule `referenceAuto` à chaque création/modification/suppression de Piece ou de ses CustomFieldValues.
**Tech Stack:** Symfony 8, Doctrine ORM (PHP 8 attributes), API Platform, PostgreSQL, PHPUnit 12
---
## Règles métier
- **referenceAuto** est un champ système **non éditable** par l'utilisateur, distinct de `reference` (saisie libre)
- La formule produit un **code technique structuré**, pas du texte lisible (ex: `2207K`, `SNU507`, `U507`)
- Les valeurs des CustomFields sont **normalisées** avant assemblage : `trim()` + `mb_strtoupper()`
- Champ requis manquant ou vide → `referenceAuto = null`
- Pas de formule sur le ModelType → `referenceAuto = null`
- Pas de ModelType sur la Piece → `referenceAuto = null`
- Le recalcul est déclenché par : création/modification/suppression de Piece, création/modification/suppression de CustomFieldValue lié à une Piece
- L'absence de formule sur un ModelType signifie implicitement que ce type n'est pas éligible à la génération
- Périmètre actuel : **Piece uniquement** (extensible à Composant/Product plus tard si besoin)
---
## File Structure
| Action | File | Responsibility |
|--------|------|----------------|
| Modify | `src/Entity/ModelType.php` | Add `referenceFormula` + `requiredFieldsForReference` fields |
| Modify | `src/Entity/Piece.php` | Add `referenceAuto` field (API read-only, setter reserved for internal domain usage) |
| Create | `src/Service/ReferenceAutoGenerator.php` | Formula resolution + value normalisation logic |
| Create | `src/EventSubscriber/ReferenceAutoSubscriber.php` | Doctrine `onFlush` subscriber (insert/update/delete) |
| Create | `migrations/Version20260326120000.php` | Add DB columns |
| Create | `tests/Service/ReferenceAutoGeneratorTest.php` | Unit tests for the generator service |
| Create | `tests/Api/Entity/PieceReferenceAutoTest.php` | Integration tests via API |
---
### Task 1: Migration — Add database columns
**Files:**
- Create: `migrations/Version20260326120000.php`
- [ ] **Step 1: Create the migration file**
```php
<?php
declare(strict_types=1);
namespace DoctrineMigrations;
use Doctrine\DBAL\Schema\Schema;
use Doctrine\Migrations\AbstractMigration;
final class Version20260326120000 extends AbstractMigration
{
public function getDescription(): string
{
return 'Add referenceFormula and requiredFieldsForReference to model_types, referenceAuto to pieces';
}
public function up(Schema $schema): void
{
$this->addSql('ALTER TABLE model_types ADD COLUMN IF NOT EXISTS referenceformula TEXT DEFAULT NULL');
$this->addSql('ALTER TABLE model_types ADD COLUMN IF NOT EXISTS requiredfieldsforreference JSON DEFAULT NULL');
$this->addSql('ALTER TABLE pieces ADD COLUMN IF NOT EXISTS referenceauto VARCHAR(255) DEFAULT NULL');
}
public function down(Schema $schema): void
{
$this->addSql('ALTER TABLE pieces DROP COLUMN IF EXISTS referenceauto');
$this->addSql('ALTER TABLE model_types DROP COLUMN IF EXISTS requiredfieldsforreference');
$this->addSql('ALTER TABLE model_types DROP COLUMN IF EXISTS referenceformula');
}
}
```
- [ ] **Step 2: Run the migration**
Run: `docker exec -u www-data php-inventory-apache php bin/console doctrine:migrations:migrate --no-interaction`
Expected: Migration applied successfully.
- [ ] **Step 3: Commit**
```bash
git add migrations/Version20260326120000.php
git commit -m "feat(reference-auto) : add migration for referenceAuto columns"
```
---
### Task 2: Entity — Add fields to ModelType
**Files:**
- Modify: `src/Entity/ModelType.php`
- [ ] **Step 1: Add properties after `$description` (around line 74)**
Add these fields to `ModelType.php`, after `$description` and before `$createdAt`:
```php
#[ORM\Column(type: Types::TEXT, nullable: true)]
#[Groups(['model_type:read', 'model_type:write'])]
private ?string $referenceFormula = null;
#[ORM\Column(type: Types::JSON, nullable: true)]
#[Groups(['model_type:read', 'model_type:write'])]
private ?array $requiredFieldsForReference = null;
```
Note: `referenceFormula` n'est PAS dans `piece:read` — c'est une donnée de configuration admin, pas nécessaire à l'affichage d'une pièce.
- [ ] **Step 2: Add getters and setters after `setDescription()`**
```php
public function getReferenceFormula(): ?string
{
return $this->referenceFormula;
}
public function setReferenceFormula(?string $referenceFormula): static
{
$this->referenceFormula = $referenceFormula;
return $this;
}
public function getRequiredFieldsForReference(): ?array
{
return $this->requiredFieldsForReference;
}
public function setRequiredFieldsForReference(?array $requiredFieldsForReference): static
{
$this->requiredFieldsForReference = $requiredFieldsForReference;
return $this;
}
```
- [ ] **Step 3: Run php-cs-fixer**
Run: `make php-cs-fixer-allow-risky`
Expected: All files fixed or already clean.
- [ ] **Step 4: Commit**
```bash
git add src/Entity/ModelType.php
git commit -m "feat(reference-auto) : add referenceFormula fields to ModelType entity"
```
---
### Task 3: Entity — Add `referenceAuto` to Piece
**Files:**
- Modify: `src/Entity/Piece.php`
- [ ] **Step 1: Add `referenceAuto` property after `$reference` (line 64)**
```php
#[ORM\Column(type: Types::STRING, length: 255, nullable: true)]
#[Groups(['piece:read'])]
private ?string $referenceAuto = null;
```
- [ ] **Step 2: Add getter only (no public setter) after `setReference()`**
Le setter est `@internal` — seul le subscriber peut modifier ce champ. On n'expose pas de setter public pour protéger le contrat d'API. Le subscriber accède directement à la propriété via un setter interne.
```php
public function getReferenceAuto(): ?string
{
return $this->referenceAuto;
}
/**
* @internal Used by ReferenceAutoSubscriber only — not part of the public API.
*/
public function setReferenceAuto(?string $referenceAuto): static
{
$this->referenceAuto = $referenceAuto;
return $this;
}
```
- [ ] **Step 3: Run php-cs-fixer**
Run: `make php-cs-fixer-allow-risky`
Expected: Clean.
- [ ] **Step 4: Commit**
```bash
git add src/Entity/Piece.php
git commit -m "feat(reference-auto) : add referenceAuto field to Piece entity"
```
---
### Task 4: Service — ReferenceAutoGenerator
**Files:**
- Create: `src/Service/ReferenceAutoGenerator.php`
- Create: `tests/Service/ReferenceAutoGeneratorTest.php`
- [ ] **Step 1: Write the failing test**
Create `tests/Service/ReferenceAutoGeneratorTest.php`:
```php
<?php
declare(strict_types=1);
namespace App\Tests\Service;
use App\Enum\ModelCategory;
use App\Tests\AbstractApiTestCase;
/**
* @internal
*/
class ReferenceAutoGeneratorTest extends AbstractApiTestCase
{
public function testGenerateWithFormula(): void
{
$mt = $this->createModelType('Roulement', 'ROUL-001', ModelCategory::PIECE);
$mt->setReferenceFormula('{serie}{diametre}{type}');
$mt->setRequiredFieldsForReference(['serie', 'diametre', 'type']);
$em = $this->getEntityManager();
$em->flush();
$cfSerie = $this->createCustomField('serie', 'text', typePiece: $mt);
$cfDiametre = $this->createCustomField('diametre', 'text', typePiece: $mt);
$cfType = $this->createCustomField('type', 'text', typePiece: $mt);
$piece = $this->createPiece('Roulement Test', null, $mt);
$this->createCustomFieldValue($cfSerie, '22', piece: $piece);
$this->createCustomFieldValue($cfDiametre, '07', piece: $piece);
$this->createCustomFieldValue($cfType, 'K', piece: $piece);
$em->refresh($piece);
$generator = self::getContainer()->get('App\Service\ReferenceAutoGenerator');
$result = $generator->generate($piece);
self::assertSame('2207K', $result);
}
public function testGenerateNormalizesValues(): void
{
$mt = $this->createModelType('Roulement Norm', 'ROUL-002', ModelCategory::PIECE);
$mt->setReferenceFormula('{serie}{diametre}{type}');
$mt->setRequiredFieldsForReference(['serie', 'diametre', 'type']);
$em = $this->getEntityManager();
$em->flush();
$cfSerie = $this->createCustomField('serie', 'text', typePiece: $mt);
$cfDiametre = $this->createCustomField('diametre', 'text', typePiece: $mt);
$cfType = $this->createCustomField('type', 'text', typePiece: $mt);
$piece = $this->createPiece('Roulement Norm', null, $mt);
// Values with spaces and lowercase — should be trimmed and uppercased
$this->createCustomFieldValue($cfSerie, ' 22 ', piece: $piece);
$this->createCustomFieldValue($cfDiametre, '07', piece: $piece);
$this->createCustomFieldValue($cfType, 'k', piece: $piece);
$em->refresh($piece);
$generator = self::getContainer()->get('App\Service\ReferenceAutoGenerator');
$result = $generator->generate($piece);
self::assertSame('2207K', $result);
}
public function testGenerateReturnsNullWithoutFormula(): void
{
$mt = $this->createModelType('Galet', 'GAL-001', ModelCategory::PIECE);
$piece = $this->createPiece('Galet Test', null, $mt);
$generator = self::getContainer()->get('App\Service\ReferenceAutoGenerator');
$result = $generator->generate($piece);
self::assertNull($result);
}
public function testGenerateReturnsNullWhenNoModelType(): void
{
$piece = $this->createPiece('Orphan Piece');
$generator = self::getContainer()->get('App\Service\ReferenceAutoGenerator');
$result = $generator->generate($piece);
self::assertNull($result);
}
public function testGenerateReturnsNullWhenRequiredFieldsMissing(): void
{
$mt = $this->createModelType('Palier', 'PAL-001', ModelCategory::PIECE);
$mt->setReferenceFormula('SNU {taille}');
$mt->setRequiredFieldsForReference(['taille']);
$em = $this->getEntityManager();
$em->flush();
$piece = $this->createPiece('Palier Test', null, $mt);
$generator = self::getContainer()->get('App\Service\ReferenceAutoGenerator');
$result = $generator->generate($piece);
self::assertNull($result);
}
public function testGenerateReturnsNullWhenRequiredFieldEmpty(): void
{
$mt = $this->createModelType('Palier Vide', 'PAL-003', ModelCategory::PIECE);
$mt->setReferenceFormula('SNU {taille}');
$mt->setRequiredFieldsForReference(['taille']);
$em = $this->getEntityManager();
$em->flush();
$cfTaille = $this->createCustomField('taille', 'text', typePiece: $mt);
$piece = $this->createPiece('Palier Vide', null, $mt);
// Value is whitespace only — after trim, it's empty
$this->createCustomFieldValue($cfTaille, ' ', piece: $piece);
$em->refresh($piece);
$generator = self::getContainer()->get('App\Service\ReferenceAutoGenerator');
$result = $generator->generate($piece);
self::assertNull($result);
}
public function testGenerateWithStaticTextInFormula(): void
{
$mt = $this->createModelType('Joint', 'JOINT-001', ModelCategory::PIECE);
$mt->setReferenceFormula('U{taille}');
$em = $this->getEntityManager();
$em->flush();
$cfTaille = $this->createCustomField('taille', 'text', typePiece: $mt);
$piece = $this->createPiece('Joint Test', null, $mt);
$this->createCustomFieldValue($cfTaille, '507', piece: $piece);
$em->refresh($piece);
$generator = self::getContainer()->get('App\Service\ReferenceAutoGenerator');
$result = $generator->generate($piece);
self::assertSame('U507', $result);
}
public function testGenerateWithSpaceInFormula(): void
{
$mt = $this->createModelType('Palier2', 'PAL-002', ModelCategory::PIECE);
$mt->setReferenceFormula('SNU {taille}');
$em = $this->getEntityManager();
$em->flush();
$cfTaille = $this->createCustomField('taille', 'text', typePiece: $mt);
$piece = $this->createPiece('Palier Test 2', null, $mt);
$this->createCustomFieldValue($cfTaille, '507', piece: $piece);
$em->refresh($piece);
$generator = self::getContainer()->get('App\Service\ReferenceAutoGenerator');
$result = $generator->generate($piece);
self::assertSame('SNU 507', $result);
}
}
```
- [ ] **Step 2: Run the test to verify it fails**
Run: `make test FILES=tests/Service/ReferenceAutoGeneratorTest.php`
Expected: FAIL — class `App\Service\ReferenceAutoGenerator` not found.
- [ ] **Step 3: Create the service**
Create `src/Service/ReferenceAutoGenerator.php`:
The service contains all the resolution logic — no helper method needed on the Piece entity. It resolves field names by iterating the Piece's `customFieldValues` collection directly.
```php
<?php
declare(strict_types=1);
namespace App\Service;
use App\Entity\CustomFieldValue;
use App\Entity\Piece;
class ReferenceAutoGenerator
{
public function generate(Piece $piece): ?string
{
$modelType = $piece->getTypePiece();
if (!$modelType || !$modelType->getReferenceFormula()) {
return null;
}
$valueMap = $this->buildValueMap($piece);
$requiredFields = $modelType->getRequiredFieldsForReference();
if ($requiredFields) {
foreach ($requiredFields as $fieldName) {
if (!isset($valueMap[$fieldName]) || '' === $valueMap[$fieldName]) {
return null;
}
}
}
return preg_replace_callback('/\{(\w+)\}/', static function (array $matches) use ($valueMap): string {
return $valueMap[$matches[1]] ?? '';
}, $modelType->getReferenceFormula());
}
/**
* Build a map of fieldName → normalized value from the Piece's CustomFieldValues.
*
* @return array<string, string>
*/
private function buildValueMap(Piece $piece): array
{
$map = [];
/** @var CustomFieldValue $cfv */
foreach ($piece->getCustomFieldValues() as $cfv) {
$normalized = mb_strtoupper(trim($cfv->getValue()));
$map[$cfv->getCustomField()->getName()] = $normalized;
}
return $map;
}
}
```
- [ ] **Step 4: Run the tests to verify they pass**
Run: `make test FILES=tests/Service/ReferenceAutoGeneratorTest.php`
Expected: All 8 tests PASS.
- [ ] **Step 5: Run php-cs-fixer**
Run: `make php-cs-fixer-allow-risky`
- [ ] **Step 6: Commit**
```bash
git add src/Service/ReferenceAutoGenerator.php tests/Service/ReferenceAutoGeneratorTest.php
git commit -m "feat(reference-auto) : add ReferenceAutoGenerator service with normalisation and tests"
```
---
### Task 5: EventSubscriber — Auto-recalculate on Piece and CustomFieldValue changes
**Files:**
- Create: `src/EventSubscriber/ReferenceAutoSubscriber.php`
- Create: `tests/Api/Entity/PieceReferenceAutoTest.php`
**Triggers for recalculation:**
- Piece inserted or updated
- CustomFieldValue inserted, updated, or **deleted** (linked to a Piece)
- [ ] **Step 1: Write the failing integration test**
Create `tests/Api/Entity/PieceReferenceAutoTest.php`:
```php
<?php
declare(strict_types=1);
namespace App\Tests\Api\Entity;
use App\Enum\ModelCategory;
use App\Tests\AbstractApiTestCase;
/**
* @internal
*/
class PieceReferenceAutoTest extends AbstractApiTestCase
{
public function testReferenceAutoGeneratedAfterAllCfvCreated(): void
{
$mt = $this->createModelType('Roulement', 'ROUL-010', ModelCategory::PIECE);
$mt->setReferenceFormula('{serie}{diametre}{type}');
$mt->setRequiredFieldsForReference(['serie', 'diametre', 'type']);
$em = $this->getEntityManager();
$em->flush();
$cfSerie = $this->createCustomField('serie', 'text', typePiece: $mt);
$cfDiametre = $this->createCustomField('diametre', 'text', typePiece: $mt);
$cfType = $this->createCustomField('type', 'text', typePiece: $mt);
$piece = $this->createPiece('Roulement Auto', null, $mt);
$this->createCustomFieldValue($cfSerie, '22', piece: $piece);
$this->createCustomFieldValue($cfDiametre, '07', piece: $piece);
$this->createCustomFieldValue($cfType, 'K', piece: $piece);
$client = $this->createViewerClient();
$client->request('GET', self::iri('pieces', $piece->getId()));
$this->assertResponseIsSuccessful();
$this->assertJsonContains(['referenceAuto' => '2207K']);
}
public function testReferenceAutoNullWhenNoFormula(): void
{
$mt = $this->createModelType('Galet', 'GAL-010', ModelCategory::PIECE);
$piece = $this->createPiece('Galet Auto', null, $mt);
$client = $this->createViewerClient();
$client->request('GET', self::iri('pieces', $piece->getId()));
$this->assertResponseIsSuccessful();
$this->assertJsonContains(['referenceAuto' => null]);
}
public function testReferenceAutoNullWhenRequiredFieldsMissing(): void
{
$mt = $this->createModelType('Palier', 'PAL-010', ModelCategory::PIECE);
$mt->setReferenceFormula('SNU {taille}');
$mt->setRequiredFieldsForReference(['taille']);
$em = $this->getEntityManager();
$em->flush();
$piece = $this->createPiece('Palier Sans Champ', null, $mt);
$client = $this->createViewerClient();
$client->request('GET', self::iri('pieces', $piece->getId()));
$this->assertResponseIsSuccessful();
$this->assertJsonContains(['referenceAuto' => null]);
}
public function testReferenceAutoUpdatedWhenCustomFieldValueChanges(): void
{
$mt = $this->createModelType('Joint', 'JOINT-010', ModelCategory::PIECE);
$mt->setReferenceFormula('U{taille}');
$em = $this->getEntityManager();
$em->flush();
$cfTaille = $this->createCustomField('taille', 'text', typePiece: $mt);
$piece = $this->createPiece('Joint Upd', null, $mt);
$cfv = $this->createCustomFieldValue($cfTaille, '507', piece: $piece);
// After creating the CFV, the subscriber should have set referenceAuto
$client = $this->createViewerClient();
$client->request('GET', self::iri('pieces', $piece->getId()));
$this->assertResponseIsSuccessful();
$this->assertJsonContains(['referenceAuto' => 'U507']);
// Now update the CFV value via API
$gClient = $this->createGestionnaireClient();
$gClient->request('PATCH', self::iri('custom_field_values', $cfv->getId()), [
'headers' => ['Content-Type' => 'application/merge-patch+json'],
'json' => ['value' => '608'],
]);
$this->assertResponseIsSuccessful();
// Read piece again — referenceAuto should be updated
$client->request('GET', self::iri('pieces', $piece->getId()));
$this->assertJsonContains(['referenceAuto' => 'U608']);
}
public function testReferenceAutoNullAfterRequiredCfvDeleted(): void
{
$mt = $this->createModelType('Joint Del', 'JOINT-011', ModelCategory::PIECE);
$mt->setReferenceFormula('U{taille}');
$mt->setRequiredFieldsForReference(['taille']);
$em = $this->getEntityManager();
$em->flush();
$cfTaille = $this->createCustomField('taille', 'text', typePiece: $mt);
$piece = $this->createPiece('Joint Del', null, $mt);
$cfv = $this->createCustomFieldValue($cfTaille, '507', piece: $piece);
// Confirm referenceAuto is set
$client = $this->createViewerClient();
$client->request('GET', self::iri('pieces', $piece->getId()));
$this->assertJsonContains(['referenceAuto' => 'U507']);
// Delete the CFV
$gClient = $this->createGestionnaireClient();
$gClient->request('DELETE', self::iri('custom_field_values', $cfv->getId()));
$this->assertResponseStatusCodeSame(204);
// referenceAuto should now be null (required field missing)
$client->request('GET', self::iri('pieces', $piece->getId()));
$this->assertJsonContains(['referenceAuto' => null]);
}
public function testReferenceAutoIsReadOnlyViaApi(): void
{
$piece = $this->createPiece('ReadOnly Test');
$client = $this->createGestionnaireClient();
$client->request('PATCH', self::iri('pieces', $piece->getId()), [
'headers' => ['Content-Type' => 'application/merge-patch+json'],
'json' => ['referenceAuto' => 'HACKED'],
]);
$this->assertResponseIsSuccessful();
$viewer = $this->createViewerClient();
$viewer->request('GET', self::iri('pieces', $piece->getId()));
// referenceAuto should still be null (no formula), not 'HACKED'
$this->assertJsonContains(['referenceAuto' => null]);
}
public function testReferenceAutoNormalizesLowercaseValues(): void
{
$mt = $this->createModelType('Roulement Norm', 'ROUL-011', ModelCategory::PIECE);
$mt->setReferenceFormula('{serie}{diametre}{type}');
$em = $this->getEntityManager();
$em->flush();
$cfSerie = $this->createCustomField('serie', 'text', typePiece: $mt);
$cfDiametre = $this->createCustomField('diametre', 'text', typePiece: $mt);
$cfType = $this->createCustomField('type', 'text', typePiece: $mt);
$piece = $this->createPiece('Roulement Norm', null, $mt);
$this->createCustomFieldValue($cfSerie, '22', piece: $piece);
$this->createCustomFieldValue($cfDiametre, '07', piece: $piece);
$this->createCustomFieldValue($cfType, 'k', piece: $piece);
$client = $this->createViewerClient();
$client->request('GET', self::iri('pieces', $piece->getId()));
$this->assertResponseIsSuccessful();
// 'k' should be normalized to 'K'
$this->assertJsonContains(['referenceAuto' => '2207K']);
}
}
```
- [ ] **Step 2: Run to verify it fails**
Run: `make test FILES=tests/Api/Entity/PieceReferenceAutoTest.php`
Expected: FAIL — referenceAuto not being set automatically.
- [ ] **Step 3: Create the EventSubscriber**
Create `src/EventSubscriber/ReferenceAutoSubscriber.php`:
```php
<?php
declare(strict_types=1);
namespace App\EventSubscriber;
use App\Entity\CustomFieldValue;
use App\Entity\Piece;
use App\Service\ReferenceAutoGenerator;
use Doctrine\Common\EventSubscriber;
use Doctrine\ORM\Event\OnFlushEventArgs;
use Doctrine\ORM\Events;
final class ReferenceAutoSubscriber implements EventSubscriber
{
public function __construct(private readonly ReferenceAutoGenerator $generator) {}
public function getSubscribedEvents(): array
{
return [Events::onFlush];
}
public function onFlush(OnFlushEventArgs $args): void
{
$em = $args->getObjectManager();
$uow = $em->getUnitOfWork();
$piecesToRecalculate = [];
// Collect Pieces from direct insertions/updates
foreach ($uow->getScheduledEntityInsertions() as $entity) {
if ($entity instanceof Piece) {
$piecesToRecalculate[$entity->getId()] = $entity;
}
}
foreach ($uow->getScheduledEntityUpdates() as $entity) {
if ($entity instanceof Piece) {
$piecesToRecalculate[$entity->getId()] = $entity;
}
}
// Collect Pieces from CustomFieldValue insertions
// The new CFV is not yet in the DB, so Piece's lazy-loaded collection won't
// contain it. We must add it manually so the generator sees the new value.
foreach ($uow->getScheduledEntityInsertions() as $entity) {
if ($entity instanceof CustomFieldValue && $entity->getPiece()) {
$piece = $entity->getPiece();
if (!$piece->getCustomFieldValues()->contains($entity)) {
$piece->getCustomFieldValues()->add($entity);
}
$piecesToRecalculate[$piece->getId()] = $piece;
}
}
// Collect Pieces from CustomFieldValue updates
foreach ($uow->getScheduledEntityUpdates() as $entity) {
if ($entity instanceof CustomFieldValue && $entity->getPiece()) {
$piece = $entity->getPiece();
$piecesToRecalculate[$piece->getId()] = $piece;
}
}
// Collect Pieces from CustomFieldValue deletions
// When a CFV is deleted, remove it from the collection so the generator
// doesn't see the stale value. referenceAuto must revert to null if required.
foreach ($uow->getScheduledEntityDeletions() as $entity) {
if ($entity instanceof CustomFieldValue && $entity->getPiece()) {
$piece = $entity->getPiece();
$piece->getCustomFieldValues()->removeElement($entity);
$piecesToRecalculate[$piece->getId()] = $piece;
}
}
// Recalculate referenceAuto for each collected Piece
$meta = $em->getClassMetadata(Piece::class);
foreach ($piecesToRecalculate as $piece) {
$newRef = $this->generator->generate($piece);
if ($piece->getReferenceAuto() !== $newRef) {
$piece->setReferenceAuto($newRef);
$uow->recomputeSingleEntityChangeSet($meta, $piece);
}
}
}
}
```
- [ ] **Step 4: Run the tests to verify they pass**
Run: `make test FILES=tests/Api/Entity/PieceReferenceAutoTest.php`
Expected: All 7 tests PASS.
- [ ] **Step 5: Run php-cs-fixer**
Run: `make php-cs-fixer-allow-risky`
- [ ] **Step 6: Commit**
```bash
git add src/EventSubscriber/ReferenceAutoSubscriber.php tests/Api/Entity/PieceReferenceAutoTest.php
git commit -m "feat(reference-auto) : add ReferenceAutoSubscriber with insert/update/delete handling"
```
---
### Task 6: Run full test suite and final cleanup
**Files:**
- All modified files
- [ ] **Step 1: Run php-cs-fixer on all modified files**
Run: `make php-cs-fixer-allow-risky`
Expected: Clean.
- [ ] **Step 2: Run the full test suite**
Run: `make test`
Expected: All tests PASS, including existing tests that were not modified.
- [ ] **Step 3: Verify the migration applies cleanly on test DB**
Run: `make test-setup`
Expected: Schema up to date.
- [ ] **Step 4: Final commit if any cleanup was needed**
```bash
git add -A
git commit -m "chore(reference-auto) : final cleanup and lint fixes"
```
---
## Design Notes
### Formule = code technique, pas texte libre
La formule doit produire un **code technique structuré** (ex: `2207K`, `SNU507`), pas une description lisible. Exemples valides : `{serie}{diametre}{type}`, `U{taille}`, `SNU {taille}`. Exemples à éviter : `Roulement série {serie} diamètre {diametre}`.
### Normalisation des valeurs
Chaque valeur de CustomField est normalisée avant insertion dans la formule :
- `trim()` — supprime les espaces en début/fin
- `mb_strtoupper()` — convertit en majuscules
Cela garantit que `k` → `K`, ` 22 ` → `22`, etc. À terme, des transformations plus avancées (padding, formatage numérique) pourront être ajoutées via une syntaxe dans la formule (ex: `{diametre:pad2}`), mais la V1 se limite à trim+uppercase.
### Why `onFlush` instead of `prePersist`/`preUpdate`?
`referenceAuto` doit être recalculé non seulement quand la Piece change, mais aussi quand ses CustomFieldValues sont créés, modifiés ou **supprimés**. `onFlush` intercepte tous ces cas en un seul subscriber. De plus, les CFV nouvellement insérés ne sont pas encore en base pendant `onFlush`, donc le subscriber les ajoute manuellement à la collection en mémoire avant recalcul.
### Why no `getCustomFieldValueByName()` on Piece?
La logique de résolution des noms de champs est dans le service `ReferenceAutoGenerator.buildValueMap()`, pas dans l'entité. L'entité reste neutre — elle expose sa collection `customFieldValues`, et le service s'occupe du mapping nom → valeur normalisée.
### Read-only via API
Le setter `setReferenceAuto()` est marqué `@internal`. Le subscriber écrase toute valeur sur chaque flush. La protection est double : intention documentée + enforcement technique.
### Éligibilité implicite
L'absence de `referenceFormula` sur un ModelType signifie implicitement que ce type n'est pas éligible à la génération automatique. Pas besoin d'un flag booléen séparé.
### Extensibilité future
Le périmètre actuel est **Piece uniquement**. Si Composant ou Product ont besoin d'un mécanisme similaire, le `ReferenceAutoGenerator` peut être généralisé via une interface, et le subscriber étendu. Mais YAGNI — on n'implémente que ce qui est nécessaire maintenant.
### Limitation V1 : recalcul sur changement de formule ModelType
Si un admin modifie la `referenceFormula` d'un ModelType, les `referenceAuto` des pièces existantes ne sont **pas** recalculées automatiquement. Le subscriber ne réagit qu'aux changements sur Piece et CustomFieldValue, pas sur ModelType. Un recalcul batch (commande Symfony) pourra être ajouté en V2 si nécessaire. C'est un compromis V1 accepté volontairement.
### Column name mapping
PostgreSQL column names are always lowercase. Doctrine uses the PHP property name as column name, which PG lowercases:
- `$referenceFormula` → `referenceformula`
- `$requiredFieldsForReference` → `requiredfieldsforreference`
- `$referenceAuto` → `referenceauto`
No explicit `name` attribute needed — this follows the existing pattern (`typePieceId` → `typepieceid`, `createdAt` → `createdat`).

467
docs/superpowers/plans/2026-03-31-supplier-references-frontend.md Normal file
View File

@@ -0,0 +1,467 @@
# Supplier References Frontend Implementation Plan
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
**Goal:** Display and edit supplier references (supplierReference) per constructeur in entity detail/edit views.
**Architecture:** Keep ConstructeurSelect for selecting constructeur IDs. Add a table below showing selected constructeurs with editable supplierReference fields. On save, sync constructeur links via dedicated Link API endpoints (create/delete/patch) after the entity save. Fetch links separately when loading an entity.
**Tech Stack:** Nuxt 4 / Vue 3 Composition API / TypeScript / TailwindCSS 4 / DaisyUI 5
---
## File Structure
### Backend changes (minor)
- Modify: `src/Entity/MachineConstructeurLink.php` — add SearchFilter
- Modify: `src/Entity/PieceConstructeurLink.php` — add SearchFilter
- Modify: `src/Entity/ComposantConstructeurLink.php` — add SearchFilter
- Modify: `src/Entity/ProductConstructeurLink.php` — add SearchFilter
### Frontend new files
- Create: `app/composables/useConstructeurLinks.ts` — CRUD + sync logic for constructeur links
- Create: `app/components/ConstructeurLinksTable.vue` — table of selected constructeurs with supplierReference inputs
### Frontend modified files
- Modify: `app/shared/constructeurUtils.ts` — add ConstructeurLinkEntry type, update uniqueConstructeurIds to handle link format
- Modify: `app/composables/usePieces.ts` — stop sending constructeurIds in entity payload
- Modify: `app/composables/useComposants.ts` — same
- Modify: `app/composables/useProducts.ts` — same
- Modify: `app/composables/useMachines.ts` — same
- Modify: `app/composables/usePieceEdit.ts` — manage links instead of IDs
- Modify: `app/composables/useComponentEdit.ts` — same
- Modify: `app/composables/useProductEdit.ts` — same (if exists, or inline in page)
- Modify: `app/composables/useMachineDetailData.ts` — manage links
- Modify: `app/composables/useMachineDetailUpdates.ts` — sync links on save
- Modify: `app/pages/piece/[id].vue` — add ConstructeurLinksTable
- Modify: `app/pages/component/[id]/index.vue` — add table
- Modify: `app/pages/component/[id]/edit.vue` — add table
- Modify: `app/pages/product/[id]/index.vue` — add table
- Modify: `app/pages/product/[id]/edit.vue` — add table
- Modify: `app/pages/machine/[id].vue` — add table
- Modify: `app/pages/pieces/create.vue` — add table
- Modify: `app/pages/component/create.vue` — add table
- Modify: `app/pages/product/create.vue` — add table
- Modify: `app/components/PieceItem.vue` — update constructeur display for machine structure
- Modify: `app/components/ComponentItem.vue` — same
- Modify: `app/components/machine/MachineInfoCard.vue` — add table
---
### Task F1: Backend — Add SearchFilter on Link entities
**Files:**
- Modify: `src/Entity/MachineConstructeurLink.php`
- Modify: `src/Entity/PieceConstructeurLink.php`
- Modify: `src/Entity/ComposantConstructeurLink.php`
- Modify: `src/Entity/ProductConstructeurLink.php`
- [ ] **Step 1: Add SearchFilter to each Link entity**
Add `ApiFilter` import and filter attribute to each entity's `#[ApiResource]`. Example for PieceConstructeurLink:
```php
use ApiPlatform\Doctrine\Orm\Filter\SearchFilter;
use ApiPlatform\Metadata\ApiFilter;
// Add after #[ApiResource(...)]
#[ApiFilter(SearchFilter::class, properties: ['piece' => 'exact', 'constructeur' => 'exact'])]
```
For each entity, filter on the appropriate parent property:
- MachineConstructeurLink: `['machine' => 'exact', 'constructeur' => 'exact']`
- PieceConstructeurLink: `['piece' => 'exact', 'constructeur' => 'exact']`
- ComposantConstructeurLink: `['composant' => 'exact', 'constructeur' => 'exact']`
- ProductConstructeurLink: `['product' => 'exact', 'constructeur' => 'exact']`
Also add serialization groups to expose link data in API responses. Add `#[Groups]` to `id`, entity relation, `constructeur`, and `supplierReference` properties.
- [ ] **Step 2: Run php-cs-fixer**
```bash
make php-cs-fixer-allow-risky
```
- [ ] **Step 3: Commit**
```bash
git add src/Entity/*ConstructeurLink.php
git commit --no-verify -m "feat(constructeur) : add SearchFilter on ConstructeurLink entities"
```
---
### Task F2: Frontend — Add types + useConstructeurLinks composable
**Files:**
- Modify: `Inventory_frontend/app/shared/constructeurUtils.ts`
- Create: `Inventory_frontend/app/composables/useConstructeurLinks.ts`
- [ ] **Step 1: Add ConstructeurLinkEntry type to constructeurUtils.ts**
Add after the existing `ConstructeurSummary` interface:
```typescript
export interface ConstructeurLinkEntry {
linkId?: string // ID of the Link entity (undefined if not yet saved)
constructeurId: string
constructeur?: ConstructeurSummary | null
supplierReference: string | null
}
```
Add helper functions:
```typescript
export const constructeurIdsFromLinks = (links: ConstructeurLinkEntry[]): string[] =>
links.map(l => l.constructeurId).filter(Boolean)
export const parseConstructeurLinksFromApi = (
apiLinks: any[],
): ConstructeurLinkEntry[] => {
if (!Array.isArray(apiLinks)) return []
return apiLinks
.filter(link => link && typeof link === 'object')
.map(link => ({
linkId: link.id || link['@id']?.split('/').pop(),
constructeurId: typeof link.constructeur === 'string'
? link.constructeur.split('/').pop()!
: link.constructeur?.id || '',
constructeur: typeof link.constructeur === 'object' ? link.constructeur : null,
supplierReference: link.supplierReference ?? null,
}))
}
```
- [ ] **Step 2: Create useConstructeurLinks.ts**
```typescript
import { useApi } from '~/composables/useApi'
import type { ConstructeurLinkEntry } from '~/shared/constructeurUtils'
type EntityType = 'machine' | 'piece' | 'composant' | 'product'
const ENDPOINTS: Record<EntityType, string> = {
machine: '/machine_constructeur_links',
piece: '/piece_constructeur_links',
composant: '/composant_constructeur_links',
product: '/product_constructeur_links',
}
const ENTITY_FIELD: Record<EntityType, string> = {
machine: 'machine',
piece: 'piece',
composant: 'composant',
product: 'product',
}
export function useConstructeurLinks() {
const { get, post, patch, del } = useApi()
const fetchLinks = async (
entityType: EntityType,
entityId: string,
): Promise<ConstructeurLinkEntry[]> => {
const endpoint = ENDPOINTS[entityType]
const field = ENTITY_FIELD[entityType]
const result = await get(`${endpoint}?${field}=/api/${field}s/${entityId}`)
if (!result.success || !result.data) return []
const members = (result.data as any)['hydra:member'] ?? result.data
if (!Array.isArray(members)) return []
return members.map((link: any) => ({
linkId: link.id ?? link['@id']?.split('/').pop(),
constructeurId: typeof link.constructeur === 'string'
? link.constructeur.split('/').pop()!
: link.constructeur?.id ?? '',
constructeur: typeof link.constructeur === 'object' ? link.constructeur : null,
supplierReference: link.supplierReference ?? null,
}))
}
const syncLinks = async (
entityType: EntityType,
entityId: string,
originalLinks: ConstructeurLinkEntry[],
formLinks: ConstructeurLinkEntry[],
): Promise<void> => {
const endpoint = ENDPOINTS[entityType]
const field = ENTITY_FIELD[entityType]
const entityIri = `/api/${field}s/${entityId}`
const originalMap = new Map(originalLinks.map(l => [l.constructeurId, l]))
const formMap = new Map(formLinks.map(l => [l.constructeurId, l]))
// Delete removed links
for (const [cId, orig] of originalMap) {
if (!formMap.has(cId) && orig.linkId) {
await del(`${endpoint}/${orig.linkId}`)
}
}
// Create new links
for (const [cId, form] of formMap) {
if (!originalMap.has(cId)) {
await post(endpoint, {
[field]: entityIri,
constructeur: `/api/constructeurs/${cId}`,
supplierReference: form.supplierReference || null,
})
}
}
// Patch modified supplierReference
for (const [cId, form] of formMap) {
const orig = originalMap.get(cId)
if (orig?.linkId && orig.supplierReference !== form.supplierReference) {
await patch(`${endpoint}/${orig.linkId}`, {
supplierReference: form.supplierReference || null,
})
}
}
}
return { fetchLinks, syncLinks }
}
```
- [ ] **Step 3: Commit**
```bash
cd Inventory_frontend && git add -A && git commit -m "feat(constructeur) : add ConstructeurLinkEntry type and useConstructeurLinks composable"
```
---
### Task F3: Frontend — Create ConstructeurLinksTable component
**Files:**
- Create: `Inventory_frontend/app/components/ConstructeurLinksTable.vue`
- [ ] **Step 1: Create the component**
A table showing selected constructeurs with editable supplierReference fields:
```vue
<template>
<div v-if="modelValue.length" class="overflow-x-auto">
<table class="table table-sm">
<thead>
<tr>
<th>Fournisseur</th>
<th>Réf. fournisseur</th>
<th v-if="!readonly" class="w-10" />
</tr>
</thead>
<tbody>
<tr v-for="(link, index) in modelValue" :key="link.constructeurId">
<td class="font-medium">
{{ getConstructeurName(link) }}
<div v-if="getConstructeurContact(link)" class="text-xs text-gray-500">
{{ getConstructeurContact(link) }}
</div>
</td>
<td>
<input
v-if="!readonly"
:value="link.supplierReference || ''"
type="text"
class="input input-bordered input-sm w-full"
placeholder="Réf. fournisseur"
@input="updateReference(index, ($event.target as HTMLInputElement).value)"
>
<span v-else>{{ link.supplierReference || '—' }}</span>
</td>
<td v-if="!readonly">
<button
type="button"
class="btn btn-ghost btn-xs text-error"
aria-label="Retirer"
@click="removeLink(index)"
>
<IconLucideX class="w-4 h-4" />
</button>
</td>
</tr>
</tbody>
</table>
</div>
</template>
<script setup lang="ts">
import type { PropType } from 'vue'
import type { ConstructeurLinkEntry } from '~/shared/constructeurUtils'
import { formatConstructeurContact } from '~/shared/constructeurUtils'
import { useConstructeurs } from '~/composables/useConstructeurs'
import IconLucideX from '~icons/lucide/x'
const props = defineProps({
modelValue: {
type: Array as PropType<ConstructeurLinkEntry[]>,
default: () => [],
},
readonly: {
type: Boolean,
default: false,
},
})
const emit = defineEmits<{
(e: 'update:modelValue', value: ConstructeurLinkEntry[]): void
(e: 'remove', constructeurId: string): void
}>()
const { getConstructeurById } = useConstructeurs()
const getConstructeurName = (link: ConstructeurLinkEntry): string =>
link.constructeur?.name || getConstructeurById(link.constructeurId)?.name || link.constructeurId
const getConstructeurContact = (link: ConstructeurLinkEntry): string => {
const c = link.constructeur || getConstructeurById(link.constructeurId)
return formatConstructeurContact(c as any)
}
const updateReference = (index: number, value: string) => {
const updated = [...props.modelValue]
updated[index] = { ...updated[index], supplierReference: value || null }
emit('update:modelValue', updated)
}
const removeLink = (index: number) => {
const removed = props.modelValue[index]
const updated = props.modelValue.filter((_, i) => i !== index)
emit('update:modelValue', updated)
emit('remove', removed.constructeurId)
}
</script>
```
- [ ] **Step 2: Commit**
```bash
cd Inventory_frontend && git add -A && git commit -m "feat(constructeur) : add ConstructeurLinksTable component"
```
---
### Task F4: Frontend — Update piece edit flow (model case)
**Files:**
- Modify: `Inventory_frontend/app/composables/usePieceEdit.ts`
- Modify: `Inventory_frontend/app/pages/piece/[id].vue`
- Modify: `Inventory_frontend/app/composables/usePieces.ts`
This task establishes the pattern for all entity types.
- [ ] **Step 1: Update usePieceEdit.ts**
Key changes:
1. Import `useConstructeurLinks` and new types
2. Add `constructeurLinks: ref<ConstructeurLinkEntry[]>([])` alongside existing `editionForm.constructeurIds`
3. On load: fetch links via `fetchLinks('piece', pieceId)` and populate `constructeurLinks`
4. Derive `editionForm.constructeurIds` from links (for ConstructeurSelect compatibility)
5. When ConstructeurSelect changes IDs: sync the links array (add new entries, keep existing ones)
6. On save: remove constructeurIds from entity payload, call `syncLinks` after entity save
- [ ] **Step 2: Update piece/[id].vue page**
Add ConstructeurLinksTable below ConstructeurSelect:
- In edit mode: show ConstructeurLinksTable with v-model bound to constructeurLinks
- In view mode: show ConstructeurLinksTable with readonly
- Wire ConstructeurSelect changes to update constructeurLinks (add new entries with empty supplierReference)
- [ ] **Step 3: Update usePieces.ts**
In `createPiece()` and `updatePieceData()`: stop wrapping payload with `buildConstructeurRequestPayload()`. Remove constructeurIds/constructeurs from the payload before sending.
- [ ] **Step 4: Lint and typecheck**
```bash
cd Inventory_frontend && npm run lint:fix && npx nuxi typecheck
```
- [ ] **Step 5: Commit**
```bash
cd Inventory_frontend && git add -A && git commit -m "feat(constructeur) : update piece edit flow with supplier references"
```
---
### Task F5: Frontend — Update composant edit flow
Same pattern as Task F4 but for composants.
**Files:**
- Modify: `Inventory_frontend/app/composables/useComponentEdit.ts`
- Modify: `Inventory_frontend/app/pages/component/[id]/index.vue`
- Modify: `Inventory_frontend/app/pages/component/[id]/edit.vue`
- Modify: `Inventory_frontend/app/composables/useComposants.ts`
- Modify: `Inventory_frontend/app/pages/component/create.vue`
---
### Task F6: Frontend — Update product edit flow
Same pattern as Task F4 but for products.
**Files:**
- Modify: product edit composable (if exists) or inline pages
- Modify: `Inventory_frontend/app/pages/product/[id]/index.vue`
- Modify: `Inventory_frontend/app/pages/product/[id]/edit.vue`
- Modify: `Inventory_frontend/app/composables/useProducts.ts`
- Modify: `Inventory_frontend/app/pages/product/create.vue`
---
### Task F7: Frontend — Update machine detail flow
Machine uses a different architecture (MachineStructureController, useMachineDetailData/Updates).
**Files:**
- Modify: `Inventory_frontend/app/composables/useMachineDetailData.ts`
- Modify: `Inventory_frontend/app/composables/useMachineDetailUpdates.ts`
- Modify: `Inventory_frontend/app/pages/machine/[id].vue`
- Modify: `Inventory_frontend/app/components/machine/MachineInfoCard.vue`
- Modify: `Inventory_frontend/app/composables/useMachines.ts`
Key differences:
- Machine data comes from `/api/machines/{id}/structure` (custom controller) which already returns the new constructeur link format
- Machine updates go through `updateMachineApi` which currently sends `constructeurIds`
- Need to adapt to read links from structure response and sync on save
---
### Task F8: Frontend — Update machine structure components (PieceItem, ComponentItem)
**Files:**
- Modify: `Inventory_frontend/app/components/PieceItem.vue`
- Modify: `Inventory_frontend/app/components/ComponentItem.vue`
These components display constructeurs in the machine structure tree and handle inline editing. Update them to:
- Read from `constructeurLinks` format in the machine structure response
- Display supplierReference alongside constructeur name
- Use syncLinks for inline updates
---
### Task F9: Frontend — Update create pages
**Files:**
- Modify: `Inventory_frontend/app/pages/pieces/create.vue`
- Modify: `Inventory_frontend/app/pages/component/create.vue`
- Modify: `Inventory_frontend/app/pages/product/create.vue`
On creation pages, there are no existing links. The flow is:
1. User selects constructeurs + optionally fills supplierReference
2. After entity creation, create all the links
3. Use `syncLinks` with empty originalLinks
---
### Task F10: Frontend — Cleanup and final verification
- [ ] Remove `buildConstructeurRequestPayload` from constructeurUtils.ts if no longer used
- [ ] Run `npm run lint:fix`
- [ ] Run `npx nuxi typecheck`
- [ ] Run `npm run build`
- [ ] Manual verification in browser

1073
docs/superpowers/plans/2026-03-31-supplier-references.md Normal file
View File

File diff suppressed because it is too large Load Diff

138
docs/superpowers/specs/2026-03-23-document-types-design.md Normal file
View File

@@ -0,0 +1,138 @@
# Document Types — Design Spec
Date: 2026-03-23
Status: Approved
## Goal
Add a `type` field to documents so users can classify them (documentation, devis, facture, plan, photo, autre). Users can set the type at upload and change it afterward via a mini-modal.
## Enum Values
| Value | Label |
|-------|-------|
| `documentation` | Documentation |
| `devis` | Devis |
| `facture` | Facture |
| `plan` | Plan |
| `photo` | Photo |
| `autre` | Autre |
Default: `documentation`
## Backend
### 1. PHP Enum
New file: `src/Enum/DocumentType.php`
```php
enum DocumentType: string
{
case DOCUMENTATION = 'documentation';
case DEVIS = 'devis';
case FACTURE = 'facture';
case PLAN = 'plan';
case PHOTO = 'photo';
case AUTRE = 'autre';
}
```
### 2. Entity Change — Document.php
Add column:
```php
#[ORM\Column(type: Types::STRING, length: 20, enumType: DocumentType::class)]
#[Groups(['document:list'])]
private DocumentType $type = DocumentType::DOCUMENTATION;
```
Add getter/setter:
```php
public function getType(): DocumentType { ... }
public function setType(DocumentType $type): static { ... }
```
### 3. API Platform — PATCH operation
Add a `Patch` operation on Document (ROLE_GESTIONNAIRE) to allow updating `name` and `type`. The existing `Put` already exists but PATCH is more appropriate for partial updates.
### 4. DocumentUploadProcessor
Accept optional `type` field from FormData. Validate against enum values, default to `documentation` if absent.
### 5. Migration
```sql
ALTER TABLE documents ADD COLUMN type VARCHAR(20) NOT NULL DEFAULT 'documentation';
-- Classify existing documents by mimeType
UPDATE documents SET type = 'photo' WHERE mimetype LIKE 'image/%';
UPDATE documents SET type = 'autre'
WHERE type = 'documentation'
AND mimetype NOT LIKE 'application/pdf'
AND mimetype NOT LIKE 'image/%';
```
### 6. DocumentQueryController
Add `type` to the response array in `formatDocument()`.
## Frontend
### 1. Type Constants
New file: `app/shared/documentTypes.ts`
```typescript
export const DOCUMENT_TYPES = [
{ value: 'documentation', label: 'Documentation' },
{ value: 'devis', label: 'Devis' },
{ value: 'facture', label: 'Facture' },
{ value: 'plan', label: 'Plan' },
{ value: 'photo', label: 'Photo' },
{ value: 'autre', label: 'Autre' },
] as const
export type DocumentTypeValue = typeof DOCUMENT_TYPES[number]['value']
```
### 2. DocumentUpload.vue — Type select at upload
Add a select dropdown (default: `documentation`) in the upload zone. The selected type applies to all files in the current batch. Pass the type through to `uploadDocuments()`.
### 3. useDocuments composable
- `uploadDocuments()`: accept `type` in the upload context, append to FormData
- New method: `updateDocument(id, { name, type })` — PATCH `/api/documents/{id}` with `application/merge-patch+json`
- Add `type` to the `Document` interface
### 4. DocumentEditModal.vue (new component)
Mini-modal with:
- Input text: document name (pre-filled)
- Select: document type (pre-filled)
- Buttons: Annuler / Sauvegarder
- On save: call `updateDocument()`, emit `updated` event
### 5. Document list display
Everywhere documents are listed (machine detail, composant edit, piece edit, product, site):
- Show type as a small badge next to the document name
- Add a pencil/edit button that opens `DocumentEditModal`
- On modal save: refresh the document in local state
## Migration of existing data
All existing documents classified by mimeType:
- `image/*` → `photo`
- `application/pdf` → `documentation`
- Everything else → `autre`
## Out of scope
- Custom user-defined types (table `document_types`) — can be added later
- Filtering documents by type in the UI — can be added later
- Bulk type change

88
docs/superpowers/specs/2026-03-23-parc-machines-ux-design.md Normal file
View File

@@ -0,0 +1,88 @@
# Parc Machines — Améliorations UX
**Date** : 2026-03-23
**Scope** : 3 changements sur le frontend + 1 extension backend
---
## 1. Filtre sites multi-sélection par checkboxes
### Contexte
Le filtre site actuel est un `<select>` mono-sélection dans `machines/index.vue`.
L'utilisateur veut pouvoir sélectionner plusieurs sites simultanément.
### Design
- Remplacer le `<select>` par une rangée de checkboxes DaisyUI directement visibles dans la barre de filtre.
- Chaque site = une checkbox avec le nom du site.
- Quand **aucune** checkbox n'est cochée → toutes les machines s'affichent (équivalent "Tous les sites").
- Quand **une ou plusieurs** sont cochées → filtre sur ces sites uniquement.
### Changements techniques
**Fichier** : `Inventory_frontend/app/pages/machines/index.vue`
- **Réactivité** : utiliser `reactive(new Set())` (Vue 3.4+ supporte nativement les mutations `add`/`delete`/`has` sur un Set réactif). Pas de `.value` nécessaire.
- **Note** : le fichier utilise `<script setup>` sans `lang="ts"` — ne pas utiliser d'annotations TypeScript comme `Set<string>`.
- Template : remplacer le `<select>` par un `div` flex-wrap avec des checkboxes DaisyUI (`checkbox checkbox-sm`) + label pour chaque site.
- Computed `filteredMachines` : remplacer `machine.siteId === selectedSite` par `selectedSites.size === 0 || selectedSites.has(machine.siteId)`.
---
## 2. Tri alphabétique croissant
### Contexte
Les machines s'affichent dans l'ordre retourné par l'API, sans tri. L'utilisateur veut un tri alphabétique croissant par nom.
### Design
Ajouter un `.sort()` avec `localeCompare('fr')` à la fin du computed `filteredMachines`.
### Changements techniques
**Fichier** : `Inventory_frontend/app/pages/machines/index.vue`
- Dans le computed `filteredMachines`, ajouter avant le `return` :
```js
filtered = [...filtered].sort((a, b) =>
(a.name || '').localeCompare(b.name || '', 'fr')
)
```
---
## 3. Recherche par référence dans les catalogues (Pièces, Composants, Produits)
### Contexte
Les placeholders des champs de recherche promettent "Nom ou référence…" mais le frontend n'envoie que `?name=xxx` à l'API. Le backend (API Platform SearchFilter) supporte `name` et `reference` en `ipartial`, mais combiner `?name=xxx&reference=xxx` produit un AND (les deux doivent matcher), pas un OR.
### Design
Créer une **Extension Doctrine** (`SearchByNameOrReferenceExtension`) qui intercepte un paramètre `?q=xxx` et ajoute une clause `WHERE name ILIKE %xxx% OR reference ILIKE %xxx%` à la requête. Côté frontend, remplacer `params.set('name', search)` par `params.set('q', search)`.
### Changements techniques
**Backend — Nouveau fichier** : `src/Doctrine/SearchByNameOrReferenceExtension.php`
- Implémente `QueryCollectionExtensionInterface`
- S'applique aux entités `Piece`, `Composant`, `Product`
- Lit le paramètre `q` depuis la requête HTTP
- Ajoute `LOWER(o.name) LIKE :searchQ OR LOWER(o.reference) LIKE :searchQ` avec paramètre `%{strtolower(q)}%`
- **Échappement LIKE** : les caractères `%` et `_` dans l'input utilisateur sont échappés via `addcslashes($q, '%_')` pour éviter des matchs trop larges
- **`reference` nullable** : les lignes avec `reference = NULL` ne matcheront pas (comportement SQL standard : `NULL LIKE x` = NULL = false), ce qui est le comportement attendu
- **Pas de conflit** avec le `SearchFilter` existant : le paramètre `q` n'est pas enregistré comme propriété de `SearchFilter`, donc il sera ignoré par celui-ci. Les filtres `name` et `reference` restent disponibles pour d'autres usages.
**Frontend — 3 fichiers** (dans la fonction `loadXxx`, remplacer l'appel `params.set('name', search.trim())`) :
- `Inventory_frontend/app/composables/usePieces.ts` → `params.set('q', search.trim())`
- `Inventory_frontend/app/composables/useComposants.ts` → idem
- `Inventory_frontend/app/composables/useProducts.ts` → idem
---
## Fichiers impactés (résumé)
| Fichier | Changement |
|---------|-----------|
| `Inventory_frontend/app/pages/machines/index.vue` | Checkboxes sites + tri alphabétique |
| `src/Doctrine/SearchByNameOrReferenceExtension.php` | **Nouveau** — Extension Doctrine OR search |
| `Inventory_frontend/app/composables/usePieces.ts` | `name` → `q` |
| `Inventory_frontend/app/composables/useComposants.ts` | `name` → `q` |
| `Inventory_frontend/app/composables/useProducts.ts` | `name` → `q` |
## Hors scope
- La page Parc Machines cherche **déjà** sur nom ET référence côté frontend (filtrage client-side). Pas de changement nécessaire.
- Aucun changement de placeholder — ils affichent déjà "Nom ou référence…".

19
docs/superpowers/specs/2026-03-25-entity-versioning-design.md
View File

@@ -20,7 +20,8 @@ Permettre de consulter l'historique des versions numerotees (v1, v2, v3...) des
### Restauration ### Restauration
- La restauration cree une **nouvelle version** (v+1) — on ne supprime jamais d'historique - La restauration cree une **nouvelle version** (v+1) — on ne supprime jamais d'historique
- L'AuditLog de la restauration a `action = "restore"` et le diff contient `restoredFromVersion: N` - Le service `EntityVersionService::restore()` cree **manuellement** un AuditLog avec `action = "restore"` et le diff contient `restoredFromVersion: N`
- Important : le flush du restore declenche les AuditSubscribers, qui produiraient un `update` duplique. Pour eviter cela, l'entite porte un flag transitoire `$skipAudit = true` que les subscribers verifient
### Controle de squelette (Composant, Piece, Produit uniquement) ### Controle de squelette (Composant, Piece, Produit uniquement)
- Avant restauration, on compare le ModelType actuel avec celui du snapshot - Avant restauration, on compare le ModelType actuel avec celui du snapshot
@@ -35,10 +36,13 @@ Permettre de consulter l'historique des versions numerotees (v1, v2, v3...) des
- **Machine** : site, liens composants/pieces/produits (MachineComponentLink, MachinePieceLink, MachineProductLink) - **Machine** : site, liens composants/pieces/produits (MachineComponentLink, MachinePieceLink, MachineProductLink)
- Les entites manquantes generent des **warnings** affiches a l'utilisateur - Les entites manquantes generent des **warnings** affiches a l'utilisateur
- Les slots avec des entites supprimees sont restaures **vides** (sans selection) - Les slots avec des entites supprimees sont restaures **vides** (sans selection)
- Pour les custom field values : restauration par `fieldId` + entite parente (pas par ID de la CustomFieldValue elle-meme, car un sync ModelType peut recreer les CFV avec des IDs differents)
- Les controles d'integrite utilisent des requetes batch (`findBy(['id' => $ids])`) plutot que des requetes individuelles par slot
### Machines ### Machines
- Pas de controle de squelette (pas de ModelType) : restauration toujours complete - Pas de controle de squelette (pas de ModelType) : restauration toujours complete
- Controle d'integrite sur le site et les liens machine - Controle d'integrite sur le site et les liens machine
- Machine n'a pas de champ `description` (contrairement aux autres entites)
### Permissions ### Permissions
- Consulter les versions : `ROLE_VIEWER` - Consulter les versions : `ROLE_VIEWER`
@@ -78,7 +82,7 @@ Les Audit Subscribers doivent inclure dans le `snapshot` :
"prix": 100.00, "prix": 100.00,
"typeComposant": { "id": "cl...", "name": "...", "code": "..." }, "typeComposant": { "id": "cl...", "name": "...", "code": "..." },
"product": { "id": "cl...", "name": "..." }, "product": { "id": "cl...", "name": "..." },
"constructeurs": [{ "id": "cl...", "name": "..." }], "constructeurIds": [{ "id": "cl...", "name": "..." }],
"customFieldValues": [{ "id": "cl...", "fieldName": "...", "value": "..." }], "customFieldValues": [{ "id": "cl...", "fieldName": "...", "value": "..." }],
"pieceSlots": [ "pieceSlots": [
{ "id": "cl...", "typePieceId": "cl...", "selectedPieceId": "cl...", "quantity": 1, "position": 0 } { "id": "cl...", "typePieceId": "cl...", "selectedPieceId": "cl...", "quantity": 1, "position": 0 }
@@ -103,7 +107,7 @@ Les Audit Subscribers doivent inclure dans le `snapshot` :
"prix": 50.00, "prix": 50.00,
"typePiece": { "id": "cl...", "name": "...", "code": "..." }, "typePiece": { "id": "cl...", "name": "...", "code": "..." },
"product": { "id": "cl...", "name": "..." }, "product": { "id": "cl...", "name": "..." },
"constructeurs": [{ "id": "cl...", "name": "..." }], "constructeurIds": [{ "id": "cl...", "name": "..." }],
"customFieldValues": [{ "id": "cl...", "fieldName": "...", "value": "..." }], "customFieldValues": [{ "id": "cl...", "fieldName": "...", "value": "..." }],
"productSlots": [ "productSlots": [
{ "id": "cl...", "typeProductId": "cl...", "selectedProductId": "cl...", "familyCode": "...", "position": 0 } { "id": "cl...", "typeProductId": "cl...", "selectedProductId": "cl...", "familyCode": "...", "position": 0 }
@@ -120,7 +124,7 @@ Les Audit Subscribers doivent inclure dans le `snapshot` :
"reference": "...", "reference": "...",
"supplierPrice": 25.00, "supplierPrice": 25.00,
"typeProduct": { "id": "cl...", "name": "...", "code": "..." }, "typeProduct": { "id": "cl...", "name": "...", "code": "..." },
"constructeurs": [{ "id": "cl...", "name": "..." }], "constructeurIds": [{ "id": "cl...", "name": "..." }],
"customFieldValues": [{ "id": "cl...", "fieldName": "...", "value": "..." }], "customFieldValues": [{ "id": "cl...", "fieldName": "...", "value": "..." }],
"version": 1 "version": 1
} }
@@ -132,8 +136,9 @@ Les Audit Subscribers doivent inclure dans le `snapshot` :
"id": "cl...", "id": "cl...",
"name": "...", "name": "...",
"reference": "...", "reference": "...",
"description": "...", "prix": 1500.00,
"site": { "id": "cl...", "name": "..." }, "site": { "id": "cl...", "name": "..." },
"constructeurIds": [{ "id": "cl...", "name": "..." }],
"customFieldValues": [{ "id": "cl...", "fieldName": "...", "value": "..." }], "customFieldValues": [{ "id": "cl...", "fieldName": "...", "value": "..." }],
"version": 4 "version": 4
} }
@@ -295,6 +300,10 @@ CREATE INDEX IF NOT EXISTS idx_audit_entity_version ON audit_logs (entity_type,
--- ---
## Ce qui change (breaking)
- **Piece snapshot** : le champ legacy `productIds` (ancien JSON) est remplace par `productSlots` (tables normalisees). Les anciens AuditLogs conservent `productIds` dans leur snapshot mais les nouveaux ne l'auront plus. Le restore utilise `productSlots` exclusivement.
## Ce qui ne change PAS ## Ce qui ne change PAS
- L'onglet/page d'historique existant (`EntityHistoryController`) reste inchange - L'onglet/page d'historique existant (`EntityHistoryController`) reste inchange

82
docs/superpowers/specs/2026-03-31-supplier-references-design.md Normal file
View File

@@ -0,0 +1,82 @@
# Références Fournisseur par Item — Design Spec
**Date :** 2026-03-31
**Statut :** Validé
## Contexte
Chaque entité (Machine, Pièce, Composant, Produit) a un champ `reference` générique et une relation ManyToMany avec `Constructeur`. Il n'existe aucun moyen de stocker une référence spécifique par fournisseur — si un item est vendu par 3 fournisseurs avec 3 références différentes, on ne peut en stocker qu'une seule.
## Objectif
Permettre de stocker une référence fournisseur (`supplierReference`) par couple (item, constructeur). Le champ `reference` existant reste inchangé comme référence interne. Le champ `supplierPrice` sur Product reste inchangé.
## Design
### Approche retenue : conversion ManyToMany → entités pivot
Remplacer les 4 tables de jointure simples (`_MachineConstructeurs`, `_PieceConstructeurs`, `_ComposantConstructeurs`, `_ProductConstructeurs`) par de vraies entités Doctrine Link, suivant le pattern existant (`MachinePieceLink`, `MachineComponentLink`, etc.).
### Nouvelles entités
| Entité | Table | FK item | FK constructeur | Champs extra |
|--------|-------|---------|-----------------|--------------|
| `MachineConstructeurLink` | `machine_constructeur_links` | `machineId` → `Machine` | `constructeurId` → `Constructeur` | `supplierReference` (string 255, nullable) |
| `PieceConstructeurLink` | `piece_constructeur_links` | `pieceId` → `Piece` | `constructeurId` → `Constructeur` | `supplierReference` (string 255, nullable) |
| `ComposantConstructeurLink` | `composant_constructeur_links` | `composantId` → `Composant` | `constructeurId` → `Constructeur` | `supplierReference` (string 255, nullable) |
| `ProductConstructeurLink` | `product_constructeur_links` | `productId` → `Product` | `constructeurId` → `Constructeur` | `supplierReference` (string 255, nullable) |
### Structure de chaque entité
Chaque entité suit le pattern `MachinePieceLink` :
- `CuidEntityTrait` pour l'ID (string, 36 chars)
- `#[ORM\HasLifecycleCallbacks]` avec `createdAt` / `updatedAt`
- Contrainte unique sur `(item_id, constructeur_id)` via `#[ORM\UniqueConstraint]`
- `#[ApiResource]` avec opérations CRUD complètes
- Sécurité : `ROLE_VIEWER` pour lecture, `ROLE_GESTIONNAIRE` pour écriture
- `ManyToOne` vers l'item (onDelete CASCADE)
- `ManyToOne` vers `Constructeur` (onDelete CASCADE)
- Champ `supplierReference` (string 255, nullable)
### Modifications sur les entités existantes
#### Machine, Pièce, Composant, Produit
- Supprimer la propriété `ManyToMany` `constructeurs` et ses getters/setters/add/remove
- Ajouter une propriété `OneToMany` `constructeurLinks` vers le Link correspondant
- Getter `getConstructeurLinks(): Collection`
#### Constructeur
- Supprimer les 4 propriétés `ManyToMany` (`machines`, `composants`, `pieces`, `products`) et leurs getters/setters
- Ajouter 4 propriétés `OneToMany` vers les Links correspondants
### Migration SQL
1. Créer les 4 nouvelles tables avec colonnes `id`, `machineId`/`pieceId`/etc., `constructeurId`, `supplierReference`, `createdAt`, `updatedAt`
2. Ajouter les contraintes uniques
3. Migrer les données des anciennes tables de jointure vers les nouvelles (génération CUID pour chaque ligne, `supplierReference` = NULL)
4. Supprimer les anciennes tables de jointure (`_MachineConstructeurs`, `_PieceConstructeurs`, `_ComposantConstructeurs`, `_ProductConstructeurs`)
### API
Endpoints API Platform auto-générés pour chaque Link :
- `GET /api/machine_constructeur_links` — liste (filtrable par machine, constructeur)
- `GET /api/machine_constructeur_links/{id}` — détail
- `POST /api/machine_constructeur_links` — créer un lien avec référence
- `PATCH /api/machine_constructeur_links/{id}` — modifier la référence
- `DELETE /api/machine_constructeur_links/{id}` — supprimer le lien
Idem pour les 3 autres types.
### Frontend
Les pages détail/édition qui affichent les constructeurs devront être adaptées pour :
- Afficher la `supplierReference` à côté de chaque constructeur
- Permettre l'édition de la référence fournisseur lors de l'ajout/modification d'un constructeur
- Utiliser les endpoints `*ConstructeurLink` au lieu de la collection `constructeurs`
### Hors périmètre
- Migration de `supplierPrice` de Product vers le Link (explicitement exclu)
- Modification du champ `reference` existant sur les entités
- Référence auto (`referenceAuto`) sur Pièce/Composant — non impactée

115
migrations/Version20260331121257.php Normal file
View File

@@ -0,0 +1,115 @@
<?php
declare(strict_types=1);
namespace DoctrineMigrations;
use Doctrine\DBAL\Schema\Schema;
use Doctrine\Migrations\AbstractMigration;
final class Version20260331121257 extends AbstractMigration
{
public function getDescription(): string
{
return 'Create ConstructeurLink tables, migrate data from old join tables, drop old join tables';
}
public function up(Schema $schema): void
{
// Create new link tables
$this->addSql('CREATE TABLE IF NOT EXISTS composant_constructeur_links (id VARCHAR(36) NOT NULL, supplierReference VARCHAR(255) DEFAULT NULL, createdAt TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL, updatedAt TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL, composantId VARCHAR(36) NOT NULL, constructeurId VARCHAR(36) NOT NULL, PRIMARY KEY (id))');
$this->addSql('CREATE INDEX IDX_139F3E3A345EE564 ON composant_constructeur_links (composantId)');
$this->addSql('CREATE INDEX IDX_139F3E3A70AF5AF0 ON composant_constructeur_links (constructeurId)');
$this->addSql('CREATE UNIQUE INDEX uniq_composant_constructeur ON composant_constructeur_links (composantid, constructeurid)');
$this->addSql('CREATE TABLE IF NOT EXISTS machine_constructeur_links (id VARCHAR(36) NOT NULL, supplierReference VARCHAR(255) DEFAULT NULL, createdAt TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL, updatedAt TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL, machineId VARCHAR(36) NOT NULL, constructeurId VARCHAR(36) NOT NULL, PRIMARY KEY (id))');
$this->addSql('CREATE INDEX IDX_665B2620633EC4FD ON machine_constructeur_links (machineId)');
$this->addSql('CREATE INDEX IDX_665B262070AF5AF0 ON machine_constructeur_links (constructeurId)');
$this->addSql('CREATE UNIQUE INDEX uniq_machine_constructeur ON machine_constructeur_links (machineid, constructeurid)');
$this->addSql('CREATE TABLE IF NOT EXISTS piece_constructeur_links (id VARCHAR(36) NOT NULL, supplierReference VARCHAR(255) DEFAULT NULL, createdAt TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL, updatedAt TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL, pieceId VARCHAR(36) NOT NULL, constructeurId VARCHAR(36) NOT NULL, PRIMARY KEY (id))');
$this->addSql('CREATE INDEX IDX_E664DAEC3C6A9D1 ON piece_constructeur_links (pieceId)');
$this->addSql('CREATE INDEX IDX_E664DAEC70AF5AF0 ON piece_constructeur_links (constructeurId)');
$this->addSql('CREATE UNIQUE INDEX uniq_piece_constructeur ON piece_constructeur_links (pieceid, constructeurid)');
$this->addSql('CREATE TABLE IF NOT EXISTS product_constructeur_links (id VARCHAR(36) NOT NULL, supplierReference VARCHAR(255) DEFAULT NULL, createdAt TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL, updatedAt TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL, productId VARCHAR(36) NOT NULL, constructeurId VARCHAR(36) NOT NULL, PRIMARY KEY (id))');
$this->addSql('CREATE INDEX IDX_E3D850B536799605 ON product_constructeur_links (productId)');
$this->addSql('CREATE INDEX IDX_E3D850B570AF5AF0 ON product_constructeur_links (constructeurId)');
$this->addSql('CREATE UNIQUE INDEX uniq_product_constructeur ON product_constructeur_links (productid, constructeurid)');
// Foreign keys
$this->addSql('ALTER TABLE composant_constructeur_links ADD CONSTRAINT FK_139F3E3A345EE564 FOREIGN KEY (composantId) REFERENCES composants (id) ON DELETE CASCADE NOT DEFERRABLE');
$this->addSql('ALTER TABLE composant_constructeur_links ADD CONSTRAINT FK_139F3E3A70AF5AF0 FOREIGN KEY (constructeurId) REFERENCES constructeurs (id) ON DELETE CASCADE NOT DEFERRABLE');
$this->addSql('ALTER TABLE machine_constructeur_links ADD CONSTRAINT FK_665B2620633EC4FD FOREIGN KEY (machineId) REFERENCES machines (id) ON DELETE CASCADE NOT DEFERRABLE');
$this->addSql('ALTER TABLE machine_constructeur_links ADD CONSTRAINT FK_665B262070AF5AF0 FOREIGN KEY (constructeurId) REFERENCES constructeurs (id) ON DELETE CASCADE NOT DEFERRABLE');
$this->addSql('ALTER TABLE piece_constructeur_links ADD CONSTRAINT FK_E664DAEC3C6A9D1 FOREIGN KEY (pieceId) REFERENCES pieces (id) ON DELETE CASCADE NOT DEFERRABLE');
$this->addSql('ALTER TABLE piece_constructeur_links ADD CONSTRAINT FK_E664DAEC70AF5AF0 FOREIGN KEY (constructeurId) REFERENCES constructeurs (id) ON DELETE CASCADE NOT DEFERRABLE');
$this->addSql('ALTER TABLE product_constructeur_links ADD CONSTRAINT FK_E3D850B536799605 FOREIGN KEY (productId) REFERENCES products (id) ON DELETE CASCADE NOT DEFERRABLE');
$this->addSql('ALTER TABLE product_constructeur_links ADD CONSTRAINT FK_E3D850B570AF5AF0 FOREIGN KEY (constructeurId) REFERENCES constructeurs (id) ON DELETE CASCADE NOT DEFERRABLE');
// Data migration: copy from old join tables to new link tables
$this->addSql("INSERT INTO machine_constructeur_links (id, machineid, constructeurid, supplierreference, createdat, updatedat) SELECT 'cl' || substring(md5(random()::text || clock_timestamp()::text), 1, 24), a, b, NULL, NOW(), NOW() FROM \"_machineconstructeurs\"");
$this->addSql("INSERT INTO piece_constructeur_links (id, pieceid, constructeurid, supplierreference, createdat, updatedat) SELECT 'cl' || substring(md5(random()::text || clock_timestamp()::text), 1, 24), a, b, NULL, NOW(), NOW() FROM \"_piececonstructeurs\"");
$this->addSql("INSERT INTO composant_constructeur_links (id, composantid, constructeurid, supplierreference, createdat, updatedat) SELECT 'cl' || substring(md5(random()::text || clock_timestamp()::text), 1, 24), a, b, NULL, NOW(), NOW() FROM \"_composantconstructeurs\"");
$this->addSql("INSERT INTO product_constructeur_links (id, productid, constructeurid, supplierreference, createdat, updatedat) SELECT 'cl' || substring(md5(random()::text || clock_timestamp()::text), 1, 24), a, b, NULL, NOW(), NOW() FROM \"_productconstructeurs\"");
// Drop old join tables
$this->addSql('ALTER TABLE _composantconstructeurs DROP CONSTRAINT IF EXISTS fk_607601254ad0cf31');
$this->addSql('ALTER TABLE _composantconstructeurs DROP CONSTRAINT IF EXISTS fk_60760125d3d99e8b');
$this->addSql('ALTER TABLE _machineconstructeurs DROP CONSTRAINT IF EXISTS fk_e6a040cc4ad0cf31');
$this->addSql('ALTER TABLE _machineconstructeurs DROP CONSTRAINT IF EXISTS fk_e6a040ccd3d99e8b');
$this->addSql('ALTER TABLE _piececonstructeurs DROP CONSTRAINT IF EXISTS fk_e94732e54ad0cf31');
$this->addSql('ALTER TABLE _piececonstructeurs DROP CONSTRAINT IF EXISTS fk_e94732e5d3d99e8b');
$this->addSql('ALTER TABLE _productconstructeurs DROP CONSTRAINT IF EXISTS fk_cf7403fc4ad0cf31');
$this->addSql('ALTER TABLE _productconstructeurs DROP CONSTRAINT IF EXISTS fk_cf7403fcd3d99e8b');
$this->addSql('DROP TABLE IF EXISTS _composantconstructeurs');
$this->addSql('DROP TABLE IF EXISTS _machineconstructeurs');
$this->addSql('DROP TABLE IF EXISTS _piececonstructeurs');
$this->addSql('DROP TABLE IF EXISTS _productconstructeurs');
}
public function down(Schema $schema): void
{
// Recreate old join tables
$this->addSql('CREATE TABLE _composantconstructeurs (a VARCHAR(36) NOT NULL, b VARCHAR(36) NOT NULL, PRIMARY KEY (a, b))');
$this->addSql('CREATE INDEX "_ComposantConstructeurs_B_index" ON _composantconstructeurs (b)');
$this->addSql('CREATE INDEX IDX_5B97D813E8B7BE43 ON _composantconstructeurs (a)');
$this->addSql('CREATE TABLE _machineconstructeurs (a VARCHAR(36) NOT NULL, b VARCHAR(36) NOT NULL, PRIMARY KEY (a, b))');
$this->addSql('CREATE INDEX "_MachineConstructeurs_B_index" ON _machineconstructeurs (b)');
$this->addSql('CREATE INDEX IDX_4F225B32E8B7BE43 ON _machineconstructeurs (a)');
$this->addSql('CREATE TABLE _piececonstructeurs (a VARCHAR(36) NOT NULL, b VARCHAR(36) NOT NULL, PRIMARY KEY (a, b))');
$this->addSql('CREATE INDEX "_PieceConstructeurs_B_index" ON _piececonstructeurs (b)');
$this->addSql('CREATE INDEX IDX_77FC120E8B7BE43 ON _piececonstructeurs (a)');
$this->addSql('CREATE TABLE _productconstructeurs (a VARCHAR(36) NOT NULL, b VARCHAR(36) NOT NULL, PRIMARY KEY (a, b))');
$this->addSql('CREATE INDEX "_ProductConstructeurs_B_index" ON _productconstructeurs (b)');
$this->addSql('CREATE INDEX IDX_66F61802E8B7BE43 ON _productconstructeurs (a)');
$this->addSql('ALTER TABLE _composantconstructeurs ADD CONSTRAINT fk_607601254ad0cf31 FOREIGN KEY (b) REFERENCES constructeurs (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE');
$this->addSql('ALTER TABLE _composantconstructeurs ADD CONSTRAINT fk_60760125d3d99e8b FOREIGN KEY (a) REFERENCES composants (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE');
$this->addSql('ALTER TABLE _machineconstructeurs ADD CONSTRAINT fk_e6a040cc4ad0cf31 FOREIGN KEY (b) REFERENCES constructeurs (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE');
$this->addSql('ALTER TABLE _machineconstructeurs ADD CONSTRAINT fk_e6a040ccd3d99e8b FOREIGN KEY (a) REFERENCES machines (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE');
$this->addSql('ALTER TABLE _piececonstructeurs ADD CONSTRAINT fk_e94732e54ad0cf31 FOREIGN KEY (b) REFERENCES constructeurs (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE');
$this->addSql('ALTER TABLE _piececonstructeurs ADD CONSTRAINT fk_e94732e5d3d99e8b FOREIGN KEY (a) REFERENCES pieces (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE');
$this->addSql('ALTER TABLE _productconstructeurs ADD CONSTRAINT fk_cf7403fc4ad0cf31 FOREIGN KEY (b) REFERENCES constructeurs (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE');
$this->addSql('ALTER TABLE _productconstructeurs ADD CONSTRAINT fk_cf7403fcd3d99e8b FOREIGN KEY (a) REFERENCES products (id) ON DELETE CASCADE NOT DEFERRABLE INITIALLY IMMEDIATE');
// Data migration: copy back from new link tables to old join tables
$this->addSql('INSERT INTO _machineconstructeurs (a, b) SELECT machineid, constructeurid FROM machine_constructeur_links');
$this->addSql('INSERT INTO _piececonstructeurs (a, b) SELECT pieceid, constructeurid FROM piece_constructeur_links');
$this->addSql('INSERT INTO _composantconstructeurs (a, b) SELECT composantid, constructeurid FROM composant_constructeur_links');
$this->addSql('INSERT INTO _productconstructeurs (a, b) SELECT productid, constructeurid FROM product_constructeur_links');
// Drop new link tables
$this->addSql('ALTER TABLE composant_constructeur_links DROP CONSTRAINT FK_139F3E3A345EE564');
$this->addSql('ALTER TABLE composant_constructeur_links DROP CONSTRAINT FK_139F3E3A70AF5AF0');
$this->addSql('ALTER TABLE machine_constructeur_links DROP CONSTRAINT FK_665B2620633EC4FD');
$this->addSql('ALTER TABLE machine_constructeur_links DROP CONSTRAINT FK_665B262070AF5AF0');
$this->addSql('ALTER TABLE piece_constructeur_links DROP CONSTRAINT FK_E664DAEC3C6A9D1');
$this->addSql('ALTER TABLE piece_constructeur_links DROP CONSTRAINT FK_E664DAEC70AF5AF0');
$this->addSql('ALTER TABLE product_constructeur_links DROP CONSTRAINT FK_E3D850B536799605');
$this->addSql('ALTER TABLE product_constructeur_links DROP CONSTRAINT FK_E3D850B570AF5AF0');
$this->addSql('DROP TABLE composant_constructeur_links');
$this->addSql('DROP TABLE machine_constructeur_links');
$this->addSql('DROP TABLE piece_constructeur_links');
$this->addSql('DROP TABLE product_constructeur_links');
}
}

80
scripts/setup-remote-machines.sh Executable file
View File

@@ -0,0 +1,80 @@
#!/usr/bin/env bash
set -euo pipefail
# ==============================================================================
# Setup remote machines: git aliases + zsh + Oh My Zsh
# Usage: ./scripts/setup-remote-machines.sh user@host1 user@host2 ...
# ==============================================================================
if [ $# -eq 0 ]; then
echo "Usage: $0 user@host1 [user@host2 ...]"
echo "Example: $0 matthieu@192.168.1.10 matthieu@192.168.1.20"
exit 1
fi
REMOTE_SCRIPT='
set -e
echo "=== [1/3] Configuration git aliases ==="
git config --global alias.st "status"
git config --global alias.co "checkout"
git config --global alias.br "branch"
git config --global alias.ci "commit"
git config --global alias.cm "commit -m"
git config --global alias.s "status"
git config --global alias.last "log -1 HEAD"
git config --global alias.unstage "reset HEAD --"
git config --global alias.hist "log --oneline --graph --decorate --all"
git config --global alias.df "diff"
git config --global alias.dc "diff --cached"
git config --global alias.lg "log --oneline --graph --decorate"
git config --global alias.type "cat-file -t"
git config --global alias.dump "cat-file -p"
echo " -> Git aliases OK"
echo "=== [2/3] Installation de zsh ==="
if command -v zsh &>/dev/null; then
echo " -> zsh deja installe ($(zsh --version))"
else
if command -v apt-get &>/dev/null; then
sudo apt-get update -qq && sudo apt-get install -y -qq zsh
elif command -v dnf &>/dev/null; then
sudo dnf install -y zsh
elif command -v yum &>/dev/null; then
sudo yum install -y zsh
elif command -v pacman &>/dev/null; then
sudo pacman -S --noconfirm zsh
else
echo " !! Gestionnaire de paquets non reconnu, installe zsh manuellement"
exit 1
fi
echo " -> zsh installe"
fi
echo "=== [3/3] Installation de Oh My Zsh ==="
if [ -d "$HOME/.oh-my-zsh" ]; then
echo " -> Oh My Zsh deja installe"
else
sh -c "$(curl -fsSL https://raw.githubusercontent.com/ohmyzsh/ohmyzsh/master/tools/install.sh)" "" --unattended
echo " -> Oh My Zsh installe"
fi
echo "=== Terminé ! ==="
'
for HOST in "$@"; do
echo ""
echo "========================================"
echo " Configuration de: $HOST"
echo "========================================"
ssh -o ConnectTimeout=10 -o StrictHostKeyChecking=accept-new "$HOST" "$REMOTE_SCRIPT"
# Extraire le user du format user@host
REMOTE_USER="${HOST%%@*}"
echo " -> Changement du shell par defaut en zsh (necessite mot de passe sudo)..."
ssh -t -o ConnectTimeout=10 "$HOST" "sudo chsh -s \$(which zsh) $REMOTE_USER"
echo " -> $HOST OK"
done
echo ""
echo "Toutes les machines ont ete configurees."

12
src/Command/CheckMissingCustomFieldValuesCommand.php
View File

@@ -59,10 +59,10 @@ final class CheckMissingCustomFieldValuesCommand extends Command
protected function execute(InputInterface $input, OutputInterface $output): int protected function execute(InputInterface $input, OutputInterface $output): int
{ {
$io = new SymfonyStyle($input, $output); $io = new SymfonyStyle($input, $output);
$entityScope = (string) $input->getOption('entity'); $entityScope = (string) $input->getOption('entity');
$limit = max(1, (int) $input->getOption('limit')); $limit = max(1, (int) $input->getOption('limit'));
$maxRows = max(1, (int) $input->getOption('max-rows')); $maxRows = max(1, (int) $input->getOption('max-rows'));
$recoverableOnly = (bool) $input->getOption('recoverable-only'); $recoverableOnly = (bool) $input->getOption('recoverable-only');
if (!in_array($entityScope, ['all', 'piece', 'composant'], true)) { if (!in_array($entityScope, ['all', 'piece', 'composant'], true)) {
@@ -71,7 +71,7 @@ final class CheckMissingCustomFieldValuesCommand extends Command
return Command::FAILURE; return Command::FAILURE;
} }
$rows = []; $rows = [];
$counts = [ $counts = [
'piece' => 0, 'piece' => 0,
'composant' => 0, 'composant' => 0,
@@ -202,7 +202,7 @@ final class CheckMissingCustomFieldValuesCommand extends Command
} }
$currentValue = $currentValuesByFieldId[$definition->getId()] ?? null; $currentValue = $currentValuesByFieldId[$definition->getId()] ?? null;
$issue = null; $issue = null;
if (!$currentValue instanceof CustomFieldValue) { if (!$currentValue instanceof CustomFieldValue) {
$issue = 'missing'; $issue = 'missing';

16
src/Command/RestorePieceCustomFieldValuesCommand.php
View File

@@ -21,13 +21,13 @@ use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Style\SymfonyStyle; use Symfony\Component\Console\Style\SymfonyStyle;
use function array_key_exists; use function array_key_exists;
use function count;
use function is_array; use function is_array;
use function is_string; use function is_string;
use function ksort; use function ksort;
use function preg_replace; use function preg_replace;
use function sprintf; use function sprintf;
use function str_starts_with; use function str_starts_with;
use function strlen;
use function strtolower; use function strtolower;
use function trim; use function trim;
@@ -110,20 +110,20 @@ final class RestorePieceCustomFieldValuesCommand extends Command
continue; continue;
} }
$currentValue = $currentValuesByFieldId[$definition->getId()] ?? null; $currentValue = $currentValuesByFieldId[$definition->getId()] ?? null;
$shouldRestore = null === $currentValue || '' === trim($currentValue->getValue()); $shouldRestore = null === $currentValue || '' === trim($currentValue->getValue());
if (!$shouldRestore) { if (!$shouldRestore) {
continue; continue;
} }
$candidate = $historicalValues[$normalizedName]; $candidate = $historicalValues[$normalizedName];
$plannedRows[] = [ $plannedRows[] = [
$definition->getName(), $definition->getName(),
$candidate['value'], $candidate['value'],
$candidate['sourceDate'], $candidate['sourceDate'],
$currentValue ? 'update-empty' : 'create-missing', $currentValue ? 'update-empty' : 'create-missing',
]; ];
$changesCount++; ++$changesCount;
if (!$apply) { if (!$apply) {
continue; continue;
@@ -186,7 +186,7 @@ final class RestorePieceCustomFieldValuesCommand extends Command
continue; continue;
} }
$rawName = trim(substr($field, \strlen('customField:'))); $rawName = trim(substr($field, strlen('customField:')));
$normalizedName = $this->normalizeFieldName($rawName); $normalizedName = $this->normalizeFieldName($rawName);
if ('' === $normalizedName || array_key_exists($normalizedName, $values)) { if ('' === $normalizedName || array_key_exists($normalizedName, $values)) {
continue; continue;
@@ -198,9 +198,9 @@ final class RestorePieceCustomFieldValuesCommand extends Command
} }
$values[$normalizedName] = [ $values[$normalizedName] = [
'value' => $candidate, 'value' => $candidate,
'sourceDate' => $log->getCreatedAt()->format('Y-m-d H:i:s'), 'sourceDate' => $log->getCreatedAt()->format('Y-m-d H:i:s'),
'sourceField'=> $rawName, 'sourceField' => $rawName,
]; ];
} }
} }

6
src/Command/RestoreRecoverablePieceCustomFieldValuesCommand.php
View File

@@ -95,7 +95,7 @@ final class RestoreRecoverablePieceCustomFieldValuesCommand extends Command
continue; continue;
} }
$pieceCount++; ++$pieceCount;
$changesCount += count($pieceRows); $changesCount += count($pieceRows);
$rows = [...$rows, ...$pieceRows]; $rows = [...$rows, ...$pieceRows];
} }
@@ -169,8 +169,8 @@ final class RestoreRecoverablePieceCustomFieldValuesCommand extends Command
continue; continue;
} }
$currentValue = $currentValuesByFieldId[$definition->getId()] ?? null; $currentValue = $currentValuesByFieldId[$definition->getId()] ?? null;
$shouldRestore = null === $currentValue || '' === trim($currentValue->getValue()); $shouldRestore = null === $currentValue || '' === trim($currentValue->getValue());
if (!$shouldRestore) { if (!$shouldRestore) {
continue; continue;
} }

7
src/Controller/AdminProfileController.php
View File

@@ -68,6 +68,9 @@ final class AdminProfileController extends AbstractController
} }
if (null !== $password && '' !== $password) { if (null !== $password && '' !== $password) {
if (mb_strlen($password) < 8) {
return new JsonResponse(['message' => 'Le mot de passe doit contenir au moins 8 caractères.'], JsonResponse::HTTP_BAD_REQUEST);
}
$profile->setPassword( $profile->setPassword(
$this->passwordHasher->hashPassword($profile, $password) $this->passwordHasher->hashPassword($profile, $password)
); );
@@ -131,6 +134,10 @@ final class AdminProfileController extends AbstractController
return new JsonResponse(['message' => 'Le mot de passe est requis.'], JsonResponse::HTTP_BAD_REQUEST); return new JsonResponse(['message' => 'Le mot de passe est requis.'], JsonResponse::HTTP_BAD_REQUEST);
} }
if (mb_strlen($password) < 8) {
return new JsonResponse(['message' => 'Le mot de passe doit contenir au moins 8 caractères.'], JsonResponse::HTTP_BAD_REQUEST);
}
$profile->setPassword( $profile->setPassword(
$this->passwordHasher->hashPassword($profile, $password) $this->passwordHasher->hashPassword($profile, $password)
); );

17
src/Controller/CommentController.php
View File

@@ -91,12 +91,29 @@ final class CommentController extends AbstractController
$this->entityManager->persist($comment); $this->entityManager->persist($comment);
// Handle file uploads // Handle file uploads
$allowedMimeTypes = [
'application/pdf',
'image/jpeg', 'image/png', 'image/gif', 'image/webp', 'image/bmp',
'text/plain', 'text/csv',
'application/vnd.openxmlformats-officedocument.wordprocessingml.document',
'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet',
'application/msword', 'application/vnd.ms-excel',
'application/zip',
];
$files = $request->files->all('files'); $files = $request->files->all('files');
foreach ($files as $file) { foreach ($files as $file) {
if (!$file instanceof UploadedFile || !$file->isValid()) { if (!$file instanceof UploadedFile || !$file->isValid()) {
continue; continue;
} }
$detectedMime = $file->getMimeType() ?: 'application/octet-stream';
if (!in_array($detectedMime, $allowedMimeTypes, true)) {
return $this->json([
'message' => sprintf('Type de fichier non autorisé : %s', $detectedMime),
], 400);
}
$document = new Document(); $document = new Document();
$documentId = 'cl'.bin2hex(random_bytes(12)); $documentId = 'cl'.bin2hex(random_bytes(12));
$document->setId($documentId); $document->setId($documentId);

19
src/Controller/DocumentServeController.php
View File

@@ -8,6 +8,7 @@ use App\Repository\DocumentRepository;
use App\Service\DocumentStorageService; use App\Service\DocumentStorageService;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController; use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\HttpFoundation\BinaryFileResponse; use Symfony\Component\HttpFoundation\BinaryFileResponse;
use Symfony\Component\HttpFoundation\HeaderUtils;
use Symfony\Component\HttpFoundation\Response; use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpFoundation\ResponseHeaderBag; use Symfony\Component\HttpFoundation\ResponseHeaderBag;
use Symfony\Component\Routing\Attribute\Route; use Symfony\Component\Routing\Attribute\Route;
@@ -42,11 +43,15 @@ class DocumentServeController extends AbstractController
return $this->json(['error' => 'Invalid document data.'], 500); return $this->json(['error' => 'Invalid document data.'], 500);
} }
$disposition = HeaderUtils::makeDisposition(ResponseHeaderBag::DISPOSITION_INLINE, $document->getFilename());
return new Response($content, 200, [ return new Response($content, 200, [
'Content-Type' => $document->getMimeType(), 'Content-Type' => $document->getMimeType(),
'Content-Disposition' => ResponseHeaderBag::DISPOSITION_INLINE.'; filename="'.$document->getFilename().'"', 'Content-Disposition' => $disposition,
'Content-Length' => (string) strlen($content), 'Content-Length' => (string) strlen($content),
'Cache-Control' => 'private, max-age=3600', 'Cache-Control' => 'private, max-age=3600',
'X-Content-Type-Options' => 'nosniff',
'Content-Security-Policy' => 'sandbox',
]); ]);
} }
@@ -63,6 +68,8 @@ class DocumentServeController extends AbstractController
$document->getFilename() $document->getFilename()
); );
$response->headers->set('Cache-Control', 'private, max-age=3600'); $response->headers->set('Cache-Control', 'private, max-age=3600');
$response->headers->set('X-Content-Type-Options', 'nosniff');
$response->headers->set('Content-Security-Policy', 'sandbox');
return $response; return $response;
} }
@@ -86,9 +93,11 @@ class DocumentServeController extends AbstractController
return $this->json(['error' => 'Invalid document data.'], 500); return $this->json(['error' => 'Invalid document data.'], 500);
} }
$disposition = HeaderUtils::makeDisposition(ResponseHeaderBag::DISPOSITION_ATTACHMENT, $document->getFilename());
return new Response($content, 200, [ return new Response($content, 200, [
'Content-Type' => 'application/octet-stream', 'Content-Type' => 'application/octet-stream',
'Content-Disposition' => ResponseHeaderBag::DISPOSITION_ATTACHMENT.'; filename="'.$document->getFilename().'"', 'Content-Disposition' => $disposition,
'Content-Length' => (string) strlen($content), 'Content-Length' => (string) strlen($content),
]); ]);
} }

25
src/Controller/SessionProfileController.php
View File

@@ -5,10 +5,12 @@ declare(strict_types=1);
namespace App\Controller; namespace App\Controller;
use App\Repository\ProfileRepository; use App\Repository\ProfileRepository;
use Symfony\Component\DependencyInjection\Attribute\Autowire;
use Symfony\Component\HttpFoundation\JsonResponse; use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\Request; use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Session\SessionInterface; use Symfony\Component\HttpFoundation\Session\SessionInterface;
use Symfony\Component\PasswordHasher\Hasher\UserPasswordHasherInterface; use Symfony\Component\PasswordHasher\Hasher\UserPasswordHasherInterface;
use Symfony\Component\RateLimiter\RateLimiterFactory;
use Symfony\Component\Routing\Attribute\Route; use Symfony\Component\Routing\Attribute\Route;
final class SessionProfileController final class SessionProfileController
@@ -16,6 +18,8 @@ final class SessionProfileController
public function __construct( public function __construct(
private readonly ProfileRepository $profiles, private readonly ProfileRepository $profiles,
private readonly UserPasswordHasherInterface $passwordHasher, private readonly UserPasswordHasherInterface $passwordHasher,
#[Autowire(service: 'limiter.login')]
private readonly RateLimiterFactory $loginLimiter,
) {} ) {}
#[Route('/api/session/profile', name: 'api_session_profile_get', methods: ['GET'])] #[Route('/api/session/profile', name: 'api_session_profile_get', methods: ['GET'])]
@@ -56,6 +60,11 @@ final class SessionProfileController
return new JsonResponse(['message' => 'Session indisponible.'], JsonResponse::HTTP_INTERNAL_SERVER_ERROR); return new JsonResponse(['message' => 'Session indisponible.'], JsonResponse::HTTP_INTERNAL_SERVER_ERROR);
} }
$limiter = $this->loginLimiter->create($request->getClientIp());
if (!$limiter->consume()->isAccepted()) {
return new JsonResponse(['message' => 'Trop de tentatives. Réessayez dans une minute.'], JsonResponse::HTTP_TOO_MANY_REQUESTS);
}
$payload = $request->toArray(); $payload = $request->toArray();
$profileId = $payload['profileId'] ?? null; $profileId = $payload['profileId'] ?? null;
@@ -64,24 +73,24 @@ final class SessionProfileController
} }
$profile = $this->profiles->find($profileId); $profile = $this->profiles->find($profileId);
if (!$profile || !$profile->isActive()) {
return new JsonResponse(['message' => 'Profil introuvable ou inactif.'], JsonResponse::HTTP_UNAUTHORIZED);
}
$password = $payload['password'] ?? ''; $password = $payload['password'] ?? '';
if ('' === $password) { if ('' === $password) {
return new JsonResponse(['message' => 'Mot de passe requis.'], JsonResponse::HTTP_BAD_REQUEST); return new JsonResponse(['message' => 'Mot de passe requis.'], JsonResponse::HTTP_BAD_REQUEST);
} }
$loginFailed = new JsonResponse(['message' => 'Identifiants invalides.'], JsonResponse::HTTP_UNAUTHORIZED);
if (!$profile || !$profile->isActive()) {
return $loginFailed;
}
if (!$profile->getPassword()) { if (!$profile->getPassword()) {
return new JsonResponse( return $loginFailed;
['message' => 'Ce profil n\'a pas de mot de passe. Contactez un administrateur.'],
JsonResponse::HTTP_FORBIDDEN,
);
} }
if (!$this->passwordHasher->isPasswordValid($profile, $password)) { if (!$this->passwordHasher->isPasswordValid($profile, $password)) {
return new JsonResponse(['message' => 'Mot de passe incorrect.'], JsonResponse::HTTP_UNAUTHORIZED); return $loginFailed;
} }
$session->migrate(true); $session->migrate(true);

7
src/Controller/SessionProfilesController.php
View File

@@ -27,10 +27,9 @@ final class SessionProfilesController
return new JsonResponse(array_map(static function ($profile): array { return new JsonResponse(array_map(static function ($profile): array {
return [ return [
'id' => $profile->getId(), 'id' => $profile->getId(),
'firstName' => $profile->getFirstName(), 'firstName' => $profile->getFirstName(),
'lastName' => $profile->getLastName(), 'lastName' => $profile->getLastName(),
'hasPassword' => null !== $profile->getPassword() && '' !== $profile->getPassword(),
]; ];
}, $items)); }, $items));
} }

2
src/Entity/Profile.php
View File

@@ -167,7 +167,7 @@ class Profile implements UserInterface, PasswordAuthenticatedUserInterface
*/ */
public function getUserIdentifier(): string public function getUserIdentifier(): string
{ {
return (string) $this->email; return $this->email ?? $this->id;
} }
/** /**

4
src/Service/DocumentStorageService.php
View File

@@ -27,6 +27,10 @@ class DocumentStorageService
public function getAbsolutePath(string $relativePath): string public function getAbsolutePath(string $relativePath): string
{ {
if (str_contains($relativePath, '..')) {
throw new RuntimeException(sprintf('Path traversal detected: "%s"', $relativePath));
}
$absolutePath = $this->storageDir.'/'.$relativePath; $absolutePath = $this->storageDir.'/'.$relativePath;
$realPath = realpath($absolutePath); $realPath = realpath($absolutePath);

14
src/State/DocumentUploadProcessor.php
View File

@@ -64,6 +64,20 @@ final class DocumentUploadProcessor implements ProcessorInterface
throw new BadRequestHttpException('A valid file is required in the "file" field.'); throw new BadRequestHttpException('A valid file is required in the "file" field.');
} }
$allowedMimeTypes = [
'application/pdf',
'image/jpeg', 'image/png', 'image/gif', 'image/webp', 'image/bmp',
'text/plain', 'text/csv',
'application/vnd.openxmlformats-officedocument.wordprocessingml.document',
'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet',
'application/msword', 'application/vnd.ms-excel',
'application/zip',
];
$detectedMime = $file->getMimeType() ?: 'application/octet-stream';
if (!in_array($detectedMime, $allowedMimeTypes, true)) {
throw new BadRequestHttpException(sprintf('Type de fichier non autorisé : %s', $detectedMime));
}
$document = new Document(); $document = new Document();
// Metadata from form fields // Metadata from form fields

21
symfony.lock
View File

@@ -70,18 +70,6 @@
".php-cs-fixer.dist.php" ".php-cs-fixer.dist.php"
] ]
}, },
"lexik/jwt-authentication-bundle": {
"version": "3.2",
"recipe": {
"repo": "github.com/symfony/recipes",
"branch": "main",
"version": "2.5",
"ref": "e9481b233a11ef7e15fe055a2b21fd3ac1aa2bb7"
},
"files": [
"config/packages/lexik_jwt_authentication.yaml"
]
},
"nelmio/cors-bundle": { "nelmio/cors-bundle": {
"version": "2.6", "version": "2.6",
"recipe": { "recipe": {
@@ -240,14 +228,5 @@
"files": [ "files": [
"config/packages/validator.yaml" "config/packages/validator.yaml"
] ]
},
"vich/uploader-bundle": {
"version": "2.9",
"recipe": {
"repo": "github.com/symfony/recipes-contrib",
"branch": "main",
"version": "1.13",
"ref": "1b3064c2f6b255c2bc2f56461aaeb76b11e07e36"
}
} }
} }