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 dropWeapon, dropMagazine, add dropItem, weaponComponents #296

Merged
merged 3 commits into from
Mar 19, 2016
Merged

fix dropWeapon, dropMagazine, add dropItem, weaponComponents #296

merged 3 commits into from
Mar 19, 2016

Conversation

commy2
Copy link
Contributor

@commy2 commy2 commented Mar 6, 2016

ref #289

Fixes broken CBA_fnc_dropWeapon and CBA_fnc_dropMagazine. Instead of using the broken drop action, these remove the item from the inventory and create a GWH with all items instead.
Unfortunately the weapon is broken up into it's parts, but there is no other way currently.
Also adds optional paramter to dropMagazine, to use a mag with a specified ammo count.
Also adds CBA_fnc_dropItem for first aid kits etc.
dropWeapon needs a functional weaponComponets function. CBA_fnc_weaponComponents is a much improved version of the BI variant (which reports the black MX as tan MX, the Opfor DLC laserdesignator as the normal free laserdesignator, etc. etc.)


params [["_unit", objNull, [objNull]], ["_item", "", [""]]];

private _return = [_unit, _item, _ammo] call CBA_fnc_removeItem;
Copy link
Contributor

Choose a reason for hiding this comment

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

_ammo?

Killswitch00 added a commit that referenced this pull request Mar 19, 2016
fix dropWeapon, dropMagazine, add dropItem, weaponComponents
@Killswitch00 Killswitch00 merged commit 21dd81e into CBATeam:master Mar 19, 2016
@commy2 commy2 added this to the 2.3.2 milestone May 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants