search

LSS-Manager / lss-manager-v2

Diese Version wird nicht mehr Supported!
www.lss-manager.de
2 stars 0 forks source link

Bugfixes, Style Changes #13

Closed eaglefsd closed 8 years ago

eaglefsd commented 8 years ago

Manche Funktionen wurden doppelt aufgerufen, jQuery Animation war in einer Schleife von 200 mal, viele Semikolons fehlten, Switches sind schöner als ifelse, manche Variablen wurden nicht definiert. building_maps_draw nicht korrigiert, da zu unübersichtlich und quasi alles falsch...

Tsumiki-Chan commented 8 years ago

Erstmal vielen Dank für deine Mühe. Ich bin grad mal deinen Merge-Request überflogen und muss erstmal sagen, dass alles soweit clean ist. Gut, bei building_maps_redraw musste ich zwangsweise die LSS-Funktion überschreiben, da es mir sonst nicht möglich war die Fahrzeuge anzeigen zu lassen. Habe also quasi den Code 1:1 übernommen und nicht weiter auseinander gefummelt, weil ich faul war.

Mir sind aber 1-2 Sachen aufgefallen: 1 http://prntscr.com/9tidkw Wenn du das kurz mit unserem Code vergleichst, wirst du festgestellt haben, dass ich mit einer sehr langen CSS-Definition gearbeitet hatte. Das hatte den Grund, dass bei der Originalfunktion (so wie du sie requestet hast), immer der erste Verbandseinsatz ausgewählt wurde.

2 http://prntscr.com/9tieqp Du hattest die Variable "org_radioMessage" entfernt. Hattest du das auch kurzgetestet? Afaik würde der jetzt in einem infinite loop stecken bleiben, weil die Funktion "radioMessage" überschrieben wurde, du jetzt aber Immer wieder "radioMessage(e);" aufrufst, also die Funktion rekursiv ist.

Auch fehlt da jetzt die Anzeige von neuen Chatnachrichten und Sprechwünschen. Das ist nicht dein Fehler, sondern meiner, weil ich das Mergen verpennt habe :(

eaglefsd commented 8 years ago

1. Die Verbandeinsätze liegen (normalerweise) in #mission_list_alliance. Wenn ihr das geändert habt, dann trag ich gleich wieder euren Code ein.

2. Aaaaach, jetzt ist mir erst klar geworden, was ihr machen wollt. Einfacher wäre es da, wenn ihr das Script per script-Tag direkt in die Seite packt und dann per Faye alles abfangt.

Tsumiki-Chan commented 8 years ago

Zu 1: Okay, ich habe deinen Code falsch gelesen. Ne, das ist okay so. Du hast den komplizierten weg den ich gegangen bin vereinfacht.

Zu 2: Vielleicht hast du mal Zeit zu skypen? Also wir binden ja unser script über den autoupdater mittlerweile direkt in die Seite ein. Wenn man mit Tampermonkey o.Ä das Script direkt einbindet, dann funktionieren einige Sachen (wie z.b. das Überschreiben der "building_maps_redraw") nicht. Ich bin aber noch nicht hinter Faye gestiegen. Hab es in deinem Script gesehen, bin aber zu blond gewesen um es zu verstehen.