src/main/java/ch/zhaw/gartenverwaltung/MainFXMLController.java

@@ -10,16 +10,10 @@ import javafx.scene.layout.AnchorPane;
 
 import java.io.IOException;
 import java.net.URL;
-import java.util.HashMap;
-import java.util.Map;
 import java.util.Objects;
 import java.util.ResourceBundle;
 
 public class MainFXMLController implements Initializable {
-    /**
-     * Caching the panes
-     */
-    private final Map<String, AnchorPane> panes = new HashMap<>();
 
     @FXML
     private Button home_button;
@@ -68,22 +62,15 @@ public class MainFXMLController implements Initializable {
      * @throws IOException exception when file does not exist
      */
     public void loadPane(String fxmlFile) throws IOException {
-
-        AnchorPane anchorPane = panes.get(fxmlFile);
-        if (anchorPane == null) {
+        //ToDo HGrow and VGrow of new node
+        Node node;
         FXMLLoader loader = new FXMLLoader(Objects.requireNonNull(HelloApplication.class.getResource(fxmlFile)));
-            anchorPane = loader.load();
-            panes.put(fxmlFile, anchorPane);
-
+        node = (Node)loader.load();
         if(fxmlFile.equals("MyPlants.fxml")) {
             MyPlantsController myPlantsController = loader.getController();
             myPlantsController.getMainController(this);
         }
-        }
-        mainPane.getChildren().setAll(anchorPane);
-        anchorPane.prefWidthProperty().bind(mainPane.widthProperty());
-        anchorPane.prefHeightProperty().bind(mainPane.heightProperty());
-
+        mainPane.getChildren().setAll(node);
     }
 
     private void styleChangeButton(Button button) {

src/main/java/ch/zhaw/gartenverwaltung/PlantsController.java

@@ -4,26 +4,19 @@ import ch.zhaw.gartenverwaltung.io.HardinessZoneNotSetException;
 import ch.zhaw.gartenverwaltung.plantList.PlantListModel;
 import ch.zhaw.gartenverwaltung.types.HardinessZone;
 import ch.zhaw.gartenverwaltung.types.Plant;
-import ch.zhaw.gartenverwaltung.types.Seasons;
-import javafx.application.Platform;
-import javafx.beans.property.ListProperty;
-import javafx.beans.property.SimpleListProperty;
 import javafx.beans.value.ChangeListener;
 import javafx.beans.value.ObservableValue;
-import javafx.collections.FXCollections;
 import javafx.event.ActionEvent;
 import javafx.fxml.FXML;
 import javafx.fxml.Initializable;
-import javafx.geometry.Insets;
-import javafx.geometry.Bounds;
 import javafx.scene.control.*;
-import javafx.scene.image.Image;
 import javafx.scene.image.ImageView;
-import javafx.scene.input.KeyEvent;
+import javafx.scene.input.InputMethodEvent;
 import javafx.scene.layout.VBox;
 
 import java.io.IOException;
 import java.net.URL;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.ResourceBundle;
 
@@ -32,12 +25,8 @@ public class PlantsController implements Initializable {
     private Plant selectedPlant = null;
     private final HardinessZone DEFAULT_HARDINESS_ZONE = HardinessZone.ZONE_8A;
 
-    // TODO: move to model
-
-    private final ListProperty<Plant> plantListProperty = new SimpleListProperty<>(FXCollections.observableArrayList());
-
     @FXML
-    private VBox seasons;
+    private CheckBox autum_filter;
 
     @FXML
     private VBox climate_zones;
@@ -57,45 +46,79 @@ public class PlantsController implements Initializable {
     @FXML
     private TextField search_plants;
 
-    /**
-     * saves the current selected plant in new JSON database
-     * @param event event
-     */
+    @FXML
+    private CheckBox sommer_filter;
+
+    @FXML
+    private CheckBox spring_filter;
+
+    @FXML
+    private CheckBox winter_filter;
+
+    @FXML
+    void filterAutum(ActionEvent event) {
+        //ToDo
+
+    }
+
+    @FXML
+    void filterSommer(ActionEvent event) {
+        //ToDo
+
+    }
+
+    @FXML
+    void filterSpring(ActionEvent event) {
+        //ToDo
+
+    }
+
+    @FXML
+    void filterWinter(ActionEvent event) {
+        //ToDo
+
+    }
+
     @FXML
     void saveToMyPlant(ActionEvent event) {
         //ToDo model save selectedPlant to mySelectedPlant(IO)
     }
 
+    @FXML
+    void searchForPlant(InputMethodEvent event) {
+        viewFilteredListBySearch(search_plants.getText());
+    }
+
     /**
-     * fill list view with current hardiness zone
-     * set default values
-     * create filter of season and hardiness zone
-     * create event listener for selected list entry and search by query
      * {@inheritDoc}
      */
     @Override
     public void initialize(URL url, ResourceBundle resourceBundle) {
-        setListCellFactory();
-        fillPlantListWithHardinessZone();
-        list_plants.itemsProperty().bind(plantListProperty);
+        List<Plant> plantList = new LinkedList<>();
+        try {
+            plantList = plantListModel.getPlantList(DEFAULT_HARDINESS_ZONE);
+        } catch (HardinessZoneNotSetException | IOException e) {
+            e.printStackTrace();
+        }
+        fillListViewWithData(plantList);
 
         description_plant.setText("");
         saveToMyPlant_button.setDisable(true);
 
-        createFilterSeasons();
         createFilterHardinessZone();
         lookForSelectedListEntry();
-        try {
-            viewFilteredListBySearch();
-        } catch (HardinessZoneNotSetException | IOException e) {
-            e.printStackTrace();
-        }
     }
 
     /**
-     *  set text of list view to plant name
+     * update the ListView according to the plant list provided
+     * Entry in ListView is plant name
+     * @param list plantList which fill the ListView
      */
-    private void setListCellFactory() {
+    private void fillListViewWithData(List<Plant> list) {
+        clearListView();
+        for (Plant plant : list) {
+            list_plants.getItems().add(plant);
+        }
         list_plants.setCellFactory(param -> new ListCell<Plant>() {
             @Override
             protected void updateItem(Plant plant, boolean empty) {
@@ -110,139 +133,44 @@ public class PlantsController implements Initializable {
         });
     }
 
-    /**
-     * get plant list according to param season and hardiness zone
-     * fill list view with plant list
-     * @param season enum of seasons
-     * @throws HardinessZoneNotSetException throws exception
-     * @throws IOException throws exception
-     */
-    private void viewFilteredListBySeason(Seasons season) throws HardinessZoneNotSetException, IOException {
-        clearListView();
-        plantListProperty.addAll(plantListModel.getFilteredPlantListBySaisonWithoutGrowthPhase(plantListModel.getCurrentZone(), season.getStartDate(), season.getEndDate()));
+    private void viewFilteredListByFilters() {
+        boolean springValue = spring_filter.isSelected();
+        boolean sommerValue = sommer_filter.isSelected();
+        boolean autumValue = autum_filter.isSelected();
+        boolean winterValue = winter_filter.isSelected();
+        //ToDo getFilteredPlantList with (plantListModel.getFilteredPlantList(DEFAULT_HARDINESS_ZONE, <predicate>))
+        //List<Plant> plantList = new LinkedList<>();
+        //fillListViewWithData(plantList);
     }
 
-    /**
-     * get plant list filtered by search plant entry and hardiness zone
-     * fill list view with plant list
-     * @throws HardinessZoneNotSetException throws exception when no hardiness zone is defined
-     * @throws IOException throws exception
-     */
-    private void viewFilteredListBySearch() throws HardinessZoneNotSetException, IOException {
-        search_plants.textProperty().addListener((observable, oldValue, newValue) -> {
-            if (newValue.isEmpty()) {
-                fillPlantListWithHardinessZone();
-            }else {
-                try {
-                    List<Plant> filteredPlants = plantListModel.getFilteredPlantListByString(DEFAULT_HARDINESS_ZONE, newValue);
-                    clearListView();
-                    plantListProperty.addAll(filteredPlants);
-                } catch (HardinessZoneNotSetException | IOException e) {
-                    e.printStackTrace();
-                }
-            }
-        });
+    private void viewFilteredListBySearch(String query) {
+        //ToDo getFilteredPlantList with (plantListModel.getFilteredPlantList(DEFAULT_HARDINESS_ZONE, <predicate>))
+        //List<Plant> plantList = new LinkedList<>();
+        //fillListViewWithData(plantList);
     }
 
-    /**
-     * get plant list of current hardiness zone
-     * fill list view with plant list
-     */
-    private void fillPlantListWithHardinessZone() {
-        try {
-            clearListView();
-            plantListProperty.addAll(plantListModel.getPlantList(plantListModel.getCurrentZone()));
-        } catch (HardinessZoneNotSetException | IOException e) {
-            e.printStackTrace();
-        }
-    }
-
-    /**
-     * creates radio buttons for the hardiness zones defined in enum HardinessZone
-     * defines default value as selected
-     * when selected filter viewList according to hardiness zone
-     */
     private void createFilterHardinessZone() {
-        ToggleGroup hardinessGroup = new ToggleGroup();
-        for (HardinessZone zone : HardinessZone.values()) {
-            RadioButton radioButton = new RadioButton(zone.name());
-            radioButton.setToggleGroup(hardinessGroup);
-            radioButton.setPadding(new Insets(0,0,10,0));
-            if (zone.equals(DEFAULT_HARDINESS_ZONE)) {
-                radioButton.setSelected(true);
-            }
-            radioButton.selectedProperty().addListener(new ChangeListener<Boolean>() {
-                @Override
-                public void changed(ObservableValue<? extends Boolean> observable, Boolean oldValue, Boolean newValue) {
-                    plantListModel.setCurrentZone(zone);
-                    fillPlantListWithHardinessZone();
-                }
-            });
-            climate_zones.getChildren().add(radioButton);
-        }
+        //ToDo create radioList of hardinessZone in VBox climate_zones
     }
 
     /**
-     * creates radio buttons for the seasons defined in enum Seasons
-     * defines default value as selected
-     * when selected filter viewList according to seasons
-     */
-    private void createFilterSeasons() {
-        ToggleGroup seasonGroup = new ToggleGroup();
-        for (Seasons season : Seasons.values()) {
-            RadioButton radioButton = new RadioButton(season.name());
-            radioButton.setToggleGroup(seasonGroup);
-            radioButton.setPadding(new Insets(0,0,10,0));
-            if (season.equals(Seasons.AllSEASONS)) {
-                radioButton.setSelected(true);
-            }
-            radioButton.selectedProperty().addListener(new ChangeListener<Boolean>() {
-                @Override
-                public void changed(ObservableValue<? extends Boolean> observable, Boolean oldValue, Boolean newValue) {
-                    if (season.equals(Seasons.AllSEASONS)) {
-                        fillPlantListWithHardinessZone();
-                    } else {
-                        try {
-                            viewFilteredListBySeason(season);
-                        } catch (HardinessZoneNotSetException | IOException e) {
-                            e.printStackTrace();
-                        }
-                    }
-
-                }
-            });
-            seasons.getChildren().add(radioButton);
-        }
-    }
-
-    /**
-     * observes changes in the selectedProperty of ListView and updates:
-     * the description label
-     * image of the plant
+     * observes changes in the selectedProperty of ListView and updates the description label
      */
     private void lookForSelectedListEntry() {
         list_plants.getSelectionModel().selectedItemProperty().addListener(new ChangeListener<Plant>() {
             @Override
             public void changed(ObservableValue<? extends Plant> observable, Plant oldValue, Plant newValue) {
+                //ToDo
                 if(newValue != null) {
                     selectedPlant = newValue;
                     description_plant.setText(selectedPlant.description());
                     saveToMyPlant_button.setDisable(false);
-                    Image img;
-                    if(selectedPlant.image() != null) {
-                        img = selectedPlant.image();
-                    } else {
-                        img = new Image(String.valueOf(PlantsController.class.getResource("placeholder.png")));
-                    }
-                    img_plant.setImage(img);
-
-
+                    //update img plant
                 } else {
                     selectedPlant = null;
                     description_plant.setText("");
                     saveToMyPlant_button.setDisable(true);
-                    Image img = new Image(String.valueOf(PlantsController.class.getResource("placeholder.png")));
-                    img_plant.setImage(img);
+                    //update img when null placeholder PNG
                 }
             }
         });
@@ -253,6 +181,6 @@ public class PlantsController implements Initializable {
      * clears the ListView of entries
      */
     private void clearListView() {
-        plantListProperty.clear();
+        list_plants.getItems().clear();
     }
 }

src/main/java/ch/zhaw/gartenverwaltung/json/PlantImageDeserializer.java

@@ -6,11 +6,9 @@ import com.fasterxml.jackson.databind.DeserializationContext;
 import com.fasterxml.jackson.databind.JsonDeserializer;
 import javafx.scene.image.Image;
 
-import java.io.File;
 import java.io.FileInputStream;
 import java.io.IOException;
 import java.io.InputStream;
-import java.net.URISyntaxException;
 import java.net.URL;
 
 public class PlantImageDeserializer extends JsonDeserializer<Image> {
@@ -20,9 +18,9 @@ public class PlantImageDeserializer extends JsonDeserializer<Image> {
         Image result = null;
         URL imageUrl = PlantDatabase.class.getResource(String.format("images/%s", parser.getText()));
         if (imageUrl != null) {
-            try (InputStream is = new FileInputStream(new File(imageUrl.toURI()))) {
+            try (InputStream is = new FileInputStream(imageUrl.getFile())) {
                 result = new Image(is);
-            } catch (IllegalArgumentException | URISyntaxException e) {
+            } catch (IllegalArgumentException e) {
                 // TODO: Log
                 e.printStackTrace();
                 System.err.printf("Cannot find Image \"%s\"\n", imageUrl.getFile());

src/main/java/ch/zhaw/gartenverwaltung/plantList/PlantListModel.java

@@ -33,12 +33,12 @@ public class PlantListModel {
         setDefaultZone();
     }
 
-    public PlantListModel(PlantDatabase plantDatabase) {
+    public PlantListModel(PlantDatabase plantDatabase){
         this.plantDatabase = plantDatabase;
         setDefaultZone();
     }
 
-    private void setDefaultZone() {
+    private void setDefaultZone(){
         currentZone = HardinessZone.ZONE_8A; // TODO: get Default Zone from Config
     }
 
@@ -52,7 +52,6 @@ public class PlantListModel {
 
     /**
      * Method to get actual Plant List in alphabetic Order
-     *
      * @return actual Plant List in alphabetic Order
      */
     public List<Plant> getPlantList(HardinessZone zone) throws HardinessZoneNotSetException, IOException {
@@ -63,7 +62,6 @@ public class PlantListModel {
 
     /**
      * Method to get the actual Plant list in custom Order
-     *
      * @param zone selected hardiness zone
      * @param comparator comparator to sort the list
      * @return sorted list with plants in the given hardiness zone
@@ -77,7 +75,6 @@ public class PlantListModel {
 
     /**
      * Method to get Filtered plant list
-     *
      * @param predicate predicate to filter the list
      * @param zone selected hardiness zone
      * @return filterd list with plants in the hardinness zone
@@ -91,7 +88,6 @@ public class PlantListModel {
 
     /**
      * Method to get Filtered plant list by id by exact match
-     *
      * @param zone selected hardiness zone
      * @param id id of plant
      * @return if id doesn't exist: empty List, else list with 1 plant entry.
@@ -106,6 +102,7 @@ public class PlantListModel {
     }
 
     /**
+     *
      * @param zone selected hardiness zone
      * @param searchString the string to search plant List, set '#' as first char the search by id.
      * @return List of plants found in Plant List which contain the search String in the name or description
@@ -113,25 +110,21 @@ public class PlantListModel {
      * @throws IOException If the database cannot be accessed
      */
     public List<Plant> getFilteredPlantListByString(HardinessZone zone, String searchString) throws HardinessZoneNotSetException, IOException {
-        if (searchString.length() == 0) {
+        if(searchString.length() == 0){
             return getPlantList(zone);
-        } else if (searchString.charAt(0) == '#') {
+        } else if(searchString.charAt(0) == '#') {
             try {
                 return getFilteredPlantListById(zone, Long.parseLong(searchString.substring(1)));
             } catch (NumberFormatException e) {
                 return new ArrayList<>();
             }
         } else {
-            String caseInsensitiveSearchString = searchString.toLowerCase();
-            return getFilteredPlantList(zone, plant ->
-                            plant.name().toLowerCase().contains(caseInsensitiveSearchString) ||
-                                    plant.description().toLowerCase().contains(caseInsensitiveSearchString)
-            );
+            return getFilteredPlantList(zone, plant -> plant.name().contains(searchString) || plant.description().contains(searchString));
         }
     }
 
-
     /**
+     *
      * @param type GrowPhaseType to filter
      * @param zone selected hardiness zone
      * @param from the earliest date to for the filter
@@ -145,6 +138,7 @@ public class PlantListModel {
     }
 
     /**
+     *
      * @param zone selected hardiness zone
      * @param from the earliest date to for the filter
      * @param to the lastest date for the filter
@@ -157,6 +151,7 @@ public class PlantListModel {
     }
 
     /**
+     *
      * @param zone selected hardiness zone
      * @param from the earliest date to for the filter
      * @param to the lastest date for the filter
@@ -167,17 +162,4 @@ public class PlantListModel {
     public List<Plant> getFilteredPlantListByHarvestSaison(HardinessZone zone, MonthDay from, MonthDay to) throws HardinessZoneNotSetException, IOException {
         return getFilteredPlantListBySaison(GrowthPhaseType.HARVEST, zone, from, to);
     }
-
-    /**
-     *
-     * @param zone selected hardiness zone
-     * @param from the earliest date to for the filter
-     * @param to the lastest date for the filter
-     * @return List of Plants with selected saison
-     * @throws HardinessZoneNotSetException If no {@link HardinessZone} was specified
-     * @throws IOException If the database cannot be accessed
-     */
-    public List<Plant> getFilteredPlantListBySaisonWithoutGrowthPhase(HardinessZone zone, MonthDay from, MonthDay to) throws HardinessZoneNotSetException, IOException {
-        return getFilteredPlantList(zone, plant -> plant.lifecycle().stream().anyMatch(growthPhase -> growthPhase.startDate().compareTo(from) >= 0 && (growthPhase.startDate().compareTo(to) <= 0)));
-    }
 }

src/main/java/ch/zhaw/gartenverwaltung/types/Seasons.java

@@ -1,26 +0,0 @@
-package ch.zhaw.gartenverwaltung.types;
-
-import java.time.MonthDay;
-
-public enum Seasons {
-    AllSEASONS("--01-01", "--12-31"),
-    SPRING("--03-01", "--05-30"),
-    SOMMER("--06-01", "--08-30"),
-    AUTUM("--09-01", "--11-30"),
-    WINTER("--12-01", "--02-28");
-
-    public final String startDate;
-    public final String endDate;
-
-    Seasons(String startDate, String endDate) {
-        this.startDate = startDate;
-        this.endDate = endDate;
-    }
-
-    public MonthDay getStartDate() {
-        return MonthDay.parse(this.startDate);
-    }
-    public MonthDay getEndDate() {
-        return MonthDay.parse(this.endDate);
-    }
-}

src/main/resources/ch/zhaw/gartenverwaltung/Plants.fxml

@@ -3,6 +3,7 @@
 <?import javafx.geometry.Insets?>
 <?import javafx.geometry.Rectangle2D?>
 <?import javafx.scene.control.Button?>
+<?import javafx.scene.control.CheckBox?>
 <?import javafx.scene.control.Label?>
 <?import javafx.scene.control.ListView?>
 <?import javafx.scene.control.SplitPane?>
@@ -15,9 +16,8 @@
 <?import javafx.scene.layout.VBox?>
 <?import javafx.scene.text.Font?>
 
-<AnchorPane maxHeight="-Infinity" maxWidth="1.7976931348623157E308" minHeight="-Infinity" minWidth="1000.0" prefHeight="853.0"
-            prefWidth="1219.0" xmlns="http://javafx.com/javafx/17" xmlns:fx="http://javafx.com/fxml/1"
-            fx:controller="ch.zhaw.gartenverwaltung.PlantsController">
+<AnchorPane maxHeight="-Infinity" maxWidth="1.7976931348623157E308" minHeight="-Infinity" minWidth="1000.0" prefHeight="853.0" prefWidth="1219.0"
+            xmlns="http://javafx.com/javafx/17" xmlns:fx="http://javafx.com/fxml/1" fx:controller="ch.zhaw.gartenverwaltung.PlantsController">
    <children>
       <SplitPane dividerPositions="0.7377363661277062" layoutX="539.0" layoutY="266.0" prefHeight="853.0" prefWidth="1219.0" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0">
         <items>
@@ -30,7 +30,7 @@
                               <Font name="System Bold" size="30.0" />
                            </font>
                         </Label>
-                        <TextField fx:id="search_plants" promptText="Search for Plant Name" />
+                        <TextField fx:id="search_plants" onInputMethodTextChanged="#searchForPlant" promptText="Search for Plant Name" />
                         <HBox alignment="CENTER_LEFT" prefHeight="480.0" prefWidth="881.0" VBox.vgrow="ALWAYS">
                            <children>
                               <ListView fx:id="list_plants" maxWidth="1.7976931348623157E308" prefHeight="497.0" prefWidth="580.0" HBox.hgrow="ALWAYS" />
@@ -75,7 +75,30 @@
                           <content>
                             <AnchorPane minHeight="0.0" minWidth="0.0" prefHeight="180.0" prefWidth="200.0">
                                  <children>
-                                    <VBox fx:id="seasons" layoutX="37.0" layoutY="-19.0" prefHeight="180.66666666666666" prefWidth="310.0" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0" />
+                                    <VBox layoutX="37.0" layoutY="-19.0" prefHeight="180.66666666666666" prefWidth="310.0" AnchorPane.bottomAnchor="0.0" AnchorPane.leftAnchor="0.0" AnchorPane.rightAnchor="0.0" AnchorPane.topAnchor="0.0">
+                                       <children>
+                                          <CheckBox fx:id="spring_filter" mnemonicParsing="false" onAction="#filterSpring" text="Spring">
+                                             <padding>
+                                                <Insets top="10.0" />
+                                             </padding>
+                                          </CheckBox>
+                                          <CheckBox fx:id="sommer_filter" mnemonicParsing="false" onAction="#filterSommer" text="Sommer">
+                                             <padding>
+                                                <Insets top="10.0" />
+                                             </padding>
+                                          </CheckBox>
+                                          <CheckBox fx:id="autum_filter" mnemonicParsing="false" onAction="#filterAutum" text="Autum">
+                                             <padding>
+                                                <Insets top="10.0" />
+                                             </padding>
+                                          </CheckBox>
+                                          <CheckBox fx:id="winter_filter" mnemonicParsing="false" onAction="#filterWinter" text="Winter">
+                                             <padding>
+                                                <Insets top="10.0" />
+                                             </padding>
+                                          </CheckBox>
+                                       </children>
+                                    </VBox>
                                  </children>
                               </AnchorPane>
                           </content>