Bugfix in log event processor

VadereSimulator/src/org/vadere/simulator/models/seating/LogEventProcessor.java

@@ -131,7 +131,12 @@ public class LogEventProcessor extends DataProcessor<IdDataKey, LogEventEntry> {
 
 		@Override
 		public void reachedTarget(Target target, Agent agent) {
-			writeSitDownEvent((Pedestrian) agent, seatNumber);
+			final Seat seat = trainModel.getSeatForTarget(target);
+			// This check is necessary because a second person could reach the
+			// target (and find that the seat is already taken).
+			// Order of listener invocations is not predictable, therefore both options are checked.
+			if (seat.isAvailable() || seat.getSittingPerson() == agent)
+				writeSitDownEvent((Pedestrian) agent, seatNumber);
 		}
 	}
 

VadereState/src/org/vadere/state/scenario/Target.java

@@ -15,7 +15,10 @@ public class Target implements ScenarioElement, Comparable<Target> {
 	private final AttributesTarget attributes;
 	private final Map<Integer, Double> enteringTimes;
 	
-	/** Collection of listeners - order does not play a role. */
+	/**
+	 * Collection of listeners - unordered because it's order is not predictable
+	 * (at least not for clients).
+	 */
 	private final Collection<TargetListener> targetListeners = new LinkedList<>();
 
 	public Target(AttributesTarget attributes) {