feat: new global event OnSave

[luanluciano93]

Description

• Add new global event: On Save.

Expected

Why don't we call in saveGameState? In situations like shutdown, things happen beforehand (players being sent off, etc.), so we call individually.

Type of change

Please delete options that are not relevant.

• New feature (non-breaking change which adds functionality)

How Has This Been Tested

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• the GAME_STATE_SHUTDOWN and GAME_STATE_CLOSED methods were executed without prejudice.

Test Configuration:

• Server Version: Canary Main
• Client: 13.21
• Operating System: Windows

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I checked the PR checks reports
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works

Credits: ramon-bernardo

[dudantas]

• Certifique-se de realizar testes de compilação e corrigir quaisquer problemas antes de criar um pull request. Isso está documentado no template do pull request, onde você deve listar os passos para testar a sua alteração e marcar as caixas indicando que realizou esses testes. Parece que essas etapas não foram seguidas aqui, uma vez que as compilações estão quebradas.

canary/.github/PULL_REQUEST_TEMPLATE.md

Line 30 in 5c87e81

- [ ] Test A

canary/.github/PULL_REQUEST_TEMPLATE.md

Line 42 in 5c87e81

- [ ] I have performed a self-review of my own code

canary/.github/PULL_REQUEST_TEMPLATE.md

Line 43 in 5c87e81

- [ ] I checked the PR checks reports

canary/.github/PULL_REQUEST_TEMPLATE.md

Line 47 in 5c87e81

- [ ] I have added tests that prove my fix is effective or that my feature works

Além disso, seria benéfico adicionar um script de teste para que outros desenvolvedores possam facilmente testar suas mudanças, caso desejem fazê-lo. Isso tornará o processo de revisão e integração mais eficiente.

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[github-actions]

This PR is stale because it has been open 45 days with no activity.

[sonarqubecloud]

Quality Gate failed

Failed conditions

D Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

[luanluciano93]

@dudantas Is something missing here?

[github-actions]

This PR is stale because it has been open 45 days with no activity.

[majestyotbr]

Is this ready for tests?

[luanluciano93]

Is this ready for tests?

yes

[majestyotbr]

Do you have a example to test?

[luanluciano93]

Do you have a example to test?

just save the game, the event itself is to map conditions for saving the game