Move the sorting of the tagList into the LogicManager

AY2425S1-CS2103T-F14a-4 · Oct 21, 2024 · 98f50e1 · 98f50e1
1 parent 8d1e6d0
commit 98f50e1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/main/java/seedu/address/logic/Logic.java b/src/main/java/seedu/address/logic/Logic.java
@@ -34,7 +34,7 @@ public interface Logic {
     /** Returns an unmodifiable view of the filtered list of persons */
     ObservableList<Person> getFilteredPersonList();
 
-    /** Returns a list of currently defined tags */
+    /** Returns a list of currently defined tags alphabetically sorted */
     ObservableList<Tag> getListOfCurrentTags();
 
     /**

diff --git a/src/main/java/seedu/address/logic/LogicManager.java b/src/main/java/seedu/address/logic/LogicManager.java
@@ -74,7 +74,7 @@ public ObservableList<Person> getFilteredPersonList() {
 
     @Override
     public ObservableList<Tag> getListOfCurrentTags() {
-        return model.getListOfCurrentTags();
+        return model.getListOfCurrentTags().sorted();
     }
 
     @Override

diff --git a/src/main/java/seedu/address/ui/TagListPanel.java b/src/main/java/seedu/address/ui/TagListPanel.java
@@ -28,7 +28,7 @@ public class TagListPanel extends UiPart<Region> {
      */
     public TagListPanel(ObservableList<Tag> tagList) {
         super(FXML);
-        tagListView.setItems(FXCollections.observableArrayList(tagList.sorted()));
+        tagListView.setItems(FXCollections.observableArrayList(tagList));
         tagListView.setCellFactory(listView -> new TagListViewCell());
     }