pmayd / OPT

Github der Operation Pandora Trigger
GNU General Public License v3.0
6 stars 3 forks source link

Flaggenziehradius zu groß #45

Closed Frozen-byte closed 5 years ago

Frozen-byte commented 5 years ago

Ich kann mit dem FlaggenZieh-Hotkey die Flagge aus 20 Metern ziehen, selbst wenn der Radius auf 5m eingestellt wurde.

Der Radius aus der Konfiguration sollte funktionieren.

In jeder Testmission reproduzierbar, wenn der Hotkey gesetzt und betätigt wird.

GNCLORD-MDB commented 5 years ago

Ich habe mir gerade das script angesehen und da wir nirgenswo eine distance abgefragt dafür, ich kann nix finden. Da wird ungefilter die max länge ob cursorObject genomme, und die kann auch mehr als 20m dann sein.

senshi-x commented 5 years ago

Klar steht die Distanz im Skript. https://github.com/pmayd/OPT/blob/b5d69edd54795235c5b4facc91cddb6b785ade12/[J]OPT.lythium/sectorcontrol/functions/fnc_setupFlag.sqf#L74

Ob sie funktioniert und richtig ausgelesen wird ist eine andere Frage.

pmayd commented 5 years ago

Danke, hätte mich auch sehr stark gewundert, sowas triviales übersehen zu haben.

Ich teste das mal eben erstmal lokal, ehe man sich im Code verliert

pmayd commented 5 years ago

Nein, @WGPSenshi du hast die AddAction-Einträge verlinkt, es geht aber um das zusätzliche Ziehen per Hotkey, das findet in der Datei postInit bzw. die Überprüfung in fnc_captureFlagCondition statt

pmayd commented 5 years ago

Oh super ich kann die Mission nicht mehr testen, da wir kein Lythium mehr im Repo haben

pmayd commented 5 years ago

Dh wir müssen erstmal irgendwie in Git eine lauffähige Karte aufnehmen damit man überhaupt wieder testen kann, ich schau mal was es da aktuell gibt...man das ist nervig.

Ah ok @lythium ist wohl noch im Ordner, nur nicht mehr aktuell geladen, zum Testen reichts damit. Aber vielleicht finden wir mal ne cleverere Lösung um Mission auf neue Karten zu portieren

pmayd commented 5 years ago

Und siehe da, die Karte wirft natürlich Fehler weil jbads fehlt :p. Einfach mitladen via additional options in ArmASync und man kann erstmal wieder arbeiten

pmayd commented 5 years ago

Ok Fehler ist tatsächlich vorhanden und übel, danke für das Issue! Wird sofort behoben

GNCLORD-MDB commented 5 years ago

Wegen der Karte, würde ich Altis oder Tanoa als Basis nehmen. für neue Karten brauch man ja immer nur die Scripte kopieren. Die Karte brauche keine Addons und geht immer.

pmayd commented 5 years ago

Genau, das besprechen wir Do mit Kalle, so habe ich das mit ihm auch besprochen. Allerdings ist es mit den Skripten leider nicht getan, da wir einige Abhängigkeiten wie Variablennamen (Radar) oder Trigger haben. Da müssen wir überlegen, wie man das geschickter lösen kann oder ob alles dann eben auf der Basiskarte in Altis vorhanden sein muss. Wenn man dann aber plötzlich zwei Basen will muss man wissen wo man z.B. neue Trigger einträgt etc. Also am besten gehen wir alle Systeme durch, die aktuell ein Objekt im Editor benötigen (Service-Pad, Beam, etc). Einige brauchen Objekte wie Trigger, die nicht so offensichtlich sind wie Skripte, die über Trigger direkt per Init aufgerufen werden. Vielleicht können wir das optimieren.

@WGPSenshi hierzu Ideen?

GNCLORD-MDB commented 5 years ago

man kann auch die Mission,sqm mit kopieren und die dann in Editor alles verschieben. Muss nur in der Mission.sqm die neu Karte eintragen Man muss nur bei Editor Änderungen in der Basis Mission dann aufpassen, die müssen dann auch in die Kampagekarte übertragen werden

pmayd commented 5 years ago

Ja eben und deswegen wäre es ideal wenn die Mission eigentlich nur aus Skripten besteht und die Karte eben auf einer beliebigen Insel sein kann, ohne zu viele Abhängigkeiten. Man könnte z.B. die Namen von Objekten per CBA settings einstellbar machen, ebenso notwendige Trigger. Aber besprechen wir am Do, was da geht und ob kopieren eine sinnvolle Lösung ist. Ich weiß, dass man den Ordner einfach umbenennen kann, also mission.sqm mitnehmen, aber beim Verschieben geht normalerweise auch der genaue Offset der Gegenstände verloren, davon bin ich nicht so ein Freund, aber bis auf die Drohnenstation haben wir kaum Objekte, die super fein ausgerichtet werden müssen.