diff --git a/Emulator/src/main/java/com/eu/habbo/messages/rcon/GiveRespect.java b/Emulator/src/main/java/com/eu/habbo/messages/rcon/GiveRespect.java index 05c57de7..3b9a57b2 100644 --- a/Emulator/src/main/java/com/eu/habbo/messages/rcon/GiveRespect.java +++ b/Emulator/src/main/java/com/eu/habbo/messages/rcon/GiveRespect.java @@ -30,8 +30,8 @@ public class GiveRespect extends RCONMessage { habbo.getClient().sendResponse(new UserDataComposer(habbo)); } else { try (Connection connection = Emulator.getDatabase().getDataSource().getConnection(); PreparedStatement statement = connection.prepareStatement("UPDATE users_settings SET respects_given = respects_given + ?, respects_received = respects_received + ?, daily_respect_points = daily_respect_points + ? WHERE user_id = ? LIMIT 1")) { - statement.setInt(1, object.respect_received); - statement.setInt(2, object.respect_given); + statement.setInt(1, object.respect_given); + statement.setInt(2, object.respect_received); statement.setInt(3, object.daily_respects); statement.setInt(4, object.user_id); statement.execute(); @@ -50,4 +50,4 @@ public class GiveRespect extends RCONMessage { public int respect_received = 0; public int daily_respects = 0; } -} \ No newline at end of file +} diff --git a/Emulator/src/test/java/com/eu/habbo/messages/rcon/GiveRespectContractTest.java b/Emulator/src/test/java/com/eu/habbo/messages/rcon/GiveRespectContractTest.java new file mode 100644 index 00000000..b36999c3 --- /dev/null +++ b/Emulator/src/test/java/com/eu/habbo/messages/rcon/GiveRespectContractTest.java @@ -0,0 +1,24 @@ +package com.eu.habbo.messages.rcon; + +import org.junit.jupiter.api.Test; + +import java.nio.file.Files; +import java.nio.file.Path; + +import static org.junit.jupiter.api.Assertions.assertTrue; + +class GiveRespectContractTest { + private static String giveRespectSource() throws Exception { + return Files.readString(Path.of("src/main/java/com/eu/habbo/messages/rcon/GiveRespect.java")); + } + + @Test + void offlineRespectGrantBindsGivenAndReceivedToMatchingColumns() throws Exception { + String source = giveRespectSource(); + + assertTrue(source.contains("statement.setInt(1, object.respect_given);"), + "respects_given must be incremented with respect_given"); + assertTrue(source.contains("statement.setInt(2, object.respect_received);"), + "respects_received must be incremented with respect_received"); + } +}