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.

Sign up for GitHub

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

Jump to bottom

Using CBA_fnc_removeItemCargo removes all vests of specified type, not the quantity asked #1153

Closed
armazac opened this issue Jun 2, 2019 · 2 comments · Fixed by #1168
Closed

Using CBA_fnc_removeItemCargo removes all vests of specified type, not the quantity asked #1153

armazac opened this issue Jun 2, 2019 · 2 comments · Fixed by #1168
Assignees
@jonpas
Labels
Bug
Milestone
3.12

Comments

@armazac
Copy link

armazac commented Jun 2, 2019 •
edited
Loading

Arma 3 Version: 1.92 (stable)
CBA Version: 3.11.2.190515 (stable)

Mods:

- CBA_A3

Description:

  • Removing vests using CBA_fnc_removeItemCargo on crate will delete all the vests in that crate of that type, even if you only specify to delete 1

Steps to reproduce:

  • Look at a container
  • Run [cursorObject,"V_PlateCarrier1_rgr",5] call CBA_fnc_addItemCargo
  • Open inventory, see 5 vests
  • Run [cursorObject,"V_PlateCarrier1_rgr",1] call CBA_fnc_removeItemCargo
  • Open inventory, see no vests

Where did the issue occur?

  • Dedicated

RPT log file:
N/A
@commy2
Copy link
Contributor

commy2 commented Jun 7, 2019

Can confirm that this is bugged.

@commy2 commy2 added the Bug label Jun 7, 2019
@commy2 commy2 added this to the 3.12 milestone Jun 7, 2019
@jonpas
Copy link
Member

jonpas commented Jun 7, 2019

@alganthe just confirmed for me last night. I will fix it soonish.

jonpas added a commit that referenced this issue Jun 24, 2019
@jonpas
Fix CBA_fnc_removeItemCargo removing all containers instead of specif…
18314c3 
…ied quantity - fix #1153
@jonpas jonpas mentioned this issue Jun 24, 2019
Merged
jonpas added a commit that referenced this issue Jun 29, 2019
@jonpas
Fix CBA_fnc_removeItemCargo removing all containers instead of specif…
07a2e04 
…ied quantity (#1168)

* Fix CBA_fnc_removeBackpackCargo unit test

* Fix CBA_fnc_removeItemCargo removing all containers instead of specified quantity - fix #1153
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jonpas jonpas

Labels
Bug
Projects
None yet
Milestone
3.12
Development

Successfully merging a pull request may close this issue.

Fix CBA_fnc_removeItemCargo removing all containers instead of specified quantity CBATeam/CBA_A3
3 participants
@commy2 @jonpas @armazac