taskjuggler 0.0.6 → 0.0.7

data/lib/SimpleQueryExpander.rb

@@ -0,0 +1,58 @@
+#!/usr/bin/env ruby -w
+# encoding: UTF-8
+#
+# = SimpleQueryExpander.rb -- The TaskJuggler III Project Management Software
+#
+# Copyright (c) 2006, 2007, 2008, 2009, 2010 by Chris Schlaeger <cs@kde.org>
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of version 2 of the GNU General Public License as
+# published by the Free Software Foundation.
+#
+
+require 'strscan'
+
+class TaskJuggler
+
+  # The SimpleQueryExpander class is used to replace embedded attribute
+  # queries in a string with the value of the attribute. The embedded queries
+  # must have the form <-name-> where name is the name of the attribute. The
+  # Query class is used to determine the value of the attribute within the
+  # context of the query.
+  class SimpleQueryExpander
+
+    # _inputStr_ is the String with the embedded queries. _query_ is the Query
+    # with that provides the evaluation context. _messageHandle_ is a
+    # MessageHandler that will be used for error reporting. _sourceFileInfo_
+    # is a SourceFileInfo reference used for error reporting.
+    def initialize(inputStr, query, messageHandler, sourceFileInfo)
+      @inputStr = inputStr
+      @query = query.dup
+      @messageHandler = messageHandler
+      @sourceFileInfo = sourceFileInfo
+    end
+
+    def expand
+      # Create a copy of the input string since we will modify it.
+      str = @inputStr.dup
+      # Replace all occurences of <-name->.
+      str.gsub!(/<-[a-zA-Z][_a-zA-Z]*->/) do |match|
+        len = match.size
+        attribute = match[2..-3]
+        @query.attributeId = attribute
+        @query.process
+        if @query.ok
+          @query.to_s
+        else
+          # The query failed. We report an error.
+          @messageHandler.error('sqe_expand_failed',
+                                "Unknown attribute #{attribute}",
+                                @sourceFileInfo)
+        end
+      end
+      str
+    end
+
+  end
+
+end

data/lib/SyntaxReference.rb

@@ -24,10 +24,14 @@ class TaskJuggler
   # output as well as HTML files are supported.
   class SyntaxReference
 
-    # The constructor is the most important function of this class. It creates a
-    # parser object and then traverses all rules and extracts the documented
-    # patterns. In a second pass the extracted KeywordDocumentation objects are
-    # then cross referenced to capture their relationships.
+    attr_reader :keywords
+
+    # The constructor is the most important function of this class. It creates
+    # a parser object and then traverses all rules and extracts the documented
+    # patterns. In a second pass the extracted KeywordDocumentation objects
+    # are then cross referenced to capture their relationships. _manual_ is an
+    # optional reference to the  UserManual object that uses this
+    # SyntaxReference.
     def initialize(manual = nil)
       @manual = manual
       @messageHandler = MessageHandler.new(true)
@@ -69,7 +73,7 @@ class TaskJuggler
       end
     end
 
-    # Return a sorted Array with all keywords.
+    # Return a sorted Array with all keywords (as String objects).
     def all
       sorted = @keywords.keys.sort
       # Register the neighbours with each keyword so we can use this info in

data/lib/TOCEntry.rb

@@ -21,6 +21,8 @@ class TaskJuggler
   # be turned into an HTML tree.
   class TOCEntry
 
+    attr_reader :number, :title, :tag
+
     # Create a TOCEntry object.
     # _number_: The section number as String, e. g. '1.2.3' or 'A.3'.
     # _title_: The section title as String.

data/lib/TableOfContents.rb

@@ -30,6 +30,10 @@ class TaskJuggler
       @entries << entry
     end
 
+    def each
+      @entries.each { |e| yield e }
+    end
+
     # Return HTML elements that represent the content of the TableOfContents
     # object. The result is a tree of XMLElement objects.
     def to_html

data/lib/Task.rb

@@ -32,25 +32,37 @@ class TaskJuggler
     end
 
     def query_journal(query)
-      journalText(query, true)
+      journalText(query, true, false)
+    end
+
+    def query_journal_sub(query)
+      journalText(query, true, true)
     end
 
     private
 
     # Create a blog-style list of all alert messages that match the Query.
-    def journalText(query, longVersion)
+    def journalText(query, longVersion, recursive)
       # The components of the message are either UTF-8 text or RichText. For
       # the RichText components, we use the originally provided markup since
       # we compose the result as RichText markup first.
       rText = ''
-      list = @project['journal'].entriesByTask(self, query.start, query.end)
+      if recursive
+        list = @project['journal'].entriesByTaskR(self, query.start, query.end)
+      else
+        list = @project['journal'].entriesByTask(self, query.start, query.end)
+      end
       list.reverse.each do |entry|
         tsRecord = entry.timeSheetRecord
 
         if entry.property.is_a?(Task)
           levelRecord = @project['alertLevels'][entry.alertLevel]
-          alertName = "[[File:icons/flag-#{levelRecord[0]}.png|" +
-                      "alt=[#{levelRecord[1]}]|text-bottom]]"
+          if query.selfContained
+            alertName = "<nowiki>[#{levelRecord[1]}]</nowiki>"
+          else
+            alertName = "[[File:icons/flag-#{levelRecord[0]}.png|" +
+                        "alt=[#{levelRecord[1]}]|text-bottom]]"
+          end
           rText += "== #{alertName} <nowiki>#{entry.headline}</nowiki> ==\n" +
                    "''Reported on #{entry.date.to_s(query.timeFormat)}'' "
           if entry.author
@@ -84,15 +96,14 @@ class TaskJuggler
 
       # Now convert the RichText markup String into RichTextIntermediate
       # format.
-      begin
-        rti = RichText.new(rText, RTFHandlers.create(@project)).
-          generateIntermediateFormat
-      rescue RichTextException => msg
-        $stderr.puts "Error while processing Rich Text\n" +
-                     "Line #{msg.lineNo}: #{msg.text}\n" +
-                     "#{msg.line}"
+      unless (rti = RichText.new(rText, RTFHandlers.create(@project),
+                                 @project.messageHandler).
+                                 generateIntermediateFormat)
+        @project.messageHandler.warning(
+          'task_journal_text', 'Syntax error in journal text')
         return nil
       end
+
       # No section numbers, please!
       rti.sectionNumbers = false
       # We use a special class to allow CSS formating.

data/lib/TaskJuggler.rb

@@ -45,17 +45,29 @@ class TaskJuggler
     files.each do |file|
       begin
         @parser.open(file, master)
-      rescue TjException
+      rescue TjException => msg
+        if msg.message && !msg.message.empty?
+          @messageHandler.critical('parse', msg.message)
+        end
         Log.exit('parser')
         return false
       end
       if master
-        @project = @parser.parse('project')
+        # The first file is considered the master file.
+        if (@project = @parser.parse('project')) == false
+          Log.exit('parser')
+          return false
+        end
         master = false
       else
+        # All other files.
         @parser.setGlobalMacros
-        @parser.parse('properties')
+        if @parser.parse('properties') == false
+          Log.exit('parser')
+          return false
+        end
       end
+      @project.inputFiles << file
       @parser.close
     end
 
@@ -70,9 +82,19 @@ class TaskJuggler
   # Parse a file and add the content to the existing project. _fileName_ is
   # the name of the file. _rule_ is the TextParser::Rule to start with.
   def parseFile(fileName, rule)
-    @parser.open(fileName, false)
+    begin
+      @parser.open(fileName, false)
+      @project.inputFiles << fileName
+    rescue TjException => msg
+      if msg.message && !msg.message.empty?
+        @messageHandler.critical('parse_file', msg.message)
+      end
+      return nil
+    end
+
     @parser.setGlobalMacros
-    return nil if (res = @parser.parse(rule)).nil?
+    return nil if (res = @parser.parse(rule)) == false
+
     @parser.close
     res
   end
@@ -83,7 +105,16 @@ class TaskJuggler
     Log.enter('scheduler', 'Scheduling project ...')
     #puts @project.to_s
     @project.warnTsDeltas = @warnTsDeltas
-    res = @project.schedule
+
+    begin
+      res = @project.schedule
+    rescue TjException => msg
+      if msg.message && !msg.message.empty?
+        @messageHandler.critical('scheduling_error', msg.message)
+      end
+      return false
+    end
+
     Log.exit('scheduler')
     res
   end
@@ -95,9 +126,18 @@ class TaskJuggler
     outputDir += '/' unless outputDir.empty? || outputDir[-1] == '/'
     @project.outputDir = outputDir
     Log.enter('reports', 'Generating reports ...')
-    res = @project.generateReports(@maxCpuCores)
+
+    begin
+      @project.generateReports(@maxCpuCores)
+    rescue TjException => msg
+      if msg.message && !msg.message.empty?
+        @messageHandler.critical('generate_reports', msg.message)
+      end
+      return false
+    end
+
     Log.exit('reports')
-    res
+    true
   end
 
   # Generate the report with the ID _reportId_. If _regExpMode_ is true,
@@ -109,7 +149,10 @@ class TaskJuggler
     begin
       Log.enter('generateReport', 'Generating report #{reportId} ...')
       @project.generateReport(reportId, regExpMode, dynamicAttributes)
-    rescue TjException
+    rescue TjException => msg
+      if msg.message && !msg.message.empty?
+        @messageHandler.critical('generate_report', msg.message)
+      end
       Log.exit('generateReport')
       return false
     end
@@ -117,14 +160,16 @@ class TaskJuggler
     true
   end
 
-  # Generate the report with the ID _reportId_. If _regExpMode_ is true,
-  # _reportId_ is interpreted as a Regular Expression and all reports with
-  # matching IDs are listed.
+  # List the details of the report with _reportId_ or if _regExpMode_ the
+  # reports that match the regular expression in _reportId_.
   def listReports(reportId, regExpMode)
     begin
       Log.enter('listReports', 'Generating report list for #{reportId} ...')
       @project.listReports(reportId, regExpMode)
-    rescue TjException
+    rescue TjException => msg
+      if msg.message && !msg.message.empty?
+        @messageHandler.critical('list_reports', msg.message)
+      end
       Log.exit('listReports')
       return false
     end
@@ -149,17 +194,14 @@ class TaskJuggler
                      'scenarioIdx' => @project['trackingScenarioIdx'],
                      'start' => ts.interval.start,
                      'end' => ts.interval.end,
-                     'timeFormat' => '%Y-%m-%d' }
+                     'timeFormat' => '%Y-%m-%d',
+                     'selfContained' => true }
       query = Query.new(queryAttrs)
-      rti = ts.resource.query_journal(query)
-      rti.lineWidth = 72
-      rti.indent = 2
-      rti.titleIndent = 0
-      rti.listIndent = 2
-      rti.parIndent = 2
-      rti.preIndent = 4
-      puts rti.to_s
-    rescue TjException
+      puts ts.resource.query_journal(query).richText.inputText
+    rescue TjException => msg
+      if msg.message && !msg.message.empty?
+        @messageHandler.critical('check_time_sheet', msg.message)
+      end
       Log.exit('checkTimeSheet')
       return false
     end
@@ -181,17 +223,14 @@ class TaskJuggler
                      'timeFormat' => '%Y-%m-%d',
                      'start' => ss[1],
                      'end' => ss[2],
-                     'timeFormat' => '%Y-%m-%d' }
+                     'timeFormat' => '%Y-%m-%d',
+                     'selfContained' => true }
       query = Query.new(queryAttrs)
-      rti = ss[0].query_dashboard(query)
-      rti.lineWidth = 72
-      rti.indent = 2
-      rti.titleIndent = 0
-      rti.listIndent = 2
-      rti.parIndent = 2
-      rti.preIndent = 4
-      puts rti.to_s
-    rescue TjException
+      puts ss[0].query_dashboard(query).richText.inputText
+    rescue TjException => msg
+      if msg.message && !msg.message.empty?
+        @messageHandler.critical('check_status_sheet', msg.message)
+      end
       Log.exit('checkStatusSheet')
       return false
     end

data/lib/TaskScenario.rb

@@ -324,7 +324,10 @@ class TaskJuggler
     # be called to do some more housekeeping. It computes some derived data
     # based on the just scheduled values.
     def finishScheduling
-      calcCompletion
+      # The completion degree calculation is done recursively. So we only call
+      # it for top-level tasks.
+      calcCompletion unless @property.parent
+
       # This list is no longer needed, so let's save some memory. Set it to
       # nil so we can detect accidental use.
       @candidates = nil
@@ -434,12 +437,33 @@ class TaskJuggler
         task = dependency.task
         limit = task[dependency.onEnd ? 'end' : 'start', @scenarioIdx]
         next if limit.nil?
-        if limit > a('start')
+        if a('start') < limit
           error('task_pred_before',
                 "Task #{@property.fullId} must start after " +
                 "#{dependency.onEnd ? 'end' : 'start'} of task " +
                 "#{task.fullId}.")
         end
+        if dependency.gapDuration > 0
+          if limit + dependency.gapDuration > a('start')
+            error('task_pred_before_gd',
+                  "Task #{@property.fullId} must start " +
+                  "#{dependency.gapDuration / (60 * 60 * 24)} days after " +
+                  "#{dependency.onEnd ? 'end' : 'start'} of task " +
+                  "#{task.fullId}. TaskJuggler cannot enforce this condition " +
+                  "because the task is scheduled ALAP (finish-to-start).")
+          end
+        end
+        if dependency.gapLength > 0
+          if calcLength(limit, a('start')) < dependency.gapLength
+            error('task_pred_before_gl',
+                  "Task #{@property.fullId} must start " +
+                  "#{@project.slotsToDays(dependency.gapLength)} " +
+                  "working days after " +
+                  "#{dependency.onEnd ? 'end' : 'start'} of task " +
+                  "#{task.fullId}. TaskJuggler cannot enforce this condition " +
+                  "because the task is scheduled ALAP (finish-to-start).")
+          end
+        end
       end
 
       # Check that all following tasks end before this task
@@ -452,6 +476,27 @@ class TaskJuggler
                 "Task #{@property.fullId} must end before " +
                 "#{dependency.onEnd ? 'end' : 'start'} of task #{task.fullId}.")
         end
+        if dependency.gapDuration > 0
+          if limit - dependency.gapDuration < a('end')
+            error('task_succ_after_gd',
+                  "Task #{@property.fullId} must end " +
+                  "#{dependency.gapDuration / (60 * 60 * 24)} days before " +
+                  "#{dependency.onEnd ? 'end' : 'start'} of task " +
+                  "#{task.fullId}. TaskJuggler cannot enforce this condition " +
+                  "because the task is scheduled ASAP (start-to-finish).")
+          end
+        end
+        if dependency.gapLength > 0
+          if calcLength(a('end'), limit) < dependency.gapLength
+            error('task_succ_after_gl',
+                  "Task #{@property.fullId} must end " +
+                  "#{@project.slotsToDays(dependency.gapLength)} " +
+                  "working days before " +
+                  "#{dependency.onEnd ? 'end' : 'start'} of task " +
+                  "#{task.fullId}. TaskJuggler cannot enforce this condition " +
+                  "because the task is scheduled ASAP (start-to-finish).")
+          end
+        end
       end
 
       if a('milestone') && a('start') != a('end')
@@ -497,8 +542,9 @@ class TaskJuggler
     def checkForLoops(path, atEnd, fromOutside)
       # Check if we have been here before on this path.
       if path.include?([ @property, atEnd ])
-        error('loop_detected', "Loop detected at #{atEnd ? 'end' : 'start'} " +
-                               "of task #{@property.fullId}", false)
+        warning('loop_detected',
+                "Loop detected at #{atEnd ? 'end' : 'start'} " +
+                "of task #{@property.fullId}", false)
         skip = true
         path.each do |t, e|
           if t == @property && e == atEnd
@@ -507,7 +553,8 @@ class TaskJuggler
           end
           next if skip
           info("loop_at_#{e ? 'end' : 'start'}",
-               "Loop ctnd. at #{e ? 'end' : 'start'} of task #{t.fullId}", t)
+               "Loop ctnd. at #{e ? 'end' : 'start'} of task #{t.fullId}",
+               t.sourceFileInfo)
         end
         error('loop_end', "Aborting")
       end
@@ -1098,12 +1145,8 @@ class TaskJuggler
     end
 
     def query_complete(query)
-      if @property.leaf?
-        query.sortable = query.numerical = complete = a('complete').to_i
-        query.string = "#{complete}%"
-      else
-        query.string = ''
-      end
+      query.sortable = query.numerical = complete = a('complete').to_i
+      query.string = "#{complete}%"
     end
 
     # Compute the cost generated by this Task for a given Account during a given
@@ -1172,10 +1215,10 @@ class TaskJuggler
         str += "* <nowiki>#{task.name}</nowiki> (#{task.fullId}) "
         if onEnd
           taskEnd = task['end', query.scenarioIdx].to_s(query.timeFormat)
-          str += "[->] #{taskEnd}"
+          str += "<nowiki>[->]</nowiki> #{taskEnd}"
         else
           taskStart = task['start', query.scenarioIdx].to_s(query.timeFormat)
-          str += "[->[ #{taskStart}"
+          str += "<nowiki>[->[</nowiki> #{taskStart}"
         end
         str += "\n"
       end
@@ -1184,16 +1227,19 @@ class TaskJuggler
         str += "* <nowiki>#{task.name}</nowiki> (#{task.fullId}) "
         if onEnd
           taskEnd = task['end', query.scenarioIdx].to_s(query.timeFormat)
-          str += "]->] #{taskEnd}"
+          str += "<nowiki>]->]</nowiki> #{taskEnd}"
         else
           taskStart = task['start', query.scenarioIdx].to_s(query.timeFormat)
-          str += "]->[ #{taskStart}"
+          str += "<nowiki>]->[</nowiki> #{taskStart}"
         end
         str += "\n"
       end
 
-      rText = RichText.new(str)
-      query.rti = rText.generateIntermediateFormat
+      rText = RichText.new(str, [], @property.project.messageHandler)
+      unless (query.rti = rText.generateIntermediateFormat)
+        @property.project.messageHandler.warning(
+          'task_sc_followers', 'Syntax error in followers')
+      end
     end
 
     def query_precursors(query)
@@ -1204,10 +1250,10 @@ class TaskJuggler
         str += "* <nowiki>#{task.name}</nowiki> (#{task.fullId}) "
         if onEnd
           taskEnd = task['end', query.scenarioIdx].to_s(query.timeFormat)
-          str += "]->] #{taskEnd}"
+          str += "<nowiki>]->]</nowiki> #{taskEnd}"
         else
           taskStart = task['start', query.scenarioIdx].to_s(query.timeFormat)
-          str += "[->[ #{taskStart}"
+          str += "<nowiki>[->[</nowiki> #{taskStart}"
         end
         str += "\n"
       end
@@ -1216,10 +1262,10 @@ class TaskJuggler
         str += "* <nowiki>#{task.name}</nowiki> (#{task.fullId}) "
         if onEnd
           taskEnd = task['end', query.scenarioIdx].to_s(query.timeFormat)
-          str += "[->] #{taskEnd}"
+          str += "<nowiki>[->]</nowiki> #{taskEnd}"
         else
           taskStart = task['start', query.scenarioIdx].to_s(query.timeFormat)
-          str += "]->[ #{taskStart}"
+          str += "<nowiki>]->[</nowiki> #{taskStart}"
         end
         str += "\n"
       end
@@ -1259,7 +1305,7 @@ class TaskJuggler
 
     def query_targets(query)
       targetList = PropertyList.new(@project.tasks, false)
-      targets(targetList)
+      targets(targetList, true)
       targetList.delete(@property)
       targetList.setSorting([['start', true, @scenarioIdx],
                              ['seqno', true, -1 ]])
@@ -1569,7 +1615,6 @@ class TaskJuggler
         return unless found
       end
 
-      iv = Interval.new(date, date + slotDuration)
       a('allocate').each do |allocation|
         next unless allocation.onShift?(date)
 
@@ -1640,6 +1685,16 @@ class TaskJuggler
       true
     end
 
+    # Calculate the number of general working time slots between the TjTime
+    # objects _d1_ and _d2_.
+    def calcLength(d1, d2)
+      slots = 0
+      while d1 < d2
+        slots += 1 if @project.isWorkingTime(d1)
+        d1 += @project['scheduleGranularity']
+      end
+      slots
+    end
 
     # Register the user provided bookings with the Resource scoreboards. A
     # booking describes the assignment of a Resource to a certain Task for a
@@ -1649,7 +1704,7 @@ class TaskJuggler
       a('booking').each do |booking|
         unless booking.resource.leaf?
           error('booking_resource_not_leaf',
-                "Booked resources may not be group resources", true,
+                "Booked resources may not be group resources",
                 booking.sourceFileInfo)
         end
         unless a('forward') || scheduled
@@ -1866,51 +1921,59 @@ class TaskJuggler
       # If the user provided a completion degree we are not touching it.
       if @property.provided('complete', @scenarioIdx)
         calcStatus
-        return
+        return a('complete')
       end
 
+      # We cannot compute a completion degree without a start or end date.
       if a('start').nil? || a('end').nil?
         @property['complete', @scenarioIdx] = 0.0
         @property['status', @scenarioIdx] = 'unknown'
-        return
+        return nil
       end
 
+      completion = 0.0
       if a('milestone')
-        @property['complete', @scenarioIdx] =
+        # Milestones are either 0% or 100% complete.
+        @property['complete', @scenarioIdx] = completion =
           @property['end', @scenarioIdx] <= @project['now'] ? 100.0 : 0.0
         @property['status', @scenarioIdx] =
           a('end') <= @project['now'] ? 'done' : 'not reached'
       else
-        completion = 0.0
-        if a('end') <= @project['now']
+        # The task is in progress. Calculate the current completion
+        # degree.
+        if !property.leaf?
+          # For container task the completion degree is the average of the
+          # sub tasks.
+          completion = 0.0
+          @property.children.each do |child|
+            return nil unless (comp = child.calcCompletion(@scenarioIdx))
+            completion += comp
+          end
+          completion /= @property.children.length
+        elsif a('end') <= @project['now']
           # The task has ended already. It's 100% complete.
           completion = 100.0
         elsif @project['now'] <= a('start')
           # The task has not started yet. Its' 0% complete.
           completion = 0.0
+        elsif a('effort') > 0
+          # Effort based leaf tasks. The completion degree is the percentage
+          # of effort that has been done already.
+          done = getEffectiveWork(@project.dateToIdx(a('start')),
+                                  @project.dateToIdx(@project['now']))
+          total = @project.convertToDailyLoad(
+            a('effort') * @project['scheduleGranularity'])
+          completion = done / total * 100.0
         else
-          # The task is in progress. Calculate the current completion
-          # degree.
-          if @property.leaf? && a('effort') > 0
-            # Effort based leaf tasks. The completion degree is the percantage
-            # of effort that has been done already.
-            done = getEffectiveWork(@project.dateToIdx(a('start')),
-                                    @project.dateToIdx(@project['now']))
-            total = @project.convertToDailyLoad(
-              a('effort') * @project['scheduleGranularity'])
-            completion = done / total * 100.0
-          else
-            # Container tasks and length/duration leaf tasks. There is no way
-            # we can compute the completion degree of a container task with a
-            # mix of effort and duration task in a meaningful way.  So, we
-            # just go by duration.
-            completion = ((@project['now'] - a('start')) /
-                          (a('end') - a('start'))) * 100.0
-          end
+          # Length/duration leaf tasks.
+          completion = ((@project['now'] - a('start')) /
+                        (a('end') - a('start'))) * 100.0
         end
         @property['complete', @scenarioIdx] = completion
         calcStatus
       end
+
+      completion
     end
 
     # Calculate the status of the task based on the 'complete' attribute.
@@ -1927,20 +1990,35 @@ class TaskJuggler
 
     # Recursively compile a list of Task properties which depend on the
     # current task.
-    def targets(list)
+    def targets(list, includeChildren)
+      # Ignore tasks that we have already included in the list.
+      return if list.include?(@property)
+
       # A target must be a leaf function that has no direct or indirect
-      # (through parent) following tasks.
-      if @property.leaf? && !hasSuccessors && !list.include?(@property)
+      # (through parent) following tasks. Only milestones are recognized as
+      # targets.
+      if @property.leaf? && !hasSuccessors && a('milestone')
         list << @property
         return
       end
 
       a('endsuccs').each do |t, onEnd|
-        t.targets(@scenarioIdx, list)
+        t.targets(@scenarioIdx, list, false)
+      end
+
+      # Check for indirect followers.
+      if @property.parent
+        @property.parent.targets(@scenarioIdx, list, false)
       end
 
-      # Check of indirect followers.
-      @property.parent.targets(@scenarioIdx, list) if @property.parent
+      # Also include targets of child tasks. The recursive iteration of child
+      # tasks is limited to the tested task only. The followers are not
+      # iterated.
+      if includeChildren
+        @property.children.each do |child|
+          child.targets(@scenarioIdx, list, true)
+        end
+      end
     end
 
     # Compute the turnover generated by this Task for a given Account _account_