Skip to content
Snippets Groups Projects
Commit 4bad8ad7 authored by Matej Dipčár's avatar Matej Dipčár
Browse files

Fix `TaskModel#updateFiltered` to not add tasks as new

parent 935154df
No related branches found
No related tags found
2 merge requests!52Final project MR,!46UI <--> DB refactor
Pipeline #
...@@ -5,6 +5,7 @@ import cz.muni.fi.pv168.project.db.DataAccessException; ...@@ -5,6 +5,7 @@ import cz.muni.fi.pv168.project.db.DataAccessException;
import cz.muni.fi.pv168.project.db.interfaces.DataAccessObject; import cz.muni.fi.pv168.project.db.interfaces.DataAccessObject;
import cz.muni.fi.pv168.project.ui.dialog.error.ErrorDialog; import cz.muni.fi.pv168.project.ui.dialog.error.ErrorDialog;
import java.util.ArrayList;
import java.util.function.Predicate; import java.util.function.Predicate;
public class TaskModel extends UpdatableDataModel<Task> { public class TaskModel extends UpdatableDataModel<Task> {
...@@ -16,11 +17,13 @@ public class TaskModel extends UpdatableDataModel<Task> { ...@@ -16,11 +17,13 @@ public class TaskModel extends UpdatableDataModel<Task> {
public void updateFiltered(Predicate<Task> predicate) { public void updateFiltered(Predicate<Task> predicate) {
clear(); clear();
try { try {
var tasks = new ArrayList<Task>();
for (Task element : dataAccessObject.getAll()) { for (Task element : dataAccessObject.getAll()) {
if (predicate.test(element)) { if (predicate.test(element)) {
super.addElement(element); tasks.add(element);
} }
} }
addAll(tasks);
} catch (DataAccessException e) { } catch (DataAccessException e) {
e.printStackTrace(); e.printStackTrace();
ErrorDialog.show("Error when querying all elements.", e); ErrorDialog.show("Error when querying all elements.", e);
......
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