feat(apps): التقييم والمحفظتان ومسار الرحلة — المرحلة 5
## التقييم مفروض لا اختياري
SessionCubit يفحص GET /trips/rating/pending بعد إثبات الجلسة، ووجود رحلة
معلّقة يحجز التطبيق كله عند /rating حتى تُحسم. دور المستخدم في الرحلة يأتي
من الخادم لا من نوع التطبيق — نفس الرقم قد يكون له حسابان (docs/38 §1).
التقييم المزدوج يرجّع 400، وهو نجاح من منظور المستخدم: الرحلة مُقيَّمة فعلاً
فلا يُحبس في شاشة بلا مخرج.
الجلسة تحتاج معرفة التقييم المعلّق، و core لا يجوز أن يستورد ميزة التقييم —
فأُضيفت الطريقة لواجهة SessionSource ونفّذها AuthRepository.
## core/format/money.dart
كل المبالغ تصل نصوصاً ("0.000")؛ التحويل والعرض من نقطة واحدة. عدد المنازل
من العملة (JOD ثلاث)، والعملة من إعداد المستأجر لا من ثابت في التطبيق.
## المحفظتان لا تُوحَّدان
- الراكب WalletPage: رصيد + شحن = التزام على المنصّة
- السائق EarningsPage: أرباح + سحب = إيراد له
شكلهما متقارب وحسابهما مختلف؛ دمجهما خطأ محاسبي لا اختصار (docs/39 §4).
## السحب بخطوتين
request يرسل رمزاً بلا خصم، و confirm هو ما يخصم ويحجز. شاشة سيرو كانت خطوة
واحدة ولم تُقلَّد. الرمز الخاطئ لا يُرجع المستخدم لخطوة المبلغ.
الرصيد التشغيلي معروض منفصلاً عن الأرباح: منه تُخصم العمولة ونفاده يوقف
وصول الطلبات — خلطه بالأرباح يخفي سبب توقّف العمل عن السائق.
FareSummary يعرض رقم كلّ طرف: price_for_passenger للراكب و price_for_driver
للسائق — حقلان منفصلان والفرق عمولة (docs/38 §4).
الإلغاء بتأكيد صريح مع سبب؛ السبب لا يقبله عقد الخادم بعد لكن السؤال يخفض
الإلغاء العرضي ويجهّز الحقل.
flutter analyze نظيف · الراكب 86 ملف/6,280 سطر · السائق 83 ملف/5,818 سطر.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
Hamza-Ayed
co-authored by
Claude Opus 5
Claude Opus 5
parent
1156299d09
commit
66e28acc58
@@ -0,0 +1,40 @@
|
||||
import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
|
||||
import '../../../core/api/api_exception.dart';
|
||||
import '../data/rating_repository.dart';
|
||||
import 'rating_state.dart';
|
||||
|
||||
class RatingCubit extends Cubit<RatingState> {
|
||||
RatingCubit(this._repo) : super(const RatingState());
|
||||
|
||||
final RatingRepository _repo;
|
||||
|
||||
void attach(PendingRating pending) => emit(RatingState(pending: pending));
|
||||
|
||||
void setStars(int stars) => emit(state.copyWith(stars: stars));
|
||||
|
||||
void setComment(String comment) => emit(state.copyWith(comment: comment));
|
||||
|
||||
Future<void> submit() async {
|
||||
final pending = state.pending;
|
||||
if (pending == null || !state.canSubmit) return;
|
||||
|
||||
emit(state.copyWith(status: RatingStatus.submitting));
|
||||
try {
|
||||
await _repo.rate(
|
||||
pending.tripId,
|
||||
state.stars,
|
||||
comment: state.comment.trim().isEmpty ? null : state.comment.trim(),
|
||||
);
|
||||
emit(state.copyWith(status: RatingStatus.done));
|
||||
} on ApiException catch (e) {
|
||||
// 400 هنا = تقييم مزدوج، وهو نجاحٌ من منظور المستخدم: الرحلة مُقيَّمة
|
||||
// فعلاً، فلا يُحبس في شاشة لا مخرج منها.
|
||||
emit(state.copyWith(
|
||||
status: e.statusCode == 400
|
||||
? RatingStatus.alreadyRated
|
||||
: RatingStatus.failed,
|
||||
));
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,38 @@
|
||||
import 'package:equatable/equatable.dart';
|
||||
|
||||
import '../data/rating_repository.dart';
|
||||
|
||||
enum RatingStatus { idle, submitting, done, alreadyRated, failed }
|
||||
|
||||
class RatingState extends Equatable {
|
||||
const RatingState({
|
||||
this.status = RatingStatus.idle,
|
||||
this.pending,
|
||||
this.stars = 0,
|
||||
this.comment = '',
|
||||
});
|
||||
|
||||
final RatingStatus status;
|
||||
final PendingRating? pending;
|
||||
final int stars;
|
||||
final String comment;
|
||||
|
||||
bool get canSubmit => stars > 0 && status != RatingStatus.submitting;
|
||||
|
||||
RatingState copyWith({
|
||||
RatingStatus? status,
|
||||
PendingRating? pending,
|
||||
int? stars,
|
||||
String? comment,
|
||||
}) {
|
||||
return RatingState(
|
||||
status: status ?? this.status,
|
||||
pending: pending ?? this.pending,
|
||||
stars: stars ?? this.stars,
|
||||
comment: comment ?? this.comment,
|
||||
);
|
||||
}
|
||||
|
||||
@override
|
||||
List<Object?> get props => [status, pending?.tripId, stars, comment];
|
||||
}
|
||||
@@ -0,0 +1,40 @@
|
||||
import '../../../core/api/api_client.dart';
|
||||
|
||||
/// رحلة تنتظر تقييماً — يرجّعها الخادم عند كل فتح للتطبيق (docs/38 §7).
|
||||
class PendingRating {
|
||||
const PendingRating({required this.tripId, required this.role});
|
||||
|
||||
final String tripId;
|
||||
|
||||
/// الدور يحدّد نصّ السؤال: الراكب يقيّم السائق والعكس.
|
||||
final String role;
|
||||
|
||||
bool get isDriver => role == 'driver';
|
||||
}
|
||||
|
||||
class RatingRepository {
|
||||
RatingRepository(this._api);
|
||||
|
||||
final ApiClient _api;
|
||||
|
||||
/// `{"pending": null, "count": 0}` حين لا شيء — شكل مُتحقَّق حيّاً.
|
||||
Future<PendingRating?> pending() async {
|
||||
final res = await _api.get<Map<String, dynamic>>('/trips/rating/pending');
|
||||
final pending = res['pending'];
|
||||
if (pending is! Map<String, dynamic>) return null;
|
||||
final tripId = pending['tripId'] as String?;
|
||||
if (tripId == null) return null;
|
||||
return PendingRating(
|
||||
tripId: tripId,
|
||||
role: pending['role'] as String? ?? 'rider',
|
||||
);
|
||||
}
|
||||
|
||||
/// التقييم المزدوج يرجّع **400** — تُعامَل كحالة معروفة لا كعطل.
|
||||
Future<void> rate(String tripId, int stars, {String? comment}) {
|
||||
return _api.post<Map<String, dynamic>>(
|
||||
'/trips/$tripId/rate',
|
||||
body: {'stars': stars, 'comment': ?comment},
|
||||
);
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,123 @@
|
||||
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/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/rating_cubit.dart';
|
||||
import '../cubit/rating_state.dart';
|
||||
import '../data/rating_repository.dart';
|
||||
import 'star_row.dart';
|
||||
|
||||
/// شاشة التقييم — **تُفرض عند فتح التطبيق** إن كانت هناك رحلة معلّقة
|
||||
/// (`GET /trips/rating/pending`، docs/38 §7). لذلك بلا زرّ رجوع في الشريط:
|
||||
/// الطريق الوحيد للخروج هو التقييم أو التخطّي الصريح.
|
||||
class RatingPage extends StatefulWidget {
|
||||
const RatingPage({super.key, required this.pending, required this.onDone});
|
||||
|
||||
final PendingRating pending;
|
||||
final VoidCallback onDone;
|
||||
|
||||
@override
|
||||
State<RatingPage> createState() => _RatingPageState();
|
||||
}
|
||||
|
||||
class _RatingPageState extends State<RatingPage> {
|
||||
late final RatingCubit _cubit = sl<RatingCubit>()..attach(widget.pending);
|
||||
final _comment = TextEditingController();
|
||||
|
||||
@override
|
||||
void dispose() {
|
||||
_comment.dispose();
|
||||
_cubit.close();
|
||||
super.dispose();
|
||||
}
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
final l10n = context.l10n;
|
||||
// من يقيّم مَن: دور **هذا** المستخدم في الرحلة يأتي من الخادم، ولا
|
||||
// يُستنتج من نوع التطبيق — الرقم نفسه قد يكون له حسابان (docs/38 §1).
|
||||
final title = widget.pending.isDriver ? l10n.rateTitleDriver : l10n.rateTitle;
|
||||
|
||||
return BlocProvider.value(
|
||||
value: _cubit,
|
||||
child: BlocConsumer<RatingCubit, RatingState>(
|
||||
listenWhen: (p, n) => p.status != n.status,
|
||||
listener: (context, state) {
|
||||
if (state.status == RatingStatus.done ||
|
||||
state.status == RatingStatus.alreadyRated) {
|
||||
widget.onDone();
|
||||
}
|
||||
},
|
||||
builder: (context, state) {
|
||||
return TripzScaffold(
|
||||
showAppBar: false,
|
||||
bottomAction: Column(
|
||||
mainAxisSize: MainAxisSize.min,
|
||||
children: [
|
||||
TripzButton.primary(
|
||||
label: l10n.rateSubmit,
|
||||
loading: state.status == RatingStatus.submitting,
|
||||
onPressed: state.canSubmit ? _cubit.submit : null,
|
||||
),
|
||||
TripzButton.text(
|
||||
label: l10n.rateSkip,
|
||||
onPressed: widget.onDone,
|
||||
),
|
||||
],
|
||||
),
|
||||
child: ListView(
|
||||
children: [
|
||||
const SizedBox(height: Space.xxl),
|
||||
Icon(
|
||||
Icons.star_rounded,
|
||||
size: 56,
|
||||
color: context.tripzColors.warning,
|
||||
),
|
||||
const SizedBox(height: Space.md),
|
||||
Text(
|
||||
title,
|
||||
textAlign: TextAlign.center,
|
||||
style: context.texts.headlineSmall,
|
||||
),
|
||||
const SizedBox(height: Space.xs),
|
||||
Text(
|
||||
l10n.rateLead,
|
||||
textAlign: TextAlign.center,
|
||||
style: context.texts.bodyMedium?.copyWith(
|
||||
color: context.colors.onSurfaceVariant,
|
||||
),
|
||||
),
|
||||
const SizedBox(height: Space.lg),
|
||||
StarRow(value: state.stars, onChanged: _cubit.setStars),
|
||||
const SizedBox(height: Space.lg),
|
||||
TripzTextField(
|
||||
controller: _comment,
|
||||
hint: l10n.rateComment,
|
||||
onChanged: _cubit.setComment,
|
||||
),
|
||||
if (state.status == RatingStatus.failed) ...[
|
||||
const SizedBox(height: Space.sm),
|
||||
Text(
|
||||
l10n.errorGeneric,
|
||||
textAlign: TextAlign.center,
|
||||
style: context.texts.bodySmall?.copyWith(
|
||||
color: context.tripzColors.danger,
|
||||
),
|
||||
),
|
||||
],
|
||||
// مساحة للـCTA السفلي كي لا يغطّي الحقل عند فتح الكيبورد.
|
||||
const SizedBox(height: Sizes.control),
|
||||
],
|
||||
),
|
||||
);
|
||||
},
|
||||
),
|
||||
);
|
||||
}
|
||||
}
|
||||
@@ -0,0 +1,43 @@
|
||||
import 'package:flutter/material.dart';
|
||||
|
||||
import '../../../core/design/tokens.dart';
|
||||
import '../../../core/design/tripz_colors.dart';
|
||||
|
||||
/// خمس نجوم — أهداف لمس ≥ 48 لأن الشاشة تُستعمل عادةً والمستخدم مستعجل.
|
||||
class StarRow extends StatelessWidget {
|
||||
const StarRow({super.key, required this.value, required this.onChanged});
|
||||
|
||||
final int value;
|
||||
final ValueChanged<int> onChanged;
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
return Row(
|
||||
mainAxisAlignment: MainAxisAlignment.center,
|
||||
// النجوم تُقرأ يساراً-ليميناً في اللغتين: ترتيبها ليس نصّاً.
|
||||
textDirection: TextDirection.ltr,
|
||||
children: List.generate(5, (i) {
|
||||
final star = i + 1;
|
||||
final filled = star <= value;
|
||||
return IconButton(
|
||||
onPressed: () => onChanged(star),
|
||||
iconSize: Sizes.iconLg,
|
||||
constraints: const BoxConstraints(
|
||||
minWidth: Sizes.touchTarget,
|
||||
minHeight: Sizes.touchTarget,
|
||||
),
|
||||
icon: AnimatedSwitcher(
|
||||
duration: Motion.tap,
|
||||
child: Icon(
|
||||
filled ? Icons.star_rounded : Icons.star_outline_rounded,
|
||||
key: ValueKey(filled),
|
||||
color: filled
|
||||
? context.tripzColors.warning
|
||||
: context.colors.onSurfaceVariant,
|
||||
),
|
||||
),
|
||||
);
|
||||
}),
|
||||
);
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user