rafalpietraszewicz
commented
5 months ago Czy przesłana przeze mnie poprawka rózni się czymś logicznie (pod względem wyników) od Twojej?
Logicznie nie różni się. Jednak kiedy mamy już zawieszone zobowiązania u klienta, a następnie dodajemy 'zawieś wszystkie zobowiązania', to już dla tych dodatkowo zawieszonych źle się wylicza zawieszenie. W tym commicie https://github.com/chilek/lms/pull/2453 połączyłem Twoje zmiany z resztą niezbędnych zmian.
Puściłem innę wersję poprawki, która rozwiązuje pierwotny problem ukazany poniższymi zapytaniami SQL (w pgsql);
lms-stable=> SELECT 20 / 100 AS suspension_percentage; suspension_percentage ----------------------- 0 (1 row) lms-stable=> SELECT 20.0 / 100 AS suspension_percentage; suspension_percentage ------------------------ 0.20000000000000000000 (1 row) lms-stable=>https://github.com/chilek/lms-plus/commit/8013b2f4e4d73dd56796ef3b71bb233857512027
Czy przesłana przeze mnie poprawka rózni się czymś logicznie (pod względem wyników) od Twojej?