Fix AddTemplateHotbar(): loading action spells

[sogladev]

changes

Add actions from db

details

• SendActionButtons(2) clears all actions. 1 commits it
  0 didn't work. No need to send (2) then (1).
• SaveToDB() is required to commit actions to db. it calls _saveActions()

test

option need to be enabled in the module config

the data is wrong in DB. I did not apply any optional sql. I only found action related data in the default.
template 1 is disabled, has actions, but no spells
template 2 is enabled, has spells, but has no actions
so, use actions from id 1 and test with 2
UPDATE acore_world.mod_ptrtemplate_action SET id = 2;

level 80 druid (or warrior)

1. .apply template 2
2. check if it matches DB actions
3. relog and see if spells are still there

TODO

fix action data,
for druid bear form spells are set in the wrong form, but they're loaded at least

[heyitsbench]

Suggested change

	player->SaveToDB(false, false); // commit action buttons
	player->SaveToDB(false, false); // Commit action buttons

Do you happen to know of any reasons this (L577) would have to fail?

[sogladev]

it fails if this check fails https://github.com/sogladev/azerothcore-wotlk/blob/7abbca0f5b4ab3a15d78db7ec40e3b4358bed7da/src/server/game/Entities/Player/Player.cpp#L5566

An example I encountered is when the template is trying to add an action spell that is not learned as the entry is not set in spells template
https://github.com/sogladev/azerothcore-wotlk/blob/7abbca0f5b4ab3a15d78db7ec40e3b4358bed7da/src/server/game/Entities/Player/Player.cpp#L5589C13-L5589C35

[sogladev]

will need to retest as SendActionButtons(1) is not send if no entry is found

[sogladev]

tested again, should be good now

[heyitsbench]

Thank you very much for the PR! ❤️

Good reminder for me to finish adding the data for the level 60 template. 😛