fix: speed and conditions

data-otservbr-global/migrations/46.lua

@@ -1,3 +1,7 @@
 function onUpdateDatabase()
-	return false -- true = There are others migrations file | false = this is the last migration file
+	logger.info("Updating database to version 47 (fix: creature speed and conditions)")
+
+	db.query("ALTER TABLE `players` MODIFY `conditions` mediumblob NOT NULL;")
+
+	return true
 end

data-otservbr-global/migrations/47.lua

@@ -0,0 +1,3 @@
+function onUpdateDatabase()
+	return false -- true = There are others migrations file | false = this is the last migration file
+end

schema.sql

@@ -70,7 +70,7 @@ CREATE TABLE IF NOT EXISTS `players` (
     `posx` int(11) NOT NULL DEFAULT '0',
     `posy` int(11) NOT NULL DEFAULT '0',
     `posz` int(11) NOT NULL DEFAULT '0',
-    `conditions` blob NOT NULL,
+    `conditions` mediumblob NOT NULL,
     `cap` int(11) NOT NULL DEFAULT '0',
     `sex` int(11) NOT NULL DEFAULT '0',
     `pronoun` int(11) NOT NULL DEFAULT '0',

src/creatures/creature.cpp

@@ -1592,6 +1592,17 @@ LightInfo Creature::getCreatureLight() const {
 	return internalLight;
 }
 
+uint16_t Creature::getSpeed() const {
+	const auto speed = baseSpeed + varSpeed;
+	if (speed >= 0 && speed <= std::numeric_limits<uint16_t>::max()) {
+		return speed;
+	} else if (speed < 0) {
+		return 0;
+	} else {
+		return std::numeric_limits<uint16_t>::max();
+	}
+}
+
 void Creature::setSpeed(int32_t varSpeedDelta) {
 	// Prevents creatures from not exceeding the maximum allowed speed
 	if (getSpeed() >= PLAYER_MAX_SPEED) {

src/creatures/creature.hpp

@@ -194,9 +194,7 @@ class Creature : virtual public Thing, public SharedObject {
 	virtual uint16_t getStepSpeed() const {
 		return getSpeed();
 	}
-	uint16_t getSpeed() const {
-		return static_cast<uint16_t>(baseSpeed + varSpeed);
-	}
+	uint16_t getSpeed() const;
 	void setSpeed(int32_t varSpeedDelta);
 
 	void setBaseSpeed(uint16_t newBaseSpeed) {