Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add keep engine of vehicles running after getting out #6885

Merged
merged 14 commits into from
Apr 1, 2019

Conversation

brainslush
Copy link
Contributor

When merged this pull request will:

  • Adds a feature which leaves the vehicle engine running after getting out. So, it requires the driver to switch off the engine of the vehicle.
    I tested it locally and have tested it numerous times as part of another mod before moving it to ACE3. Please retest as I could made have mistakes while adapting the code for ACE3.

addons/vehicles/XEH_postInit.sqf Outdated Show resolved Hide resolved
addons/vehicles/XEH_PREP.hpp Outdated Show resolved Hide resolved
addons/vehicles/CfgEventHandlers.hpp Outdated Show resolved Hide resolved
addons/vehicles/CfgEventHandlers.hpp Outdated Show resolved Hide resolved
addons/vehicles/functions/fnc_keepEngineRunning.sqf Outdated Show resolved Hide resolved
addons/vehicles/functions/fnc_keepEngineRunning.sqf Outdated Show resolved Hide resolved
addons/vehicles/initSettings.sqf Outdated Show resolved Hide resolved
addons/vehicles/stringtable.xml Outdated Show resolved Hide resolved
addons/vehicles/stringtable.xml Outdated Show resolved Hide resolved
@commy2
Copy link
Contributor

commy2 commented Mar 22, 2019

Do we even need setVariable public for this?

I would do the following things:

  • move the event handlers from config into preInit, because config is butt ugly
  • change all setVar pub to setVar
  • change Init to InitPost
  • use engineOn event from common

This should handle the correct state for JIP, because it is either the state when you connect as client, or whicher was last toggled by Engine event. InitPost and Engine are event with global args, so no strain on the network is needed.

@jonpas jonpas added the kind/feature Release Notes: **ADDED:** label Mar 22, 2019
@jonpas jonpas added this to the 3.12.7 milestone Mar 22, 2019
Copy link
Contributor

@commy2 commy2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments

@commy2 commy2 self-assigned this Mar 31, 2019
@jonpas jonpas merged commit 978049e into acemod:master Apr 1, 2019
@PabstMirror PabstMirror modified the milestones: 3.12.7, 3.13.0 Jul 5, 2019
BaerMitUmlaut pushed a commit that referenced this pull request Aug 5, 2019
Adds a feature which leaves the vehicle engine running after getting out. So, it requires the driver to switch off the engine of the vehicle.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Release Notes: **ADDED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants