Diskussion:NLT Skripte

Aus NLT2Projekt
Wechseln zu: Navigation, Suche

select_alle_ausser_gruppenführer ist jetzt doppelt vorhanden. Ich meine, die Funktion sollte so nicht funktionieren, weil eine Held-Variable verbleibt und das

<select by-name="nlt2_held_ist_gruppenfuehrer" max="1"/>

bei Nichtvorhandensein auch nicht anspringen sollte.

Ich nehme an, dass die beiden letzten Methoden aus der KdS stammen und als funktionstüchtig gelten? Zumindest die erste könnte man aber durch eine hübsche for-Schleife ersetzen. Die zweite verstehe ich noch nicht (keine Zeit). --Trokhanor (Diskussion) 21:10, 19. Dez. 2015 (CET)

Doppeltes entfernt. Hm, ich glaube schon, dass das funktioniert hätte. Aber sicherheitshalber habe ich jetzt vorher bei der ganzen Gruppe die Variable erstmal auf 1 gesetzt, ehe sie beim Gruppenführer auf 2 gesetzt wird. Dann funktioniert es auf jeden Fall. War weniger Arbeit, als es auszuprobieren.

PS: Dass so mancher Code mittlerweile kompakter zu lösen wäre, ist sicherlich richtig. KdS und etliche Skripte entstanden zu einer Zeit, als Gruppen-AOQML noch absolut in den Kinderschuhen steckte. In den letzten zwei, drei Jahren ist vieles nachgerüstet worden. Ich möchte jetzt aber nicht unbedingt alles dahingehend optimieren. Das wäre mir zuviel Arbeit. Die Skripte, die aus KdS bereits bestehen und funktionieren, übernehme ich lieber. Da ich solche Sachen eh allesamt via include auslagere, bläht es ja den Fließtext auch nicht weiter auf und stört nirgends. Aber wenn du sie optimieren willst; gerne! ;) Ich verwende überall die Skripte, also wenn du sie auf der Skriptseite änderst, werden die Änderungen auch global übernommen. --Pergor (Diskussion) 21:35, 19. Dez. 2015 (CET)

Das sollte aber dann schon in das Skript, das mit der Held-Variablen für alle. Aus Interesse habe ich das auf dem TS getestet: mit max="1" geht es, mit min="3" nicht. Fällt für mich in die Grauzone zwischen Bug und Feature. --Trokhanor (Diskussion) 22:07, 19. Dez. 2015 (CET)