Description Johan Schulz 2011-04-29 11:13:17 EDT

Build Identifier: 3.6.2 - M20110210-1200

Type at line 5723, uses counter of enclosing for loop (i) not the current one (x).


if (hasSpecialDateRanges()) {           
	for (int i = 0; i < _specDateRanges.size(); i++) {
		final GanttSpecialDateRange range = (GanttSpecialDateRange) _specDateRanges.get(i);

		ArrayList failedMoves = new ArrayList();
		
		// check if any of the moved/resized events overlap any of our date ranges
		for (int x = 0; x < _dragEvents.size(); x++) {
			// final GanttEvent ge = (GanttEvent) _dragEvents.get( i ); // <- wrong
                        final GanttEvent ge = (GanttEvent) _dragEvents.get( x );    // <- correct one

			// event is not allowed to be on these dates, undo
			if (!range.canEventOccupy(ge.getActualStartDate(), ge.getActualEndDate())) {
				ge.moveCancelled();	                	
				_dragEvents.remove(ge);
				needsRedraw = true;
				failedMoves.add(ge);
			}                
		}

This bug causes the move and resize feature to be broken

Regards

Johan Schulz

Reproducible: Always

Steps to Reproduce:
1. Resize/Move GantEvent ( first time seems to work fine )
2. Resize/Move GantEvent, this time no further action possible
3.