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

Unable to load arrested people in vehicle #2199

Closed
Knight1981 opened this issue Aug 18, 2015 · 9 comments
Closed

Unable to load arrested people in vehicle #2199

Knight1981 opened this issue Aug 18, 2015 · 9 comments
Assignees
Milestone

Comments

@Knight1981
Copy link

ACE3 Version: 3.2.1

Mods:

Placed ACE3 Modules:
basic medic system

Description:
Hey guys,

yesterday we played a rescue mission on our dedicated server. Therefore we have arrested the person and escorted him to an evac helicopter (in this case it was a Little Bird). That worked very well, but than we were not able to load the arrested person in the vehicle because no menu / option for this appeared. Could you please check this?

Thank you very much. :)

Steps to reproduce:

  1. Arrest AI
  2. Escort to a Little Bird
  3. Try to load the arrested AI into the Helicopter

Where did the issue occur?
Multiplayer, dedicated server

RPT log file:
n.a

@PabstMirror
Copy link
Contributor

You have to be Escorting the captive to have the "load in vehicle" interaction show.
The action shows on the vehicle (you have to look through the captive).

There is also an action on the captive himself while escorting, but it is extremely awkward to access.

I would break this issue into 2 parts,

  • Code failure: Do you think you where using it correctly and it failed to work?
  • Design: For ease of use, would it make sense to remove the "isEscorting" requirement. IE, make it so a captive near a vehicle will just have a "load into vehicle" interaction on him. I don't see any harm, and I think it would make things cleaner and easier to use.

@PabstMirror PabstMirror added this to the Ongoing milestone Aug 19, 2015
@PabstMirror PabstMirror self-assigned this Aug 19, 2015
@Knight1981
Copy link
Author

Thank you for your answer. It is possible that we made a mistake, I will check it later and let you know if it works as you described.

But yeah, it is a little bit confusing (4 guys didn't know how to load the captive) :D I think it would be more easier when the captive is near the vehicle and you can look at him and say "load captive into vehicle". That would be the same behaviour as for patients.

Thank you :)

@Ghost77788
Copy link

This is because the check on helicopters is broken within the doloadcaptive.sqf.
Is resolved in next update for ACE 3

@Knight1981
Copy link
Author

Ah okay, thank you for this information :) We wait eagerly for the next ACE update because of some other hotfixes (e.g. camp fire issue, slide show broken etc.).

When can we expect the next release?

@bux
Copy link
Member

bux commented Aug 19, 2015

When can we expect the next release?

You won't get any ETA's on an official version from us. The version has to be ready codewise and has to be at least somewhat tested (preferably properly tested). But feel free to build it from the source and help us to test it. That would be very helpful and could probably increase our development process hence might lead to a somewhat earlier release.

@Knight1981
Copy link
Author

Okay I checked it again today and yes, it is a problem with the little bird. I can load the captive in a Hunter etc. but not in the Little Bird. I think this has nothing to do with the captive module because I get no "option" on the little bird as Ghost77788 described before.

I didn't checked it with other helicopters but I believe this is not only with the Little Bird. Thank you.

@PabstMirror
Copy link
Contributor

@Ghost77788 thank you, yes it was a code problem (#2132) that will be fixed for next release;

I still want to remove the escorting requirement, it should match the functionally of loading unconscious.

@kripto202
Copy link

what kind of heli were you trying to use?

@Ghost77788
Copy link

https://www.dropbox.com/s/huteenhfhmw1x4m/ace_captives.rar?dl=0

I made it now so that you can load the captive when not escorting either by interacting with captive or vehicle.
Only issue is: by removing exceptions[] = "isnotescorting"; from ACE_LoadCaptive, it no longer gives the option to load captive when escorting. so you would have to first release captive and then interact.
But i might have missed something with all the alterations.

Also sorry for not putting this in a pull request or similar.
Just not experienced with github and i modified quite a few functions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants