search

ZhooL / TSX_EnhancedVehicle

FS19. Farming Simulator 19. LS19. Landwirtschafts Simulator 19. Shuttle shift, differential locks, wheel drive modes and better hydraulics controls for your vehicle. Shows more vehicle details on HUD.
https://www.modhoster.de/mods/enhancedvehicle
83 stars 24 forks source link

Kompatibilität mit KeyboardSteer #25

Closed Mogli12 closed 5 years ago

Mogli12 commented 5 years ago

Hallo,

Hier mein Vorschlag. Ich habe diese Issue und eins in KeyboardSteer angelegt. https://github.com/Mogli12/KeyboardSteer/issues/118

Hier können wir die notwendigen Änderungen und die abgestimmten Versionsnummer austauschen. Mein erster Schwung von Anpassungen ist in https://github.com/Mogli12/KeyboardSteer/commit/05b7d5d8f73234e3a2a280943f8814945cde7dec.

Gruß, Stefan

Mogli12 commented 5 years ago

Hallo,

ich überlege gerade, KeyboardSteer in 2 Mods zu zerschlagen. Eventuell reaktiviere ich dazu dann doch noch GearboxAddon, aber mit stark eingeschränkter Funktionalität. Shuttle Control wäre dann auch im anderen Mod. Wie wollen wir uns da synchronisieren?

Gruß, Stefan

ZhooL commented 5 years ago

Hi,

gerne. Ich würde meinerseits weiterhin auf Shuttle Control verzichten, wenn eins von Dir erkannt wird (es sei denn der User schaltet es aktiv bei dir aus und bei mir an). Ich nehme mal an, die Shuttle-Funktionalität steckt/bleibt bei Dir in FS19_VehicleControlAddon? Ich halte dann Ausschau nach Funktionen bei Dir, woran ich den Shuttle-Status erkennen kann. Wenn Du auch Diffs und Hydraulik bei Dir machen möchtest - auch da finden wir einen Weg.

Gruß Mario

Mogli12 commented 5 years ago

Halo Mario,

ich habe die neue Methode self:vcaExternalGetShuttleCtrl() angelegt. Du brauchst also nicht nachschauen, ob der Mod geladen ist, sondern nur

    local vcaShuttleCtrl = false 
    if type( self.vcaExternalGetShuttleCtrl ) == "function" then 
      vcaShuttleCtrl = self:vcaExternalGetShuttleCtrl()
   end

Ich habe auch nochmal alle Funktionen unter dem alten Name (z.B. ksmExternalSetShuttleCtrl) bei self eingefügt.

Gruß, Stefan

Mogli12 commented 5 years ago

Hallo Mario,

Ich denke, dass wir das jetzt schließen können.

Viele Grüße, Stefan

ZhooL commented 5 years ago

agreed :-)