-
Notifications
You must be signed in to change notification settings - Fork 739
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 Switching of Weapon from gunbag. #6212
Conversation
params ["_unit", "_target"]; | ||
|
||
private _gunbag = backpackContainer _target; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this really need a empty line after each line of script?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just combined the other 2 functions in that module and there are written the same way.
What is the goal of this? Switching weapon takes time anyways, and using interaction menu for that kind of plays into that. |
Seems to me that instead of having to drop your current primary weapon on the ground, use ace interact to get your gunbag weapon out, drop that on the ground, pick up the previous weapon, and put that in the gunbag, it's combined into one action. |
Ah, there was a big discussion about that a while ago IIRC. |
Currently to "switch" a weapon with a Gunbag you need to Open your Inventory and drop your weapon, take your weapon out of your gun bag, drop the new weapon and take the other one, put it into the gun bag and then finally get the last weapon from the ground. and I think that is currently a very tedious progress. this PR was the idea to improve that. |
IIRC from the original PR which added this feature, it was decided against adding this specifically because you can't add a weapon in same perfect order as you had it in hands (with attachments, preset attachments and all those shenanigans). Even with @commy2 probably knows more details since he was vocal about it in the original PR. |
@jonpas the weapon is not stored in the backpack as normal Arma weapon its stored as a virtual item. so there is no problem with weapons cargo. |
I see, however that doesn't solve the problem of preset weapons. Say you have a preset weapon (that's one of those with attachments already on them without adding them yourself), and a player takes off one of those preset attachments, it will just reappear with your method if I am reading this correctly. |
but then the general system also have that issue. it is then not a issue this PR introduces. and no it first removes all attachments from the weapon that was added and then readds them. that is the same system that the other function also uses |
No, it "exploits" the issues, as in introduces a way to make it happen. |
When merged this pull request will: