fix: eventcallbacks which returns a ReturnValue

[lamonato29]

Description

This PR fixes eventcallbacks whose returns were in type conflict (creatureOnAreaCombat and creatureOnTargetCombat)

Behaviour

Actual

The eventcallback does not start.

Expected

Now, we can use ReturnValue, instead of true or false, to stop an attack execution, for example.
The old events system (data/events/scripts/creature.lua for example) runs first, then, if it succeed, the new eventcallback system runs too.

Tibia.-.Palaum.2024-04-08.16-31-58.mp4

Be Careful with custom scripts returning true or false. It may fail.

Fixes #2529

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)

[luanluciano93]

Tested with the proposed changes and it's working fine! Just change the README to the new operation.
https://github.com/opentibiabr/canary/blob/main/data/scripts/eventcallbacks/README.md?plain=1

[lamonato29]

add the changes but it gives me an error when trying to compile in event_callback_functions.cpp

Hello. I think you made something wrong. I didn't touched this file event_callback_functions.cpp.

[luanluciano93]

in the README file the example of a callback with Boolean return is creatureOnAreaCombat

[lamonato29]

in the README file the example of a callback with Boolean return is creatureOnAreaCombat

Fixed! Thank you, Luan.

[luanluciano93]

@lamonato29 Correct the return in this script https://github.com/opentibiabr/canary/blob/main/data/scripts/eventcallbacks/creature/on_area_combat.lua#L4

[lamonato29]

@lamonato29 Correct the return in this script https://github.com/opentibiabr/canary/blob/main/data/scripts/eventcallbacks/creature/on_area_combat.lua#L4

Done! Thx.