From a75705a80b0ac834155a01923f59b7065b1f76c0 Mon Sep 17 00:00:00 2001 From: Robin Kupper Date: Thu, 7 Apr 2011 18:37:53 +0200 Subject: [PATCH] Added check for empty result sets --- src/de/diddiz/LogBlock/Consumer.java | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/de/diddiz/LogBlock/Consumer.java b/src/de/diddiz/LogBlock/Consumer.java index d8ed53c..bd8456f 100644 --- a/src/de/diddiz/LogBlock/Consumer.java +++ b/src/de/diddiz/LogBlock/Consumer.java @@ -74,12 +74,16 @@ public class Consumer implements Runnable state.execute("INSERT INTO `" + b.table + "` (date, playerid, replaced, type, data, x, y, z) SELECT now(), playerid, " + b.replaced + ", " + b.type + ", " + b.data + ", '" + b.x + "', " + b.y + ", '" + b.z + "' FROM `lb-players` WHERE playername = '" + b.name + "'", Statement.RETURN_GENERATED_KEYS); if (b.signtext != null) { ResultSet keys = state.getGeneratedKeys(); - keys.next(); - state.execute("INSERT INTO `" + b.table + "-sign` (id, signtext) values (" + keys.getInt(1) + ", '" + b.signtext + "')"); + if (keys.next()) + state.execute("INSERT INTO `" + b.table + "-sign` (id, signtext) values (" + keys.getInt(1) + ", '" + b.signtext + "')"); + else + LogBlock.log.severe("[LogBlock Consumer] Failed to get generated keys"); } else if (b.ca != null) { ResultSet keys = state.getGeneratedKeys(); - keys.next(); - state.execute("INSERT INTO `" + b.table + "-chest` (id, intype, inamount, outtype, outamount) values (" + keys.getInt(1) + ", " + b.ca.inType + ", " + b.ca.inAmount + ", " + b.ca.outType + ", " + b.ca.outAmount + ")"); + if (keys.next()) + state.execute("INSERT INTO `" + b.table + "-chest` (id, intype, inamount, outtype, outamount) values (" + keys.getInt(1) + ", " + b.ca.inType + ", " + b.ca.inAmount + ", " + b.ca.outType + ", " + b.ca.outAmount + ")"); + else + LogBlock.log.severe("[LogBlock Consumer] Failed to get generated keys"); } count++; }