#44 Vertretungslehrer in den Benachrichtigungen und Karten anzeigen

Closed
opened 4 years ago by Bixilon · 8 comments

Mir fehlt eine wichtige Information im Kartenmodus: Die Spalte "Vertreter". Diese bitte noch anzeigen

Mir fehlt eine wichtige Information im Kartenmodus: Die Spalte "Vertreter". Diese bitte noch anzeigen
fynngodau commented 4 years ago
Owner

Hat nichts mit dem Kartenmodus an sich zu tun, sondern mit dieser Zeile: dir wird stattdessen die (leere??) Spalte "Vertr. von" angezeigt.

Hat nichts mit dem Kartenmodus an sich zu tun, sondern mit [dieser Zeile](https://notabug.org/fynngodau/DSBDirect/src/b7e882c38961762fabdc5173870c2a33bac56a05/app/src/main/java/godau/fynn/dsbdirect/table/reader/Untis.java#L255): dir wird stattdessen die (leere??) Spalte "Vertr. von" angezeigt.
Moritz Zwerger commented 4 years ago
Poster

Lustig. Die Spalte ist bei uns tatsächlich leer. Dann könnten (sollten?) wir wohl die 2 Zeilen (254 und 255) vertauschen, damit die richtige Spalte eine höhere Priorität hat, oder? Oder überprüfen, ob die Spalte leer ist und dann einfach ignorieren.

Lustig. Die Spalte ist bei uns tatsächlich leer. Dann könnten (sollten?) wir wohl die 2 Zeilen (254 und 255) vertauschen, damit die richtige Spalte eine höhere Priorität hat, oder? Oder überprüfen, ob die Spalte leer ist und dann einfach ignorieren.
fynngodau commented 4 years ago
Owner

Vertauschen würde nicht helfen, da die Spalten zeilenweise von vorne nach hinten in die "Master-Tabelle" geschrieben werden und somit "Vert. von" immer "Vertreter" überschreibt.

Vertauschen würde nicht helfen, da die Spalten zeilenweise von vorne nach hinten in die "Master-Tabelle" geschrieben werden und somit "Vert. von" immer "Vertreter" überschreibt.
Moritz Zwerger commented 4 years ago
Poster

Update: Diese Spalte ist bei uns zu 99% leer. Allerdings, wenn eine Klasse Verlegung hat, wird in der Spalte die Stunde angezeigt, wann die Stunde eigentlich statt finden würde.

Das sieht wie folgt aus: "Mi-11.9. / 7". Evtl sollten wir per regex überprüfen (oder einfach, wenn die Spalte leer ist oder ob Nummern, Punkte, ein Slash und ein Minus enthalten sind.), ob die eigentliche Spalte nicht Vertreter ist. Betrifft evtl. ja auch andere Schulen. Wenn dies nicht der Fall ist, wird diese für den Vertretungslehrer benutzt.

Ich würde ja gerne Fragen, warum wir DSB verwenden und ob wir diese Spalte nicht evtl. umbenennen könnten, kann mich aber die nächste Zeit nicht vor dem Sekreteriat blicken lassen ^)

Update: Diese Spalte ist bei uns zu 99% leer. Allerdings, wenn eine Klasse Verlegung hat, wird in der Spalte die Stunde angezeigt, wann die Stunde eigentlich statt finden würde. Das sieht wie folgt aus: "Mi-11.9. / 7". Evtl sollten wir per regex überprüfen (oder einfach, wenn die Spalte leer ist oder ob Nummern, Punkte, ein Slash und ein Minus enthalten sind.), ob die eigentliche Spalte nicht Vertreter ist. Betrifft evtl. ja auch andere Schulen. Wenn dies nicht der Fall ist, wird diese für den Vertretungslehrer benutzt. Ich würde ja gerne Fragen, warum wir DSB verwenden und ob wir diese Spalte nicht evtl. umbenennen könnten, kann mich aber die nächste Zeit nicht vor dem Sekreteriat blicken lassen ^)
fynngodau commented 4 years ago
Owner

"Mi-11.9. / 7"

Ich glaube nicht, dass dieses Format standartisiert ist und zwangsläufig bei anderen Schulen, die sie haben, genau so aussieht. Es kann auch gut sein, dass die Spalte von Hand ausgefüllt wird.

Ich weiß nicht mehr, warum die Spalte als Vertretungslehrerspalte in die Master-Tabelle eingeht. Wie es aussieht, ist das eine Zeitangabe, die also in die Info-Spalte sollte.

Also wäre es sinnvoll, sie in der Master-Tabelle als neue Spalte einzutragen und mit der Info-Spalte zu mergen (smartConcatenate).

> "Mi-11.9. / 7" Ich glaube nicht, dass dieses Format standartisiert ist und zwangsläufig bei anderen Schulen, die sie haben, genau so aussieht. Es kann auch gut sein, dass die Spalte von Hand ausgefüllt wird. Ich weiß nicht mehr, warum die Spalte als Vertretungslehrerspalte in die Master-Tabelle eingeht. Wie es aussieht, ist das eine Zeitangabe, die also in die Info-Spalte sollte. Also wäre es sinnvoll, sie in der Master-Tabelle als neue Spalte einzutragen und mit der Info-Spalte zu mergen (`smartConcatenate`).
Moritz Zwerger commented 4 years ago
Poster

Würde Sinn machen, allerdings kann es auch sein, dass beide ausgefüllt sind. Wenn die Spalten gemergt werden, kann es sein, dass der User keine Ahnung mehr hat, da evtl. 2 Sätze zusammenhanglos in der App stehen

Würde Sinn machen, allerdings kann es auch sein, dass beide ausgefüllt sind. Wenn die Spalten gemergt werden, kann es sein, dass der User keine Ahnung mehr hat, da evtl. 2 Sätze zusammenhanglos in der App stehen
fynngodau commented 4 years ago
Owner

Im Fall, dass zwei befüllte Spalten in eine vereint werden müssen, werden die beiden ursprünglichen Spalten im Augenblick durch ein · getrennt.

Im Fall, dass zwei befüllte Spalten in eine vereint werden müssen, werden die beiden ursprünglichen Spalten im Augenblick durch ein ` · ` getrennt.
fynngodau commented 4 years ago
Owner

"Vert. von" wird jetzt auf die neue Master-Spale OLD_TIME remappt, was in der Info-Spalte auftaucht, die sich jetzt so zusammensetzt: TYPE · OLD_TIME · ROOM · INFO

Dabei ergibt sich folgendes Bild. Die Vertretungslehrer werden jetzt also auch bei diesem Plan richtig dargestellt.

"Vert. von" wird jetzt auf die neue Master-Spale `OLD_TIME` remappt, was in der Info-Spalte auftaucht, die sich jetzt so zusammensetzt: `TYPE · OLD_TIME · ROOM · INFO` Dabei ergibt sich folgendes Bild. Die Vertretungslehrer werden jetzt also auch bei diesem Plan richtig dargestellt.
Sign in to join this conversation.
No Milestone
No assignee
2 Participants
Loading...
Cancel
Save
There is no content yet.