feat(apps): الدراور وطبقتا الميزات والإشعارات — المرحلة 6
## طبقتا الميزات مُطبَّقتان معاً
AppDrawer يفحص: Features.wallet && context.tenantHas('wallet')
- Features.x ثابت const: يقرّر ما يُبنى في الـbinary. إطفاؤه يحذف الكود
فلا يُستخرج (docs/23 §5)
- tenantHas من GET /tenant/config/:slug: يقرّر ما يُعرض ممّا بُني حسب اشتراك
المستأجر (docs/38 §10)
هذا هو ربط lite/pro/max عملياً. TenantCubit يعرض النسخة المخزّنة فوراً ثم
يحدّثها: إقلاع بلا إنترنت يعرض الميزات المعروفة آخر مرة لا شاشة فارغة.
## الشاشات
الدراور · تعديل الحساب · سجلّ الرحلات · الإعدادات (مظهر + لغة) · المساعدة.
الرقم في تعديل الحساب معطّل مع سبب معروض لا مخفي: هو هوية الحساب على الخادم
وتغييره يعني حساباً آخر (docs/38 §1).
## الإشعارات
PushService معزول — الملف الوحيد الذي يعرف Firebase. التسجيل بعد إثبات
الجلسة عبر خطّاف في SessionCubit، ومع اشتراك onTokenRefresh: بدونه تتوقّف
الإشعارات بصمت بعد أسابيع. الإعداد الأصلي كان جاهزاً مسبقاً.
## تصحيحات بنيوية كشفها فحص آلي
docs/23 §1 يمنع أن تستورد ميزة من أخرى — وكانت خمس مخالفات:
1. features/history أُدمجت في features/trip — السجلّ جزء من ميزة الرحلة
2. AppDrawer صعد إلى core/ui — هيكل تطبيق لا ميزة
3. AuthFailure → core/api/api_failure.dart، ومترجمه → core/ui/failure_text.dart
4. AuthHeader → core/ui/form_header.dart
5. ProfileRepository مستقلّ عن AuthRepository: تعديل الملف ليس مصادقة
flutter analyze نظيف · الراكب 98 ملف/7,497 سطر · السائق 95 ملف/6,931 سطر.
فحوص آلية نظيفة: صفر استيراد بين ميزتين · core يستورد features من di و
router وحدهما · Firebase و intaleq_maps و geolocator و socket_io كلٌّ معزول.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
66e28acc58
commit
096acae74e
@@ -1,14 +1,22 @@
|
||||
import 'package:flutter/material.dart';
|
||||
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
|
||||
import '../../../core/design/tokens.dart';
|
||||
|
||||
import '../../../core/design/tripz_colors.dart';
|
||||
|
||||
import '../../../core/l10n/l10n.dart';
|
||||
|
||||
import '../../../core/ui/tripz_button.dart';
|
||||
|
||||
import '../../../core/ui/tripz_scaffold.dart';
|
||||
|
||||
import '../cubit/login_cubit.dart';
|
||||
|
||||
import '../cubit/login_state.dart';
|
||||
import 'widgets/auth_header.dart';
|
||||
|
||||
import '../../../core/ui/form_header.dart';
|
||||
|
||||
/// البوابة الأولى: شاشة كاملة لا حوار، وزر متابعة **معطّل حتى يُؤشَّر**
|
||||
/// (docs/39 §2). تُحفظ الموافقة فلا تتكرر.
|
||||
@@ -31,7 +39,7 @@ class AgreementView extends StatelessWidget {
|
||||
child: ListView(
|
||||
children: [
|
||||
const SizedBox(height: Space.xl),
|
||||
AuthHeader(
|
||||
FormHeader(
|
||||
icon: Icons.policy_outlined,
|
||||
title: l10n.agreementTitle,
|
||||
lead: l10n.agreementLead,
|
||||
|
||||
@@ -1,13 +1,21 @@
|
||||
import 'package:flutter/material.dart';
|
||||
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
|
||||
import '../../../core/di.dart';
|
||||
|
||||
import '../../../core/session/session_cubit.dart';
|
||||
|
||||
import '../cubit/login_cubit.dart';
|
||||
|
||||
import '../cubit/login_state.dart';
|
||||
|
||||
import 'agreement_view.dart';
|
||||
|
||||
import 'otp_view.dart';
|
||||
|
||||
import 'permission_view.dart';
|
||||
|
||||
import 'phone_view.dart';
|
||||
|
||||
/// مضيف تدفّق الدخول: خطوة واحدة معروضة في كل لحظة، والانتقال بينها من
|
||||
|
||||
@@ -1,16 +1,26 @@
|
||||
import 'package:flutter/material.dart';
|
||||
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
|
||||
import '../../../core/design/tokens.dart';
|
||||
|
||||
import '../../../core/design/tripz_colors.dart';
|
||||
|
||||
import '../../../core/l10n/l10n.dart';
|
||||
|
||||
import '../../../core/ui/failure_text.dart';
|
||||
|
||||
import '../../../core/ui/tripz_button.dart';
|
||||
|
||||
import '../../../core/ui/tripz_otp_field.dart';
|
||||
|
||||
import '../../../core/ui/tripz_scaffold.dart';
|
||||
|
||||
import '../cubit/login_cubit.dart';
|
||||
|
||||
import '../cubit/login_state.dart';
|
||||
import 'widgets/auth_failure_text.dart';
|
||||
import 'widgets/auth_header.dart';
|
||||
|
||||
import '../../../core/ui/form_header.dart';
|
||||
|
||||
class OtpView extends StatefulWidget {
|
||||
const OtpView({super.key});
|
||||
@@ -44,7 +54,7 @@ class _OtpViewState extends State<OtpView> {
|
||||
child: ListView(
|
||||
children: [
|
||||
const SizedBox(height: Space.xl),
|
||||
AuthHeader(
|
||||
FormHeader(
|
||||
icon: Icons.mark_email_read_outlined,
|
||||
title: l10n.otpTitle,
|
||||
lead: l10n.otpLead(state.phone),
|
||||
|
||||
@@ -1,14 +1,22 @@
|
||||
import 'package:flutter/material.dart';
|
||||
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
|
||||
import '../../../core/design/tokens.dart';
|
||||
|
||||
import '../../../core/l10n/l10n.dart';
|
||||
|
||||
import '../../../core/ui/status_banner.dart';
|
||||
|
||||
import '../../../core/ui/tripz_button.dart';
|
||||
|
||||
import '../../../core/ui/tripz_scaffold.dart';
|
||||
|
||||
import '../cubit/login_cubit.dart';
|
||||
|
||||
import '../cubit/login_state.dart';
|
||||
import 'widgets/auth_header.dart';
|
||||
|
||||
import '../../../core/ui/form_header.dart';
|
||||
|
||||
/// البوابة الثانية: إذن الموقع. عند الرفض النهائي لا يبقى إلا فتح إعدادات
|
||||
/// النظام — والعودة منها يلتقطها `LoginPage` بمراقبة دورة الحياة.
|
||||
@@ -35,7 +43,7 @@ class PermissionView extends StatelessWidget {
|
||||
child: ListView(
|
||||
children: [
|
||||
const SizedBox(height: Space.xl),
|
||||
AuthHeader(
|
||||
FormHeader(
|
||||
icon: Icons.my_location_rounded,
|
||||
title: l10n.permissionTitle,
|
||||
lead: l10n.permissionLead,
|
||||
|
||||
@@ -1,15 +1,24 @@
|
||||
import 'package:flutter/material.dart';
|
||||
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
|
||||
import '../../../core/design/tokens.dart';
|
||||
|
||||
import '../../../core/l10n/l10n.dart';
|
||||
|
||||
import '../../../core/ui/failure_text.dart';
|
||||
|
||||
import '../../../core/ui/tripz_button.dart';
|
||||
|
||||
import '../../../core/ui/tripz_scaffold.dart';
|
||||
|
||||
import '../../../core/ui/tripz_text_field.dart';
|
||||
|
||||
import '../cubit/login_cubit.dart';
|
||||
|
||||
import '../cubit/login_state.dart';
|
||||
import 'widgets/auth_failure_text.dart';
|
||||
import 'widgets/auth_header.dart';
|
||||
|
||||
import '../../../core/ui/form_header.dart';
|
||||
|
||||
class PhoneView extends StatelessWidget {
|
||||
const PhoneView({super.key});
|
||||
@@ -34,7 +43,7 @@ class PhoneView extends StatelessWidget {
|
||||
child: ListView(
|
||||
children: [
|
||||
const SizedBox(height: Space.xl),
|
||||
AuthHeader(
|
||||
FormHeader(
|
||||
icon: Icons.phone_outlined,
|
||||
title: l10n.phoneTitle,
|
||||
lead: l10n.phoneLead,
|
||||
|
||||
@@ -1,85 +0,0 @@
|
||||
import 'package:flutter/material.dart';
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
|
||||
import '../../../core/design/tokens.dart';
|
||||
import '../../../core/di.dart';
|
||||
import '../../../core/l10n/l10n.dart';
|
||||
import '../../../core/ui/tripz_button.dart';
|
||||
import '../../../core/ui/tripz_scaffold.dart';
|
||||
import '../../../core/ui/tripz_text_field.dart';
|
||||
import '../cubit/profile_cubit.dart';
|
||||
import '../../../core/session/session_cubit.dart';
|
||||
import 'widgets/auth_failure_text.dart';
|
||||
import 'widgets/auth_header.dart';
|
||||
|
||||
class ProfilePage extends StatefulWidget {
|
||||
const ProfilePage({super.key});
|
||||
|
||||
@override
|
||||
State<ProfilePage> createState() => _ProfilePageState();
|
||||
}
|
||||
|
||||
class _ProfilePageState extends State<ProfilePage> {
|
||||
final _name = TextEditingController();
|
||||
String? _error;
|
||||
|
||||
@override
|
||||
void dispose() {
|
||||
_name.dispose();
|
||||
super.dispose();
|
||||
}
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
final l10n = context.l10n;
|
||||
return BlocProvider(
|
||||
create: (_) => ProfileCubit(sl()),
|
||||
child: BlocBuilder<ProfileCubit, ProfileState>(
|
||||
builder: (context, state) {
|
||||
return TripzScaffold(
|
||||
showAppBar: false,
|
||||
bottomAction: TripzButton.primary(
|
||||
label: l10n.actionSave,
|
||||
loading: state.saving,
|
||||
onPressed: () => _submit(context),
|
||||
),
|
||||
child: ListView(
|
||||
children: [
|
||||
const SizedBox(height: Space.xl),
|
||||
AuthHeader(
|
||||
icon: Icons.person_outline_rounded,
|
||||
title: l10n.profileTitle,
|
||||
lead: l10n.profileLead,
|
||||
),
|
||||
const SizedBox(height: Space.lg),
|
||||
TripzTextField(
|
||||
controller: _name,
|
||||
label: l10n.profileNameLabel,
|
||||
autofocus: true,
|
||||
enabled: !state.saving,
|
||||
errorText: _error,
|
||||
textInputAction: TextInputAction.done,
|
||||
autofillHints: const [AutofillHints.name],
|
||||
onChanged: (_) => setState(() => _error = null),
|
||||
onSubmitted: (_) => _submit(context),
|
||||
),
|
||||
if (state.failure != null)
|
||||
AuthFailureText(failure: state.failure!),
|
||||
],
|
||||
),
|
||||
);
|
||||
},
|
||||
),
|
||||
);
|
||||
}
|
||||
|
||||
Future<void> _submit(BuildContext context) async {
|
||||
if (_name.text.trim().isEmpty) {
|
||||
setState(() => _error = context.l10n.profileNameErrEmpty);
|
||||
return;
|
||||
}
|
||||
final session = context.read<SessionCubit>();
|
||||
final user = await context.read<ProfileCubit>().save(_name.text);
|
||||
if (user != null) session.onProfileCompleted(user);
|
||||
}
|
||||
}
|
||||
@@ -1,61 +0,0 @@
|
||||
import 'package:flutter/material.dart';
|
||||
|
||||
import '../../../../core/design/tokens.dart';
|
||||
import '../../../../core/design/tripz_colors.dart';
|
||||
import '../../../../core/l10n/l10n.dart';
|
||||
import '../../cubit/auth_failure.dart';
|
||||
import '../../cubit/phone_error.dart';
|
||||
|
||||
/// النقطة **الوحيدة** التي تُترجَم فيها رموز الفشل إلى نصّ.
|
||||
/// الشاشات لا تكتب رسائل خطأ، والـCubit لا يعرف اللغة (راجع `auth_failure.dart`).
|
||||
class AuthFailureText extends StatelessWidget {
|
||||
const AuthFailureText({super.key, required this.failure});
|
||||
|
||||
final AuthFailure failure;
|
||||
|
||||
static String messageOf(BuildContext context, AuthFailure failure) {
|
||||
final l10n = context.l10n;
|
||||
return switch (failure) {
|
||||
AuthFailure.invalidCode => l10n.otpErrInvalid,
|
||||
AuthFailure.tooManyAttempts => l10n.otpErrTooManyAttempts,
|
||||
AuthFailure.rateLimited => l10n.otpErrRateLimited,
|
||||
AuthFailure.network => l10n.errorNetwork,
|
||||
AuthFailure.timeout => l10n.errorTimeout,
|
||||
AuthFailure.server => l10n.errorGeneric,
|
||||
};
|
||||
}
|
||||
|
||||
static String phoneMessageOf(BuildContext context, PhoneError error) {
|
||||
final l10n = context.l10n;
|
||||
return switch (error) {
|
||||
PhoneError.empty => l10n.phoneErrEmpty,
|
||||
PhoneError.leadingZero => l10n.phoneErrLeadingZero,
|
||||
PhoneError.tooShort => l10n.phoneErrShort,
|
||||
};
|
||||
}
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
return Padding(
|
||||
padding: const EdgeInsets.only(top: Space.sm),
|
||||
child: Row(
|
||||
children: [
|
||||
Icon(
|
||||
Icons.error_outline_rounded,
|
||||
size: Sizes.iconSm,
|
||||
color: context.tripzColors.danger,
|
||||
),
|
||||
const SizedBox(width: Space.xs),
|
||||
Expanded(
|
||||
child: Text(
|
||||
messageOf(context, failure),
|
||||
style: context.texts.bodySmall?.copyWith(
|
||||
color: context.tripzColors.danger,
|
||||
),
|
||||
),
|
||||
),
|
||||
],
|
||||
),
|
||||
);
|
||||
}
|
||||
}
|
||||
@@ -1,49 +0,0 @@
|
||||
import 'package:flutter/material.dart';
|
||||
|
||||
import '../../../../core/design/tokens.dart';
|
||||
import '../../../../core/design/tripz_colors.dart';
|
||||
|
||||
/// ترويسة موحّدة لشاشات الدخول: أيقونة ثم عنوان ثم سطر شارح.
|
||||
/// «الشاشة مهمة واحدة» — سؤال واحد وزر أساسي واحد أسفلها (docs/26 §8.1).
|
||||
class AuthHeader extends StatelessWidget {
|
||||
const AuthHeader({
|
||||
super.key,
|
||||
required this.icon,
|
||||
required this.title,
|
||||
required this.lead,
|
||||
});
|
||||
|
||||
final IconData icon;
|
||||
final String title;
|
||||
final String lead;
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
return Column(
|
||||
crossAxisAlignment: CrossAxisAlignment.start,
|
||||
children: [
|
||||
Container(
|
||||
padding: const EdgeInsets.all(Space.sm),
|
||||
decoration: BoxDecoration(
|
||||
color: context.colors.primaryContainer,
|
||||
borderRadius: Radii.card_,
|
||||
),
|
||||
child: Icon(
|
||||
icon,
|
||||
size: Sizes.icon,
|
||||
color: context.colors.onPrimaryContainer,
|
||||
),
|
||||
),
|
||||
const SizedBox(height: Space.md),
|
||||
Text(title, style: context.texts.headlineSmall),
|
||||
const SizedBox(height: Space.xs),
|
||||
Text(
|
||||
lead,
|
||||
style: context.texts.bodyMedium?.copyWith(
|
||||
color: context.colors.onSurfaceVariant,
|
||||
),
|
||||
),
|
||||
],
|
||||
);
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user