austnv/remnawave-minishop
1
0
Fork 0
Code Issues 1 Pull Requests Activity

Remove outdated comments and fallback logic in payment method history handler

Browse Source 
- Eliminated unnecessary comments regarding auto-renewal payment methods to enhance code clarity.
- Removed fallback logic for including auto-renew entries, streamlining the filtering process for user payments.
This commit is contained in:
machka-pasla
2025-09-04 19:00:03 +03:00
parent 2b855ed11b
commit 868c48ccb9
1 changed files with 0 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
bot/handlers/user/subscription.py
-4
View File
@@ -942,7 +942,6 @@ async def payment_method_history(callback: types.CallbackQuery, settings: Settin
# Filter to rows we can confidently associate with the selected method # Filter to rows we can confidently associate with the selected method
# Heuristics: # Heuristics:
# 1) Payments with yookassa_payment_id -> fetch payment info and compare payment_method.id # 1) Payments with yookassa_payment_id -> fetch payment info and compare payment_method.id
# 2) For auto-renew description, it always uses default saved method; keep those too
filtered: List[Payment] = [] filtered: List[Payment] = []
for p in user_payments: for p in user_payments:
if p.provider != 'yookassa': if p.provider != 'yookassa':
@@ -956,9 +955,6 @@ async def payment_method_history(callback: types.CallbackQuery, settings: Settin
continue continue
except Exception: except Exception:
pass pass
# Fallback: auto-renew entries initiated via default method; include them
if (p.description or "").lower().startswith("auto-renewal"):
filtered.append(p)
user_payments = filtered user_payments = filtered
if not user_payments: if not user_payments:
# Try to get pm_id from context to go one step back # Try to get pm_id from context to go one step back