taskjuggler 0.0.7 → 0.0.8

data/lib/Project.rb

@@ -78,7 +78,7 @@ class TaskJuggler
         'flags' => [],
         'journal' => Journal.new,
         'limits' => nil,
-        'loadUnit' => :shortauto,
+        'loadUnit' => :days,
         'name' => name,
         'navigators' => {},
         'now' => TjTime.now.align(3600),
@@ -97,7 +97,7 @@ class TaskJuggler
         'vacations' => [],
         'version' => version || "1.0",
         'weekStartsMonday' => true,
-        'workinghours' => WorkingHours.new,
+        'workinghours' => nil,
         'yearlyworkingdays' => 260.714
       }
 
@@ -430,6 +430,23 @@ class TaskJuggler
         raise "Unknown project attribute #{name}"
       end
       @attributes[name] = value
+
+      # If the start, end or schedule granularity have been changed, we have
+      # to reset the working hours.
+      if %w(start end scheduleGranularity timezone timingresolution).
+        include?(name)
+        if @attributes['start'] && @attributes['end']
+          @attributes['workinghours'] =
+            WorkingHours.new(@attributes['scheduleGranularity'],
+                             @attributes['start'], @attributes['end'])
+          # WorkingHours is using a copy-on-write scheme to prevent multiple
+          # copies of the same Scoreboard. All other WorkingHours objects are
+          # created as copies of this object. By calling
+          # WorkingHours::onShift? we make sure this instance has got a
+          # Scoreboard than gets reused by the other instances.
+          @attributes['workinghours'].onShift?(@attributes['start'])
+        end
+      end
     end
 
     # Return the number of defined scenarios for the project.
@@ -440,7 +457,7 @@ class TaskJuggler
     # Try to match _levelName_ to a defined alert level name and return the
     # index of it. If no level is found, nil is returned.
     def alertLevelIndex(levelName)
-      0.upto(@attributes['alertLevels'].length - 1) do |i|
+      @attributes['alertLevels'].length.times do |i|
         if @attributes['alertLevels'][i][0] == levelName
           return i
         end
@@ -602,6 +619,24 @@ class TaskJuggler
       true
     end
 
+    # Make sure that we have a least one report defined that has an output
+    # format.
+    def checkReports
+      if @reports.empty?
+        @messageHandler.warning('no_report_defined',
+                                "This project has no reports defined. " +
+                                "No output data will be generated.")
+      end
+
+      @reports.each do |report|
+        return unless report.get('formats').empty?
+      end
+
+      @messageHandler.warning('all_formats_empty',
+                              "None of the reports has a 'formats' attribute. " +
+                              "No output data will be generated.")
+    end
+
     # Call this function to generate the reports based on the scheduling result.
     # This function may only be called after Project#schedule has been called.
     def generateReports(maxCpuCores)
@@ -652,6 +687,12 @@ class TaskJuggler
           error('unknown_report_id',
                 "Request to generate unknown report #{id}")
         end
+        if report.get('formats').empty?
+          @messageHandler.error('formats_empty',
+           "The report #{report.fullId} has no 'formats' attribute. " +
+           "No output data will be generated.")
+        end
+
         Log.startProgressMeter("Report #{report.name}")
         @reportContexts.push(context = ReportContext.new(self, report))
 
@@ -834,7 +875,7 @@ class TaskJuggler
 
     # Print the attribute values. It's used for debugging only.
     def to_s
-      raise "STOP!"
+      #raise "STOP!"
       str = ''
       @attributes.each do |attribute, value|
         if value
@@ -921,19 +962,22 @@ class TaskJuggler
 
     def finishScenario(scIdx)
       Log.startProgressMeter("Checking scenario #{scenario(scIdx).get('name')}")
-      total = @tasks.items
-      i = 0
       @tasks.each do |task|
-        task.finishScheduling(scIdx)
-        i += 1
-        Log.progress((i.to_f / total) * 0.5)
+        # Recursively traverse the top-level tasks to finish all tasks.
+        task.finishScheduling(scIdx) unless task.parent
+      end
+
+      @resources.each do |resource|
+        # Recursively traverse the top-level resources to finish them all.
+        resource.finishScheduling(scIdx) unless resource.parent
       end
 
       i = 0
+      total = @tasks.items
       @tasks.each do |task|
         task.postScheduleCheck(scIdx) if task.parent.nil?
         i += 1
-        Log.progress(0.5 + (i.to_f / total) * 0.5)
+        Log.progress(i.to_f / total)
       end
 
       # This should be really fast so we don't log progess.

data/lib/ProjectFileParser.rb

@@ -39,10 +39,11 @@ class TaskJuggler
 
       # Define the token types that the ProjectFileScanner may return for
       # variable elements.
-      @variables = %w( INTEGER FLOAT DATE TIME STRING LITERAL ID ID_WITH_COLON
-                       ABSOLUTE_ID MACRO )
+      @variables = [ :INTEGER, :FLOAT, :DATE, :TIME, :STRING, :LITERAL,
+                     :ID, :ID_WITH_COLON, :ABSOLUTE_ID, :MACRO ]
 
       initRules
+      updateParserTables
 
       @project = nil
     end
@@ -196,7 +197,7 @@ class TaskJuggler
       # Add the new user-defined attribute as reportable attribute to the parser
       # rule.
       oldCurrentRule = @cr
-      @cr = @rules['reportableAttributes']
+      @cr = @rules[:reportableAttributes]
       singlePattern('_' + @val[1])
       descr(@val[2])
       @cr = oldCurrentRule
@@ -209,11 +210,10 @@ class TaskJuggler
     # data into a MessageHandler message that points to the correct location.
     # This is necessary, because the RichText parser knows nothing about the
     # actual input file. So we have to map the error location in the RichText
-    # input stream back to the position in the project file.
-    # To limit the supported set of variable tokens, a subset can be provided
-    # by _tokenSet_.
-    def newRichText(text, tokenSet = nil)
-      sfi = sourceFileInfo
+    # input stream back to the position in the project file. _sfi_ is the
+    # SourceFileInfo of the input string. To limit the supported set of
+    # variable tokens, a subset can be provided by _tokenSet_.
+    def newRichText(text, sfi, tokenSet = nil)
       rText = RichText.new(text, RTFHandlers.create(@project, sfi))
       unless (rti = rText.generateIntermediateFormat( [ 0, 0, 0], tokenSet))
         rText.messageHandler.messages.each do |msg|
@@ -357,7 +357,8 @@ class TaskJuggler
     # token.
     def descr(text)
       if @cr.patterns[-1].length != 1 ||
-         (@cr.patterns[-1][0][0] != ?_ && @cr.patterns[-1][0][0] != ?$)
+         (@cr.patterns[-1][0][0] != :literal &&
+          @cr.patterns[-1][0][0] != :variable)
         raise 'descr() may only be used for patterns with terminal tokens.'
       end
       arg(0, nil, text)

data/lib/ProjectFileScanner.rb

@@ -71,25 +71,25 @@ class TaskJuggler
         [ nil, /.*\n/, :macroCall, method('midMacroCall') ],
 
         # An ID with a colon suffix: foo:
-        [ 'ID_WITH_COLON', /[a-zA-Z_]\w*:/, :tjp, method('chop') ],
+        [ :ID_WITH_COLON, /[a-zA-Z_]\w*:/, :tjp, method('chop') ],
 
         # An absolute ID: a.b.c
-        [ 'ABSOLUTE_ID', /[a-zA-Z_]\w*(\.[a-zA-Z_]\w*)+/ ],
+        [ :ABSOLUTE_ID, /[a-zA-Z_]\w*(\.[a-zA-Z_]\w*)+/ ],
 
         # A normal ID: bar
-        [ 'ID', /[a-zA-Z_]\w*/ ],
+        [ :ID, /[a-zA-Z_]\w*/ ],
 
         # A date
-        [ 'DATE', /\d{4}-\d{1,2}-\d{1,2}(-\d{1,2}:\d{1,2}(:\d{1,2})?(-[-+]?\d{4})?)?/, :tjp, method('to_date') ],
+        [ :DATE, /\d{4}-\d{1,2}-\d{1,2}(-\d{1,2}:\d{1,2}(:\d{1,2})?(-[-+]?\d{4})?)?/, :tjp, method('to_date') ],
 
         # A time of day
-        [ 'TIME', /\d{1,2}:\d{2}/, :tjp, method('to_time') ],
+        [ :TIME, /\d{1,2}:\d{2}/, :tjp, method('to_time') ],
 
         # A floating point number (e. g. 3.143)
-        [ 'FLOAT', /\d*\.\d+/, :tjp, method('to_f') ],
+        [ :FLOAT, /\d*\.\d+/, :tjp, method('to_f') ],
 
         # An integer number
-        [ 'INTEGER', /\d+/, :tjp, method('to_i') ],
+        [ :INTEGER, /\d+/, :tjp, method('to_i') ],
 
         # Multi line string enclosed with double quotes. The string may
         # contain double quotes prefixed by a backslash. The first rule
@@ -98,14 +98,14 @@ class TaskJuggler
         # Any line not containing the start or end.
         [ 'nil', /^(\\"|[^"])*\n/, :dqString, method('midStringDQ') ],
         # The end of the string.
-        [ 'STRING', /(\\"|[^"])*"/, :dqString, method('endStringDQ') ],
+        [ :STRING, /(\\"|[^"])*"/, :dqString, method('endStringDQ') ],
 
         # Multi line string enclosed with single quotes.
         [ 'nil', /'(\\'|[^'])*/, :tjp, method('startStringSQ') ],
         # Any line not containing the start or end.
         [ 'nil', /^(\\'|[^'])*\n/, :sqString, method('midStringSQ') ],
         # The end of the string.
-        [ 'STRING', /(\\'|[^'])*'/, :sqString, method('endStringSQ') ],
+        [ :STRING, /(\\'|[^'])*'/, :sqString, method('endStringSQ') ],
 
         # Scizzors marked string -8<- ... ->8-: The opening mark must be the
         # last thing in the line. The indentation of the first line after the
@@ -115,8 +115,8 @@ class TaskJuggler
         # Since the first line can be the last line (empty string case), we
         # need to detect the end in szrString1 and szrString mode. The
         # patterns switch the scanner back to tjp mode.
-        [ 'STRING', /\s*->8-/, :szrString1, method('endStringSZR') ],
-        [ 'STRING', /\s*->8-/, :szrString, method('endStringSZR') ],
+        [ :STRING, /\s*->8-/, :szrString1, method('endStringSZR') ],
+        [ :STRING, /\s*->8-/, :szrString, method('endStringSZR') ],
         # This rule handles macros inside of sizzors strings.
         [ nil, /.*?\$\{\s*([a-zA-Z_]\w*)(\s*"(\\"|[^"])*")*/,
           [ :szrString, :szrString1 ], method('startMacroCall') ],
@@ -125,24 +125,24 @@ class TaskJuggler
         [ 'nil', /.*\n/, :szrString, method('midStringSZR') ],
 
         # Single line macro definition
-        [ 'MACRO', /\[.*\]\n/, :tjp, method('chop2nl') ],
+        [ :MACRO, /\[.*\]\n/, :tjp, method('chop2nl') ],
 
         # Multi line macro definition: The pattern switches the scanner into
         # macroDef mode.
         [ nil, /\[.*\n/, :tjp, method('startMacroDef') ],
         # The end of the macro is marked by a ']' that is immediately followed
         # by a line break. It switches the scanner back to tjp mode.
-        [ 'MACRO', /.*\]\n/, :macroDef, method('endMacroDef') ],
+        [ :MACRO, /.*\]\n/, :macroDef, method('endMacroDef') ],
         # Any line not containing the start or end.
         [ nil, /.*\n/, :macroDef, method('midMacroDef') ],
 
         # Some multi-char literals.
-        [ 'LITERAL', /<=?/ ],
-        [ 'LITERAL', />=?/ ],
-        [ 'LITERAL', /!=?/ ],
+        [ :LITERAL, /<=?/ ],
+        [ :LITERAL, />=?/ ],
+        [ :LITERAL, /!=?/ ],
 
         # Everything else is returned as a single-char literal.
-        [ 'LITERAL', /./ ]
+        [ :LITERAL, /./ ]
       ]
 
       super(masterFile, messageHandler, tokenPatterns, :tjp)
@@ -159,15 +159,28 @@ class TaskJuggler
     end
 
     def to_time(type, match)
-      h, m, s = match.split(':')
+      h, m = match.split(':')
       h = h.to_i
+      if h < 0 || h > 24
+        error('time_bad_hour', "Hour #{h} out of range (0 - 24)")
+      end
       m = m.to_i
-      s = 0 if s.nil?
-      [ type, h * 3600 + m * 60 + s ]
+      if m < 0 || h > 59
+        error('time_bad_minute', "Minute #{m} out of range (0 - 59)")
+      end
+      if h == 24 && m != 0
+        error('time_bad_time', "Time #{match} cannot be larger then 24:00")
+      end
+
+      [ type, (h * 60 + m) * 60 ]
     end
 
     def to_date(type, match)
-      [ type, TjTime.new(match) ]
+      begin
+        [ type, TjTime.new(match) ]
+      rescue TjException => msg
+        error('time_error', msg)
+      end
     end
 
     def newPos(type, match)
@@ -216,7 +229,7 @@ class TaskJuggler
       self.mode = :tjp
       # Remove the trailing " and remove the backslashes from escaped ".
       @string += match[0..-2].gsub(/\\"/, '"')
-      [ 'STRING', @string ]
+      [ :STRING, @string ]
     end
 
     def startStringSQ(type, match)
@@ -236,7 +249,7 @@ class TaskJuggler
       self.mode = :tjp
       # Remove the trailing ' and remove the backslashes from escaped '.
       @string += match[0..-2].gsub(/\\'/, "'")
-      [ 'STRING', @string ]
+      [ :STRING, @string ]
     end
 
     def startStringSZR(type, match)
@@ -274,7 +287,7 @@ class TaskJuggler
 
     def endStringSZR(type, match)
       self.mode = :tjp
-      [ 'STRING', @string ]
+      [ :STRING, @string ]
     end
 
     def startMacroDef(type, match)
@@ -293,7 +306,7 @@ class TaskJuggler
       self.mode = :tjp
       # Remove "]\n"
       @macroDef += match[0..-3]
-      [ 'MACRO', @macroDef ]
+      [ :MACRO, @macroDef ]
     end
 
     def startMacroCall(type, match)

data/lib/PropertyList.rb

@@ -168,19 +168,21 @@ class TaskJuggler
     def addSortingCriteria(criteria, up, scIdx)
       unless @propertySet.knownAttribute?(criteria) ||
              @propertySet.hasQuery?(criteria, scIdx)
-        raise "Unknown attribute #{criteria} used for sorting criterium"
+        raise TjException.new,
+              "Unknown attribute #{criteria} used for sorting criterium"
       end
       if scIdx == -1
         if @propertySet.scenarioSpecific?(criteria)
-          raise "Attribute #{criteria} is scenario specific." +
+          raise TjException.new,
+                "Attribute #{criteria} is scenario specific." +
                 "You must specify a scenario id."
         end
       else
         if @propertySet.project.scenario(scIdx).nil?
-          raise "Unknown scenario index #{scIdx} used."
+          raise TjException.new, "Unknown scenario index #{scIdx} used."
         end
         if !@propertySet.scenarioSpecific?(criteria)
-          raise "Attribute #{criteria} is not scenario specific"
+          raise TjException.new, "Attribute #{criteria} is not scenario specific"
         end
       end
       @sortingCriteria.push(criteria)

data/lib/Query.rb

@@ -70,8 +70,6 @@ class TaskJuggler
       if date.is_a?(TjTime)
         @start = date
         @startIdx = @project.dateToIdx(date, true)
-      #elsif date.is_a?(Fixnum)
-      #  self.startIdx=(date)
       else
         raise "Unsupported type #{date.class}"
       end
@@ -81,8 +79,6 @@ class TaskJuggler
       if idx.is_a?(Fixnum)
         @startIdx = idx
         @start = @project.idxToDate(idx)
-      #elsif idx.is_a?(TjTime)
-      #  self.start=(idx)
       else
         raise "Unsupported type #{idx.class}"
       end
@@ -92,8 +88,6 @@ class TaskJuggler
       if date.is_a?(TjTime)
         @end = date
         @endIdx = @project.dateToIdx(date, true)
-      #elsif date.is_a?(Fixnum)
-      #  self.endIdx=(date)
       else
         raise "Unsupported type #{date.class}"
       end
@@ -103,8 +97,6 @@ class TaskJuggler
       if idx.is_a?(Fixnum)
         @endIdx = idx
         @end = @project.idxToDate(idx)
-      #elsif idx.is_a?(TjTime)
-      #  self.end=(idx)
       else
         raise "Unsupported type #{idx.class}"
       end

data/lib/RealFormat.rb

@@ -37,7 +37,7 @@ class TaskJuggler
         end
       elsif args.length == 5
         # The argument is a list of values.
-        0.upto(4) do |i|
+        args.length.times do |i|
           instance_variable_set(iVars[i], args[i])
         end
       else

data/lib/Resource.rb

@@ -56,7 +56,7 @@ class TaskJuggler
       # Sort all entries in buckets by their alert level.
       numberOfLevels = project['alertLevels'].length
       listByLevel = []
-      0.upto(numberOfLevels - 1) { |i| listByLevel[i] = [] }
+      numberOfLevels.times { |i| listByLevel[i] = [] }
       list.each do |entry|
         listByLevel[entry.alertLevel] << entry
       end

data/lib/ResourceScenario.rb

@@ -37,15 +37,23 @@ class TaskJuggler
       # subset.
       @scoreboard = nil
 
+      # The index of the earliest booked time slot.
       @firstBookedSlot = nil
+      # Same but for each assigned resource.
+      @firstBookedSlots = {}
+      # The index of the last booked time Slot.
       @lastBookedSlot = nil
+      # Same but for each assigned resource.
+      @lastBookedSlots = {}
+
+      @dCache = DataCache.instance
     end
 
     # This method must be called at the beginning of each scheduling run. It
     # initializes variables used during the scheduling process.
     def prepareScheduling
       @property['effort', @scenarioIdx] = 0
-      initScoreboard
+      initScoreboard if @property.leaf?
 
       setDirectReports
     end
@@ -91,7 +99,6 @@ class TaskJuggler
       end
     end
 
-
     def preScheduleCheck
       a('managers').each do |manager|
         unless manager.leaf?
@@ -106,6 +113,25 @@ class TaskJuggler
       end
     end
 
+    # This method does some housekeeping work after the scheduling is
+    # completed. It's meant to be called for top-level resources and then
+    # recursively descends into all child resources.
+    def finishScheduling
+      # Recursively descend into all child resources.
+      @property.children.each do |resource|
+        resource.finishScheduling(@scenarioIdx)
+      end
+
+      if (parent = @property.parent)
+        # Add the assigned task to the parent duties list.
+        a('duties').each do |task|
+          unless parent['duties', @scenarioIdx].include?(task)
+            parent['duties', @scenarioIdx] << task
+          end
+        end
+      end
+    end
+
     def postScheduleCheck
       if a('fail') || a('warn')
         queryAttrs = { 'project' => @project,
@@ -158,6 +184,11 @@ class TaskJuggler
     def book(sbIdx, task, force = false)
       return false if !force && !available?(sbIdx)
 
+      # Make sure the task is in the list of duties.
+      unless a('duties').include?(task)
+        @property['duties', @scenarioIdx] << task
+      end
+
       #puts "Booking resource #{@property.fullId} at " +
       #     "#{@scoreboard.idxToDate(sbIdx)}/#{sbIdx} for task #{task.fullId}\n"
       @scoreboard[sbIdx] = task
@@ -170,17 +201,24 @@ class TaskJuggler
       end
       a('limits').inc(@scoreboard.idxToDate(sbIdx)) if a('limits')
 
-      # Make sure the task is in the list of duties.
-      unless a('duties').include?(task)
-        @property['duties', @scenarioIdx] << task
-      end
-
+      # Scoreboard iterations are fairly expensive but they are very frequent
+      # operations in later processing. To limit the interations to the
+      # relevant intervals, we store the interval for all bookings and for
+      # each individual task.
       if @firstBookedSlot.nil? || @firstBookedSlot > sbIdx
         @firstBookedSlot = sbIdx
       end
       if @lastBookedSlot.nil? || @lastBookedSlot < sbIdx
         @lastBookedSlot = sbIdx
       end
+      if task
+        if @firstBookedSlots[task].nil? || @firstBookedSlots[task] > sbIdx
+          @firstBookedSlots[task] = sbIdx
+        end
+        if @lastBookedSlots[task].nil? || @lastBookedSlots[task] < sbIdx
+          @lastBookedSlots[task] = sbIdx
+        end
+      end
       true
     end
 
@@ -291,6 +329,16 @@ class TaskJuggler
     # Returns the work of the resource (and its children) weighted by their
     # efficiency.
     def getEffectiveWork(startIdx, endIdx, task = nil)
+      # There can't be any effective work if the start is after the end or the
+      # todo list doesn't contain the specified task.
+      return 0.0 if startIdx >= endIdx || (task && !a('duties').include?(task))
+
+      # The unique key we use to address the result in the cache.
+      key = [ self, :ResourceScenarioEffectiveWork, startIdx, endIdx,
+              task ].hash
+      work = @dCache.load(key)
+      return work if work
+
       # Convert the interval dates to indexes if needed.
       startIdx = @project.dateToIdx(startIdx, true) if startIdx.is_a?(TjTime)
       endIdx = @project.dateToIdx(endIdx, true) if endIdx.is_a?(TjTime)
@@ -307,7 +355,7 @@ class TaskJuggler
                  getAllocatedSlots(startIdx, endIdx, task) *
                  @project['scheduleGranularity']) * a('efficiency')
       end
-      work
+      @dCache.store(work, key)
     end
 
     # Returns the allocated accumulated time of this resource and its children.
@@ -419,16 +467,13 @@ class TaskJuggler
     # the period specified with the Interval _iv_. If task is not nil
     # only allocations to this tasks are respected.
     def allocated?(iv, task = nil)
-      initScoreboard if @scoreboard.nil?
+      return false if task && !a('duties').include?(task)
 
-      startIdx = @scoreboard.dateToIdx(iv.start, true)
-      endIdx = @scoreboard.dateToIdx(iv.end, true)
+      startIdx = @project.dateToIdx(iv.start, true)
+      endIdx = @project.dateToIdx(iv.end, true)
 
-      startIdx = @firstBookedSlot if @firstBookedSlot &&
-                                     startIdx < @firstBookedSlot
-      endIdx = @lastBookedSlot + 1 if @lastBookedSlot &&
-                                      endIdx < @lastBookedSlot + 1
-      return false if startIdx > endIdx
+      startIdx, endIdx = fitIndicies(startIdx, endIdx, task)
+      return false if startIdx >= endIdx
 
       return allocatedSub(startIdx, endIdx, task)
     end
@@ -461,8 +506,6 @@ class TaskJuggler
               bookings[lastTask] = Booking.new(@property, lastTask, [])
             end
 
-            # Make sure the index is correct even for the last task block.
-            idx += 1 if idx == endIdx
             # Append the new interval to the Booking.
             bookings[lastTask].intervals <<
               Interval.new(@scoreboard.idxToDate(bookingStart),
@@ -494,18 +537,15 @@ class TaskJuggler
     # Count the booked slots between the start and end index. If _task_ is not
     # nil count only those slots that are assigned to this particular task.
     def getAllocatedSlots(startIdx, endIdx, task)
-      # To speedup the counting we start with the first booked slot and end
-      # with the last booked slot.
-      startIdx = @firstBookedSlot if @firstBookedSlot &&
-                                     startIdx < @firstBookedSlot
-      endIdx = @lastBookedSlot + 1 if @lastBookedSlot &&
-                                      endIdx > @lastBookedSlot + 1
+      # If there is no scoreboard, we don't have any allocations.
+      return 0 unless @scoreboard
+
+      startIdx, endIdx = fitIndicies(startIdx, endIdx, task)
+      return 0 if startIdx >= endIdx
 
-      initScoreboard if @scoreboard.nil?
       bookedSlots = 0
-      startIdx.upto(endIdx - 1) do |idx|
-        if (task.nil? && @scoreboard[idx].is_a?(Task)) ||
-           (task && @scoreboard[idx] == task)
+      @scoreboard.each(startIdx, endIdx) do |slot|
+        if (task.nil? && slot.is_a?(Task)) || (task && slot == task)
           bookedSlots += 1
         end
       end
@@ -515,7 +555,9 @@ class TaskJuggler
 
     # Count the free slots between the start and end index.
     def getFreeSlots(startIdx, endIdx)
-      initScoreboard if @scoreboard.nil?
+      return 0 if startIdx >= endIdx
+
+      initScoreboard unless @scoreboard
 
       freeSlots = 0
       startIdx.upto(endIdx - 1) do |idx|
@@ -528,7 +570,9 @@ class TaskJuggler
     # Count the regular work time slots between the start and end index that
     # have been blocked by a vacation.
     def getVacationSlots(startIdx, endIdx)
-      initScoreboard if @scoreboard.nil?
+      return 0 if startIdx >= endIdx
+
+      initScoreboard unless @scoreboard
 
       vacationSlots = 0
       startIdx.upto(endIdx - 1) do |idx|
@@ -603,6 +647,24 @@ class TaskJuggler
       end
     end
 
+    # Limit the _startIdx_ and _endIdx_ to the actually assigned interval.
+    # If _task_ is provided, fit it for the bookings of this particular task.
+    def fitIndicies(startIdx, endIdx, task = nil)
+      if task
+        startIdx = @firstBookedSlots[task] if @firstBookedSlots[task] &&
+                                              startIdx < @firstBookedSlots[task]
+        endIdx = @lastBookedSlots[task] + 1 if @lastBookedSlots[task] &&
+                                               endIdx >
+                                               @lastBookedSlots[task] + 1
+      else
+        startIdx = @firstBookedSlot if @firstBookedSlot &&
+                                       startIdx < @firstBookedSlot
+        endIdx = @lastBookedSlot + 1 if @lastBookedSlot &&
+                                        endIdx > @lastBookedSlot + 1
+      end
+      [ startIdx, endIdx ]
+    end
+
     def setReports_i(reports)
       if reports.include?(@property)
         # A manager must never show up in the list of his/her own reports.
@@ -638,7 +700,10 @@ class TaskJuggler
                                                task)
         end
       else
-        return false unless a('duties').include?(task)
+        return false unless @scoreboard && a('duties').include?(task)
+
+        startIdx, endIdx = fitIndicies(startIdx, endIdx, task)
+        return false if startIdx >= endIdx
 
         startIdx.upto(endIdx - 1) do |idx|
           return true if @scoreboard[idx] == task