search

BillTechPL / lms-billtech-plugin

Other
6 stars 7 forks source link

poprawki na duplikaty wplat #83

Closed interduo closed 1 year ago

interduo commented 2 years ago

dzięki temu wtyczka nie będzie smrodzić w logach błędami SQL

smiecielica-billtech commented 2 years ago

Dzięki! Mamy to na liście todo. Potrzebna jest jeszcze migracja doc/upgradedb/ z datą, zaktualizowanie w pliku Billtech.php oraz poprawka zarówno dla mysql jak i postgresa. Zrobimy to w przyszłym tygodniu.

P.S. Należałoby także zbadać przyczynę dlaczego dodają się dwie płatności. W pliku BilltechPaymentsUpdater.php od linijki 113 sprawdzane jest czy taka płatność już istnieje. Proszę sprawdzić w tabeli czy zduplikowane płatności posiadają 1:1 te same wartości kolumn: 'value', 'type','userid', 'customerid', 'comment', 'time'?

interduo commented 2 years ago

@smiecielica z mojej strony to jest łatka wystarczająca, żebym zapomniał o problemie. Pamiętaj, że MySQL jeśli go wspieracie we wtyczce ma inny format zapytania.

Przyjmuję, że schemat aktualizacji i zrobienie wariantu dla MySQL leży po waszej stronie.

Nie sprawdzę tego o co prosisz z prostego powodu - płatność nie trafiła w ogóle do mnie do DB ponieważ zadziałał tutaj CONSTRAINT UNIQUE, który dodałem ręcznie na bazie. Natomiast wcześniejsze duplikaty wam wysyłałem i uznaliście to za nieważny temat. A z racji tego że problem dotyczy większej ilości osób, które używają wtyczki postanowiłem się podzielić.

interduo commented 2 years ago

@smiecielica testuję to już z tydzień - błędy SQL znikneły :) wpuścicie to do brancha? wrzuciłbym to na jeszcze jedną instancję u znajomego

smiecielica-billtech commented 2 years ago

@interduo Tak, tez przetestowałem i działa ok. Jutro dorzucę tutaj commity z postgresem i do startowego pliku bazy, poproszę kogoś o review i wtedy wejdzie na mastera.

interduo commented 2 years ago

@smiecielica wypuścicie to?

interduo commented 1 year ago

czy w wersji 1.0 taka sytuacja może mieć miejsce? jeśli nie proszę o zamknięcie PR