nightflyza / Ubilling

Ubilling is opensource ISP billing system based on stargazer
https://ubilling.net.ua
GNU General Public License v2.0
134 stars 105 forks source link

Fix remoteapi freezedayscharge #850

Closed pautiina closed 1 month ago

pautiina commented 1 month ago

Fix commit cdc2a3d for remoteapi module freezedayscharge

nightflyza commented 1 month ago

Бачиш наскільки простіше, якщо спочатку подумати а потім зробити? :)

EyVg59uWgAIjRc5

bobr-kun commented 1 month ago

@pautiina ти все таки можеш мені розтлумачити, шо саме зламалося? і до чого тут саме cdc2a3d і реліз 1.4.6? бо все виглядає так, шо оті 2 рядки $tabUsers->orderBy('id'); $tabFrozenChargeDays->orderBy('id');

не працювали в принципі ніяк з самого релізу 1.0.5, коли той рефакторинг робився. бо я тоді чогось собі вирішив :sweat_smile:, шо orderBy() метод має значення параметру order за замовчуванням. але він його не мав ніколи. тому, все, що цей рядок робив - це звав flushOrder(), що, в принципі, не мало ніяких негативних наслідків. ну, і позитивних не мало.

чи яшось не так зрозумів?