populate position box works

SchedulerApp/src/main/java/com/example/schedulerapp/Controller.java

@@ -494,12 +494,8 @@ public class Controller {
         temp.setItems(list);
     }
     public void populatePositionBox(MouseEvent event) {
-        ArrayList<String> aList = model.returnPositions();
-        ObservableList<String> list = FXCollections.observableArrayList();
-        list.addAll(aList);
-
         ComboBox test = (ComboBox) event.getSource();
-        test.setItems(list);
+        test.setItems(FXCollections.observableArrayList(model.returnPositions()));
     }
     public void removeEmployeeBoxClicked(MouseEvent event){
         System.out.println("remove employee box clicked");
@@ -571,13 +567,8 @@ public class Controller {
         Button addPosition = new Button("Add Position");
         Button removePosition = new Button("Remove Position");
         tempBox.getChildren().addAll(tempCombo,addPosition,removePosition);
-        tempCombo.setOnAction(new EventHandler<ActionEvent>() {
-            @Override
-            public void handle(ActionEvent actionEvent) {
-                actionEvent.getEventType();
-                populatePositionBox((ComboBox) actionEvent.getSource());
-            }
-        });
+        tempCombo.setOnMouseClicked(this::populatePositionBox);
+
         addPosition.setOnMouseClicked(new EventHandler<MouseEvent>() {
             @Override
             public void handle(MouseEvent mouseEvent) {
@@ -612,13 +603,6 @@ public class Controller {
         });
         root.setCenter(tempBox);
     }
-    public void populatePositionBox(ComboBox box){
-        ArrayList<String> aList = model.returnPositions();
-        ObservableList<String> list = FXCollections.observableArrayList();
-        list.addAll(aList);
 
 
-        box.setItems(list);
-    }
-
 }
\ No newline at end of file