Skip to content

Commit 145fdf7

Browse files
committed
Fix expense history sorting to properly handle Indian locale date format
1 parent 0abac74 commit 145fdf7

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

www/js/modules/miscellaneous.js

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -197,7 +197,11 @@ export class ExpensesManager {
197197
const container = document.getElementById('businessExpenseHistoryList');
198198
const businessExpenses = AppState.expensesHistory
199199
.filter(p => p.category === 'business' && p.id)
200-
.sort((a, b) => new Date(b.date) - new Date(a.date));
200+
.sort((a, b) => {
201+
const dateA = Helpers.parseDate(a.date);
202+
const dateB = Helpers.parseDate(b.date);
203+
return (dateB?.getTime() || 0) - (dateA?.getTime() || 0);
204+
});
201205

202206
if (businessExpenses.length === 0) {
203207
container.innerHTML = '<p style="text-align: center; color: #888; margin-top: 40px;">No business expenses recorded yet</p>';
@@ -228,7 +232,11 @@ export class ExpensesManager {
228232
const container = document.getElementById('personalExpenseHistoryList');
229233
const personalExpenses = AppState.expensesHistory
230234
.filter(p => p.category === 'personal' && p.id)
231-
.sort((a, b) => new Date(b.date) - new Date(a.date));
235+
.sort((a, b) => {
236+
const dateA = Helpers.parseDate(a.date);
237+
const dateB = Helpers.parseDate(b.date);
238+
return (dateB?.getTime() || 0) - (dateA?.getTime() || 0);
239+
});
232240

233241
if (personalExpenses.length === 0) {
234242
container.innerHTML = '<p style="text-align: center; color: #888; margin-top: 40px;">No personal expenses recorded yet</p>';

0 commit comments

Comments
 (0)