Field Rations - Add CBA events for consuming items and refilling bottles

[diwako]

When merged this pull request will:
Adds local CBA events when consuming items or refilling items.

Reason for this is, there is no way to interact with this so far on mission level. If someone wants to manipulate how much an item quenches thirst or how much hunger is sated it has to be done via mod right now.
This also means scripters can add their own logic when someone consumes a certain item or refills at a certain water source.

[dedmen]

"refillItem" looks so weird :D

[diwako]

It is the functions name, so ¯_(ツ)_/¯

[jonpas]

Those should probably be named corresponding to standard of name in other events here.

Eg. acex_rationConsumed and acex_rationRefilled.

[PabstMirror]

I'm fine keeping these events, but we should probably just add settings for amount of thirst/hunger restored

[diwako]

Updated the event names to be named like other events in ace.

[PabstMirror]

Ref #160 for settings for thirst quench

[mharis001]

What about acex_itemConsumed and acex_itemRefilled?

[jonpas]

That could be any item, and this is specific to rations. Just my opinion though.

[diwako]

Did i kill travis CI? The logs only return null.

[jonpas]

Probably just a minor outage, I restarted the build.

[mharis001]

What I was thinking as well for itemConsumed, I guess itemRefilled is more clear with that. Could QGVAR prefix them for field_rations?

[diwako]

In commit one I used QGVAR prefix on the event names, it was suggested to change them to be similar to ace's naming.

[mharis001]

QGVAR solves the being specific to field rations problem. IMO itemRefilled is a better name than rationRefilled.

[diwako]

I am fine with either QGVARr or acex_* as they are now. Ultimately the maintainers of this mod have to decide this.

[thojkooi]

lgtm

[thojkooi]

Thanks @diwako

[dedmen]

Typo in title.
refilling not re-filing aka file.