Changeset 515 for trunk/grails-app

Timestamp:

Apr 30, 2010, 3:41:32 AM (15 years ago)

Author:

gav

Message:

Default task dates to today when manually creating a sub task.

File:

• trunk/grails-app/services/TaskService.groovy (modified) (3 diffs)

trunk/grails-app/services/TaskService.groovy

@@ -10 +10 @@
 
     def authService
+    def dateUtilService
     def assignedGroupService
     def assignedPersonService
@@ -167 +168 @@
     /**
     * Creates a subTask copying sane attributes from the parentTask unless otherwise specified in params.
+    * The targetStartDate and targetCompletionDate default to today since that is the sane thing to do.
     * The taskProcedure is only assigned to the sub task if supplied in params.
     * The assignedPersons and assignedGroups are only added to the sub task if supplied in params.
     * Collections in params must be supplied as new ArrayList's.
     * This method is not intended to be a copyTask method.
-    * There should be no reason to copy tasks, try to find a better solution.
+    * There should be no reason to copy tasks, recurrence can be used to create similar tasks.
     * @param parentTask The parent task to get attributes from, also set as the parent.
     * @param params Overrides the parent task values if specified.
@@ -185 +187 @@
         p.description = params.description ?: parentTask.description
         p.comment = params.comment ?: parentTask.comment
-        p.targetStartDate = params.targetStartDate ?: parentTask.targetStartDate
-        p.targetCompletionDate = params.targetCompletionDate ?: parentTask.targetCompletionDate
+        p.targetStartDate = params.targetStartDate ?: dateUtilService.today
+        p.targetCompletionDate = params.targetCompletionDate ?: dateUtilService.today
         p.scheduled = params.scheduled ?:  parentTask.scheduled