docs/14_NOTE-BACKEND-CONFIG-SETUP.md

@@ -1,37 +0,0 @@
-# Ticket #14 – Note pour modifications backend
-
-**Contexte :** Première connexion admin → panneau Paramètres, déblocage après clic sur « Sauvegarder ». Le front appelle `POST /api/v1/configuration/setup/complete` au clic sur Sauvegarder.
-
-## Problème
-
-Erreur renvoyée par le back :  
-`invalid input syntax for type uuid: "system"`
-
-- Le controller fait `const userId = req.user?.id || 'system'` puis `markSetupCompleted(userId)`.
-- Le service `set()` fait `config.modifiePar = { id: userId }` ; la colonne `modifie_par` est une FK UUID vers `users`.
-- La chaîne `"system"` n’est pas un UUID valide → erreur PostgreSQL.
-
-## Modifications à apporter au backend
-
-**Option A – Accepter l’absence d’utilisateur (recommandé si la route peut être appelée sans JWT)**
-
-1. **`config.controller.ts`** (route `completeSetup`)  
-   - Remplacer :  
-     `const userId = req.user?.id || 'system';`  
-   - Par :  
-     `const userId = req.user?.id ?? null;`
-
-2. **`config.service.ts`** (`markSetupCompleted`)  
-   - Changer la signature :  
-     `async markSetupCompleted(userId: string | null): Promise<void>`  
-   - Et appeler :  
-     `await this.set('setup_completed', 'true', userId ?? undefined);`  
-   - Dans `set()`, ne pas remplir `modifiePar` quand `userId` est absent (déjà le cas si `if (userId)`).
-
-**Option B – Imposer un utilisateur authentifié**
-
-- Activer le guard JWT (et éventuellement RolesGuard) sur `POST /configuration/setup/complete` pour que `req.user` soit toujours défini, et garder `userId = req.user.id` (plus de fallback `'system'`).
-
----
-
-Une fois le back modifié, le flux « Sauvegarder » → déblocage des panneaux fonctionne sans erreur.

frontend/lib/screens/administrateurs/admin_dashboardScreen.dart

@@ -1,5 +1,4 @@
 import 'package:flutter/material.dart';
-import 'package:p_tits_pas/services/configuration_service.dart';
 import 'package:p_tits_pas/widgets/admin/assistante_maternelle_management_widget.dart';
 import 'package:p_tits_pas/widgets/admin/gestionnaire_management_widget.dart';
 import 'package:p_tits_pas/widgets/admin/parent_managmant_widget.dart';
@@ -15,34 +14,12 @@ class AdminDashboardScreen extends StatefulWidget {
 }
 
 class _AdminDashboardScreenState extends State<AdminDashboardScreen> {
-  /// null = chargement du statut setup, true/false = connu
-  bool? _setupCompleted;
-
   /// 0 = Gestion des utilisateurs, 1 = Paramètres
   int mainTabIndex = 0;
 
   /// Sous-onglet quand mainTabIndex == 0 : 0=Gestionnaires, 1=Parents, 2=AM, 3=Administrateurs
   int subIndex = 0;
 
-  @override
-  void initState() {
-    super.initState();
-    _loadSetupStatus();
-  }
-
-  Future<void> _loadSetupStatus() async {
-    try {
-      final completed = await ConfigurationService.getSetupStatus();
-      if (!mounted) return;
-      setState(() {
-        _setupCompleted = completed;
-        if (!completed) mainTabIndex = 1;
-      });
-    } catch (_) {
-      if (mounted) setState(() => _setupCompleted = true);
-    }
-  }
-
   void onMainTabChange(int index) {
     setState(() {
       mainTabIndex = index;
@@ -57,11 +34,6 @@ class _AdminDashboardScreenState extends State<AdminDashboardScreen> {
 
   @override
   Widget build(BuildContext context) {
-    if (_setupCompleted == null) {
-      return const Scaffold(
-        body: Center(child: CircularProgressIndicator()),
-      );
-    }
     return Scaffold(
       appBar: PreferredSize(
         preferredSize: const Size.fromHeight(60.0),
@@ -74,7 +46,6 @@ class _AdminDashboardScreenState extends State<AdminDashboardScreen> {
           child: DashboardAppBarAdmin(
             selectedIndex: mainTabIndex,
             onTabChange: onMainTabChange,
-            setupCompleted: _setupCompleted!,
           ),
         ),
       ),
@@ -96,9 +67,7 @@ class _AdminDashboardScreenState extends State<AdminDashboardScreen> {
 
   Widget _getBody() {
     if (mainTabIndex == 1) {
-      return ParametresPanel(
-        onSetupCompleted: () => setState(() => _setupCompleted = true),
-      );
+      return const ParametresPanel();
     }
     switch (subIndex) {
       case 0:

frontend/lib/services/configuration_service.dart

@@ -106,9 +106,8 @@ class ConfigurationService {
       body: jsonEncode(body),
     );
     if (response.statusCode != 200) {
-      final err = jsonDecode(response.body) as Map<String, dynamic>?;
-      final msg = err != null ? (err['message'] as String? ?? err['error'] as String?) : null;
-      throw Exception(msg ?? 'Erreur lors de la sauvegarde');
+      final err = jsonDecode(response.body) as Map;
+      throw Exception(err['message'] ?? 'Erreur lors de la sauvegarde');
     }
   }
 
@@ -119,12 +118,11 @@ class ConfigurationService {
       headers: await _headers(),
       body: jsonEncode({'testEmail': testEmail}),
     );
-    final data = jsonDecode(response.body) as Map<String, dynamic>?;
-    if (response.statusCode == 200 && (data?['success'] == true)) {
-      return data!['message'] as String? ?? 'Test SMTP réussi.';
+    final data = jsonDecode(response.body) as Map;
+    if (response.statusCode == 200 && data['success'] == true) {
+      return data['message'] as String? ?? 'Test SMTP réussi.';
     }
-    final msg = data != null ? (data['error'] as String? ?? data['message'] as String?) : null;
-    throw Exception(msg ?? 'Échec du test SMTP');
+    throw Exception(data['error'] ?? data['message'] ?? 'Échec du test SMTP');
   }
 
   /// POST /api/v1/configuration/setup/complete (après première config)
@@ -134,9 +132,8 @@ class ConfigurationService {
       headers: await _headers(),
     );
     if (response.statusCode != 200) {
-      final err = jsonDecode(response.body) as Map<String, dynamic>?;
-      final msg = err != null ? (err['message'] as String? ?? err['error'] as String?) : null;
-      throw Exception(msg ?? 'Erreur finalisation configuration');
+      final err = jsonDecode(response.body) as Map;
+      throw Exception(err['message'] ?? 'Erreur finalisation configuration');
     }
   }
 }

frontend/lib/widgets/admin/dashboard_admin.dart

@@ -4,14 +4,11 @@ import 'package:flutter/material.dart';
 class DashboardAppBarAdmin extends StatelessWidget implements PreferredSizeWidget {
   final int selectedIndex;
   final ValueChanged<int> onTabChange;
-  /// Si false, l'onglet "Gestion des utilisateurs" est grisé et inaccessible.
-  final bool setupCompleted;
 
   const DashboardAppBarAdmin({
     Key? key,
     required this.selectedIndex,
     required this.onTabChange,
-    this.setupCompleted = true,
   }) : super(key: key);
 
   @override
@@ -35,9 +32,9 @@ class DashboardAppBarAdmin extends StatelessWidget implements PreferredSizeWidge
               child: Row(
                 mainAxisSize: MainAxisSize.min,
                 children: [
-                  _buildNavItem(context, 'Gestion des utilisateurs', 0, enabled: setupCompleted),
+                  _buildNavItem(context, 'Gestion des utilisateurs', 0),
                   const SizedBox(width: 24),
-                  _buildNavItem(context, 'Paramètres', 1, enabled: true),
+                  _buildNavItem(context, 'Paramètres', 1),
                 ],
               ),
             ),
@@ -77,12 +74,10 @@ class DashboardAppBarAdmin extends StatelessWidget implements PreferredSizeWidge
     );
   }
 
-  Widget _buildNavItem(BuildContext context, String title, int index, {bool enabled = true}) {
+  Widget _buildNavItem(BuildContext context, String title, int index) {
     final bool isActive = index == selectedIndex;
     return InkWell(
-      onTap: enabled ? () => onTabChange(index) : null,
-      child: Opacity(
-        opacity: enabled ? 1.0 : 0.5,
+      onTap: () => onTabChange(index),
       child: Container(
         padding: const EdgeInsets.symmetric(horizontal: 16, vertical: 8),
         decoration: BoxDecoration(
@@ -99,7 +94,6 @@ class DashboardAppBarAdmin extends StatelessWidget implements PreferredSizeWidge
           ),
         ),
       ),
-      ),
     );
   }
 

frontend/lib/widgets/admin/parametres_panel.dart

@@ -3,10 +3,7 @@ import 'package:p_tits_pas/services/configuration_service.dart';
 
 /// Panneau Paramètres / Configuration (ticket #15) : 3 sections sur une page.
 class ParametresPanel extends StatefulWidget {
-  /// Appelé après une sauvegarde réussie (pour débloquer le reste du dashboard si config initiale).
-  final VoidCallback? onSetupCompleted;
-
-  const ParametresPanel({super.key, this.onSetupCompleted});
+  const ParametresPanel({super.key});
 
   @override
   State<ParametresPanel> createState() => _ParametresPanelState();
@@ -107,12 +104,6 @@ class _ParametresPanelState extends State<ParametresPanel> {
     return payload;
   }
 
-  /// Enregistre en base sans marquer la config initiale comme terminée (utilisé avant test SMTP).
-  Future<void> _saveBulkOnly() async {
-    await ConfigurationService.updateBulk(_buildPayload());
-  }
-
-  /// Sauvegarde + marque la config initiale comme terminée + débloque les panneaux.
   Future<void> _save() async {
     setState(() {
       _message = null;
@@ -121,9 +112,6 @@ class _ParametresPanelState extends State<ParametresPanel> {
     try {
       await ConfigurationService.updateBulk(_buildPayload());
       if (!mounted) return;
-      await ConfigurationService.completeSetup();
-      if (!mounted) return;
-      widget.onSetupCompleted?.call();
       setState(() {
         _isSaving = false;
         _message = 'Configuration enregistrée.';
@@ -172,7 +160,7 @@ class _ParametresPanelState extends State<ParametresPanel> {
     if (email == null || !mounted) return;
     setState(() => _message = null);
     try {
-      await _saveBulkOnly();
+      await _save();
       if (!mounted) return;
       final msg = await ConfigurationService.testSmtp(email);
       if (!mounted) return;