added log statement in controller. renamed variable.

Author: Death111

src/main/java/de/doubleslash/keeptime/controller/Controller.java

@@ -209,24 +209,23 @@ public void editWork(final Work workToBeEdited, final Work newValuedWork) {
    }
 
    public void deleteWork(final Work workToBeDeleted) {
+      LOG.info("Deleting work '{}'.", workToBeDeleted);
 
       model.getPastWorkItems().removeIf(w -> (w.getId() == workToBeDeleted.getId()));
-
       model.getWorkRepository().delete(workToBeDeleted);
-
    }
 
    /**
     * Changes the indexes of the originalList parameter to have a consistent order.
     * 
     * @param originalList
-    *                          list of all projects to adapt the indexes for
+    *           list of all projects to adapt the indexes for
     * @param changedProject
-    *                          the project which has changed which already has the new index
+    *           the project which has changed which already has the new index
     * @param oldIndex
-    *                          the old index of the changed project
+    *           the old index of the changed project
     * @param newIndex
-    *                          the new index of the changed project (which the projects also already has)
+    *           the new index of the changed project (which the projects also already has)
     * @return all projects whose index has been adapted
     */
    List<Project> resortProjectIndexes(final List<Project> originalList, final Project changedProject,
@@ -263,9 +262,9 @@ List<Project> resortProjectIndexes(final List<Project> originalList, final Proje
     * Decreases all indexes by one, after the removed index
     * 
     * @param originalList
-    *                        list of all projects to adapt the indexes for
+    *           list of all projects to adapt the indexes for
     * @param removedIndex
-    *                        the index which has been removed
+    *           the index which has been removed
     * @return all projects whose index has been adapted
     */
    List<Project> adaptProjectIndexesAfterRemoving(final List<Project> originalList, final int removedIndex) {

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

@@ -272,14 +272,12 @@ public void updateItem(final LocalDate item, final boolean empty) {
    }
 
    private Button createDeleteWorkButton(final Work w) {
-
-      // XXX Auto-generated method stub
-      final Button editButton = new Button("delete");
-      editButton.setOnAction(e -> {
+      final Button deleteButton = new Button("delete");
+      deleteButton.setOnAction(e -> {
          LOG.info("Delete work clicked.");
          final Alert alert = new Alert(AlertType.CONFIRMATION);
          alert.setTitle("Delete Work");
-         alert.setHeaderText("You are about to delete this Work:");
+         alert.setHeaderText("Delete work item");
          alert.setContentText(w.toString());
          alert.initOwner(stage);
 
@@ -292,7 +290,7 @@ private Button createDeleteWorkButton(final Work w) {
             }
          });
       });
-      return editButton;
+      return deleteButton;
    }
 
    private Button createEditWorkButton(final Work work) {