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

fix setting direction of remote units via zeus #816

Merged
merged 2 commits into from
Dec 8, 2017

Conversation

commy2
Copy link
Contributor

@commy2 commy2 commented Dec 5, 2017

When merged this pull request will:

  • title

It has been years. They will never fix this. This work around is very simple.

@commy2 commy2 added the Feature label Dec 5, 2017
@commy2 commy2 added this to the 3.5.1 milestone Dec 5, 2017
Copy link
Contributor

@Cuel Cuel left a comment

Choose a reason for hiding this comment

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

Tested with same code on dedicated with HC

@PabstMirror
Copy link
Contributor

maybe add
if (missionNamespace getVariable [QGVAR(disableZeusDirFix), false]) exitWith {};
Just in case this causes problems or is undesired?

@commy2
Copy link
Contributor Author

commy2 commented Dec 5, 2017

We could store the ehid in a object namespace variable on the logic.


if (!local _unit) then {
private _dir = getDir _unit;
[_unit, _dir] remoteExec ["setDir", _unit];
Copy link
Contributor

Choose a reason for hiding this comment

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

With doWatch you can also let tanks turn the gun. Would be nice to have ;D

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The turret is rotating fine with this fix without having to do doWatch.

Copy link
Member

@jonpas jonpas left a comment

Choose a reason for hiding this comment

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

Love it!

@commy2 commy2 merged commit d955c74 into master Dec 8, 2017
@commy2 commy2 deleted the fix-curator-remote-unit-direction branch December 8, 2017 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants