4/5 - Provider service duplikált _award_dynamic_points() logika eltávolítása #368

New Issue

kincses · 2026-06-30T19:33:23+02:00

kincses commented

2026-06-30 19:33:23 +02:00

Mérföldkő: Epic 8 - Gamification 2.0
Cél: A provider_service.py _award_dynamic_points() fuggvenyeben talalható duplikalt point_rules olvasasi logika eltavolitasa, helyette az award_points() hivasa a megfelelo action_key-vel.

🔗 Függőségek (Dependencies)

Bemenet (Mikre támaszkodik): #365 (award_points action_key tamogatas), #366 (uj action_key-ek)
Kimenet (Mik támaszkodnak rá): provider_service.py hivoi, social_service.py

📝 Elemzés

A provider_service.py:60-135-ben talalhato _award_dynamic_points() fuggveny sajat maga olvassa ki a point_rules tablat, mielott meghivna az award_points()-t. Ez a logika MUKODIK, de DUPLIKALT, mert a process_activity() mar tartalmazza ugyanezt a logikat.

Jelenlegi folyamat:
provider_hivas -> _award_dynamic_points() -> point_rules olvasas -> award_points(hardcoded_amount) -> process_activity()

Javitott folyamat:
provider_hivas -> award_points(action_key='ADD_NEW_PROVIDER') -> process_activity() -> _get_point_rule()

A javitas leegyszerusiti a kodot, megszunteti a duplikaciot, es biztosítja, hogy a pontok egy helyen (process_activity) szamolodjanak ki.

Módosítandó fájl: backend/app/services/provider_service.py

**Mérföldkő:** Epic 8 - Gamification 2.0 **Cél:** A provider_service.py _award_dynamic_points() fuggvenyeben talalható duplikalt point_rules olvasasi logika eltavolitasa, helyette az award_points() hivasa a megfelelo action_key-vel. ### 🔗 Függőségek (Dependencies) - **Bemenet (Mikre támaszkodik):** #365 (award_points action_key tamogatas), #366 (uj action_key-ek) - **Kimenet (Mik támaszkodnak rá):** provider_service.py hivoi, social_service.py ### 📝 Elemzés A provider_service.py:60-135-ben talalhato _award_dynamic_points() fuggveny sajat maga olvassa ki a point_rules tablat, mielott meghivna az award_points()-t. Ez a logika MUKODIK, de DUPLIKALT, mert a process_activity() mar tartalmazza ugyanezt a logikat. Jelenlegi folyamat: provider_hivas -> _award_dynamic_points() -> point_rules olvasas -> award_points(hardcoded_amount) -> process_activity() Javitott folyamat: provider_hivas -> award_points(action_key='ADD_NEW_PROVIDER') -> process_activity() -> _get_point_rule() A javitas leegyszerusiti a kodot, megszunteti a duplikaciot, es biztosítja, hogy a pontok egy helyen (process_activity) szamolodjanak ki. Módosítandó fájl: backend/app/services/provider_service.py

kincses added this to the Epic 8 Gamification 2.0, Verseny és Önvéde milestone 2026-06-30 19:33:23 +02:00

kincses added the Status: To Do Scope: Backend Role: Admin Type: Refactor labels 2026-06-30 19:33:24 +02:00

kincses added Status: Done and removed Status: To Do labels 2026-06-30 19:44:40 +02:00

kincses closed this issue

2026-06-30 19:44:40 +02:00

kincses commented

2026-06-30 19:44:40 +02:00

✅ Munka befejezve: 2026-06-30 19:44:40

Technikai Összefoglaló:
Refactored _award_provider_points() in provider_service.py: removed duplicate PointRule reading logic, now delegates to gamification_service.award_points() with action_key parameter. Only keeps providers_added_count stats increment.

⏱️ Az idő rögzítve.

✅ **Munka befejezve:** 2026-06-30 19:44:40 **Technikai Összefoglaló:** Refactored _award_provider_points() in provider_service.py: removed duplicate PointRule reading logic, now delegates to gamification_service.award_points() with action_key parameter. Only keeps providers_added_count stats increment. ⏱️ *Az idő rögzítve.*

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kincses/service-finder#368