Assign rooms correctly

In a coupled lesson, if a room is not assigned to a learning group (in the example FrK), Chronos fills the gap from below so that the rooms are no longer assigned to the correct lesson. The expected representation can be seen in Figure 2.

assigned to @hansegucker

moved from AlekSIS-App-Chronos#148 (moved)

The problem could also be that in this case two courses are taught by one teacher in the same room. I have now solved this for the time being by swapping the order of the courses in Untis.

assigned to @hansegucker

added part::backend prio::1 source::customer type::bug labels

Here are some sample data from Untis:

25~0~26~47;~0;~0~0~0~~~~~~~~n~~;17;18;19;20;~~~0~0~137~0~0~,
25~0~123~47;~0;~0~0~13000~~~~~~~~n~~;17;18;19;20;~~~0~0~512~0~0~,
32~0~58~33;~0;~0~0~0~~~~~~~~n~~;17;18;19;20;~~~0~0~416~0~0~,
94~0~45~27;~0;~0~0~0~~~~~~~~n~~;17;18;19;20;~~~0~0~138~0~0~ 

~2~5~;47;33;27;~~,
~2~6~;47;33;27;~~,
~4~1~;47;33;27;~~ 

The only difference I can see is the 13000 at the line which shouldn't have a room. But I am unsure whether this is a safe mark.

Okay, in general there seem to be these rules:

1. The number in the fourth field (47/47/33/27) is a room id, but not of the real room, but of the base room.
2. If the id is repeated, the id will appear only one time in the time data (47) – so both lesson periods are in the same room.
3. If there is no room in the fourth field, there will be also no room in the time data.

(Untis' data structure is the best one I have ever seen.)

created merge request !84 (merged) to address this issue

mentioned in merge request !84 (merged)

Correction: If there are two lesson parts with the same teacher, they will use the same room, but the room will appear just one time in the room list.

closed with merge request !84 (merged)

mentioned in commit 29b6c4ca