Common - Replace canAdd with CBA_fnc_canAddItem #1561
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When merged this pull request will:
canAdd
withCBA_fnc_canAddItem
;CBA_fnc_canAddItem
;CBA_fnc_canAddItem
result.Mostly because ACE towing rope can disappear in the ground because of full load.
Adding backpack support is required for
addBackpackCargo
function.In some cases current function implementation can't count correctly. This code puts 28 designators to quadbike and 2 designators to player inventory (or near player):
The result is
false
but you can put exactly 2 designators to quadbike inventory.Here is why (
cursorObject
is quadbike with 28 designators):To work around this problem I added 0.0001 to number before
floor
. I think it's small enough not to make troubles.Also
floor parseNumber (maxLoad cursorObject * (1 - load cursorObject) / 20 toFixed 5)
can be used as workaround. It's slower but maybe looks better.