fix: expiration update

Author: jprzimba

src/creatures/players/player.cpp

@@ -12113,10 +12113,10 @@ void Player::addScheduledUpdates(uint32_t flags) {
 	scheduledUpdates |= flags;
 
 	if (shouldSchedule) {
-		g_dispatcher().addEvent(
+		g_dispatcher().scheduleEvent(
+			SCHEDULER_MINTICKS,
 			[playerId = getID()]() { g_game().updatePlayerEvent(playerId); },
-			__FUNCTION__,
-			PLAYER_UPDATE_TICKS
+			__FUNCTION__
 		);
 
 		scheduledUpdate = true;

src/game/scheduling/dispatcher.hpp

@@ -23,7 +23,6 @@
 static constexpr uint16_t DISPATCHER_TASK_EXPIRATION = 2000;
 static constexpr uint16_t SCHEDULER_MINTICKS = 50;
 static constexpr uint16_t NPC_SELL_TICKS = 150;
-static constexpr uint16_t PLAYER_UPDATE_TICKS = 1000;
 
 enum class TaskGroup : int8_t {
 	ThreadPool = -1,

src/game/scheduling/task.cpp

@@ -24,8 +24,9 @@
 std::atomic_uint_fast64_t Task::LAST_EVENT_ID = 0;
 
 Task::Task(uint32_t expiresAfterMs, std::function<void(void)> &&f, std::string_view context) :
-	func(std::move(f)), context(context), utime(OTSYS_TIME()),
-	expiration(expiresAfterMs > 0 ? OTSYS_TIME() + expiresAfterMs : 0) {
+	func(std::move(f)), context(context),
+	utime(OTSYS_TIME(true)),
+	expiration(expiresAfterMs > 0 ? OTSYS_TIME(true) + expiresAfterMs : 0) {
 	if (this->context.empty()) {
 		g_logger().error("[{}]: task context cannot be empty!", __FUNCTION__);
 		return;
@@ -46,7 +47,7 @@ Task::Task(std::function<void(void)> &&f, std::string_view context, uint32_t del
 }
 
 [[nodiscard]] bool Task::hasExpired() const {
-	return expiration != 0 && expiration < OTSYS_TIME();
+	return expiration != 0 && expiration < OTSYS_TIME(true);
 }
 
 bool Task::execute() const {