mirror of
https://github.com/R0m1k3/LogiFlow.git
synced 2026-10-14 18:59:18 +02:00
michaelschal
00e3c98550
Implement backend logic to ensure correct product group assignment, resolving an issue where products were incorrectly placed in the wrong store due to missing group ID data from the frontend, and add detailed logging for better debugging. Replit-Commit-Author: Agent Replit-Commit-Session-Id: b163d4c0-de5e-4f4e-a9c0-aed4c7049718 Replit-Commit-Checkpoint-Type: full_checkpoint Replit-Commit-Screenshot-Url: https://storage.googleapis.com/screenshot-production-us-central1/1957c339-2757-4d1f-8e92-e9f71a1ce58e/b163d4c0-de5e-4f4e-a9c0-aed4c7049718/PoPN0Ru
95 lines
3.0 KiB
Markdown
95 lines
3.0 KiB
Markdown
# Fix Final DLC Production - Employee Magasin #2 → Magasin #1
|
|
|
|
## Problème Identifié
|
|
|
|
**EXACT** : Employé magasin #2 crée DLC → apparaît dans magasin #1
|
|
|
|
## Cause Racine Trouvée
|
|
|
|
```javascript
|
|
// Erreur de validation backend:
|
|
{"message":"Validation error","errors":[{"code":"invalid_type","expected":"number","received":"undefined","path":["groupId"],"message":"Required"}]}
|
|
```
|
|
|
|
**CAUSE** : Frontend ne transmet pas `groupId` dans la requête → Backend validation échoue
|
|
**CONSÉQUENCE** : DLC pas créée OU créée avec mauvais groupId par défaut
|
|
|
|
## Solutions Appliquées
|
|
|
|
### 1. Fix Backend - Force GroupId Assignment
|
|
**Fichier:** `server/routes.ts` ligne 2325-2335
|
|
|
|
```javascript
|
|
// Fix groupId if missing - use user's assigned group or fallback
|
|
let finalGroupId = req.body.groupId;
|
|
if (!finalGroupId) {
|
|
if (user.userGroups?.[0]?.groupId) {
|
|
finalGroupId = user.userGroups[0].groupId;
|
|
console.log("🔧 Backend Fix: Using user's assigned group:", finalGroupId);
|
|
} else {
|
|
finalGroupId = 1; // Emergency fallback
|
|
console.log("🚨 Backend Fix: Using emergency fallback groupId:", finalGroupId);
|
|
}
|
|
}
|
|
```
|
|
|
|
### 2. Frontend DLC Logic Déjà Corrigée
|
|
**Fichier:** `client/src/pages/DlcPage.tsx` ligne 201-234
|
|
|
|
```javascript
|
|
// Déterminer le groupId correctement selon le rôle utilisateur
|
|
let groupId;
|
|
if (user?.role === 'admin' && selectedStoreId) {
|
|
groupId = selectedStoreId;
|
|
} else if (user?.userGroups?.[0]?.groupId) {
|
|
// UTILISATEUR NON-ADMIN: utiliser son groupe assigné ✅
|
|
groupId = user.userGroups[0].groupId;
|
|
} else if (user?.role === 'admin') {
|
|
groupId = stores[0]?.id || 1;
|
|
} else {
|
|
groupId = 1; // Fallback par défaut
|
|
}
|
|
```
|
|
|
|
### 3. Debug Logs Production
|
|
```javascript
|
|
console.log("🏪 DLC GroupId Selection DEBUG:", {
|
|
userRole: user?.role,
|
|
selectedStoreId,
|
|
userGroups: user?.userGroups?.map(ug => ({groupId: ug.groupId, groupName: ug.group?.name})),
|
|
availableStores: stores.map(s => ({id: s.id, name: s.name})),
|
|
userGroupsRaw: user?.userGroups,
|
|
firstUserGroup: user?.userGroups?.[0],
|
|
finalGroupId: groupId,
|
|
logicPath: !groupId ? 'need-fallback' : 'already-set'
|
|
});
|
|
```
|
|
|
|
## Priorité de Sélection GroupId
|
|
|
|
**BACKEND (Sécurité):**
|
|
1. `req.body.groupId` si fourni par frontend
|
|
2. `user.userGroups[0].groupId` si utilisateur assigné à un groupe ✅ **FIX PRINCIPAL**
|
|
3. `1` en fallback d'urgence
|
|
|
|
**FRONTEND (Logique UI):**
|
|
1. Admin avec magasin sélectionné → `selectedStoreId`
|
|
2. **Utilisateur avec groupe assigné → `user.userGroups[0].groupId`** ✅ **FIX PRINCIPAL**
|
|
3. Admin sans sélection → Premier magasin disponible
|
|
4. Fallback d'urgence → `1`
|
|
|
|
## Résultat Attendu
|
|
|
|
Employé assigné au magasin #2 :
|
|
- Frontend calcule `groupId = 2` depuis `user.userGroups[0].groupId`
|
|
- Backend valide et crée DLC avec `groupId = 2`
|
|
- DLC apparaît dans magasin #2 ✅
|
|
|
|
## Tests de Validation
|
|
|
|
✅ Backend force groupId si manquant
|
|
✅ Frontend utilise groupe utilisateur
|
|
✅ Logs debug pour traçabilité production
|
|
✅ Double sécurité frontend + backend
|
|
|
|
**DÉPLOIEMENT PRODUCTION REQUIS** |