Core/Movement: Fixed issue with waypoint movements generator

[sanctum32]

Fixes issues with non repeatable waypoints Also this fixed script related hooks

Changes proposed:

• Fixed issue related to waypoints
• Did minor refactor (no effect on rest functionality)
• In addition I removed non needed _pauseTimer initialization
• Fixed script hooks not being triggered correctly with actual waypoint id (node id 0 but waypoint.Id actually begins with 1)

Issues addressed: Fixes #

This fixes issue where waypoint may could be repeated several times even when waypoints is set to not be repeatable

Tests performed: (Does it build, tested in-game, etc)

This mainly can be tested with https://gist.github.com/sanctum32/3eda5c5841cf9be69ad08f6ff0138013, i'm still developing this quest and grandma SAI script is missing but mainly testing with lucius, test results could be seen before and after.

Known issues and TODO list:

None

[sanctum32]

This issue mainly is triggered because when https://github.com/The-Cataclysm-Preservation-Project/TrinityCore/blob/master/src/server/game/Movement/MovementGenerators/WaypointMovementGenerator.cpp#L286 is called, it sets path as done but https://github.com/The-Cataclysm-Preservation-Project/TrinityCore/blob/master/src/server/game/Movement/MovementGenerators/WaypointMovementGenerator.cpp#L306 is missing check for _done variable, so it still attempts to continue step anyways

[Ovahlord]

c783cf7d0f8673001db124743c624ddb5f19db0c