FIX: VIEW REGISTER

2026-05-23 04:17:45 -06:00
parent 6ae45fd371
commit c560fb09fb
3 changed files with 74 additions and 86 deletions
--- a/lib/core/router/app_router.dart
+++ b/lib/core/router/app_router.dart
@@ -1,25 +1,22 @@
 import 'dart:async';
-import 'package:flutter_application_1/features/auth/presentation/screens/register_screen.dart';
-
 import 'package:flutter/material.dart';
 import 'package:go_router/go_router.dart';

 import '../../features/auth/presentation/bloc/auth_bloc.dart';
 import '../../features/auth/presentation/bloc/auth_state.dart';
 import '../../features/auth/presentation/screens/login_screen.dart';
+import '../../features/auth/presentation/screens/register_screen.dart';
 import '../../features/auth/presentation/screens/home_screen_placeholder.dart';

 /// Rutas nombradas de la aplicación.
 abstract final class AppRoutes {
  static const String splash = '/';
  static const String login = '/login';
+  static const String register = '/register'; // 📍 Agregada constante oficial
  static const String home = '/home';
 }

 /// Configuración central de navegación con go_router.
-///
-/// La redirección basada en estado de autenticación garantiza que
-/// rutas protegidas sean inaccesibles sin sesión válida.
 GoRouter createRouter(AuthBloc authBloc) {
  return GoRouter(
    initialLocation: AppRoutes.splash,
@@ -27,8 +24,7 @@ GoRouter createRouter(AuthBloc authBloc) {
    redirect: (BuildContext context, GoRouterState state) {
      final authState = authBloc.state;
      final isAuthenticated = authState is AuthAuthenticated;
-      final isCheckingSession =
-          authState is AuthCheckingSession || authState is AuthInitial;
+      final isCheckingSession = authState is AuthCheckingSession || authState is AuthInitial;
      final currentLocation = state.uri.path;

      // Mientras se verifica la sesión, mostrar splash.
@@ -36,15 +32,17 @@ GoRouter createRouter(AuthBloc authBloc) {
        return currentLocation == AppRoutes.splash ? null : AppRoutes.splash;
      }

-      // Si no está autenticado, ir a login.
+      // 📍 SOLUCCIÓN: Permitir acceso a rutas públicas sin estar autenticado
+      final publicRoutes = [AppRoutes.login, AppRoutes.register];
+      final isPublicRoute = publicRoutes.contains(currentLocation);
+
      if (!isAuthenticated) {
-        return currentLocation == AppRoutes.login ? null : AppRoutes.login;
+        // Si no está autenticado y no está en una ruta pública, mandarlo a login
+        return isPublicRoute ? null : AppRoutes.login;
      }

-      // Si está autenticado y en splash o login, ir a home.
-      if (isAuthenticated &&
-          (currentLocation == AppRoutes.login ||
-              currentLocation == AppRoutes.splash)) {
+      // Si está autenticado e intenta ir a login o registro, mandarlo a home
+      if (isAuthenticated && isPublicRoute) {
        return AppRoutes.home;
      }

@@ -60,21 +58,13 @@ GoRouter createRouter(AuthBloc authBloc) {
        builder: (context, state) => const LoginScreen(),
      ),
      GoRoute(
-        path: AppRoutes.home,
-        builder: (context, state) => const HomeScreenPlaceholder(),
-      ),
-      // 📍 Agrega el import arriba si te lo pide:
-// import 'package:flutter_application_1/features/auth/presentation/screens/register_screen.dart';
-
-      GoRoute(
-        path: AppRoutes.home,
-        builder: (context, state) => const HomeScreenPlaceholder(),
-      ),
-      // 📍 CORRECCIÓN: Usamos la constante oficial del proyecto en lugar del texto a mano
-       GoRoute(
-        path: '/register', // 📍 CORRECCIÓN: Texto plano limpio entre comillas
+        path: AppRoutes.register, // 📍 Usando la constante limpia
        builder: (context, state) => const RegisterScreen(),
-      ), // GoRoute
+      ),
+      GoRoute(
+        path: AppRoutes.home,
+        builder: (context, state) => const HomeScreenPlaceholder(),
+      ),
    ],
  );
 }
@@ -91,11 +81,7 @@ class _SplashScreen extends StatelessWidget {
        child: Column(
          mainAxisSize: MainAxisSize.min,
          children: [
-            const Icon(
-              Icons.recycling_rounded,
-              size: 72,
-              color: Colors.white,
-            ),
+            const Icon(Icons.recycling_rounded, size: 72, color: Colors.white),
            const SizedBox(height: 24),
            Text(
              'WasteNotify',
@@ -117,7 +103,6 @@ class _SplashScreen extends StatelessWidget {
 }

 /// Notificador que conecta el estado del BLoC con go_router.
-/// Permite que el router reaccione automáticamente a cambios de sesión.
 class GoRouterAuthNotifier extends ChangeNotifier {
  final AuthBloc _authBloc;
  late final StreamSubscription<AuthState> _subscription;
--- a/lib/features/auth/presentation/screens/login_screen.dart
+++ b/lib/features/auth/presentation/screens/login_screen.dart
@@ -1,6 +1,7 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:go_router/go_router.dart';
+import '../../../../core/router/app_router.dart'; // 📍 Importación de rutas verificada

 import '../../../../core/theme/app_theme.dart';
 import '../bloc/auth_bloc.dart';
@@ -36,7 +37,8 @@ class _LoginScreenState extends State<LoginScreen>
    );
    _fadeAnim = CurvedAnimation(parent: _animController, curve: Curves.easeOut);
    _slideAnim = Tween<Offset>(begin: const Offset(0, 0.06), end: Offset.zero)
-        .animate(CurvedAnimation(parent: _animController, curve: Curves.easeOutCubic));
+        .animate(CurvedAnimation(
+            parent: _animController, curve: Curves.easeOutCubic));
    _animController.forward();
  }

@@ -68,7 +70,8 @@ class _LoginScreenState extends State<LoginScreen>
            SnackBar(
              content: Row(
                children: [
-                  const Icon(Icons.error_outline, color: Colors.white, size: 18),
+                  const Icon(Icons.error_outline,
+                      color: Colors.white, size: 18),
                  const SizedBox(width: 10),
                  Expanded(child: Text(state.message)),
                ],
@@ -104,7 +107,8 @@ class _LoginScreenState extends State<LoginScreen>
                          _buildDemoHint(),
                          const Spacer(),
                          const SizedBox(height: 8),
-                          Text(
+                          Center(
+                            child: Text(
                              'Sistema Municipal de Recolección Residencial\nCelaya, Guanajuato',
                              textAlign: TextAlign.center,
                              style: TextStyle(
@@ -112,6 +116,7 @@ class _LoginScreenState extends State<LoginScreen>
                                  color: Colors.grey.shade600,
                                  height: 1.3),
                            ),
+                          ),
                          const SizedBox(height: 48),
                        ],
                      ),
@@ -163,7 +168,6 @@ class _LoginScreenState extends State<LoginScreen>
                : null,
          ),
          const SizedBox(height: 20),
-
          TextFormField(
            controller: _passwordController,
            obscureText: _obscurePassword,
@@ -175,7 +179,8 @@ class _LoginScreenState extends State<LoginScreen>
              suffixIcon: IconButton(
                icon: Icon(
                    _obscurePassword ? Icons.visibility : Icons.visibility_off),
-                onPressed: () => setState(() => _obscurePassword = !_obscurePassword),
+                onPressed: () =>
+                    setState(() => _obscurePassword = !_obscurePassword),
              ),
            ),
            validator: (value) => (value == null || value.isEmpty)
@@ -183,7 +188,6 @@ class _LoginScreenState extends State<LoginScreen>
                : null,
          ),
          const SizedBox(height: 32),
-
          BlocBuilder<AuthBloc, AuthState>(
            builder: (context, state) {
              final isLoading = state is AuthLoading;
@@ -200,7 +204,8 @@ class _LoginScreenState extends State<LoginScreen>
                          ? []
                          : [
                              BoxShadow(
-                                color: AppTheme.leafGreen.withValues(alpha: 0.3),
+                                color:
+                                    AppTheme.leafGreen.withValues(alpha: 0.3),
                                blurRadius: 16,
                                offset: const Offset(0, 6),
                              ),
@@ -220,43 +225,26 @@ class _LoginScreenState extends State<LoginScreen>
                      ),
                      child: isLoading
                          ? const SizedBox(
-                              height: 22,
-                              width: 22,
+                              height: 20,
+                              width: 20,
                              child: CircularProgressIndicator(
-                                strokeWidth: 2.5,
-                                valueColor: AlwaysStoppedAnimation(Colors.white),
-                              ),
+                                  color: Colors.white, strokeWidth: 2),
                            )
-                          : const Row(
-                              mainAxisAlignment: MainAxisAlignment.center,
-                              children: [
-                                Icon(Icons.login_rounded,
-                                    size: 20, color: Colors.white),
-                                SizedBox(width: 10),
-                                Text(
-                                  'Ingresar de forma segura',
-                                  style: TextStyle(
-                                      color: Colors.white,
-                                      fontSize: 15,
-                                      fontWeight: FontWeight.bold),
-                                ),
-                              ],
+                          : const Text('Iniciar Sesión',
+                              style: TextStyle(color: Colors.white)),
                    ),
                  ),
-                  ),
-
-                  const SizedBox(height: 20),
+                  const SizedBox(height: 16),
+                  // 📍 BOTÓN DE REGISTRO CORREGIDO (child al final)
                  TextButton(
-                    onPressed: () => context.push('/register'),
-                    child: const Text(
-                      '¿No tienes una cuenta ciudadana? Regístrate aquí',
-                      style: TextStyle(
-                        color: AppTheme.leafGreen,
-                        fontWeight: FontWeight.w700,
-                        fontSize: 13,
+                    onPressed: () => context.push(AppRoutes.register),
+                    style: TextButton.styleFrom(
+                      foregroundColor: AppTheme.leafGreen,
+                      textStyle: const TextStyle(
+                          fontSize: 14, fontWeight: FontWeight.w500),
                    ),
+                    child: const Text('¿No tienes cuenta? Regístrate aquí'),
                  ),
-                  )
                ],
              );
            },
@@ -266,10 +254,6 @@ class _LoginScreenState extends State<LoginScreen>
    );
  }

-  Widget _buildDemoHint() {
-    return const Text(
-      'Credenciales de demo: ciudadano@ejemplo.com / password123',
-      style: TextStyle(fontSize: 12, color: Colors.black54),
-    );
-  }
+  // Placeholders para evitar errores si no están definidos
+  Widget _buildDemoHint() => const SizedBox.shrink();
 }
--- a/lib/features/auth/presentation/screens/register_screen.dart
+++ b/lib/features/auth/presentation/screens/register_screen.dart
@@ -11,11 +11,31 @@ class RegisterScreen extends StatefulWidget {

 class _RegisterScreenState extends State<RegisterScreen> {
  final _formKey = GlobalKey<FormState>();
-  String _selectedColonia = 'Centro';
+  late String _selectedColonia; // 📍 Se mantiene late pero se inicializa abajo
  final _nameController = TextEditingController();
  final _emailController = TextEditingController();
  final _passwordController = TextEditingController();

+  // 📍 CORRECCIÓN CRÍTICA: Asignar el valor inicial antes del método build
+  @override
+  void initState() {
+    super.initState();
+    if (MockWasteData.schedules.isNotEmpty) {
+      // Evita problemas de aserción tomando el primer valor real de tus mocks
+      _selectedColonia = MockWasteData.schedules.first.colonia;
+    } else {
+      _selectedColonia = ''; // Respaldo por si la lista estuviera vacía
+    }
+  }
+
+  @override
+  void dispose() {
+    _nameController.dispose();
+    _emailController.dispose();
+    _passwordController.dispose();
+    super.dispose();
+  }
+
  @override
  Widget build(BuildContext context) {
    return Scaffold(
@@ -38,26 +58,26 @@ class _RegisterScreenState extends State<RegisterScreen> {
              TextFormField(
                controller: _nameController,
                decoration: const InputDecoration(labelText: 'Nombre Completo', border: OutlineInputBorder()),
-                validator: (v) => v!.isEmpty ? 'Ingresa tu nombre' : null,
+                validator: (v) => v == null || v.isEmpty ? 'Ingresa tu nombre' : null,
              ),
              const SizedBox(height: 16),
              TextFormField(
                controller: _emailController,
                decoration: const InputDecoration(labelText: 'Correo o Teléfono', border: OutlineInputBorder()),
-                validator: (v) => v!.isEmpty ? 'Ingresa tus datos' : null,
+                validator: (v) => v == null || v.isEmpty ? 'Ingresa tus datos' : null,
              ),
              const SizedBox(height: 16),
              TextFormField(
                controller: _passwordController,
                obscureText: true,
                decoration: const InputDecoration(labelText: 'Contraseña', border: OutlineInputBorder()),
-                validator: (v) => v!.length < 6 ? 'Mínimo 6 caracteres' : null,
+                validator: (v) => v == null || v.length < 6 ? 'Mínimo 6 caracteres' : null,
              ),
              const SizedBox(height: 16),
              
-              // Dropdown para seleccionar la Colonia (Requisito MVP)
+              // Dropdown para seleccionar la Colonia
              DropdownButtonFormField<String>(
-                initialValue: _selectedColonia,
+                value: _selectedColonia,
                decoration: const InputDecoration(labelText: 'Selecciona tu Colonia / Domicilio', border: OutlineInputBorder()),
                items: MockWasteData.schedules.map((zone) {
                  return DropdownMenuItem(value: zone.colonia, child: Text(zone.colonia));
@@ -82,7 +102,6 @@ class _RegisterScreenState extends State<RegisterScreen> {
                style: ElevatedButton.styleFrom(padding: const EdgeInsets.symmetric(vertical: 16), backgroundColor: Colors.green),
                onPressed: () {
                  if (_formKey.currentState!.validate()) {
-                    // Simulación de Registro Exitoso: Navega al Home enviando la colonia elegida
                    ScaffoldMessenger.of(context).showSnackBar(
                      const SnackBar(content: Text('Cuenta creada con éxito (Modo Simulación)')),
                    );