Michael/LogiFlow
1
0 Fork 0
mirror of https://github.com/R0m1k3/LogiFlow.git synced 2026-10-11 17:27:31 +02:00
Code Issues Packages Projects Releases Wiki Activity

Fix task access and visibility for employee role users

Browse Source 
Update client/src/pages/Tasks.tsx to restrict user fetching and action button visibility based on role ('admin', 'manager', 'directeur'). Also addresses 403 errors for employees in server routes.

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/vMSppUD
This commit is contained in:
michaelschal committed 2025-08-12 09:05:02 +00:00
1 parent b986763b6c
commit b9b85fba78
4 files changed
+163 -27
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

PRODUCTION-FIXES-FINAL.md
+13 -1
View File
@@ -6,7 +6,19 @@
## CORRECTIONS À APPLIQUER SUR VOTRE SERVEUR PRIVÉ
### 1️⃣ FICHIER: server/routes.ts SEULEMENT
### 1️⃣ FICHIER: client/src/pages/Tasks.tsx
**Correction erreur 403 /api/users**
#### A) Ligne 84-85 (requête des utilisateurs):
```typescript
// REMPLACER:
enabled: !!user,
// PAR:
enabled: !!user && (user.role === 'admin' || user.role === 'manager' || user.role === 'directeur'),
```
### 2️⃣ FICHIER: server/routes.ts
**Les permissions dans shared/permissions.ts restent inchangées**
#### A) Route GET /api/suppliers (environ ligne 188-194):
attached_assets/{F8DF25B7-AAB8-4266-9F4E-3717C4CA43C3}_1754989295935.png
BIN
View File
Binary file not shown.
Side by Side

After

Width:  |  Height:  |  Size: 195 KiB

client/src/pages/Tasks.tsx
+33 -26
View File
@@ -70,7 +70,7 @@ export default function Tasks() {
enabled: !!user,
});
// Fetch users for task assignment
// Fetch users for task assignment - seulement pour admin/manager/directeur
const { data: users = [] } = useQuery({
queryKey: ["/api/users"],
queryFn: () => fetch('/api/users', {
@@ -81,7 +81,7 @@ export default function Tasks() {
}
return res.json();
}),
enabled: !!user,
enabled: !!user && (user.role === 'admin' || user.role === 'manager' || user.role === 'directeur'),
});
@@ -218,7 +218,8 @@ export default function Tasks() {
}
};
const canCreateTasks = user?.role === 'admin' || user?.role === 'manager';
const canCreateTasks = user?.role === 'admin' || user?.role === 'manager' || user?.role === 'directeur';
const canEditTasks = user?.role === 'admin' || user?.role === 'manager' || user?.role === 'directeur';
if (isLoading) {
return (
@@ -382,29 +383,35 @@ export default function Tasks() {
</div>
<div className="flex items-center gap-2 ml-4">
<Button
size="sm"
variant="outline"
onClick={() => handleCompleteTask(task.id)}
className="text-green-600 hover:text-green-700"
>
<CheckCircle className="w-4 h-4" />
</Button>
<Button
size="sm"
variant="outline"
onClick={() => handleEditTask(task)}
>
<Edit className="w-4 h-4" />
</Button>
<Button
size="sm"
variant="outline"
onClick={() => handleDeleteClick(task)}
className="text-red-600 hover:text-red-700"
>
<Trash2 className="w-4 h-4" />
</Button>
{canEditTasks && (
<Button
size="sm"
variant="outline"
onClick={() => handleCompleteTask(task.id)}
className="text-green-600 hover:text-green-700"
>
<CheckCircle className="w-4 h-4" />
</Button>
)}
{canEditTasks && (
<Button
size="sm"
variant="outline"
onClick={() => handleEditTask(task)}
>
<Edit className="w-4 h-4" />
</Button>
)}
{canEditTasks && (
<Button
size="sm"
variant="outline"
onClick={() => handleDeleteClick(task)}
className="text-red-600 hover:text-red-700"
>
<Trash2 className="w-4 h-4" />
</Button>
)}
</div>
</div>
</CardContent>
fix-employee-403-final.js
+117
View File
@@ -0,0 +1,117 @@
#!/usr/bin/env node
/**
* Script de correction finale des erreurs 403 pour les employés
* Applique les corrections nécessaires au serveur de production
*/
console.log('🔧 SCRIPT DE CORRECTION DES ERREURS 403 EMPLOYÉS');
console.log('================================================');
console.log('');
console.log('✅ CORRECTIONS APPLIQUÉES DANS REPLIT:');
console.log('');
console.log('1️⃣ client/src/pages/Tasks.tsx - Ligne ~84:');
console.log(' AVANT: enabled: !!user,');
console.log(' APRÈS: enabled: !!user && (user.role === "admin" || user.role === "manager" || user.role === "directeur"),');
console.log(' ➜ Évite l\'appel à /api/users pour les employés');
console.log('');
console.log('2️⃣ Boutons d\'action conditionnels pour les employés');
console.log(' ➜ Les employés ne voient plus les boutons modifier/supprimer des tâches');
console.log('');
console.log('📋 CORRECTIONS À APPLIQUER SUR VOTRE SERVEUR PRIVÉ:');
console.log('');
console.log('💡 SOLUTION SIMPLE - 2 FICHIERS À MODIFIER:');
console.log('');
console.log('🔸 A) client/src/pages/Tasks.tsx (environ ligne 84):');
console.log('');
console.log('// REMPLACER:');
console.log('const { data: users = [] } = useQuery({');
console.log(' queryKey: ["/api/users"],');
console.log(' queryFn: () => fetch(\'/api/users\', {');
console.log(' credentials: \'include\'');
console.log(' }).then(res => {');
console.log(' if (!res.ok) {');
console.log(' throw new Error(`HTTP error! status: ${res.status}`);');
console.log(' }');
console.log(' return res.json();');
console.log(' }),');
console.log(' enabled: !!user, // ← LIGNE À CHANGER');
console.log('});');
console.log('');
console.log('// PAR:');
console.log('const { data: users = [] } = useQuery({');
console.log(' queryKey: ["/api/users"],');
console.log(' queryFn: () => fetch(\'/api/users\', {');
console.log(' credentials: \'include\'');
console.log(' }).then(res => {');
console.log(' if (!res.ok) {');
console.log(' throw new Error(`HTTP error! status: ${res.status}`);');
console.log(' }');
console.log(' return res.json();');
console.log(' }),');
console.log(' enabled: !!user && (user.role === \'admin\' || user.role === \'manager\' || user.role === \'directeur\'),');
console.log('});');
console.log('');
console.log('🔸 B) server/routes.ts - 5 corrections de permissions fournisseurs:');
console.log('');
console.log('B.1) Route GET /api/suppliers (environ ligne 188-194):');
console.log('// REMPLACER:');
console.log('if (!user || (user.role !== \'admin\' && user.role !== \'manager\')) {');
console.log('// PAR:');
console.log('if (!user || (user.role !== \'admin\' && user.role !== \'manager\' && user.role !== \'directeur\' && user.role !== \'employee\')) {');
console.log('');
console.log('B.2) Route POST /api/suppliers (environ ligne 240-242):');
console.log('// REMPLACER:');
console.log('if (user.role !== \'admin\' && user.role !== \'manager\') {');
console.log('// PAR:');
console.log('if (user.role !== \'admin\' && user.role !== \'manager\' && user.role !== \'directeur\') {');
console.log('');
console.log('B.3) Route PUT /api/suppliers/:id (environ ligne 281-283):');
console.log('// REMPLACER:');
console.log('if (!user || (user.role !== \'admin\' && user.role !== \'manager\')) {');
console.log('// PAR:');
console.log('if (!user || (user.role !== \'admin\' && user.role !== \'manager\' && user.role !== \'directeur\')) {');
console.log('');
console.log('B.4) Route DELETE /api/suppliers/:id (environ ligne 298-300):');
console.log('// REMPLACER:');
console.log('if (!user || (user.role !== \'admin\' && user.role !== \'manager\')) {');
console.log('// PAR:');
console.log('if (!user || (user.role !== \'admin\' && user.role !== \'directeur\')) {');
console.log('');
console.log('B.5) Route POST /api/dlc-products/:id/validate (environ ligne 2332-2334):');
console.log('// REMPLACER:');
console.log('if (!user || ![\'admin\', \'manager\'].includes(user.role)) {');
console.log('// PAR:');
console.log('if (!user || ![\'admin\', \'manager\', \'directeur\'].includes(user.role)) {');
console.log('');
console.log('⚡ ÉTAPES D\'APPLICATION:');
console.log('1. Sauvegarde: cp client/src/pages/Tasks.tsx client/src/pages/Tasks.tsx.backup');
console.log('2. Sauvegarde: cp server/routes.ts server/routes.ts.backup');
console.log('3. Effectuer les corrections A) et B) ci-dessus');
console.log('4. Redémarrer votre serveur Node.js');
console.log('5. Tester avec un compte employé');
console.log('');
console.log('✅ RÉSULTAT ATTENDU:');
console.log('- Plus d\'erreurs 403 dans la console pour les employés');
console.log('- Employés peuvent voir les tâches (mais pas les créer/modifier)');
console.log('- Employés peuvent voir les fournisseurs (pour Commandes Client et DLC)');
console.log('- Directeurs peuvent valider les produits DLC');
console.log('');
console.log('🚨 IMPORTANT:');
console.log('Les permissions dans shared/permissions.ts restent inchangées:');
console.log('tasks: { employee: [\'view\'] } // ← Correct, ne pas modifier');