Skip to content
Snippets Groups Projects
Commit faed44ba authored by Harry Linrui Xu's avatar Harry Linrui Xu
Browse files

Updated faulty pie chart updates

parent 967f7281
No related branches found
No related tags found
4 merge requests!43Merging frontend-testing into master,!38"Made progressbar dynamic in accordance to spending. Added balance field....,!37Made the sub progress bars respond to changes in expense,!32Added input validation to add dialog boxes.
Pipeline #214577 passed
......@@ -401,6 +401,9 @@ public class IncomeExpenseController implements FinanceController {
dialog.setTitle(dialogTitle);
// Show the Dialog and wait for the user to close it
dialog.showAndWait();
this.incomePieChart.setData(createIncomePieChart());
}
@FXML
......@@ -429,7 +432,7 @@ public class IncomeExpenseController implements FinanceController {
// Show the Dialog and wait for the user to close it
dialog.showAndWait();
this.expensePieChart.setData(createIncomePieChart());
this.expensePieChart.setData(createExpensePieChart());
}
@FXML
......@@ -447,7 +450,7 @@ public class IncomeExpenseController implements FinanceController {
if (isConfirmed.isPresent() && isConfirmed.get() == ButtonType.OK) {
expenseRegister.removeItem(chosenExpense);
}
this.incomePieChart.setData(createIncomePieChart());
this.expensePieChart.setData(createExpensePieChart());
}
/**
* Returns an optional, which is a popup alert box, asking for confirmation for deleting an
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment