fix(rider): البحث عن مكان والمحفظة الحقيقية وأسماء الخريطة

## البحث لم يكن يستجيب — غلافان مختلفان مثبت بـcurl: /geocoding/search يرجّع {"results":[…]} بينما /geocoding/reverse يرجّع مصفوفة عارية. افترضتُ كليهما مصفوفة، فكان البحث يفشل صامتاً — ولهذا «تحديد من الخريطة» يعمل (يستعمل reverse) والبحث لا. أُضيفت رسالة «لا نتائج» كي لا يبقى الفراغ غامضاً، وأيقونة لكل صنف مكان. ## المحفظة صارت دفعاً حقيقياً لا كاش-إن كنت أستعمل POST /wallet/topup — وهو بوصف الخادم كاش-إن تجريبي يزيد الرصيد بلا دفع. المسار الصحيح موجود أصلاً: POST /payments/charge مع provider وpurpose، وسجلّ بوابات فيه cash · paymob · syriatel · mtn ومحوّل فاتورة لـ cliq · shamcash · zaincash. كليك وشام كاش وزين كاش تعمل بنمط **الفاتورة** — نفس ما يفعله سيرو حرفياً (create_cliq_invoice.php): الخادم يولّد مرجعاً ويعرض حساب الاستلام، ويحوّل المستخدم، والتسوية تصل لاحقاً بمطابقة رسالة المزوّد. لا بوابة تحويل حيّة. - InvoiceSheet يعرض المرجع وحساب الاستلام بزرّي نسخ — الرقم يُحوَّل في تطبيق بنك آخر فالنسخ ليس رفاهية - يُقال صراحةً أن الرصيد لا يزيد الآن، قبل الضغط لا بعده - حساب استلام غير مضبوط للمستأجر يُعرض كخطأ لا كتعليمات ناقصة - أسماء المزوّدين صُحّحت لتطابق PaymentGatewayRegistry حرفياً (syriatel لا syriatel_cash…) — أي اسم آخر يرجّع 400 - qi_card حُذف: لا محوّل له على الخادم - السائق: chargeCredit بنيّة credit_topup للرصيد التشغيلي ## الخريطة وواجهة «إلى أين» - أسماء البداية والنهاية تُرسم فوق الخريطة (نمط سيرو) كطبقة فلاتر لا marker - البطاقة المطوية صارت حقل بحث بارزاً مع الموقع الحالي فوقه - حقل البحث يفتح مركّزاً مع زرّ مسح ## أداة tool/sync_from_rider.sh كان يستثني ملفَّي ARB فيُسقط كل مفتاح مشترك جديد ويكسر بناء السائق — تكرّر ثلاث مرات. صارا يُنسخان ثم تُفرض نصوص السائق فوقهما. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-08-05 01:26:05 +03:00
parent e22cc5c7e4
commit 6a33cc7241
26 changed files with 965 additions and 121 deletions
@@ -10,9 +10,13 @@ class MapsRepository {

  final AntlaqApi _api;

+  /// ⚠️ **الغلافان مختلفان بين النقطتين** (مثبت بـcurl 2026-08-05):
+  /// `search` يرجّع `{"results":[…]}` بينما `reverse` يرجّع **مصفوفة عارية**.
+  /// افتراض أن كليهما مصفوفة كان يُفشل البحث صامتاً — فيبدو للمستخدم أن
+  /// «البحث لا يستجيب» بينما التحديد من الخريطة يعمل.
  Future<List<Place>> search(String query, {GeoPoint? near}) async {
    if (query.trim().isEmpty) return const [];
-    final res = await _api.get<List<dynamic>>(
+    final res = await _api.get<Map<String, dynamic>>(
      '/geocoding/search',
      query: {
        'q': query.trim(),
@@ -20,7 +24,9 @@ class MapsRepository {
        if (near != null) 'lng': near.lng,
      },
    );
-    return res
+    final results = res['results'];
+    if (results is! List) return const [];
+    return results
        .whereType<Map<String, dynamic>>()
        .map(Place.fromJson)
        .toList(growable: false);
@@ -26,6 +26,7 @@ class Place extends Equatable {
      name: (json['name_ar'] as String?)?.trim().isNotEmpty == true
          ? json['name_ar'] as String
          : (json['name'] as String? ?? ''),
+      // `address` قد يصل `null` صراحةً من map-saas — لا يُسقِط النتيجة.
      address: json['address'] as String? ?? '',
      category: json['category'] as String? ?? '',
      point: GeoPoint(
@@ -0,0 +1,66 @@
+import 'package:flutter/material.dart';
+
+import '../../../../core/design/tokens.dart';
+import '../../../../core/design/tripz_colors.dart';
+
+/// بطاقة اسم فوق نقطة على الخريطة — نمط سيرو: الراكب يقرأ «دوّار عبدون» على
+/// الخريطة نفسها لا في الورقة السفلية وحدها، فيتأكّد أن النقطة صحيحة قبل
+/// أن يطلب.
+///
+/// تُرسم في طبقة فلاتر فوق الخريطة لا كـmarker: نصّ عربي داخل صورة علامة
+/// يصعب ضبطه، وطبقة فلاتر تعطينا الخط والاتجاه والثيم مجاناً.
+class MapLabel extends StatelessWidget {
+  const MapLabel({
+    super.key,
+    required this.text,
+    required this.color,
+    this.icon,
+  });
+
+  final String text;
+  final Color color;
+  final IconData? icon;
+
+  @override
+  Widget build(BuildContext context) {
+    if (text.isEmpty) return const SizedBox.shrink();
+    return Container(
+      constraints: const BoxConstraints(maxWidth: 200),
+      padding: const EdgeInsets.symmetric(
+        horizontal: Space.xs,
+        vertical: Space.xxs,
+      ),
+      decoration: BoxDecoration(
+        color: context.tripzColors.surfaceRaised,
+        borderRadius: Radii.field_,
+        border: Border.all(color: color, width: 1.5),
+        boxShadow: [
+          BoxShadow(
+            color: Colors.black.withValues(alpha: 0.18),
+            blurRadius: 6,
+            offset: const Offset(0, 2),
+          ),
+        ],
+      ),
+      child: Row(
+        mainAxisSize: MainAxisSize.min,
+        children: [
+          Container(
+            width: 8,
+            height: 8,
+            decoration: BoxDecoration(color: color, shape: BoxShape.circle),
+          ),
+          const SizedBox(width: Space.xxs),
+          Flexible(
+            child: Text(
+              text,
+              maxLines: 1,
+              overflow: TextOverflow.ellipsis,
+              style: context.texts.labelMedium,
+            ),
+          ),
+        ],
+      ),
+    );
+  }
+}
@@ -9,6 +9,7 @@ import '../../../../core/design/tripz_colors.dart';
 import '../../data/models/geo_point.dart';

 import '../../data/models/route_info.dart';
+import 'map_label.dart';

 /// الخريطة — **الطبقة الوحيدة التي تعرف `intaleq_maps`**. ما فوقها يتعامل
 /// مع `GeoPoint` وحده، فتبديل محرّك الخريطة لاحقاً لا يلمس منطقاً.
@@ -25,6 +26,8 @@ class RideMap extends StatefulWidget {
    this.cameraTarget,
    this.onCameraIdle,
    this.bottomInset = 280,
+    this.originLabel = '',
+    this.destinationLabel = '',
  });

  final GeoPoint? origin;
@@ -39,6 +42,10 @@ class RideMap extends StatefulWidget {
  /// ارتفاع ما تغطّيه الورقة السفلية — يُحسب منه إطار المسار.
  final double bottomInset;

+  /// أسماء النقطتين — تُرسم بطاقاتها فوق الخريطة (نمط سيرو).
+  final String originLabel;
+  final String destinationLabel;
+
  @override
  State<RideMap> createState() => _RideMapState();
 }
@@ -46,12 +53,14 @@ class RideMap extends StatefulWidget {
 class _RideMapState extends State<RideMap> {
  im.IntaleqMapController? _controller;
  im.LatLng? _center;
+  bool _isStyleLoaded = false;

  static const _fallback = im.LatLng(31.9539, 35.9106); // عمّان

  @override
  void didUpdateWidget(RideMap old) {
    super.didUpdateWidget(old);
+    if (!_isStyleLoaded) return;
    if (widget.route != null && widget.route != old.route) {
      _fitRoute();
      return;
@@ -110,12 +119,53 @@ class _RideMapState extends State<RideMap> {
    );
  }

+  /// موضع النقطتين على الشاشة — يُحدَّث عند استقرار الكاميرا كي تتبع
+  /// البطاقاتُ الخريطةَ بلا حساب في كل إطار.
+  Offset? _originPx;
+  Offset? _destPx;
+
+  Future<void> _syncLabelPositions() async {
+    final controller = _controller;
+    if (controller == null) return;
+    final origin = widget.origin;
+    final dest = widget.destination;
+    try {
+      final o = origin == null
+          ? null
+          : await controller
+              .getScreenCoordinate(im.LatLng(origin.lat, origin.lng));
+      final d = dest == null
+          ? null
+          : await controller.getScreenCoordinate(im.LatLng(dest.lat, dest.lng));
+      if (!mounted) return;
+      setState(() {
+        _originPx = o == null ? null : Offset(o.x.toDouble(), o.y.toDouble());
+        _destPx = d == null ? null : Offset(d.x.toDouble(), d.y.toDouble());
+      });
+    } catch (_) {
+      // الخريطة قد تكون بين حالتين — البطاقات تُخفى بدل أن تُرسم في مكان خطأ.
+      if (mounted) setState(() => _originPx = _destPx = null);
+    }
+  }
+
+  /// بطاقة اسم فوق نقطة، مرفوعة عن الدائرة كي لا تغطّيها.
+  Widget _label(Offset? px, String text, Color color) {
+    if (px == null || text.isEmpty) return const SizedBox.shrink();
+    return Positioned(
+      left: px.dx - 100,
+      top: px.dy - 46,
+      width: 200,
+      child: Center(child: MapLabel(text: text, color: color)),
+    );
+  }
+
  @override
  Widget build(BuildContext context) {
    final colors = context.tripzColors;
    final isDark = Theme.of(context).brightness == Brightness.dark;

-    return im.IntaleqMap(
+    return Stack(children: [
+      im.IntaleqMap(
      apiKey: AppConfig.mapApiKey,
      initialCameraPosition: im.CameraPosition(
        target: widget.origin == null
@@ -132,32 +182,51 @@ class _RideMapState extends State<RideMap> {
      compassEnabled: false,
      zoomControlsEnabled: false,
      onMapCreated: _onCreated,
+      onStyleLoaded: () {
+        if (mounted && !_isStyleLoaded) {
+          setState(() => _isStyleLoaded = true);
+        }
+      },
      onCameraMove: (position) => _center = position.target,
      onCameraIdle: () {
        final c = _center;
-        if (c != null) widget.onCameraIdle?.call(GeoPoint(c.latitude, c.longitude));
+        if (c != null) {
+          widget.onCameraIdle?.call(GeoPoint(c.latitude, c.longitude));
+        }
+        _syncLabelPositions();
      },
      // العلامات دوائر ملوّنة تطابق ألوان الخط الزمني في الورقة السفلية،
      // فيربط المستخدم بين «من» و«إلى» وما يراه على الخريطة بلا شرح.
-      circles: {
-        if (widget.origin != null)
-          _dot('origin', widget.origin!, colors.mapOrigin),
-        if (widget.destination != null)
-          _dot('destination', widget.destination!, colors.mapDestination),
-      },
-      markers: {
-        if (widget.driver != null) _marker('driver', widget.driver!),
-      },
-      polylines: {
-        if (widget.route != null && widget.route!.encodedPoints.isNotEmpty)
-          im.Polyline(
-            polylineId: const im.PolylineId('route'),
-            points: im.PolylineUtils.decode(widget.route!.encodedPoints),
-            color: colors.mapRoute,
-            width: 5,
-          ),
-      },
-    );
+      circles: _isStyleLoaded
+          ? {
+              if (widget.origin != null)
+                _dot('origin', widget.origin!, colors.mapOrigin),
+              if (widget.destination != null)
+                _dot('destination', widget.destination!, colors.mapDestination),
+            }
+          : const {},
+      markers: _isStyleLoaded
+          ? {
+              if (widget.driver != null) _marker('driver', widget.driver!),
+            }
+          : const {},
+      polylines: _isStyleLoaded
+          ? {
+              if (widget.route != null && widget.route!.encodedPoints.isNotEmpty)
+                im.Polyline(
+                  polylineId: const im.PolylineId('route'),
+                  points: im.PolylineUtils.decode(widget.route!.encodedPoints),
+                  color: colors.mapRoute,
+                  width: 5,
+                ),
+            }
+          : const {},
+      ),
+      // أسماء النقطتين فوق الخريطة (نمط سيرو) — طبقة فلاتر لا marker، كي
+      // نرث الخط والاتجاه والثيم.
+      _label(_originPx, widget.originLabel, colors.mapOrigin),
+      _label(_destPx, widget.destinationLabel, colors.mapDestination),
+    ]);
  }

  im.Marker _marker(String id, GeoPoint p) => im.Marker(
@@ -28,16 +28,30 @@ class WalletCubit extends Cubit<WalletState> {
    }
  }

-  Future<void> topup(num amount) async {
+  /// شحن حقيقي. **لا يزيد الرصيد فوراً** إلا في نمط الدفع الفوري — الفاتورة
+  /// تُعرض للمستخدم ويحوّل، والتسوية تصل لاحقاً عبر مطابقة رسالة المزوّد.
+  Future<void> topup({required num amount, required String provider}) async {
    if (amount <= 0) return;
    emit(state.copyWith(action: WalletAction.toppingUp));
    try {
-      await _repo.topup(amount);
+      final result = await _repo.charge(
+        amount: amount,
+        provider: provider,
+        currency: state.currency,
+      );
      if (isClosed) return;
-      emit(state.copyWith(action: WalletAction.topupDone));
-      // الرصيد يُعاد جلبه من الخادم لا يُحسب محلياً: المال لا يُقدَّر
-      // في التطبيق (docs/23 §0.2).
-      await load();
+
+      if (result.isInstant) {
+        emit(state.copyWith(action: WalletAction.topupDone));
+        // الرصيد يُعاد جلبه من الخادم لا يُحسب محلياً: المال لا يُقدَّر
+        // في التطبيق (docs/23 §0.2).
+        await load();
+        return;
+      }
+      emit(state.copyWith(
+        action: WalletAction.invoiceReady,
+        invoice: result,
+      ));
    } on ApiException {
      if (!isClosed) emit(state.copyWith(action: WalletAction.topupFailed));
    }
@@ -3,8 +3,9 @@ import 'package:equatable/equatable.dart';
 import '../../../core/ui/view_status.dart';
 import '../data/models/wallet.dart';
 import '../data/models/wallet_txn.dart';
+import '../data/wallet_repository.dart';

-enum WalletAction { none, toppingUp, topupFailed, topupDone }
+enum WalletAction { none, toppingUp, topupFailed, topupDone, invoiceReady }

 class WalletState extends Equatable {
  const WalletState({
@@ -12,6 +13,7 @@ class WalletState extends Equatable {
    this.wallet,
    this.transactions = const [],
    this.action = WalletAction.none,
+    this.invoice,
    this.error,
  });

@@ -23,6 +25,9 @@ class WalletState extends Equatable {
  /// المعروض خلفه.
  final WalletAction action;

+  /// فاتورة تحويل بانتظار الدفع — تُعرض للمستخدم بمرجعها وحساب الاستلام.
+  final TopupResult? invoice;
+
  final String? error;

  String get currency => wallet?.currency ?? '';
@@ -32,6 +37,7 @@ class WalletState extends Equatable {
    Wallet? wallet,
    List<WalletTxn>? transactions,
    WalletAction? action,
+    TopupResult? invoice,
    String? error,
  }) {
    return WalletState(
@@ -39,10 +45,12 @@ class WalletState extends Equatable {
      wallet: wallet ?? this.wallet,
      transactions: transactions ?? this.transactions,
      action: action ?? this.action,
+      invoice: invoice ?? this.invoice,
      error: error ?? this.error,
    );
  }

  @override
-  List<Object?> get props => [status, wallet, transactions, action, error];
+  List<Object?> get props =>
+      [status, wallet, transactions, action, invoice?.reference, error];
 }
@@ -2,6 +2,33 @@ import '../../../core/api/api_client.dart';
 import 'models/wallet.dart';
 import 'models/wallet_txn.dart';

+/// نتيجة الشحن — ثلاثة أنماط: فوري · فاتورة · إعادة توجيه.
+class TopupResult {
+  const TopupResult({
+    required this.status,
+    this.reference,
+    this.transferTarget,
+    this.instructions,
+    this.redirectUrl,
+  });
+
+  final String status;
+
+  /// مرجع الفاتورة الذي يذكره المستخدم عند التحويل — به تُطابَق رسالة
+  /// المزوّد لاحقاً.
+  final String? reference;
+
+  /// حساب/رقم استلام المستأجر عند هذا المزوّد.
+  final String? transferTarget;
+
+  final String? instructions;
+  final String? redirectUrl;
+
+  bool get isInvoice => reference != null;
+  bool get isInstant => status == 'success';
+  bool get isRedirect => redirectUrl != null;
+}
+
 class WalletRepository {
  WalletRepository(this._api);

@@ -20,8 +47,51 @@ class WalletRepository {
        .toList(growable: false);
  }

-  Future<void> topup(num amount) =>
-      _api.post<Map<String, dynamic>>('/wallet/topup', body: {'amount': amount});
+  /// شحن حقيقي عبر `POST /payments/charge`.
+  ///
+  /// **لا يزيد الرصيد فوراً.** كليك وشام كاش وزين كاش تعمل بنمط الفاتورة:
+  /// الخادم يولّد مرجعاً ويعرض حساب الاستلام، ثم يحوّل المستخدم، والتسوية
+  /// تصل لاحقاً عبر مطابقة رسالة المزوّد. هذا **نفس مسار سيرو** حرفياً.
+  /// PayMob وحده يرجّع رابط إعادة توجيه، و`cash` وحده فوري.
+  /// شحن **الرصيد التشغيلي** للسائق — نيّة مختلفة عن شحن المحفظة:
+  /// `credit_topup` إيرادٌ يُقيَّد على المستأجر، والخادم يرفضها من غير
+  /// السائق (`payments.service.ts:67`).
+  Future<TopupResult> chargeCredit({
+    required num amount,
+    required String provider,
+    required String currency,
+  }) =>
+      charge(
+        amount: amount,
+        provider: provider,
+        currency: currency,
+        purpose: 'credit_topup',
+      );
+
+  Future<TopupResult> charge({
+    required num amount,
+    required String provider,
+    required String currency,
+    String purpose = 'topup',
+  }) async {
+    final res = await _api.post<Map<String, dynamic>>(
+      '/payments/charge',
+      body: {
+        'amount': amount,
+        'provider': provider,
+        'purpose': purpose,
+        'currency': currency,
+      },
+    );
+    final payment = res['payment'] as Map<String, dynamic>?;
+    return TopupResult(
+      status: payment?['status'] as String? ?? 'pending',
+      reference: res['reference'] as String?,
+      transferTarget: res['transferTarget'] as String?,
+      instructions: res['instructions'] as String?,
+      redirectUrl: res['redirectUrl'] as String?,
+    );
+  }

  // ── السحب: خطوتان لا خطوة ────────────────────────────────────────────
  // `request` يرسل رمزاً **بلا خصم**، و`confirm` هو ما يخصم ويحجز
@@ -0,0 +1,105 @@
+import 'package:flutter/material.dart';
+import 'package:flutter/services.dart';
+
+import '../../../../core/design/tokens.dart';
+import '../../../../core/design/tripz_colors.dart';
+import '../../../../core/design/typography.dart';
+import '../../../../core/l10n/l10n.dart';
+import '../../../../core/ui/status_banner.dart';
+import '../../data/wallet_repository.dart';
+
+/// فاتورة التحويل — نمط سيرو حرفياً (`create_cliq_invoice.php`).
+///
+/// المستخدم يحوّل بنفسه إلى حساب المشغّل ويذكر المرجع، والتسوية تصل لاحقاً
+/// بمطابقة رسالة المزوّد. **الرصيد لا يزيد الآن** — وهذا يُقال صراحةً بدل
+/// أن ينتظر المستخدم رقماً لن يتغيّر.
+class InvoiceSheet extends StatelessWidget {
+  const InvoiceSheet({super.key, required this.invoice});
+
+  final TopupResult invoice;
+
+  @override
+  Widget build(BuildContext context) {
+    final l10n = context.l10n;
+    final target = invoice.transferTarget;
+
+    return Column(
+      mainAxisSize: MainAxisSize.min,
+      crossAxisAlignment: CrossAxisAlignment.stretch,
+      children: [
+        Text(l10n.walletInvoiceTitle, style: context.texts.titleLarge),
+        const SizedBox(height: Space.md),
+
+        // حساب الاستلام قد لا يكون مضبوطاً للمستأجر — يُقال بوضوح بدل تعليمات
+        // ناقصة تُرسل المال إلى لا مكان.
+        if (target == null || target.isEmpty)
+          StatusBanner(message: l10n.walletNoTarget, tone: BannerTone.danger)
+        else
+          _CopyRow(label: l10n.walletInvoiceTarget, value: target),
+
+        const SizedBox(height: Space.sm),
+        if (invoice.reference != null)
+          _CopyRow(label: l10n.walletInvoiceRef, value: invoice.reference!),
+
+        const SizedBox(height: Space.md),
+        StatusBanner(
+          message: l10n.walletInvoicePending,
+          tone: BannerTone.info,
+          icon: Icons.schedule_rounded,
+        ),
+      ],
+    );
+  }
+}
+
+/// قيمة مع زرّ نسخ — الرقم يُحوَّل يدوياً في تطبيق بنك آخر، فالنسخ ليس رفاهية.
+class _CopyRow extends StatelessWidget {
+  const _CopyRow({required this.label, required this.value});
+
+  final String label;
+  final String value;
+
+  @override
+  Widget build(BuildContext context) {
+    return Container(
+      padding: Space.page,
+      decoration: BoxDecoration(
+        color: context.colors.surfaceContainerHighest,
+        borderRadius: Radii.card_,
+      ),
+      child: Row(
+        children: [
+          Expanded(
+            child: Column(
+              crossAxisAlignment: CrossAxisAlignment.start,
+              children: [
+                Text(
+                  label,
+                  style: context.texts.labelSmall?.copyWith(
+                    color: context.colors.onSurfaceVariant,
+                  ),
+                ),
+                const SizedBox(height: Space.xxs),
+                SelectableText(
+                  value,
+                  textDirection: TextDirection.ltr,
+                  style: numericStyle(context.texts.titleMedium),
+                ),
+              ],
+            ),
+          ),
+          IconButton(
+            icon: const Icon(Icons.copy_rounded),
+            tooltip: context.l10n.walletCopy,
+            onPressed: () {
+              Clipboard.setData(ClipboardData(text: value));
+              ScaffoldMessenger.of(context).showSnackBar(
+                SnackBar(content: Text(context.l10n.walletCopied)),
+              );
+            },
+          ),
+        ],
+      ),
+    );
+  }
+}