fix(shopinbit): show car research request retry errors as a dialog
What the developer wrote
Authored by sneurlax
fix(shopinbit): show car research request retry errors as a dialog
What changed, and why it matters
This commit changes how an error message is shown to the user when a car research payment retry fails. It swaps a temporary floating notification bar for a popup dialog. There is no security issue here—this is purely a user-interface improvement.
No security action needed. Treat as a normal UI/UX fix.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch replaces two calls to showFloatingFlushBar with showDialog<void> using StackOkDialog in shopinbit_car_research_payment_view.dart. The change affects only UI presentation of retry-failure errors and exceptions. No logic, authentication, cryptography, network handling, or data validation is modified.
Changed components
lib/pages/shopinbit/shopinbit_car_research_payment_view.dartInspect captured patch +15 / −9
diff --git a/lib/pages/shopinbit/shopinbit_car_research_payment_view.dart b/lib/pages/shopinbit/shopinbit_car_research_payment_view.dart
index 349b045..fc24f8c 100644
--- a/lib/pages/shopinbit/shopinbit_car_research_payment_view.dart
+++ b/lib/pages/shopinbit/shopinbit_car_research_payment_view.dart
@@ -566,11 +566,14 @@ class _ShopInBitCarResearchPaymentViewState
if (reqResp.hasError || reqResp.value == null) {
if (mounted) {
setState(() => _flowState = _PaymentFlowState.error);
- unawaited(
- showFloatingFlushBar(
- type: FlushBarType.warning,
- message: reqResp.exception?.message ?? "Retry failed",
- context: context,
+ await showDialog<void>(
+ context: context,
+ useRootNavigator: Util.isDesktop,
+ builder: (context) => StackOkDialog(
+ title: "Retry failed",
+ maxWidth: Util.isDesktop ? 500 : null,
+ message: reqResp.exception?.message,
+ desktopPopRootNavigator: Util.isDesktop,
),
);
}
@@ -608,11 +611,14 @@ class _ShopInBitCarResearchPaymentViewState
} catch (e) {
if (mounted) {
setState(() => _flowState = _PaymentFlowState.error);
- unawaited(
- showFloatingFlushBar(
- type: FlushBarType.warning,
+ await showDialog<void>(
+ context: context,
+ useRootNavigator: Util.isDesktop,
+ builder: (context) => StackOkDialog(
+ title: "Retry failed",
+ maxWidth: Util.isDesktop ? 500 : null,
message: e.toString(),
- context: context,
+ desktopPopRootNavigator: Util.isDesktop,
),
);
}
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.