fix(apps): إصلاحات من أول تجربة حقيقية على الجهاز
## تعذّر حساب الأجرة — الخطأ في التطبيق لا في جدول الكازان الخادم يبذر التعرفة بـ city: 'default' (tenants.service.ts:156) ويستعمل dto.city ?? 'default' عند إنشاء الرحلة، وكنت أرسل city: '' فيرجّع 404 «No active tariff for /economy». تحقّق حيّ: city='default' → total 1.55 JOD ✅ city='' → 404 ❌ city='amman' → 404 ❌ لا تغيير مطلوب على السيرفر. أُضيف AppConfig.defaultCity موثّقاً بالسبب. ## التسعير مرة واحدة لكل الأنواع كان نداء شبكة عند كل ضغطة نوع. صار _priceAll يسعّر كل الأنواع بالتوازي بعد اكتمال المسار — المسافة والمدة ثابتتان والفرق في التعرفة وحدها. الراكب يرى سعر الاقتصادي والمريح معاً تحت كل خيار ويقارن بنظرة، وتبديل النوع بلا نداء. ## الكاميرا لا تتبع الموقع سببان: الموقع يصل قبل إنشاء الخريطة فيضيع التحريك، ولا فحص للإذن/الخدمة قبل getCurrentPosition فيُبتلع الخطأ صامتاً. - _onCreated يلحق بما فات عند جهوز الخريطة - LocationService.ensureReady يفحص الخدمة والإذن أولاً - didChangeAppLifecycleState يعيد المحاولة عند العودة من إعدادات النظام - حالة noLocation تُعرض للمستخدم مع زر إعادة بدل خريطة صامتة ## الخريطة والتنقّل - إطار المسار من حدوده الفعلية مع حشوة سفلية بمقدار ما تغطّيه الورقة - علامتا البداية والنهاية دائرتان ملوّنتان تطابقان ألوان الخط الزمني - زر رجوع في ورقة التأكيد + زر إعادة حساب عند فشل التسعيرة: كان المستخدم يعلق في الشاشة بلا مخرج ## المحفظة — على أي أساس يُشحن POST /wallet/topup شحنٌ تجريبي بلا مزوّد دفع خلفه (الخادم نفسه يصفه كذلك). أُضيف PaymentChannel مشتقّاً من countryPack المستأجر لا من لغة الجهاز: jo → CliQ · sy → سيرياتيل/MTN/شام كاش · eg → PayMob. ويُعرض تنبيه صريح بأن الشحن بلا تحصيل فعلي — إخفاؤه يجعل المستخدم يظنّ أنه دفع. وسحب السائق صار يستعمل القناة المختارة بدل 'cliq' المكتوبة في الكود. flutter analyze نظيف في التطبيقين. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
29f375b122
commit
8976821a93
@@ -8,7 +8,20 @@ import '../config.dart';
|
||||
class LocationService {
|
||||
StreamSubscription<Position>? _sub;
|
||||
|
||||
/// بلا هذا الفحص يرمي `getCurrentPosition` ونبتلع الخطأ فتبقى الكاميرا
|
||||
/// على الاحتياطي بلا سبب ظاهر — وهو ما كان يبدو «الخريطة لا تتبع موقعي».
|
||||
Future<bool> ensureReady() async {
|
||||
if (!await Geolocator.isLocationServiceEnabled()) return false;
|
||||
var permission = await Geolocator.checkPermission();
|
||||
if (permission == LocationPermission.denied) {
|
||||
permission = await Geolocator.requestPermission();
|
||||
}
|
||||
return permission == LocationPermission.always ||
|
||||
permission == LocationPermission.whileInUse;
|
||||
}
|
||||
|
||||
Future<({double lat, double lng})?> current() async {
|
||||
if (!await ensureReady()) return null;
|
||||
try {
|
||||
final p = await Geolocator.getCurrentPosition(
|
||||
locationSettings: const LocationSettings(
|
||||
|
||||
@@ -0,0 +1,50 @@
|
||||
/// قنوات الدفع والسحب حسب **حزمة دولة المستأجر** (`countryPack` من
|
||||
/// `GET /tenant/config/:slug` — docs/38 §10)، لا حسب لغة الجهاز ولا موقعه.
|
||||
///
|
||||
/// المستأجر هو من يحدّد السوق: مشغّل أردني يخدم مستخدماً هاتفه إنجليزي
|
||||
/// وموقعه خارج الأردن يبقى أردنيّ القنوات.
|
||||
///
|
||||
/// ⚠️ **`POST /wallet/topup` شحنٌ تجريبي بلا مزوّد دفع خلفه** — الخادم نفسه
|
||||
/// يصفه بـ«كاش-إن، للإنتاج يُربط بمزوّد دفع أو يُقيَّد للأدمن». اختيار
|
||||
/// القناة هنا يجهّز الواجهة والعقد؛ التحصيل الفعلي عبر `/payments/charge`
|
||||
/// وبوابة كل قناة عملٌ منفصل لم يُنفَّذ بعد.
|
||||
enum PaymentChannel {
|
||||
cliq('cliq', 'CliQ', 'كليك'),
|
||||
syriatel('syriatel_cash', 'Syriatel Cash', 'سيرياتيل كاش'),
|
||||
mtn('mtn_cash', 'MTN Cash', 'إم تي إن كاش'),
|
||||
shamCash('sham_cash', 'Sham Cash', 'شام كاش'),
|
||||
paymob('paymob', 'PayMob', 'باي موب'),
|
||||
cash('cash', 'Cash', 'نقداً');
|
||||
|
||||
const PaymentChannel(this.wire, this.nameEn, this.nameAr);
|
||||
|
||||
/// القيمة المرسلة للخادم في حقل `channel`.
|
||||
final String wire;
|
||||
final String nameEn;
|
||||
final String nameAr;
|
||||
|
||||
String nameFor(String languageCode) =>
|
||||
languageCode == 'ar' ? nameAr : nameEn;
|
||||
|
||||
/// القنوات المتاحة لكل حزمة دولة — مطابقة لما هو مدعوم فعلياً في السوق
|
||||
/// (نفس قنوات سيرو الميدانية).
|
||||
static List<PaymentChannel> forCountry(String countryPack) =>
|
||||
switch (countryPack) {
|
||||
'jo' => const [PaymentChannel.cliq, PaymentChannel.cash],
|
||||
'sy' => const [
|
||||
PaymentChannel.syriatel,
|
||||
PaymentChannel.mtn,
|
||||
PaymentChannel.shamCash,
|
||||
PaymentChannel.cash,
|
||||
],
|
||||
'eg' => const [PaymentChannel.paymob, PaymentChannel.cash],
|
||||
// حزمة غير معروفة: النقد وحده — لا نعرض قناة قد لا تعمل في سوقها.
|
||||
_ => const [PaymentChannel.cash],
|
||||
};
|
||||
|
||||
/// قنوات السحب — النقد ليس قناة سحب.
|
||||
static List<PaymentChannel> payoutFor(String countryPack) =>
|
||||
forCountry(countryPack)
|
||||
.where((c) => c != PaymentChannel.cash)
|
||||
.toList(growable: false);
|
||||
}
|
||||
@@ -0,0 +1,48 @@
|
||||
import 'package:flutter/material.dart';
|
||||
|
||||
import '../design/tokens.dart';
|
||||
import '../design/tripz_colors.dart';
|
||||
import '../payments/payment_channels.dart';
|
||||
|
||||
/// اختيار قناة الدفع/السحب. القائمة تأتي من حزمة دولة المستأجر، فلا يرى
|
||||
/// الأردنيُّ «شام كاش» ولا السوريُّ «كليك».
|
||||
class ChannelPicker extends StatelessWidget {
|
||||
const ChannelPicker({
|
||||
super.key,
|
||||
required this.channels,
|
||||
required this.selected,
|
||||
required this.onSelect,
|
||||
});
|
||||
|
||||
final List<PaymentChannel> channels;
|
||||
final PaymentChannel? selected;
|
||||
final ValueChanged<PaymentChannel> onSelect;
|
||||
|
||||
@override
|
||||
Widget build(BuildContext context) {
|
||||
if (channels.isEmpty) return const SizedBox.shrink();
|
||||
final lang = Localizations.localeOf(context).languageCode;
|
||||
|
||||
return Wrap(
|
||||
spacing: Space.xs,
|
||||
runSpacing: Space.xs,
|
||||
children: [
|
||||
for (final channel in channels)
|
||||
ChoiceChip(
|
||||
selected: channel == selected,
|
||||
onSelected: (_) => onSelect(channel),
|
||||
avatar: Icon(_iconOf(channel), size: Sizes.iconSm),
|
||||
label: Text(channel.nameFor(lang)),
|
||||
labelStyle: context.texts.labelLarge,
|
||||
),
|
||||
],
|
||||
);
|
||||
}
|
||||
|
||||
static IconData _iconOf(PaymentChannel channel) => switch (channel) {
|
||||
PaymentChannel.cash => Icons.payments_outlined,
|
||||
PaymentChannel.cliq => Icons.account_balance_outlined,
|
||||
PaymentChannel.paymob => Icons.credit_card_outlined,
|
||||
_ => Icons.smartphone_outlined,
|
||||
};
|
||||
}
|
||||
@@ -24,6 +24,7 @@ class RideMap extends StatefulWidget {
|
||||
this.route,
|
||||
this.cameraTarget,
|
||||
this.onCameraIdle,
|
||||
this.bottomInset = 280,
|
||||
});
|
||||
|
||||
final GeoPoint? origin;
|
||||
@@ -35,6 +36,9 @@ class RideMap extends StatefulWidget {
|
||||
/// مركز الخريطة عند استقرارها — يغذّي اختيار النقطة بالدبّوس الثابت.
|
||||
final ValueChanged<GeoPoint>? onCameraIdle;
|
||||
|
||||
/// ارتفاع ما تغطّيه الورقة السفلية — يُحسب منه إطار المسار.
|
||||
final double bottomInset;
|
||||
|
||||
@override
|
||||
State<RideMap> createState() => _RideMapState();
|
||||
}
|
||||
@@ -48,12 +52,62 @@ class _RideMapState extends State<RideMap> {
|
||||
@override
|
||||
void didUpdateWidget(RideMap old) {
|
||||
super.didUpdateWidget(old);
|
||||
final target = widget.cameraTarget;
|
||||
if (target != null && target != old.cameraTarget) {
|
||||
_controller?.animateCamera(
|
||||
im.CameraUpdate.newLatLng(im.LatLng(target.lat, target.lng)),
|
||||
);
|
||||
if (widget.route != null && widget.route != old.route) {
|
||||
_fitRoute();
|
||||
return;
|
||||
}
|
||||
final target = widget.cameraTarget;
|
||||
if (target != null && target != old.cameraTarget) _moveTo(target);
|
||||
}
|
||||
|
||||
/// **يُنادى عند جهوز الخريطة**: الموقع الحالي غالباً يصل *قبل* إنشاء
|
||||
/// الخريطة، فالتحريك في `didUpdateWidget` وحده يضيع و تبقى الكاميرا على
|
||||
/// الاحتياطي. هنا نلحق بما فات.
|
||||
void _onCreated(im.IntaleqMapController controller) {
|
||||
_controller = controller;
|
||||
final target = widget.cameraTarget ?? widget.origin;
|
||||
if (widget.route != null) {
|
||||
_fitRoute();
|
||||
} else if (target != null) {
|
||||
_moveTo(target);
|
||||
}
|
||||
}
|
||||
|
||||
void _moveTo(GeoPoint p) => _controller?.animateCamera(
|
||||
im.CameraUpdate.newLatLngZoom(im.LatLng(p.lat, p.lng), 15.5),
|
||||
);
|
||||
|
||||
/// يضبط الكاميرا على المسار كاملاً بدل نقطة واحدة.
|
||||
///
|
||||
/// الحشوة السفلية أكبر لأن الورقة السفلية تغطّي نحو ثلث الشاشة — بلا ذلك
|
||||
/// يختفي نصف المسار تحتها.
|
||||
void _fitRoute() {
|
||||
final encoded = widget.route?.encodedPoints;
|
||||
if (encoded == null || encoded.isEmpty) return;
|
||||
final points = im.PolylineUtils.decode(encoded);
|
||||
if (points.length < 2) return;
|
||||
|
||||
var minLat = points.first.latitude, maxLat = points.first.latitude;
|
||||
var minLng = points.first.longitude, maxLng = points.first.longitude;
|
||||
for (final p in points) {
|
||||
if (p.latitude < minLat) minLat = p.latitude;
|
||||
if (p.latitude > maxLat) maxLat = p.latitude;
|
||||
if (p.longitude < minLng) minLng = p.longitude;
|
||||
if (p.longitude > maxLng) maxLng = p.longitude;
|
||||
}
|
||||
|
||||
_controller?.animateCamera(
|
||||
im.CameraUpdate.newLatLngBounds(
|
||||
im.LatLngBounds(
|
||||
southwest: im.LatLng(minLat, minLng),
|
||||
northeast: im.LatLng(maxLat, maxLng),
|
||||
),
|
||||
left: 48,
|
||||
right: 48,
|
||||
top: 120,
|
||||
bottom: widget.bottomInset,
|
||||
),
|
||||
);
|
||||
}
|
||||
|
||||
@override
|
||||
@@ -77,17 +131,21 @@ class _RideMapState extends State<RideMap> {
|
||||
myLocationEnabled: true,
|
||||
compassEnabled: false,
|
||||
zoomControlsEnabled: false,
|
||||
onMapCreated: (c) => _controller = c,
|
||||
onMapCreated: _onCreated,
|
||||
onCameraMove: (position) => _center = position.target,
|
||||
onCameraIdle: () {
|
||||
final c = _center;
|
||||
if (c != null) widget.onCameraIdle?.call(GeoPoint(c.latitude, c.longitude));
|
||||
},
|
||||
markers: {
|
||||
// العلامات دوائر ملوّنة تطابق ألوان الخط الزمني في الورقة السفلية،
|
||||
// فيربط المستخدم بين «من» و«إلى» وما يراه على الخريطة بلا شرح.
|
||||
circles: {
|
||||
if (widget.origin != null)
|
||||
_marker('origin', widget.origin!),
|
||||
_dot('origin', widget.origin!, colors.mapOrigin),
|
||||
if (widget.destination != null)
|
||||
_marker('destination', widget.destination!),
|
||||
_dot('destination', widget.destination!, colors.mapDestination),
|
||||
},
|
||||
markers: {
|
||||
if (widget.driver != null) _marker('driver', widget.driver!),
|
||||
},
|
||||
polylines: {
|
||||
@@ -106,4 +164,13 @@ class _RideMapState extends State<RideMap> {
|
||||
markerId: im.MarkerId(id),
|
||||
position: im.LatLng(p.lat, p.lng),
|
||||
);
|
||||
|
||||
im.Circle _dot(String id, GeoPoint p, Color color) => im.Circle(
|
||||
circleId: im.CircleId(id),
|
||||
center: im.LatLng(p.lat, p.lng),
|
||||
radius: 18,
|
||||
fillColor: color,
|
||||
strokeColor: Colors.white,
|
||||
strokeWidth: 3,
|
||||
);
|
||||
}
|
||||
|
||||
@@ -4,6 +4,9 @@ import 'package:flutter_bloc/flutter_bloc.dart';
|
||||
import '../../../../core/config.dart';
|
||||
import '../../../../core/design/tokens.dart';
|
||||
import '../../../../core/design/tripz_colors.dart';
|
||||
import '../../../../core/payments/payment_channels.dart';
|
||||
import '../../../../core/tenant/tenant_cubit.dart';
|
||||
import '../../../../core/ui/channel_picker.dart';
|
||||
import '../../../../core/l10n/l10n.dart';
|
||||
import '../../../../core/ui/tripz_button.dart';
|
||||
import '../../../../core/ui/tripz_otp_field.dart';
|
||||
@@ -24,6 +27,7 @@ class PayoutSheet extends StatefulWidget {
|
||||
class _PayoutSheetState extends State<PayoutSheet> {
|
||||
final _amount = TextEditingController();
|
||||
final _code = TextEditingController();
|
||||
PaymentChannel? _channel;
|
||||
|
||||
@override
|
||||
void dispose() {
|
||||
@@ -75,7 +79,7 @@ class _PayoutSheetState extends State<PayoutSheet> {
|
||||
hasError: state.failure == PayoutFailure.rejected,
|
||||
onCompleted: cubit.confirm,
|
||||
)
|
||||
else
|
||||
else ...[
|
||||
TripzTextField(
|
||||
controller: _amount,
|
||||
label: l10n.payoutAmount,
|
||||
@@ -85,6 +89,22 @@ class _PayoutSheetState extends State<PayoutSheet> {
|
||||
decimal: true,
|
||||
),
|
||||
),
|
||||
const SizedBox(height: Space.md),
|
||||
Text(l10n.payoutChannel, style: context.texts.titleSmall),
|
||||
const SizedBox(height: Space.xs),
|
||||
Builder(builder: (context) {
|
||||
// قنوات السحب من حزمة دولة المستأجر — والنقد ليس قناة سحب.
|
||||
final channels = PaymentChannel.payoutFor(
|
||||
context.watch<TenantCubit>().state.countryPack,
|
||||
);
|
||||
_channel ??= channels.isEmpty ? null : channels.first;
|
||||
return ChannelPicker(
|
||||
channels: channels,
|
||||
selected: _channel,
|
||||
onSelect: (c) => setState(() => _channel = c),
|
||||
);
|
||||
}),
|
||||
],
|
||||
if (state.failure != PayoutFailure.none) ...[
|
||||
const SizedBox(height: Space.sm),
|
||||
Text(
|
||||
@@ -107,8 +127,9 @@ class _PayoutSheetState extends State<PayoutSheet> {
|
||||
cubit.confirm(_code.text.trim());
|
||||
} else {
|
||||
final amount = num.tryParse(_amount.text.trim());
|
||||
if (amount != null) {
|
||||
cubit.request(amount: amount, channel: 'cliq');
|
||||
final channel = _channel;
|
||||
if (amount != null && channel != null) {
|
||||
cubit.request(amount: amount, channel: channel.wire);
|
||||
}
|
||||
}
|
||||
},
|
||||
|
||||
Reference in New Issue
Block a user