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

Improve removeItemCargo function to retain added items properly #707

Merged
merged 2 commits into from
Jun 24, 2017
Merged

Improve removeItemCargo function to retain added items properly #707

merged 2 commits into from
Jun 24, 2017

Conversation

jonpas
Copy link
Member

@jonpas jonpas commented Jun 21, 2017

When merged this pull request will:

@jonpas
Copy link
Member Author

jonpas commented Jun 21, 2017

execVM "\x\cba\addons\common\test_inventory.sqf";

20:44:09 [CBA] (common) Test OK: (CBA_fnc_addWeapon is defined) x\cba\addons\common\test_inventory.sqf:13
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:16
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:19
20:44:09 [CBA] (common) Test OK: (_result) x\cba\addons\common\test_inventory.sqf:22
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:30
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:33
20:44:09 [CBA] (common) Test OK: (_result) x\cba\addons\common\test_inventory.sqf:36
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:39
20:44:09 [CBA] (common) Test OK: (CBA_fnc_addMagazine is defined) x\cba\addons\common\test_inventory.sqf:46
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:49
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:52
20:44:09 [CBA] (common) Test OK: (_result) x\cba\addons\common\test_inventory.sqf:55
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:63
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:66
20:44:09 [CBA] (common) Test OK: (_result) x\cba\addons\common\test_inventory.sqf:69
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:74
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:89
20:44:09 [CBA] (common) Test OK: (_result) x\cba\addons\common\test_inventory.sqf:93
20:44:09 [CBA] (common) Test OK: (count (backpackCargo _container) == 2) x\cba\addons\common\test_inventory.sqf:94
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:102
20:44:09 [CBA] (common) Test OK: (_result) x\cba\addons\common\test_inventory.sqf:106
20:44:09 [CBA] (common) Test OK: (count (itemCargo _container) == 2) x\cba\addons\common\test_inventory.sqf:107
20:44:09 [CBA] (common) Test OK: (count (itemCargo _container) == 1 && count (itemCargo (((everyContainer _container) select 0) select 1)) == 5) x\cba\addons\common\test_inventory.sqf:116
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:124
20:44:09 [CBA] (common) Test OK: (_result) x\cba\addons\common\test_inventory.sqf:128
20:44:09 [CBA] (common) Test OK: (count (magazineCargo _container) == 2) x\cba\addons\common\test_inventory.sqf:129
20:44:09 [CBA] (common) Test OK: (not (_result)) x\cba\addons\common\test_inventory.sqf:137
20:44:09 [CBA] (common) Test OK: (_result) x\cba\addons\common\test_inventory.sqf:141
20:44:09 [CBA] (common) Test OK: (count (weaponCargo _container) == 2) x\cba\addons\common\test_inventory.sqf:142

Line 116 (the longest one above) is the new test.

@Killswitch00 Killswitch00 added this to the 3.4 milestone Jun 24, 2017
@Killswitch00 Killswitch00 merged commit 3036bd6 into CBATeam:master Jun 24, 2017
@jonpas jonpas deleted the removeItemCargoWearablesFix branch June 24, 2017 17:20
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