Moved packages and fixed logging

Moved packages in a more logical order.
Fixed TransactionLogger using only event.getStock()[0] while logging the
transaction.
Also, TransactionLogger uses string formatter now, instead of a
StringBuilder.

com/Acrobot/ChestShop/ChestShop.java

@@ -15,12 +15,12 @@ import com.Acrobot.ChestShop.Listeners.ItemInfoListener;
 import com.Acrobot.ChestShop.Listeners.Player.PlayerConnect;
 import com.Acrobot.ChestShop.Listeners.Player.PlayerInteract;
 import com.Acrobot.ChestShop.Listeners.Player.ShortNameSaver;
-import com.Acrobot.ChestShop.Listeners.Shop.PostTransaction.EconomicModule;
+import com.Acrobot.ChestShop.Listeners.PostTransaction.EconomicModule;
-import com.Acrobot.ChestShop.Listeners.Shop.PostTransaction.ItemManager;
+import com.Acrobot.ChestShop.Listeners.PostTransaction.ItemManager;
-import com.Acrobot.ChestShop.Listeners.Shop.PreTransaction.*;
+import com.Acrobot.ChestShop.Listeners.PreTransaction.*;
-import com.Acrobot.ChestShop.Listeners.Shop.PostTransaction.EmptyShopDeleter;
+import com.Acrobot.ChestShop.Listeners.PostTransaction.EmptyShopDeleter;
-import com.Acrobot.ChestShop.Listeners.Shop.PostTransaction.TransactionLogger;
+import com.Acrobot.ChestShop.Listeners.PostTransaction.TransactionLogger;
-import com.Acrobot.ChestShop.Listeners.Shop.PostTransaction.TransactionMessageSender;
+import com.Acrobot.ChestShop.Listeners.PostTransaction.TransactionMessageSender;
 import com.Acrobot.ChestShop.Logging.FileFormatter;
 import com.Acrobot.ChestShop.Signs.RestrictedSign;
 import com.avaje.ebean.EbeanServer;

com/Acrobot/ChestShop/Listeners/PostTransaction/EconomicModule.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PostTransaction;
+package com.Acrobot.ChestShop.Listeners.PostTransaction;
 
 import com.Acrobot.ChestShop.Economy.Economy;
 import com.Acrobot.ChestShop.Events.TransactionEvent;

com/Acrobot/ChestShop/Listeners/PostTransaction/EmptyShopDeleter.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PostTransaction;
+package com.Acrobot.ChestShop.Listeners.PostTransaction;
 
 import com.Acrobot.ChestShop.Config.Config;
 import com.Acrobot.ChestShop.Config.Property;

com/Acrobot/ChestShop/Listeners/PostTransaction/ItemManager.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PostTransaction;
+package com.Acrobot.ChestShop.Listeners.PostTransaction;
 
 import com.Acrobot.Breeze.Utils.InventoryUtil;
 import com.Acrobot.ChestShop.Events.TransactionEvent;

com/Acrobot/ChestShop/Listeners/PostTransaction/TransactionLogger.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PostTransaction;
+package com.Acrobot.ChestShop.Listeners.PostTransaction;
 
 import com.Acrobot.ChestShop.ChestShop;
 import com.Acrobot.ChestShop.Config.Config;
@@ -19,34 +19,27 @@ import static com.Acrobot.ChestShop.Events.TransactionEvent.TransactionType.BUY;
  * @author Acrobot
  */
 public class TransactionLogger implements Listener {
+    private static final String BUY_MESSAGE = "%1$s bought %2$s for %3$.2f from %4$s at %5$s";
+    private static final String SELL_MESSAGE = "%1$s sold %2$s for %3$.2f to %4$s at %5$s";
+
     @EventHandler
     public static void onTransaction(TransactionEvent event) {
-        StringBuilder message = new StringBuilder(70);
+        String template = (event.getTransactionType() == BUY ? BUY_MESSAGE : SELL_MESSAGE);
 
-        message.append(event.getClient().getName());
+        StringBuilder items = new StringBuilder(50);
-
-        if (event.getTransactionType() == BUY) {
-            message.append(" bought ");
-        } else {
-            message.append(" sold ");
-        }
 
         for (ItemStack item : event.getStock()) {
-            message.append(item.getAmount()).append(' ').append(getSignName(item));
+            items.append(item.getAmount()).append(' ').append(getSignName(item));
         }
 
-        message.append(" for ").append(event.getPrice());
+        String message = String.format(template,
+                event.getClient().getName(),
+                items.toString(),
+                event.getPrice(),
+                event.getOwner().getName(),
+                locationToString(event.getSign().getLocation()));
 
-        if (event.getTransactionType() == BUY) {
+        ChestShop.getBukkitLogger().info(message);
-            message.append(" from ");
-        } else {
-            message.append(" to ");
-        }
-
-        message.append(event.getOwner()).append(' ');
-        message.append(locationToString(event.getSign().getLocation()));
-
-        ChestShop.getBukkitLogger().info(message.toString());
     }
 
     @EventHandler
@@ -60,7 +53,7 @@ public class TransactionLogger implements Listener {
         for (ItemStack item : event.getStock()) {
             Transaction transaction = new Transaction();
 
-            transaction.setAmount(event.getStock()[0].getAmount());
+            transaction.setAmount(item.getAmount());
 
             transaction.setItemID(item.getTypeId());
             transaction.setItemDurability(item.getDurability());

com/Acrobot/ChestShop/Listeners/PostTransaction/TransactionMessageSender.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PostTransaction;
+package com.Acrobot.ChestShop.Listeners.PostTransaction;
 
 import com.Acrobot.ChestShop.Config.Config;
 import com.Acrobot.ChestShop.Config.Language;

com/Acrobot/ChestShop/Listeners/PreTransaction/AmountAndPriceChecker.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PreTransaction;
+package com.Acrobot.ChestShop.Listeners.PreTransaction;
 
 import com.Acrobot.Breeze.Utils.InventoryUtil;
 import com.Acrobot.ChestShop.Economy.Economy;

com/Acrobot/ChestShop/Listeners/PreTransaction/CreativeModeIgnorer.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PreTransaction;
+package com.Acrobot.ChestShop.Listeners.PreTransaction;
 
 import com.Acrobot.ChestShop.Config.Config;
 import com.Acrobot.ChestShop.Events.PreTransactionEvent;

com/Acrobot/ChestShop/Listeners/PreTransaction/ErrorMessageSender.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PreTransaction;
+package com.Acrobot.ChestShop.Listeners.PreTransaction;
 
 import com.Acrobot.ChestShop.Config.Config;
 import com.Acrobot.ChestShop.Config.Language;

com/Acrobot/ChestShop/Listeners/PreTransaction/PartialTransactionModule.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PreTransaction;
+package com.Acrobot.ChestShop.Listeners.PreTransaction;
 
 import com.Acrobot.Breeze.Utils.InventoryUtil;
 import com.Acrobot.Breeze.Utils.MaterialUtil;

com/Acrobot/ChestShop/Listeners/PreTransaction/PermissionChecker.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PreTransaction;
+package com.Acrobot.ChestShop.Listeners.PreTransaction;
 
 import com.Acrobot.ChestShop.Events.PreTransactionEvent;
 import com.Acrobot.ChestShop.Events.TransactionEvent;

com/Acrobot/ChestShop/Listeners/PreTransaction/PriceValidator.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PreTransaction;
+package com.Acrobot.ChestShop.Listeners.PreTransaction;
 
 import com.Acrobot.Breeze.Utils.PriceUtil;
 import com.Acrobot.ChestShop.Events.PreTransactionEvent;

com/Acrobot/ChestShop/Listeners/PreTransaction/ShopValidator.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PreTransaction;
+package com.Acrobot.ChestShop.Listeners.PreTransaction;
 
 import com.Acrobot.ChestShop.Events.PreTransactionEvent;
 import com.Acrobot.ChestShop.Signs.ChestShopSign;

com/Acrobot/ChestShop/Listeners/PreTransaction/SpamClickProtector.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PreTransaction;
+package com.Acrobot.ChestShop.Listeners.PreTransaction;
 
 import com.Acrobot.ChestShop.Events.PreTransactionEvent;
 import org.bukkit.entity.Player;

com/Acrobot/ChestShop/Listeners/PreTransaction/StockFittingChecker.java

@@ -1,4 +1,4 @@
-package com.Acrobot.ChestShop.Listeners.Shop.PreTransaction;
+package com.Acrobot.ChestShop.Listeners.PreTransaction;
 
 import com.Acrobot.Breeze.Utils.InventoryUtil;
 import com.Acrobot.ChestShop.Events.PreTransactionEvent;