[Jtimer-commits] r2940 - trunk/src/main/java/org/chorem/jtimer/data

26 Sep 2013

Author: tchemit
Date: 2013-09-26 10:03:59 +0200 (Thu, 26 Sep 2013)
New Revision: 2940

Url: http://chorem.org/projects/jtimer/repository/revisions/2940

Log:
fix javadoc typo + improve code

Modified:
   trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java

Modified: trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java
===================================================================

--- trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java	2013-09-24 07:37:24 UTC (rev 2939)
+++ trunk/src/main/java/org/chorem/jtimer/data/TimerDataManager.java	2013-09-26 08:03:59 UTC (rev 2940)
@@ -40,7 +40,6 @@
 import org.chorem.jtimer.entities.TimerAlert;
 import org.chorem.jtimer.entities.TimerProject;
 import org.chorem.jtimer.entities.TimerTask;
-import org.nuiton.util.ArrayUtil;
 
 /**
  * Gere les donnees. Des objets peuvent s'enregistrer pour etre notifies des
@@ -201,7 +200,7 @@
             vetoableDataEventListener.checkSetTaskTime(task, date, value);
         }
 
-        task.setTime(date, Long.valueOf(value));
+        task.setTime(date, value);
 
         // send notification
         for (DataEventListener dataEventListener : dataEventListeners) {
@@ -249,9 +248,8 @@
         task.getParent().getSubTasks().remove(task);
 
         // send notification
-        Iterator<DataEventListener> itDataEventListener = dataEventListeners.iterator();
-        while (itDataEventListener.hasNext()) {
-            itDataEventListener.next().deleteTask(task);
+        for (DataEventListener dataEventListener : dataEventListeners) {
+            dataEventListener.deleteTask(task);
         }
     }
 
@@ -270,9 +268,8 @@
         projectList.remove(project);
 
         // send notification
-        Iterator<DataEventListener> itDataEventListener = dataEventListeners.iterator();
-        while (itDataEventListener.hasNext()) {
-            itDataEventListener.next().deleteProject(project);
+        for (DataEventListener dataEventListener : dataEventListeners) {
+            dataEventListener.deleteProject(project);
         }
     }
 
@@ -324,9 +321,8 @@
         task.setClosed(!task.isClosed());
 
         // send notification
-        Iterator<DataEventListener> itDataEventListener = dataEventListeners.iterator();
-        while (itDataEventListener.hasNext()) {
-            itDataEventListener.next().changeClosedState(task);
+        for (DataEventListener dataEventListener : dataEventListeners) {
+            dataEventListener.changeClosedState(task);
         }
     }
 
@@ -565,17 +561,17 @@
     }
 
     /**
-     * Find corresponding task for path.
+     * Return the corresponding task for path.
      * 
-     * @param taskPath
-     * @return
+     * @param taskPath task path to find
+     * @return corresponding task for path
      */
     public TimerTask getTaskForPath(String taskPath) {
         
         String[] paths = taskPath.split("/");
         
         // make sure can't select only project
-        if (paths == null || paths.length <= 1) {
+        if (paths.length <= 1) {
             throw new IllegalArgumentException("Can't parse argument '" + taskPath + "' as task");
         }

    

[Jtimer-commits] r2940 - trunk/src/main/java/org/chorem/jtimer/data

tchemit＠users.chorem.org