Skip to content

Commit c2e0164

Browse files
committed
Refactoring on ViewController went wrong
Some how my refactoring didn't work. Now you can't use "Edit Project" (or better you can use it but it doesn't edit the Project)
1 parent 368a29f commit c2e0164

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

src/main/java/de/doubleslash/keeptime/view/ViewController.java

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -676,11 +676,8 @@ private Node addProjectToProjectList(final Project p) {
676676
editMenuItem.setOnAction(e -> {
677677
// TODO refactor to use "add project" controls
678678
LOG.info("Edit project");
679-
Dialog<ButtonType> dialog = new Dialog<>();
680-
dialog = setUpEditProjectDialog(p);
681-
682-
GridPane grid = new GridPane();
683-
grid = setUpEditProjectGridPane(p);
679+
final Dialog<ButtonType> dialog = setUpEditProjectDialog(p);
680+
final GridPane grid = setUpEditProjectGridPane(p);
684681

685682
// TODO disable OK button if no name is set
686683
dialog.getDialogPane().setContent(grid);
@@ -697,6 +694,11 @@ private Node addProjectToProjectList(final Project p) {
697694
if (buttonType != ButtonType.OK) {
698695
return;
699696
}
697+
final ObservableList<Node> nodes = grid.getChildren();
698+
final TextField projectNameTextField = (TextField) nodes.get(6);
699+
final ColorPicker colorPicker = (ColorPicker) nodes.get(4);
700+
final CheckBox isWorkCheckBox = (CheckBox) nodes.get(2);
701+
final Spinner<Integer> indexSpinner = (Spinner<Integer>) nodes.get(0);
700702
controller.editProject(p, projectNameTextField.getText(), colorPicker.getValue(),
701703
isWorkCheckBox.isSelected(), indexSpinner.getValue());
702704

0 commit comments

Comments
 (0)