feat(apps): تسجيل السائق والدردشة والكوبون — إكمال التغطية
فحص آليّ لنقاط docs/38 مقابل الكود كشف ثغرات لم تظهر بالقراءة. ## تسجيل السائق — أخطر ثغرة لم يكن موجوداً أصلاً: سائق جديد يسجّل دخوله ثم يقف. بُنيت features/onboarding كاملة — تقديم ← ملف ← مركبة ← وثائق ← انتظار الاعتماد. - الخطوة تُشتقّ من حالة الخادم لا من تقدّم محلّي: سائق يعيد تثبيت التطبيق يعود إلى حيث وقف لا إلى البداية - الاعتماد يقع على الخادم تلقائياً حين تكتمل الوثائق؛ التطبيق لا يعتمد أحداً - عند الاعتماد خروج إجباري: الدور يتغيّر على الخادم والتوكن القديم يحمل القديم، فتفشل نقاط السائق بـ403 (مصيدة docs/38 §5) - بوابة توجيه: مستخدم دوره ليس driver يُحجز في /onboarding - الوثائق تُصوَّر بالكاميرا لا من المعرض: أصعب تزويراً ## الدردشة والكوبون - Features.chat كان مفعّلاً بلا ميزة. features/chat باستطلاع كل خمس ثوان — قائمة أحداث الخادم لا تتضمّن الرسائل (docs/38 §9)، فالاستطلاع قيد خادم لا اختيار تصميمي - الكوبون: حقل في ورقة التأكيد خلف طبقتَي الميزات، يُقيَّم على الخادم ## Features.calls أُطفئ صراحةً الخادم يدعم WebRTC والتطبيق لا. عَلَم مفعّل بلا ميزة كذبٌ على القارئ التالي. ## بنية - ApiClient.upload للرفع متعدّد الأجزاء - AuthFailure → ApiFailure في core/api: يخدم المصادقة والملف والتسجيل - tool/sync_from_rider.sh: المزامنة اليدوية بين التوأمين انكسرت أربع مرات، فصارت سكربتاً واحداً يعرّف ما يملكه كل تطبيق flutter analyze نظيف · الراكب 101 ملف/7,809 سطر · السائق 110 ملف/8,116 سطر. فحوص آلية: صفر استيراد بين ميزتين · صفر عَلَم مفعّل بلا ميزة · كل نقاط العقد المخصّصة للتطبيقين مستهلكة. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
096acae74e
commit
bb2c0de478
@@ -0,0 +1,123 @@
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
|
||||
import '../../../core/api/api_exception.dart';
|
||||
import '../../../core/api/api_failure.dart';
|
||||
import '../data/driver_repository.dart';
|
||||
import 'onboarding_state.dart';
|
||||
|
||||
/// تسجيل السائق حتى الاعتماد.
|
||||
///
|
||||
/// **الخطوة تُشتقّ من حالة الخادم لا من تقدّم محلّي**: سائق يعيد تثبيت
|
||||
/// التطبيق يجب أن يعود إلى حيث وقف، لا أن يبدأ من الصفر.
|
||||
class OnboardingCubit extends Cubit<OnboardingState> {
|
||||
OnboardingCubit(this._repo) : super(const OnboardingState());
|
||||
|
||||
final DriverRepository _repo;
|
||||
|
||||
Future<void> load() async {
|
||||
emit(state.copyWith(step: OnboardingStep.loading, clearFailure: true));
|
||||
try {
|
||||
final driver = await _repo.me();
|
||||
if (isClosed) return;
|
||||
|
||||
if (driver == null) {
|
||||
emit(state.copyWith(step: OnboardingStep.apply, driver: null));
|
||||
return;
|
||||
}
|
||||
if (driver.isApproved) {
|
||||
emit(state.copyWith(step: OnboardingStep.approved, driver: driver));
|
||||
return;
|
||||
}
|
||||
|
||||
// الملف ناقص ما لم يُدخل الاسم العربي — أول ما يطلبه الخادم.
|
||||
if (driver.nameArabic == null || driver.nameArabic!.isEmpty) {
|
||||
emit(state.copyWith(step: OnboardingStep.profile, driver: driver));
|
||||
return;
|
||||
}
|
||||
|
||||
final vehicles = await _repo.vehicles();
|
||||
if (isClosed) return;
|
||||
if (vehicles.isEmpty) {
|
||||
emit(state.copyWith(
|
||||
step: OnboardingStep.vehicle,
|
||||
driver: driver,
|
||||
vehicles: vehicles,
|
||||
));
|
||||
return;
|
||||
}
|
||||
|
||||
final reqs = await _repo.requirements();
|
||||
if (isClosed) return;
|
||||
emit(state.copyWith(
|
||||
// اكتملت الوثائق ولم يُعتمد بعد: مراجعة بشرية جارية.
|
||||
step: reqs.complete ? OnboardingStep.pending : OnboardingStep.documents,
|
||||
driver: driver,
|
||||
vehicles: vehicles,
|
||||
requirements: reqs.items,
|
||||
));
|
||||
} on ApiException catch (e) {
|
||||
if (!isClosed) {
|
||||
emit(state.copyWith(
|
||||
step: OnboardingStep.apply,
|
||||
failure: e.toApiFailure(),
|
||||
));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
Future<void> apply({required String vehicleMake, required String serviceClass}) =>
|
||||
_run(() => _repo.apply(vehicleMake: vehicleMake, serviceClass: serviceClass));
|
||||
|
||||
Future<void> saveProfile({
|
||||
required String nameArabic,
|
||||
required String nationalNumber,
|
||||
String? gender,
|
||||
}) =>
|
||||
_run(() => _repo.setProfile(
|
||||
nameArabic: nameArabic,
|
||||
nationalNumber: nationalNumber,
|
||||
gender: gender,
|
||||
));
|
||||
|
||||
Future<void> addVehicle({
|
||||
required String make,
|
||||
required String model,
|
||||
required int year,
|
||||
required String color,
|
||||
required String plate,
|
||||
}) =>
|
||||
_run(() => _repo.addVehicle(
|
||||
make: make,
|
||||
model: model,
|
||||
year: year,
|
||||
color: color,
|
||||
plate: plate,
|
||||
));
|
||||
|
||||
Future<void> uploadDocument({
|
||||
required String filePath,
|
||||
required String type,
|
||||
required String side,
|
||||
}) =>
|
||||
_run(() => _repo.uploadDocument(
|
||||
filePath: filePath,
|
||||
type: type,
|
||||
side: side,
|
||||
));
|
||||
|
||||
/// كل فعل ينتهي بإعادة قراءة الحالة من الخادم — فلا يتقدّم التطبيق خطوةً
|
||||
/// لم يقبلها الخادم فعلاً.
|
||||
Future<void> _run(Future<dynamic> Function() action) async {
|
||||
emit(state.copyWith(busy: true, clearFailure: true));
|
||||
try {
|
||||
await action();
|
||||
if (isClosed) return;
|
||||
emit(state.copyWith(busy: false));
|
||||
await load();
|
||||
} on ApiException catch (e) {
|
||||
if (!isClosed) {
|
||||
emit(state.copyWith(busy: false, failure: e.toApiFailure()));
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,51 @@
|
||||
import 'package:equatable/equatable.dart';
|
||||
|
||||
import '../../../core/api/api_failure.dart';
|
||||
import '../data/models/document_requirement.dart';
|
||||
import '../data/models/driver_profile.dart';
|
||||
import '../data/models/vehicle.dart';
|
||||
|
||||
/// خطوات تسجيل السائق بالترتيب الميداني (docs/39 §2): تقديم ← ملف ←
|
||||
/// مركبة ← وثائق ← انتظار الاعتماد.
|
||||
enum OnboardingStep { loading, apply, profile, vehicle, documents, pending, approved }
|
||||
|
||||
class OnboardingState extends Equatable {
|
||||
const OnboardingState({
|
||||
this.step = OnboardingStep.loading,
|
||||
this.driver,
|
||||
this.vehicles = const [],
|
||||
this.requirements = const [],
|
||||
this.busy = false,
|
||||
this.failure,
|
||||
});
|
||||
|
||||
final OnboardingStep step;
|
||||
final DriverProfile? driver;
|
||||
final List<Vehicle> vehicles;
|
||||
final List<DocumentRequirement> requirements;
|
||||
final bool busy;
|
||||
final ApiFailure? failure;
|
||||
|
||||
OnboardingState copyWith({
|
||||
OnboardingStep? step,
|
||||
DriverProfile? driver,
|
||||
List<Vehicle>? vehicles,
|
||||
List<DocumentRequirement>? requirements,
|
||||
bool? busy,
|
||||
ApiFailure? failure,
|
||||
bool clearFailure = false,
|
||||
}) {
|
||||
return OnboardingState(
|
||||
step: step ?? this.step,
|
||||
driver: driver ?? this.driver,
|
||||
vehicles: vehicles ?? this.vehicles,
|
||||
requirements: requirements ?? this.requirements,
|
||||
busy: busy ?? this.busy,
|
||||
failure: clearFailure ? null : (failure ?? this.failure),
|
||||
);
|
||||
}
|
||||
|
||||
@override
|
||||
List<Object?> get props =>
|
||||
[step, driver, vehicles, requirements, busy, failure];
|
||||
}
|
||||
Reference in New Issue
Block a user