#579 Updated for the stock counter to be updated on any hopper input (#591)

This is necessary to support new 1.20.5 items as well as better support some older ones.

Also use the properly formatted name when trying to valueOf get the Enum directly.

.crowdin.yml

@@ -0,0 +1,8 @@
+files:
+  - source: /src/main/resources/languages/lang.en.yml
+    translation: /src/main/resources/languages/lang.%two_letters_code%.yml
+    languages_mapping:
+      two_letters_code:
+        zh-TW: zh_TW
+        pt-BR: pt_BR
+        es-MX: es_MX

.github/FUNDING.yml

@@ -0,0 +1,6 @@
+# These are supported funding model platforms
+
+github: Phoenix616
+custom: ['https://acrobot-paypal.phoenix616.dev', 'https://tip.phoenix616.dev']
+issuehunt: ChestShop-authors/ChestShop-3
+ko_fi: Phoenix616

.github/ISSUE_TEMPLATE/bug-report.md

@@ -1,31 +0,0 @@
----
-name: Bug report
-about: An error that happens in the plugin.
-
----
-
-#### Plugin Version
-[The full plugin version that you are running from /version ChestShop]
- 
-#### Plugin Config
-[The full config.yml file of ChestShop]
- 
-#### Server Version
-[The full server version that you are running from /version]
- 
-#### Server Log
-[The full latest.log file, especially important if you have a stack trace]
-
-#### What other plugins are you running?
-[List of your plugins, ideally with the version] 
-
-#### What is happening?
-[Explain what happens and what steps should be done to reproduce the issue. Ideally with pictures and the full error log!] 
-
-#### What did you expect to happen?
-[Explain what you expected to happen after performing the previously described steps]
-
-### Additional context
-[Add any other context or screenshots about the feature request here.]
-
-[And Delete the lines in square brackets!]

.github/ISSUE_TEMPLATE/bug-report.yml

@@ -0,0 +1,89 @@
+name: Bug report
+description: Report an error that happens in the plugin.
+labels: []
+body:
+  - type: markdown
+    attributes:
+      value: >
+        Please check if there are already similar issues open, also please test if your issue
+        isn't already fixed by trying the latest development build: https://ci.minebench.de/job/ChestShop-3/
+
+  - type: textarea
+    attributes:
+      label: What is happening?
+      description: Explain what happens and what steps should be done to reproduce the issue. Ideally with pictures and the full error log!
+    validations:
+      required: true
+
+  - type: textarea
+    attributes:
+      label: What did you expect to happen?
+      description: Explain what you expected to happen after performing the previously described steps
+    validations:
+      required: true
+
+  - type: input
+    attributes:
+      label: Plugin Version
+      description: The full plugin version that you are running from `/version ChestShop`
+    validations:
+      required: true
+  - type: textarea
+    attributes:
+      label: Plugin Config
+      description: The full `config.yml` file of ChestShop.
+      value:  |
+        <details>
+          <summary>Config</summary>
+
+         ```yaml
+         [Put the config here]
+         ```
+
+        </details>
+    validations:
+      required: true
+
+  - type: input
+    attributes:
+      label: Server Version
+      description: The full server version that you are running from `/version`.
+    validations:
+      required: true
+  - type: textarea
+    attributes:
+      label: Server Log
+      description: The full `latest.log` file, especially important if you have a stack trace
+      value: |
+        <details>
+          <summary>Log</summary>
+
+          ```
+          [Your log here]
+          ```
+
+        </details>
+    validations:
+      required: true
+  - type: textarea
+    attributes:
+      label: What other plugins are you running?
+      description: List of your plugins, ideally with the version
+    validations:
+      required: true
+
+  - type: checkboxes
+    attributes:
+      label: Agreements
+      description: Please agree to the following.
+      options:
+        - label: I have checked if there are already similar issues open.
+        - label: I am running the latest development version available from https://ci.minebench.de/job/ChestShop-3/.
+        - label: The version [has support](https://github.com/ChestShop-authors/ChestShop-3/security/policy#supported-versions) for this type of issue.
+        - label: I have provided all requested information and agree to the terms of the [project's license](https://github.com/ChestShop-authors/ChestShop-3/blob/master/LICENSE)
+          required: true
+
+  - type: textarea
+    attributes:
+      label: Additional context
+      description: Add any other context or screenshots about the feature request here.

.github/ISSUE_TEMPLATE/config.yml

@@ -0,0 +1,8 @@
+blank_issues_enabled: false
+contact_links:
+  - name: "#chestshop IRC channel on irc.spi.gt"
+    url: "https://kiwiirc.com/client/irc.spi.gt/#chestshop"
+    about: Live chat with the devs and other members of the community
+  - name: SpigotMC discussion thread
+    url: https://www.spigotmc.org/threads/chestshop.295890/
+    about: Ask and answer questions or discuss the plugin in general

.github/ISSUE_TEMPLATE/enhancement.md

@@ -1,19 +0,0 @@
----
-name: Enhancement
-about: Request a feature or suggest an idea for this project
-
----
-
-#### Is your feature request related to a problem? Please describe.
-[A clear and concise description of what the problem is. Ex. I'm always frustrated when...]
-
-#### Describe the solution you'd like
-[A clear and concise description of what you want to happen.]
-
-#### Describe alternatives you've considered
-[A clear and concise description of any alternative solutions or features you've considered.]
-
-#### Additional context
-[Add any other context or screenshots about the feature request here.]
-
-[And Delete the lines in square brackets!]

.github/ISSUE_TEMPLATE/enhancement.yml

@@ -0,0 +1,43 @@
+name: Enhancement
+description: Request a feature or suggest an idea for this project
+labels: [enhancement]
+body:
+  - type: markdown
+    attributes:
+      value: >
+       Please check if there are already similar issues open, also please test if your request
+       isn't already implemented by trying the latest development build: https://ci.minebench.de/job/ChestShop-3/
+
+  - type: textarea
+    attributes:
+      label: Is your feature request related to a problem? Please describe.
+      description: A clear and concise description of what the problem is. Ex. I'm always frustrated when...
+    validations:
+      required: true
+
+  - type: textarea
+    attributes:
+      label: Describe the solution you'd like
+      description: A clear and concise description of what you want to happen.
+    validations:
+      required: true
+
+  - type: textarea
+    attributes:
+      label: Describe alternatives you've considered
+      description: A clear and concise description of any alternative solutions or features you've considered.
+
+  - type: checkboxes
+    attributes:
+      label: Agreements
+      description: Please agree to the following.
+      options:
+        - label: I have checked if there are already similar issues open.
+          required: true
+        - label: I have checked that the latest build doesn't already implement the feature.
+          required: true
+
+  - type: textarea
+    attributes:
+      label: Additional context
+      description: Add any other context or screenshots about the feature request here.

.github/workflows/code_quality.yml

@@ -0,0 +1,21 @@
+name: Qodana
+on:
+  workflow_dispatch:
+  pull_request:
+  push:
+    branches:
+    - master
+
+jobs:
+  qodana:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+      with:
+        fetch-depth: 0
+    - name: 'Qodana Scan'
+      uses: JetBrains/qodana-action@v2022.3.4
+      env:
+        QODANA_TOKEN: ${{ secrets.QODANA_TOKEN }}
+      with:
+        args: --baseline,.github/workflows/qodana/qodana.sarif.json

README.md

@@ -34,4 +34,7 @@ Links
 * [Forum Thread](http://forums.bukkit.org/threads/4150/)
 * [BukkitDev site](http://dev.bukkit.org/projects/chestshop/)
 * [SpigotMC site](https://www.spigotmc.org/resources/chestshop.51856/)
-* [Bug Tracker](http://dev.bukkit.org/server-mods/chestshop/tickets/?status=+)
+* [Bounties via IssueHunt](https://issuehunt.io/r/ChestShop-authors/ChestShop-3?tab=idle)
+* [Localization](https://crowdin.com/project/chestshop-3)
+* [Qodana code quality](https://qodana.cloud/projects/zxDG5/)
+* [Old bug Tracker](http://dev.bukkit.org/server-mods/chestshop/tickets/?status=+) (please use GitHub issues!)

SECURITY.md

@@ -7,8 +7,8 @@ and which ones are supported with security updates if necessary.
 
 | Version       | Active development | Security Fixes     |
 | ------------- | ------------------ | ------------------ |
-| 3.10 (latest) | :heavy_check_mark: | :heavy_check_mark: |
-| < 3.10        | :x:                | :x:                |
+| latest        | :heavy_check_mark: | :heavy_check_mark: |
+| < latest      | :x:                | :x:                |
 | 3.9.3-1.12    | :x:                | :heavy_check_mark: |
 | 3.9.2-1.8.8   | :x:                | :heavy_check_mark: |
 

pom.xml

@@ -6,7 +6,7 @@
 
     <groupId>com.acrobot.chestshop</groupId>
     <artifactId>chestshop</artifactId>
-    <version>3.10</version>
+    <version>3.12.3-SNAPSHOT</version>
     <description>Chest-and-sign shop plugin for Bukkit</description>
     <name>ChestShop</name>
 
@@ -18,20 +18,20 @@
 
     <repositories>
         <repository>
-            <id>sk89q-repo</id>
-            <url>http://maven.sk89q.com/repo/</url>
+            <id>enginehub-repo</id>
+            <url>https://maven.enginehub.org/repo/</url>
         </repository>
         <repository>
-            <id>bstats-repo</id>
-            <url>http://repo.bstats.org/content/repositories/releases/</url>
+            <id>codemc-repo</id>
+            <url>https://repo.codemc.org/repository/maven-public/</url>
+        </repository>
+        <repository>
+            <id>lwcx-repo</id>
+            <url>https://ci.ender.zone/plugin/repository/everything/</url>
         </repository>
         <repository>
             <id>vault-repo</id>
-            <url>http://nexus.hc.to/content/repositories/pub_releases/</url>
-        </repository>
-        <repository>
-            <id>authme-repo</id>
-            <url>https://repo.codemc.org/repository/maven-public/</url>
+            <url>https://nexus.hc.to/content/repositories/pub_releases/</url>
         </repository>
         <repository>
             <id>minebench-repo</id>
@@ -42,8 +42,20 @@
             <url>file://${project.basedir}/repo/</url>
         </repository>
         <repository>
-            <id>reserve-repo</id>
-            <url>https://dl.bintray.com/theneweconomy/java/</url>
+            <id>jitpack.io</id>
+            <url>https://jitpack.io</url>
+        </repository>
+        <repository>
+            <id>NyaaCat</id>
+            <url>https://ci.nyaacat.com/maven/</url>
+        </repository>
+        <repository>
+            <id>redprotect-repo</id>
+            <url>https://raw.githubusercontent.com/FabioZumbi12/RedProtect/mvn-repo/</url>
+        </repository>
+        <repository>
+            <id>adventure-repo</id>
+            <url>https://s01.oss.sonatype.org/content/repositories/snapshots/</url>
         </repository>
     </repositories>
 
@@ -51,46 +63,63 @@
         <dependency>
             <groupId>junit</groupId>
             <artifactId>junit</artifactId>
-            <version>4.12</version>
+            <version>4.13.2</version>
             <scope>test</scope>
         </dependency>
 
         <dependency>
             <groupId>com.j256.ormlite</groupId>
             <artifactId>ormlite-jdbc</artifactId>
-            <version>5.1</version>
+            <version>6.1</version>
             <scope>compile</scope>
         </dependency>
 
+        <dependency>
+            <groupId>de.themoep.utils</groupId>
+            <artifactId>lang-bukkit</artifactId>
+            <version>1.3-SNAPSHOT</version>
+            <scope>compile</scope>
+        </dependency>
+
+        <dependency>
+            <groupId>de.themoep</groupId>
+            <artifactId>minedown-adventure</artifactId>
+            <version>1.7.2-SNAPSHOT</version>
+            <scope>compile</scope>
+        </dependency>
+
+        <dependency>
+            <groupId>net.kyori</groupId>
+            <artifactId>adventure-platform-bukkit</artifactId>
+            <version>4.3.2</version>
+            <scope>compile</scope>
+        </dependency>
+
+        <dependency>
+            <groupId>net.kyori</groupId>
+            <artifactId>adventure-text-serializer-gson</artifactId>
+            <version>4.14.0</version>
+            <scope>compile</scope>
+            <exclusions>
+                <exclusion>
+                    <groupId>com.google.code.gson</groupId>
+                    <artifactId>gson</artifactId>
+                </exclusion>
+            </exclusions>
+        </dependency>
+
         <!-- Should be kept in sync with Mojang -->
         <dependency>
             <groupId>org.apache.logging.log4j</groupId>
             <artifactId>log4j-core</artifactId>
-            <version>2.1</version>
+            <version>2.17.2</version>
             <scope>provided</scope>
         </dependency>
 
-        <dependency>
-            <groupId>org.mcstats.bukkit</groupId>
-            <artifactId>metrics</artifactId>
-            <version>R8-SNAPSHOT</version>
-            <scope>compile</scope>
-            <exclusions>
-                <exclusion>
-                    <groupId>org.bukkit</groupId>
-                    <artifactId>bukkit</artifactId>
-                </exclusion>
-                <exclusion>
-                    <groupId>org.bukkit</groupId>
-                    <artifactId>craftbukkit</artifactId>
-                </exclusion>
-            </exclusions>
-        </dependency>
-
         <dependency>
             <groupId>org.bstats</groupId>
-            <artifactId>bstats-bukkit-lite</artifactId>
-            <version>1.2</version>
+            <artifactId>bstats-bukkit</artifactId>
+            <version>3.0.1</version>
             <scope>compile</scope>
         </dependency>
 
@@ -111,23 +140,6 @@
             </exclusions>
         </dependency>
 
-        <dependency>
-            <groupId>info.somethingodd</groupId>
-            <artifactId>odditem</artifactId>
-            <version>0.9.5</version>
-            <scope>provided</scope>
-            <exclusions>
-                <exclusion>
-                    <groupId>org.bukkit</groupId>
-                    <artifactId>bukkit</artifactId>
-                </exclusion>
-                <exclusion>
-                    <groupId>org.bukkit</groupId>
-                    <artifactId>craftbukkit</artifactId>
-                </exclusion>
-            </exclusions>
-        </dependency>
-
         
         <dependency>
             <groupId>com.herocraftonline.heroes</groupId>
@@ -145,7 +157,7 @@
         <dependency>
             <groupId>fr.xephi</groupId>
             <artifactId>authme</artifactId>
-            <version>5.5.0-SNAPSHOT</version>
+            <version>5.6.0-SNAPSHOT</version>
             <scope>provided</scope>
             <exclusions>
                 <exclusion>
@@ -155,27 +167,10 @@
             </exclusions>
         </dependency>
 
-        <dependency>
-            <groupId>com.griefcraft</groupId>
-            <artifactId>lwc</artifactId>
-            <version>4.7.0-SNAPSHOT</version>
-            <scope>provided</scope>
-            <exclusions>
-                <exclusion>
-                    <groupId>org.bukkit</groupId>
-                    <artifactId>bukkit</artifactId>
-                </exclusion>
-                <exclusion>
-                    <groupId>org.bukkit</groupId>
-                    <artifactId>craftbukkit</artifactId>
-                </exclusion>
-            </exclusions>
-        </dependency>
-
         <dependency>
             <groupId>com.griefcraft.lwc</groupId>
             <artifactId>LWCX</artifactId>
-            <version>2.2.0</version>
+            <version>2.2.5</version>
             <scope>provided</scope>
             <exclusions>
                 <exclusion>
@@ -225,6 +220,12 @@
             <artifactId>worldedit-core</artifactId>
             <version>7.0.0-SNAPSHOT</version>
             <scope>provided</scope>
+            <exclusions>
+                <exclusion>
+                    <groupId>org.yaml</groupId>
+                    <artifactId>snakeyaml</artifactId>
+                </exclusion>
+            </exclusions>
         </dependency>
 
         <dependency>
@@ -240,6 +241,38 @@
             </exclusions>
         </dependency>
 
+        <dependency>
+            <groupId>com.github.TechFortress</groupId>
+            <artifactId>GriefPrevention</artifactId>
+            <version>16.12.0</version>
+            <scope>provided</scope>
+            <exclusions>
+                <exclusion>
+                    <groupId>org.bukkit</groupId>
+                    <artifactId>bukkit</artifactId>
+                </exclusion>
+            </exclusions>
+        </dependency>
+
+        <dependency>
+            <groupId>com.github.jojodmo</groupId>
+            <artifactId>ItemBridge</artifactId>
+            <version>b0054538c1</version>
+        </dependency>
+
+        <dependency>
+            <groupId>br.net.fabiozumbi12.RedProtect</groupId>
+            <artifactId>RedProtect-Spigot</artifactId>
+            <version>7.7.3</version>
+            <scope>provided</scope>
+        </dependency>
+        <dependency>
+            <groupId>br.net.fabiozumbi12.RedProtect</groupId>
+            <artifactId>RedProtect-Core</artifactId>
+            <version>7.7.3</version>
+            <scope>provided</scope>
+        </dependency>
+
         <dependency>
             <groupId>com.webkonsept.bukkit.simplechestlock</groupId>
             <artifactId>simplechestlock</artifactId>
@@ -274,6 +307,13 @@
             </exclusions>
         </dependency>
 
+        <dependency>
+            <groupId>nl.rutgerkok</groupId>
+            <artifactId>blocklocker</artifactId>
+            <version>1.9</version>
+            <scope>provided</scope>
+        </dependency>
+
         <dependency>
             <groupId>com.bekvon.bukkit</groupId>
             <artifactId>residence</artifactId>
@@ -294,14 +334,21 @@
         <dependency>
             <groupId>de.themoep.showitem</groupId>
             <artifactId>api</artifactId>
-            <version>1.2.24</version>
+            <version>1.6.3</version>
             <scope>provided</scope>
         </dependency>
 
         <dependency>
             <groupId>net.tnemc</groupId>
             <artifactId>Reserve</artifactId>
-            <version>0.1.0.10</version>
+            <version>0.1.5.4</version>
+            <scope>provided</scope>
+        </dependency>
+
+        <dependency>
+            <groupId>me.crafter.mc</groupId>
+            <artifactId>lockettepro</artifactId>
+            <version>2.10-SNAPSHOT</version>
             <scope>provided</scope>
         </dependency>
     </dependencies>
@@ -309,9 +356,28 @@
     <build>
         <finalName>${project.name}</finalName>
         <plugins>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-jar-plugin</artifactId>
+                <version>3.2.0</version>
+                <configuration>
+                    <forceCreation>true</forceCreation>
+                    <archive>
+                        <manifest>
+                            <addDefaultEntries>true</addDefaultEntries>
+                        </manifest>
+                        <manifestEntries>
+                            <Distribution-Type>${buildType}</Distribution-Type>
+                            <Built-At>${maven.build.timestamp}</Built-At>
+                            <Build-Jdk>${java.runtime.version}</Build-Jdk>
+                            <paperweight-mappings-namespace>mojang</paperweight-mappings-namespace>
+                        </manifestEntries>
+                    </archive>
+                </configuration>
+            </plugin>
             <plugin>
                 <artifactId>maven-compiler-plugin</artifactId>
-                <version>3.6.1</version>
+                <version>3.8.1</version>
                 <configuration>
                     <source>1.8</source>
                     <target>1.8</target>
@@ -330,18 +396,27 @@
                         <configuration>
                             <artifactSet>
                                 <includes>
-                                    <include>org.mcstats.bukkit</include>
+                                    <include>de.themoep:*</include>
+                                    <include>de.themoep.utils:*</include>
+                                    <include>net.kyori:*</include>
                                     <include>org.bstats:*</include>
                                     <include>net.gravitydevelopment.updater</include>
                                     <include>com.j256.ormlite</include>
-                                    <include>org.apache.logging.log4j</include>
                                     <include>javax.persistence</include>
                                 </includes>
                             </artifactSet>
                             <relocations>
                                 <relocation>
-                                    <pattern>org.mcstats</pattern>
-                                    <shadedPattern>com.Acrobot.ChestShop.Metrics.MCStats</shadedPattern>
+                                    <pattern>de.themoep.utils.lang</pattern>
+                                    <shadedPattern>com.Acrobot.ChestShop.Libs.Lang</shadedPattern>
+                                </relocation>
+                                <relocation>
+                                    <pattern>de.themoep.minedown.adventure</pattern>
+                                    <shadedPattern>com.Acrobot.ChestShop.Libs.MineDown</shadedPattern>
+                                </relocation>
+                                <relocation>
+                                    <pattern>net.kyori</pattern>
+                                    <shadedPattern>com.Acrobot.ChestShop.Libs.Kyori</shadedPattern>
                                 </relocation>
                                 <relocation>
                                     <pattern>org.bstats</pattern>
@@ -353,11 +428,11 @@
                                 </relocation>
                                 <relocation>
                                     <pattern>com.j256.ormlite</pattern>
-                                    <shadedPattern>com.Acrobot.ChestShop.ORMlite</shadedPattern>
+                                    <shadedPattern>com.Acrobot.ChestShop.Libs.ORMlite</shadedPattern>
                                 </relocation>
                                 <relocation>
                                     <pattern>javax.persistence</pattern>
-                                    <shadedPattern>com.Acrobot.ChestShop.javax.persistence</shadedPattern>
+                                    <shadedPattern>com.Acrobot.ChestShop.Libs.javax.persistence</shadedPattern>
                                 </relocation>
                             </relocations>
                         </configuration>
@@ -371,13 +446,22 @@
                 <directory>src/main/resources</directory>
                 <filtering>true</filtering>
             </resource>
+            <resource>
+                <directory>.</directory>
+                <includes>
+                    <include>LICENSE</include>
+                    <include>README.md</include>
+                </includes>
+            </resource>
         </resources>
     </build>
 
     <properties>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
         <build.number>${buildNumber}</build.number>
+        <user.name>${buildType}</user.name>
         <bukkit.plugin.version>${project.version} ${buildDescription}</bukkit.plugin.version>
+        <bukkit.dependency.version>1.20.5-R0.1-SNAPSHOT</bukkit.dependency.version>
     </properties>
 
     <profiles>
@@ -385,13 +469,74 @@
             <id>default</id>
             <activation>
                 <property>
-                    <name>!bukkit</name>
+                    <name>!testprofile</name>
                 </property>
             </activation>
             <repositories>
                 <repository>
                     <id>paper-repo</id>
-                    <url>https://papermc.io/repo/repository/maven-public/</url>
+                    <url>https://repo.papermc.io/repository/maven-public/</url>
+                </repository>
+            </repositories>
+            <dependencies>
+                <dependency>
+                    <groupId>io.papermc.paper</groupId>
+                    <artifactId>paper-api</artifactId>
+                    <version>${bukkit.dependency.version}</version>
+                    <scope>provided</scope>
+                </dependency>
+            </dependencies>
+        </profile>
+
+        <profile>
+            <id>spigot</id>
+            <activation>
+                <property>
+                    <name>testprofile</name>
+                    <value>spigot</value>
+                </property>
+            </activation>
+            <repositories>
+                <repository>
+                    <id>spigotmc-repo</id>
+                    <url>https://hub.spigotmc.org/nexus/content/groups/public</url>
+                </repository>
+            </repositories>
+            <dependencies>
+                <dependency>
+                    <groupId>org.spigotmc</groupId>
+                    <artifactId>spigot-api</artifactId>
+                    <version>${bukkit.dependency.version}</version>
+                    <scope>provided</scope>
+                </dependency>
+            </dependencies>
+            <build>
+                <finalName>${project.name}-Spigot</finalName>
+                <plugins>
+                    <plugin>
+                        <artifactId>maven-compiler-plugin</artifactId>
+                        <configuration>
+                            <excludes>
+                                <exclude>**/Paper*.java</exclude>
+                            </excludes>
+                        </configuration>
+                    </plugin>
+                </plugins>
+            </build>
+        </profile>
+
+        <profile>
+            <id>backwards</id>
+            <activation>
+                <property>
+                    <name>testprofile</name>
+                    <value>backwards</value>
+                </property>
+            </activation>
+            <repositories>
+                <repository>
+                    <id>paper-repo</id>
+                    <url>https://repo.papermc.io/repository/maven-public/</url>
                 </repository>
             </repositories>
             <dependencies>
@@ -402,35 +547,14 @@
                     <scope>provided</scope>
                 </dependency>
             </dependencies>
-        </profile>
-
-        <profile>
-            <id>bukkit</id>
-            <repositories>
-                <repository>
-                    <id>spigotmc-repo</id>
-                    <url>https://hub.spigotmc.org/nexus/content/groups/public</url>
-                </repository>
-            </repositories>
-            <dependencies>
-                <dependency>
-                    <groupId>org.bukkit</groupId>
-                    <artifactId>bukkit</artifactId>
-                    <version>1.13.2-R0.1-SNAPSHOT</version>
-                    <scope>provided</scope>
-                </dependency>
-            </dependencies>
-            <properties>
-                <bukkit>true</bukkit>
-            </properties>
             <build>
-                <finalName>${project.name}-Bukkit</finalName>
+                <finalName>${project.name}-1.13.2</finalName>
                 <plugins>
                     <plugin>
                         <artifactId>maven-compiler-plugin</artifactId>
                         <configuration>
                             <excludes>
-                                <exclude>**/Paper*.java</exclude>
+                                <exclude>**/PaperLatest*.java</exclude>
                             </excludes>
                         </configuration>
                     </plugin>
@@ -446,6 +570,7 @@
                 </property>
             </activation>
             <properties>
+                <buildType>manual</buildType>
                 <buildNumber>0</buildNumber>
                 <buildDescription>(compiled at ${maven.build.timestamp})</buildDescription>
             </properties>
@@ -459,6 +584,7 @@
                 </property>
             </activation>
             <properties>
+                <buildType>jenkins</buildType>
                 <buildNumber>${env.BUILD_NUMBER}</buildNumber>
                 <buildDescription>(build ${env.BUILD_NUMBER})</buildDescription>
             </properties>

repo/com/griefcraft/lwc/lwc/4.3.1/lwc-4.3.1.jar.md5

@@ -1 +0,0 @@
-fe29bd77061f1ced861554d6d6d28cf7

repo/com/griefcraft/lwc/lwc/4.3.1/lwc-4.3.1.jar.sha1

@@ -1 +0,0 @@
-d9c12329efa1b76057c6ec6eb9371823369e27f5

repo/com/griefcraft/lwc/lwc/4.3.1/lwc-4.3.1.pom

@@ -1,9 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns="http://maven.apache.org/POM/4.0.0"
-    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
-  <modelVersion>4.0.0</modelVersion>
-  <groupId>com.griefcraft.lwc</groupId>
-  <artifactId>lwc</artifactId>
-  <version>4.3.1</version>
-  <description>POM was created from install:install-file</description>
-</project>

repo/com/griefcraft/lwc/lwc/4.3.1/lwc-4.3.1.pom.md5

@@ -1 +0,0 @@
-45ebcc66da30cab5c5d7b2c2e29651df

repo/com/griefcraft/lwc/lwc/4.3.1/lwc-4.3.1.pom.sha1

@@ -1 +0,0 @@
-46707be3cc027277671520396501466157b6dcce

repo/com/griefcraft/lwc/lwc/maven-metadata-local.xml

@@ -1,12 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<metadata>
-  <groupId>com.griefcraft.lwc</groupId>
-  <artifactId>lwc</artifactId>
-  <versioning>
-    <release>4.3.1</release>
-    <versions>
-      <version>4.3.1</version>
-    </versions>
-    <lastUpdated>20130308183115</lastUpdated>
-  </versioning>
-</metadata>

repo/com/griefcraft/lwc/lwc/maven-metadata-local.xml.md5

@@ -1 +0,0 @@
-eff48fcb809bfa5cef0a0660d8449ee3

repo/com/griefcraft/lwc/lwc/maven-metadata-local.xml.sha1

@@ -1 +0,0 @@
-e9c597ab4ce925154cad4e0d10aff5161d051a06

repo/info/somethingodd/odditem/0.9.5/odditem-0.9.5.jar.md5

@@ -1 +0,0 @@
-c7b3b9b7cd7cd27a50c079ce4349f628

repo/info/somethingodd/odditem/0.9.5/odditem-0.9.5.jar.sha1

@@ -1 +0,0 @@
-697ff692f06dc9064badb2610fd52399651b8274

repo/info/somethingodd/odditem/0.9.5/odditem-0.9.5.pom

@@ -1,9 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd" xmlns="http://maven.apache.org/POM/4.0.0"
-    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
-  <modelVersion>4.0.0</modelVersion>
-  <groupId>info.somethingodd</groupId>
-  <artifactId>odditem</artifactId>
-  <version>0.9.5</version>
-  <description>POM was created from install:install-file</description>
-</project>

repo/info/somethingodd/odditem/0.9.5/odditem-0.9.5.pom.md5

@@ -1 +0,0 @@
-9fa8325983cb90581156734d0500aacb

repo/info/somethingodd/odditem/0.9.5/odditem-0.9.5.pom.sha1

@@ -1 +0,0 @@
-288ed3c67d66d532eac2128c120a6d74986823d6

repo/info/somethingodd/odditem/maven-metadata-local.xml

@@ -1,12 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<metadata>
-  <groupId>info.somethingodd</groupId>
-  <artifactId>odditem</artifactId>
-  <versioning>
-    <release>0.9.5</release>
-    <versions>
-      <version>0.9.5</version>
-    </versions>
-    <lastUpdated>20130826223956</lastUpdated>
-  </versioning>
-</metadata>

repo/info/somethingodd/odditem/maven-metadata-local.xml.md5

@@ -1 +0,0 @@
-7240080156e6cd8df6f550049950a99c

repo/info/somethingodd/odditem/maven-metadata-local.xml.sha1

@@ -1 +0,0 @@
-23208c6e99b6df3664c26f771fa941a37387f428

src/main/java/com/Acrobot/Breeze/Collection/SimpleCache.java

@@ -1,20 +1,21 @@
 package com.Acrobot.Breeze.Collection;
 
+import java.util.Collections;
 import java.util.LinkedHashMap;
 import java.util.Map;
 import java.util.concurrent.Callable;
 import java.util.concurrent.ExecutionException;
 
 public class SimpleCache<K, V> {
-    private final LinkedHashMap<K, V> map;
+    private final Map<K, V> map;
 
     public SimpleCache(int cacheSize) {
-        map = new LinkedHashMap<K, V>(cacheSize * 10/9, 0.7f, true) {
+        map = Collections.synchronizedMap(new LinkedHashMap<K, V>(cacheSize * 10/9, 0.7f, true) {
             @Override
             protected boolean removeEldestEntry(Map.Entry<K, V> eldest) {
                 return size() > cacheSize;
             }
-        };
+        });
     }
 
     public V put(K key, V value) {

src/main/java/com/Acrobot/Breeze/Configuration/Configuration.java

@@ -8,10 +8,14 @@ import java.io.IOException;
 import java.lang.reflect.Field;
 import java.lang.reflect.Modifier;
 import java.util.HashMap;
+import java.util.Locale;
 import java.util.Map;
 import java.util.Scanner;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import com.Acrobot.Breeze.Configuration.Annotations.Parser;
+import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.configuration.file.FileConfiguration;
 import org.bukkit.configuration.file.YamlConfiguration;
@@ -25,7 +29,16 @@ import com.Acrobot.Breeze.Configuration.Annotations.PrecededBySpace;
  */
 public class Configuration {
     private static Map<String, ValueParser> parsers = new HashMap<>();
-    public static ValueParser DEFAULT_PARSER = new ValueParser();
+    private static final ValueParser DEFAULT_PARSER = new ValueParser();
+    private static final ValueParser ENUM_PARSER = new ValueParser() {
+        @Override
+        public <T> Object parseToJava(Class<T> type, Object object) {
+            if (object instanceof String && type.isEnum()) {
+                return Enum.valueOf((Class<? extends Enum>) type, ((String) object).toUpperCase(Locale.ROOT));
+            }
+            return object;
+        }
+    };
 
     /**
      * Loads a YAML-formatted file into a class and modifies the file if some of class's fields are missing
@@ -34,8 +47,24 @@ public class Configuration {
      * @param clazz Class to modify
      */
     public static void pairFileAndClass(File file, Class<?> clazz) {
+        pairFileAndClass(file, clazz, Bukkit.getLogger());
+    }
+
+    /**
+     * Loads a YAML-formatted file into a class and modifies the file if some of class's fields are missing
+     *
+     * @param file      File to load
+     * @param clazz     Class to modify
+     * @param logger    The logger to use to log some information about the pairing
+     */
+    public static void pairFileAndClass(File file, Class<?> clazz, Logger logger) {
         FileConfiguration config = YamlConfiguration.loadConfiguration(file);
 
+        boolean debug = config.getBoolean("DEBUG", false);
+
+        if (debug) {
+            logger.log(Level.INFO, "Loading configuration " + file.getName());
+        }
         try {
             BufferedWriter writer = new BufferedWriter(new FileWriter(file, true));
 
@@ -45,6 +74,9 @@ public class Configuration {
 
             for (Field field : clazz.getDeclaredFields()) {
                 if (!Modifier.isStatic(field.getModifiers()) || Modifier.isTransient(field.getModifiers()) || !Modifier.isPublic(field.getModifiers())) {
+                    if (debug) {
+                        logger.log(Level.WARNING, "Field " + field.getName() + " is private, transient or not static!");
+                    }
                     continue;
                 }
 
@@ -52,9 +84,9 @@ public class Configuration {
 
                 try {
                     if (config.isSet(path)) {
-                        field.set(null, getParser(field).parseToJava(config.get(path)));
-                    } else if (config.isSet(path.toLowerCase())) {
-                        field.set(null, getParser(field).parseToJava(config.get(path.toLowerCase())));
+                        field.set(null, getParser(field).parseToJava(field.getType(), config.get(path)));
+                    } else if (config.isSet(path.toLowerCase(Locale.ROOT))) {
+                        field.set(null, getParser(field).parseToJava(field.getType(), config.get(path.toLowerCase(Locale.ROOT))));
                     } else {
                         if (field.isAnnotationPresent(PrecededBySpace.class)) {
                             writer.newLine();
@@ -63,15 +95,18 @@ public class Configuration {
                         writer.write(FieldParser.parse(field));
                         writer.newLine();
                     }
+                    if (debug) {
+                        logger.log(Level.INFO, field.getName() + ": " + Configuration.getParser(field).parseToYAML(field.get(null)));
+                    }
                 } catch (IllegalArgumentException | IllegalAccessException | IOException e) {
-                    e.printStackTrace();
+                    logger.log(Level.SEVERE, "Error while loading field " + field.getName() + " in configuration " + file.getName(), e);
                 }
             }
 
             writer.close();
 
         } catch (IOException e) {
-            e.printStackTrace();
+            logger.log(Level.SEVERE, "Error while loading configuration " + file.getName(), e);
         }
     }
 
@@ -91,7 +126,7 @@ public class Configuration {
 
             return lastLine.isEmpty();
         } catch (FileNotFoundException e) {
-            e.printStackTrace();
+            Logger.getLogger("Configuration").log(Level.SEVERE, "Error while checking if file " + file.getName() + " ends with space", e);
             return false;
         }
     }
@@ -122,7 +157,7 @@ public class Configuration {
      * @param valueParser The parser itself
      */
     public static void registerParser(String name, ValueParser valueParser) {
-        parsers.put(name.toLowerCase(), valueParser);
+        parsers.put(name.toLowerCase(Locale.ROOT), valueParser);
     }
 
     /**
@@ -131,7 +166,7 @@ public class Configuration {
      * @return The parser or null if it doesn't exist
      */
     public static ValueParser getParser(String name) {
-        return parsers.get(name.toLowerCase());
+        return parsers.get(name.toLowerCase(Locale.ROOT));
     }
 
     /**
@@ -147,6 +182,9 @@ public class Configuration {
         if (parser == null) {
             parser = Configuration.getParser(field.getType().getSimpleName());
         }
+        if (parser == null && field.getType().isEnum()) {
+            parser = Configuration.ENUM_PARSER;
+        }
         if (parser == null) {
             parser = Configuration.DEFAULT_PARSER;
         }

src/main/java/com/Acrobot/Breeze/Configuration/FieldParser.java

@@ -1,9 +1,10 @@
 package com.Acrobot.Breeze.Configuration;
 
 import com.Acrobot.Breeze.Configuration.Annotations.ConfigurationComment;
-import com.Acrobot.Breeze.Configuration.Annotations.Parser;
 
 import java.lang.reflect.Field;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  * @author Acrobot
@@ -19,7 +20,9 @@ public class FieldParser {
         StringBuilder builder = new StringBuilder(50);
 
         if (field.isAnnotationPresent(ConfigurationComment.class)) {
-            builder.append('#').append(field.getAnnotation(ConfigurationComment.class).value()).append('\n');
+            for (String commentLine : field.getAnnotation(ConfigurationComment.class).value().split("\n")) {
+                builder.append("# ").append(commentLine).append('\n');
+            }
         }
 
         ValueParser parser = Configuration.getParser(field);
@@ -27,7 +30,7 @@ public class FieldParser {
         try {
             builder.append(field.getName()).append(": ").append(parser.parseToYAML(field.get(null)));
         } catch (IllegalAccessException e) {
-            e.printStackTrace();
+            Logger.getLogger("FieldParser").log(Level.SEVERE, "Error while parsing field", e);
             return "";
         }
 

src/main/java/com/Acrobot/Breeze/Configuration/ValueParser.java

@@ -2,10 +2,12 @@ package com.Acrobot.Breeze.Configuration;
 
 import org.bukkit.configuration.ConfigurationSection;
 
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.stream.Collectors;
 
 /**
  * @author Acrobot
@@ -26,6 +28,13 @@ public class ValueParser {
                 sb.append("\n- ").append(parseToYAML(o));
             }
             return sb.toString();
+        } else if (object instanceof String) {
+            String[] lines = ((String) object).split("\\R");
+            if (lines.length == 1) {
+                return '\"' + String.valueOf(object).replace("\\", "\\\\") + '\"';
+            } else {
+                return "|-\n" + Arrays.stream(lines).map(s -> "  " + s.replace("\\", "\\\\")).collect(Collectors.joining("\n"));
+            }
         } else {
             return '\"' + String.valueOf(object) + '\"';
         }
@@ -34,12 +43,13 @@ public class ValueParser {
     /**
      * Parses a YAML "object" to Java-compatible object
      *
+     * @param type The type of the returned object
      * @param object Object to parse
      * @return Java-compatible object
      */
-    public Object parseToJava(Object object) {
+    public <T> Object parseToJava(Class<T> type, Object object) {
         if (object instanceof ConfigurationSection) {
-            Map<String, List<String>> map = new HashMap<String, List<String>>();
+            Map<String, List<String>> map = new HashMap<>();
 
             for (String message : ((ConfigurationSection) object).getKeys(false)) {
                 map.put(message, ((ConfigurationSection) object).getStringList(message));

src/main/java/com/Acrobot/Breeze/Database/Database.java

@@ -1,8 +1,12 @@
 package com.Acrobot.Breeze.Database;
 
+import com.Acrobot.ChestShop.ChestShop;
+
 import java.sql.Connection;
 import java.sql.DriverManager;
 import java.sql.SQLException;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import javax.persistence.Entity;
 
@@ -60,7 +64,7 @@ public class Database {
         try {
             table.create(fields);
         } catch (SQLException e) {
-            e.printStackTrace();
+            Logger.getLogger("Database").log(Level.SEVERE, "Error while creating database from " + clazz.getName() + " (" + fields + ")", e);
             return false;
         }
 

src/main/java/com/Acrobot/Breeze/Database/EntityParser.java

@@ -36,7 +36,7 @@ public class EntityParser {
             fields.add(convertToSQL(field));
         }
 
-        return fields.stream().collect(Collectors.joining(","));
+        return String.join(",", fields);
     }
 
     /**

src/main/java/com/Acrobot/Breeze/Database/Row.java

@@ -1,6 +1,8 @@
 package com.Acrobot.Breeze.Database;
 
 import java.util.*;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  * A class representing a Row in SQL query
@@ -85,21 +87,16 @@ public class Row {
 
         try {
             object = clazz.newInstance();
-        } catch (InstantiationException e) {
-            e.printStackTrace();
-            return null;
-        } catch (IllegalAccessException e) {
-            e.printStackTrace();
+        } catch (InstantiationException | IllegalAccessException e) {
+            Logger.getLogger("Row").log(Level.SEVERE, "Error while creating new instance of class " + clazz.getName() + " for row", e);
             return null;
         }
 
         for (Map.Entry<String, String> value : values.entrySet()) {
             try {
                 clazz.getDeclaredField(value.getKey()).set(object, value.getValue());
-            } catch (NoSuchFieldException ex) {
-                ex.printStackTrace();
-            } catch (IllegalAccessException ex) {
-                ex.printStackTrace();
+            } catch (NoSuchFieldException | IllegalAccessException ex) {
+                Logger.getLogger("Row").log(Level.SEVERE, "Error while setting field " + value.getKey() + " to " + value.getValue() + " of class " + clazz.getName(), ex);
             }
         }
 

src/main/java/com/Acrobot/Breeze/Database/Table.java

@@ -132,7 +132,7 @@ public class Table {
         String statement;
 
         if (condition == null || condition.isEmpty()) {
-            String format = '\'' + row.getValues().stream().collect(Collectors.joining("', ")) + '\'';
+            String format = '\'' + String.join("', ", row.getValues()) + '\'';
             statement = String.format(INSERT_VALUES, format);
         } else {
             String format = row.getKeysAndValues().entrySet().stream()

src/main/java/com/Acrobot/Breeze/Utils/BlockUtil.java

@@ -22,6 +22,10 @@ public class BlockUtil {
      * @return Is this block a sign?
      */
     public static boolean isSign(Block block) {
+        if (!isLoaded(block)) {
+            return false;
+        }
+
         BlockData data = block.getBlockData();
         return data instanceof Sign || data instanceof WallSign;
     }
@@ -33,7 +37,7 @@ public class BlockUtil {
      * @return Is this block a chest?
      */
     public static boolean isChest(Block block) {
-        return block.getBlockData() instanceof org.bukkit.block.data.type.Chest;
+        return BlockUtil.isLoaded(block) && block.getBlockData() instanceof org.bukkit.block.data.type.Chest;
     }
 
     /**
@@ -107,4 +111,14 @@ public class BlockUtil {
 
         return true;
     }
+
+    /**
+     * Check if the chunk a block is in is loaded
+     *
+     * @param block The block to check
+     * @return Whether or not the chunk is loaded
+     */
+    public static boolean isLoaded(Block block) {
+        return block.getWorld().isChunkLoaded(block.getX() >> 4, block.getZ() >> 4);
+    }
 }

src/main/java/com/Acrobot/Breeze/Utils/Encoding/Base64.java

@@ -1,5 +1,9 @@
 package com.Acrobot.Breeze.Utils.Encoding;
 
+import java.nio.charset.StandardCharsets;
+import java.util.logging.Level;
+import java.util.logging.Logger;
+
 /**
  * <p>Encodes and decodes to and from Base64 notation.</p>
  * <p>Homepage: <a href="http://iharder.net/base64">http://iharder.net/base64</a>.</p>
@@ -559,59 +563,31 @@ public class Base64 {
             throw new NullPointerException("Cannot serialize a null object.");
         }   // end if: null
 
-        // Streams
-        java.io.ByteArrayOutputStream baos = null;
-        java.io.OutputStream b64os = null;
-        java.util.zip.GZIPOutputStream gzos = null;
-        java.io.ObjectOutputStream oos = null;
-
-
-        try {
-            // ObjectOutputStream -> (GZIP) -> Base64 -> ByteArrayOutputStream
-            baos = new java.io.ByteArrayOutputStream();
-            b64os = new Base64.OutputStream(baos, ENCODE | options);
+        // ObjectOutputStream -> (GZIP) -> Base64 -> ByteArrayOutputStream
+        try (java.io.ByteArrayOutputStream baos = new java.io.ByteArrayOutputStream();
+             java.io.OutputStream b64os = new Base64.OutputStream(baos, ENCODE | options)) {
             if ((options & GZIP) != 0) {
                 // Gzip
-                gzos = new java.util.zip.GZIPOutputStream(b64os);
-                oos = new java.io.ObjectOutputStream(gzos);
+                try (java.util.zip.GZIPOutputStream gzos = new java.util.zip.GZIPOutputStream(b64os);
+                     java.io.ObjectOutputStream oos = new java.io.ObjectOutputStream(gzos)){
+                    oos.writeObject(serializableObject);
+                }
             } else {
                 // Not gzipped
-                oos = new java.io.ObjectOutputStream(b64os);
+                try (java.io.ObjectOutputStream oos = new java.io.ObjectOutputStream(b64os)) {
+                    oos.writeObject(serializableObject);
+                }
             }
-            oos.writeObject(serializableObject);
+            // Return value according to relevant encoding.
+            try {
+                return baos.toString(PREFERRED_ENCODING);
+            }   // end try
+            catch (java.io.UnsupportedEncodingException uue) {
+                // Fall back to some Java default
+                return baos.toString();
+            }   // end catch
         }   // end try
-        catch (java.io.IOException e) {
-            // Catch it and then throw it immediately so that
-            // the finally{} block is called for cleanup.
-            throw e;
-        }   // end catch
-        finally {
-            try {
-                oos.close();
-            } catch (Exception e) {
-            }
-            try {
-                gzos.close();
-            } catch (Exception e) {
-            }
-            try {
-                b64os.close();
-            } catch (Exception e) {
-            }
-            try {
-                baos.close();
-            } catch (Exception e) {
-            }
-        }   // end finally
 
-        // Return value according to relevant encoding.
-        try {
-            return new String(baos.toByteArray(), PREFERRED_ENCODING);
-        }   // end try
-        catch (java.io.UnsupportedEncodingException uue) {
-            // Fall back to some Java default
-            return new String(baos.toByteArray());
-        }   // end catch
 
     }   // end encode
 
@@ -635,7 +611,6 @@ public class Base64 {
         } catch (java.io.IOException ex) {
             assert false : ex.getMessage();
         }   // end catch
-        assert encoded != null;
         return encoded;
     }   // end encodeBytes
 
@@ -700,7 +675,6 @@ public class Base64 {
         } catch (java.io.IOException ex) {
             assert false : ex.getMessage();
         }   // end catch
-        assert encoded != null;
         return encoded;
     }   // end encodeBytes
 
@@ -810,40 +784,13 @@ public class Base64 {
 
         // Compress?
         if ((options & GZIP) != 0) {
-            java.io.ByteArrayOutputStream baos = null;
-            java.util.zip.GZIPOutputStream gzos = null;
-            Base64.OutputStream b64os = null;
-
-            try {
-                // GZip -> Base64 -> ByteArray
-                baos = new java.io.ByteArrayOutputStream();
-                b64os = new Base64.OutputStream(baos, ENCODE | options);
-                gzos = new java.util.zip.GZIPOutputStream(b64os);
+            try (java.io.ByteArrayOutputStream baos = new java.io.ByteArrayOutputStream();
+                 Base64.OutputStream b64os = new Base64.OutputStream(baos, ENCODE | options);
+                 java.util.zip.GZIPOutputStream gzos = new java.util.zip.GZIPOutputStream(b64os)) {
 
                 gzos.write(source, off, len);
-                gzos.close();
+                return baos.toByteArray();
             }   // end try
-            catch (java.io.IOException e) {
-                // Catch it and then throw it immediately so that
-                // the finally{} block is called for cleanup.
-                throw e;
-            }   // end catch
-            finally {
-                try {
-                    gzos.close();
-                } catch (Exception e) {
-                }
-                try {
-                    b64os.close();
-                } catch (Exception e) {
-                }
-                try {
-                    baos.close();
-                } catch (Exception e) {
-                }
-            }   // end finally
-
-            return baos.toByteArray();
         }   // end if: compress
 
         // Else, don't compress. Better not to use streams at all then.
@@ -1162,20 +1109,16 @@ public class Base64 {
         // Check to see if it's gzip-compressed
         // GZIP Magic Two-Byte Number: 0x8b1f (35615)
         boolean dontGunzip = (options & DONT_GUNZIP) != 0;
-        if ((bytes != null) && (bytes.length >= 4) && (!dontGunzip)) {
+        if (bytes.length >= 4 && !dontGunzip) {
 
             int head = (bytes[0] & 0xff) | ((bytes[1] << 8) & 0xff00);
             if (java.util.zip.GZIPInputStream.GZIP_MAGIC == head) {
-                java.io.ByteArrayInputStream bais = null;
-                java.util.zip.GZIPInputStream gzis = null;
-                java.io.ByteArrayOutputStream baos = null;
                 byte[] buffer = new byte[2048];
                 int length = 0;
 
-                try {
-                    baos = new java.io.ByteArrayOutputStream();
-                    bais = new java.io.ByteArrayInputStream(bytes);
-                    gzis = new java.util.zip.GZIPInputStream(bais);
+                try (java.io.ByteArrayOutputStream baos = new java.io.ByteArrayOutputStream();
+                     java.io.ByteArrayInputStream bais = new java.io.ByteArrayInputStream(bytes);
+                     java.util.zip.GZIPInputStream gzis = new java.util.zip.GZIPInputStream(bais)) {
 
                     while ((length = gzis.read(buffer)) >= 0) {
                         baos.write(buffer, 0, length);
@@ -1184,25 +1127,10 @@ public class Base64 {
                     // No error? Get new bytes.
                     bytes = baos.toByteArray();
 
-                }   // end try
-                catch (java.io.IOException e) {
-                    e.printStackTrace();
+                } catch (java.io.IOException e) {
+                    Logger.getLogger("Base64").log(Level.SEVERE, "Unable to decode", e);
                     // Just return originally-decoded bytes
-                }   // end catch
-                finally {
-                    try {
-                        baos.close();
-                    } catch (Exception e) {
-                    }
-                    try {
-                        gzis.close();
-                    } catch (Exception e) {
-                    }
-                    try {
-                        bais.close();
-                    } catch (Exception e) {
-                    }
-                }   // end finally
+                }
 
             }   // end if: gzipped
         }   // end if: bytes.length >= 2
@@ -1252,55 +1180,30 @@ public class Base64 {
         // Decode and gunzip if necessary
         byte[] objBytes = decode(encodedObject, options);
 
-        java.io.ByteArrayInputStream bais = null;
-        java.io.ObjectInputStream ois = null;
-        Object obj = null;
-
-        try {
-            bais = new java.io.ByteArrayInputStream(objBytes);
+        try (java.io.ByteArrayInputStream bais = new java.io.ByteArrayInputStream(objBytes)) {
 
             // If no custom class loader is provided, use Java's builtin OIS.
             if (loader == null) {
-                ois = new java.io.ObjectInputStream(bais);
+                try (java.io.ObjectInputStream ois = new java.io.ObjectInputStream(bais)) {
+                    return ois.readObject();
+                } // end try
             }   // end if: no loader provided
 
             // Else make a customized object input stream that uses
             // the provided class loader.
             else {
-                ois = new java.io.ObjectInputStream(bais) {
+                try (java.io.ObjectInputStream ois = new java.io.ObjectInputStream(bais) {
                     @Override
                     public Class<?> resolveClass(java.io.ObjectStreamClass streamClass)
                             throws java.io.IOException, ClassNotFoundException {
-                        Class<?> c = Class.forName(streamClass.getName(), false, loader);
-                        if (c == null) {
-                            return super.resolveClass(streamClass);
-                        } else {
-                            return c;   // Class loader knows of this class.
-                        }   // end else: not null
+                        return Class.forName(streamClass.getName(), false, loader);
                     }   // end resolveClass
-                };  // end ois
+                }) {  // end ois
+                    return ois.readObject();
+                } // end try
             }   // end else: no custom class loader
-
-            obj = ois.readObject();
         }   // end try
-        catch (java.io.IOException e) {
-            throw e;    // Catch and throw in order to execute finally{}
-        }   // end catch
-        catch (java.lang.ClassNotFoundException e) {
-            throw e;    // Catch and throw in order to execute finally{}
-        }   // end catch
-        finally {
-            try {
-                bais.close();
-            } catch (Exception e) {
-            }
-            try {
-                ois.close();
-            } catch (Exception e) {
-            }
-        }   // end finally
 
-        return obj;
     }   // end decodeObject
 
 
@@ -1325,17 +1228,10 @@ public class Base64 {
             throw new NullPointerException("Data to encode was null.");
         }   // end iff
 
-        Base64.OutputStream bos = null;
-        try {
-            bos = new Base64.OutputStream(
-                    new java.io.FileOutputStream(filename), Base64.ENCODE);
+        try (OutputStream bos = new OutputStream(
+                new java.io.FileOutputStream(filename), Base64.ENCODE)) {
             bos.write(dataToEncode);
-        } finally {
-            try {
-                bos.close();
-            } catch (Exception e) {
-            }
-        }   // end finally
+        }
 
     }   // end encodeToFile
 
@@ -1356,21 +1252,10 @@ public class Base64 {
     public static void decodeToFile(String dataToDecode, String filename)
             throws java.io.IOException {
 
-        Base64.OutputStream bos = null;
-        try {
-            bos = new Base64.OutputStream(
-                    new java.io.FileOutputStream(filename), Base64.DECODE);
+        try (OutputStream bos = new OutputStream(
+                new java.io.FileOutputStream(filename), Base64.DECODE)) {
             bos.write(dataToDecode.getBytes(PREFERRED_ENCODING));
-        }   // end try
-        catch (java.io.IOException e) {
-            throw e; // Catch and throw to execute finally{} block
-        }   // end catch: java.io.IOException
-        finally {
-            try {
-                bos.close();
-            } catch (Exception e) {
-            }
-        }   // end finally
+        }
 
     }   // end decodeToFile
 
@@ -1393,24 +1278,22 @@ public class Base64 {
             throws java.io.IOException {
 
         byte[] decodedData = null;
-        Base64.InputStream bis = null;
-        try {
-            // Set up some useful variables
-            java.io.File file = new java.io.File(filename);
-            byte[] buffer = null;
-            int length = 0;
-            int numBytes = 0;
+        // Set up some useful variables
+        java.io.File file = new java.io.File(filename);
+        byte[] buffer = null;
+        int length = 0;
+        int numBytes = 0;
 
-            // Check for size of file
-            if (file.length() > Integer.MAX_VALUE) {
-                throw new java.io.IOException("File is too big for this convenience method (" + file.length() + " bytes).");
-            }   // end if: file too big for int index
-            buffer = new byte[(int) file.length()];
+        // Check for size of file
+        if (file.length() > Integer.MAX_VALUE) {
+            throw new java.io.IOException("File is too big for this convenience method (" + file.length() + " bytes).");
+        }   // end if: file too big for int index
+        buffer = new byte[(int) file.length()];
 
-            // Open a stream
-            bis = new Base64.InputStream(
-                    new java.io.BufferedInputStream(
-                            new java.io.FileInputStream(file)), Base64.DECODE);
+        // Open a stream
+        try (Base64.InputStream bis = new Base64.InputStream(
+                new java.io.BufferedInputStream(
+                        new java.io.FileInputStream(file)), Base64.DECODE)) {
 
             // Read until done
             while ((numBytes = bis.read(buffer, length, 4096)) >= 0) {
@@ -1420,17 +1303,7 @@ public class Base64 {
             // Save in a variable to return
             decodedData = new byte[length];
             System.arraycopy(buffer, 0, decodedData, 0, length);
-
-        }   // end try
-        catch (java.io.IOException e) {
-            throw e; // Catch and release to execute finally{}
-        }   // end catch: java.io.IOException
-        finally {
-            try {
-                bis.close();
-            } catch (Exception e) {
-            }
-        }   // end finally
+        } // end try
 
         return decodedData;
     }   // end decodeFromFile
@@ -1454,18 +1327,16 @@ public class Base64 {
             throws java.io.IOException {
 
         String encodedData = null;
-        Base64.InputStream bis = null;
-        try {
-            // Set up some useful variables
-            java.io.File file = new java.io.File(filename);
-            byte[] buffer = new byte[Math.max((int) (file.length() * 1.4 + 1), 40)]; // Need max() for math on small files (v2.2.1); Need +1 for a few corner cases (v2.3.5)
-            int length = 0;
-            int numBytes = 0;
+        // Set up some useful variables
+        java.io.File file = new java.io.File(filename);
+        byte[] buffer = new byte[Math.max((int) (file.length() * 1.4 + 1), 40)]; // Need max() for math on small files (v2.2.1); Need +1 for a few corner cases (v2.3.5)
+        int length = 0;
+        int numBytes = 0;
 
-            // Open a stream
-            bis = new Base64.InputStream(
+        // Open a stream
+        try (Base64.InputStream bis = new Base64.InputStream(
                     new java.io.BufferedInputStream(
-                            new java.io.FileInputStream(file)), Base64.ENCODE);
+                            new java.io.FileInputStream(file)), Base64.ENCODE)) {
 
             // Read until done
             while ((numBytes = bis.read(buffer, length, 4096)) >= 0) {
@@ -1476,15 +1347,6 @@ public class Base64 {
             encodedData = new String(buffer, 0, length, Base64.PREFERRED_ENCODING);
 
         }   // end try
-        catch (java.io.IOException e) {
-            throw e; // Catch and release to execute finally{}
-        }   // end catch: java.io.IOException
-        finally {
-            try {
-                bis.close();
-            } catch (Exception e) {
-            }
-        }   // end finally
 
         return encodedData;
     }   // end encodeFromFile
@@ -1501,21 +1363,10 @@ public class Base64 {
             throws java.io.IOException {
 
         String encoded = Base64.encodeFromFile(infile);
-        java.io.OutputStream out = null;
-        try {
-            out = new java.io.BufferedOutputStream(
-                    new java.io.FileOutputStream(outfile));
-            out.write(encoded.getBytes("US-ASCII")); // Strict, 7-bit output.
-        }   // end try
-        catch (java.io.IOException e) {
-            throw e; // Catch and release to execute finally{}
-        }   // end catch
-        finally {
-            try {
-                out.close();
-            } catch (Exception ex) {
-            }
-        }   // end finally
+        try (java.io.OutputStream out = new java.io.BufferedOutputStream(
+                new java.io.FileOutputStream(outfile))) {
+            out.write(encoded.getBytes(StandardCharsets.US_ASCII)); // Strict, 7-bit output.
+        }
     }   // end encodeFileToFile
 
 
@@ -1531,21 +1382,10 @@ public class Base64 {
             throws java.io.IOException {
 
         byte[] decoded = Base64.decodeFromFile(infile);
-        java.io.OutputStream out = null;
-        try {
-            out = new java.io.BufferedOutputStream(
-                    new java.io.FileOutputStream(outfile));
+        try (java.io.OutputStream out = new java.io.BufferedOutputStream(
+                new java.io.FileOutputStream(outfile))) {
             out.write(decoded);
-        }   // end try
-        catch (java.io.IOException e) {
-            throw e; // Catch and release to execute finally{}
-        }   // end catch
-        finally {
-            try {
-                out.close();
-            } catch (Exception ex) {
-            }
-        }   // end finally
+        }
     }   // end decodeFileToFile
 
 

src/main/java/com/Acrobot/Breeze/Utils/ImplementationAdapter.java

@@ -0,0 +1,78 @@
+package com.Acrobot.Breeze.Utils;
+
+import org.bukkit.block.Block;
+import org.bukkit.block.BlockState;
+import org.bukkit.block.DoubleChest;
+import org.bukkit.inventory.Inventory;
+import org.bukkit.inventory.InventoryHolder;
+
+import java.util.function.BiFunction;
+
+public class ImplementationAdapter {
+
+    private static BiFunction<Inventory, Boolean, InventoryHolder> HOLDER_PROVIDER;
+    private static BiFunction<DoubleChest, Boolean, InventoryHolder> LEFT_HOLDER_PROVIDER;
+    private static BiFunction<DoubleChest, Boolean, InventoryHolder> RIGHT_HOLDER_PROVIDER;
+    private static BiFunction<Block, Boolean, BlockState> STATE_PROVIDER;
+
+    static {
+        try {
+            Inventory.class.getMethod("getHolder", boolean.class);
+            Class c = Class.forName("com.Acrobot.Breeze.Utils.ImplementationFeatures.PaperLatestHolder");
+            HOLDER_PROVIDER = (BiFunction<Inventory, Boolean, InventoryHolder>) c.getDeclaredField("PROVIDER").get(null);
+            LEFT_HOLDER_PROVIDER = (BiFunction<DoubleChest, Boolean, InventoryHolder>) c.getDeclaredField("LEFT_PROVIDER").get(null);
+            RIGHT_HOLDER_PROVIDER = (BiFunction<DoubleChest, Boolean, InventoryHolder>) c.getDeclaredField("RIGHT_PROVIDER").get(null);
+        } catch (NoSuchMethodException | ClassNotFoundException | NoSuchFieldException | IllegalAccessException e) {
+            HOLDER_PROVIDER = (inventory, useSnapshot) -> inventory.getHolder();
+            LEFT_HOLDER_PROVIDER = (doubleChest, useSnapshot) -> doubleChest.getLeftSide();
+            RIGHT_HOLDER_PROVIDER = (doubleChest, useSnapshot) -> doubleChest.getRightSide();
+        }
+        try {
+            Block.class.getMethod("getState", boolean.class);
+            Class c = Class.forName("com.Acrobot.Breeze.Utils.ImplementationFeatures.PaperLatestState");
+            STATE_PROVIDER = (BiFunction<Block, Boolean, BlockState>) c.getDeclaredField("PROVIDER").get(null);
+        } catch (NoSuchMethodException | ClassNotFoundException | NoSuchFieldException | IllegalAccessException e) {
+            STATE_PROVIDER = (block, useSnapshot) -> block.getState();
+        }
+    }
+
+    /**
+     * Get the inventory's holder.
+     * @param inventory     The inventory
+     * @param useSnapshot   Whether or not the holder should be a snapshot (if possible)
+     * @return The inventory's holder
+     */
+    public static InventoryHolder getHolder(Inventory inventory, boolean useSnapshot) {
+        return HOLDER_PROVIDER.apply(inventory, useSnapshot);
+    }
+
+    /**
+     * Get the a DoubleChest's left side
+     * @param doubleChest   The DoubleChest
+     * @param useSnapshot   Whether or not the holder should be a snapshot (if possible)
+     * @return The left side's holder
+     */
+    public static InventoryHolder getLeftSide(DoubleChest doubleChest, boolean useSnapshot) {
+        return LEFT_HOLDER_PROVIDER.apply(doubleChest, useSnapshot);
+    }
+
+    /**
+     * Get the a DoubleChest's right side
+     * @param doubleChest   The DoubleChest
+     * @param useSnapshot   Whether or not the holder should be a snapshot (if possible)
+     * @return The left right's holder
+     */
+    public static InventoryHolder getRightSide(DoubleChest doubleChest, boolean useSnapshot) {
+        return RIGHT_HOLDER_PROVIDER.apply(doubleChest, useSnapshot);
+    }
+
+    /**
+     * Get a block state
+     * @param block         The block to get the state from
+     * @param useSnapshot   Whether or not the state should be a snapshot (if possible)
+     * @return The block's state
+     */
+    public static BlockState getState(Block block, boolean useSnapshot) {
+        return STATE_PROVIDER.apply(block, useSnapshot);
+    }
+}

src/main/java/com/Acrobot/Breeze/Utils/ImplementationFeatures/PaperLatestHolder.java

@@ -0,0 +1,17 @@
+package com.Acrobot.Breeze.Utils.ImplementationFeatures;
+
+import org.bukkit.block.DoubleChest;
+import org.bukkit.inventory.Inventory;
+import org.bukkit.inventory.InventoryHolder;
+
+import java.util.function.BiFunction;
+
+public class PaperLatestHolder {
+
+    public static final BiFunction<Inventory, Boolean, InventoryHolder> PROVIDER = Inventory::getHolder;
+
+    public static final BiFunction<DoubleChest, Boolean, InventoryHolder> LEFT_PROVIDER = DoubleChest::getLeftSide;
+
+    public static final BiFunction<DoubleChest, Boolean, InventoryHolder> RIGHT_PROVIDER = DoubleChest::getRightSide;
+
+}

src/main/java/com/Acrobot/Breeze/Utils/ImplementationFeatures/PaperLatestState.java

@@ -0,0 +1,12 @@
+package com.Acrobot.Breeze.Utils.ImplementationFeatures;
+
+import org.bukkit.block.Block;
+import org.bukkit.block.BlockState;
+
+import java.util.function.BiFunction;
+
+public class PaperLatestState {
+
+    public static final BiFunction<Block, Boolean, BlockState> PROVIDER = Block::getState;
+
+}

src/main/java/com/Acrobot/Breeze/Utils/InventoryUtil.java

@@ -351,7 +351,7 @@ public class InventoryUtil {
             itemList.add(item.clone());
         }
 
-        return itemList.toArray(new ItemStack[itemList.size()]);
+        return itemList.toArray(new ItemStack[0]);
     }
 
     /**
@@ -413,7 +413,7 @@ public class InventoryUtil {
                 stackedItems.add(item.clone());
                 continue;
             }
-            for (int i = 0; i < Math.floor(item.getAmount() / maxStackSize); i++) {
+            for (int i = 0; i < Math.floor((double) item.getAmount() / maxStackSize); i++) {
                 ItemStack itemClone = item.clone();
                 itemClone.setAmount(maxStackSize);
                 stackedItems.add(itemClone);
@@ -424,6 +424,6 @@ public class InventoryUtil {
                 stackedItems.add(rest);
             }
         }
-        return stackedItems.toArray(new ItemStack[stackedItems.size()]);
+        return stackedItems.toArray(new ItemStack[0]);
     }
 }

src/main/java/com/Acrobot/Breeze/Utils/MaterialUtil.java

@@ -2,14 +2,16 @@ package com.Acrobot.Breeze.Utils;
 
 import com.Acrobot.Breeze.Collection.SimpleCache;
 import com.Acrobot.ChestShop.ChestShop;
+import com.Acrobot.ChestShop.Configuration.Messages;
 import com.Acrobot.ChestShop.Configuration.Properties;
-import com.Acrobot.ChestShop.Events.ItemParseEvent;
 import com.Acrobot.ChestShop.Events.MaterialParseEvent;
-import com.google.common.collect.ImmutableMap;
+import com.Acrobot.ChestShop.Utils.ItemUtil;
 import de.themoep.ShowItem.api.ShowItem;
-import info.somethingodd.OddItem.OddItem;
+import de.themoep.minedown.adventure.Replacer;
+import net.kyori.adventure.text.Component;
+import net.kyori.adventure.text.TextComponent;
+import net.kyori.adventure.text.serializer.gson.GsonComponentSerializer;
 import org.bukkit.Bukkit;
-import org.bukkit.ChatColor;
 import org.bukkit.Material;
 import org.bukkit.configuration.file.YamlConstructor;
 import org.bukkit.configuration.file.YamlRepresenter;
@@ -18,19 +20,18 @@ import org.bukkit.inventory.ItemStack;
 import org.bukkit.inventory.meta.Damageable;
 import org.bukkit.inventory.meta.ItemMeta;
 import org.bukkit.plugin.Plugin;
-import org.json.simple.JSONObject;
 import org.yaml.snakeyaml.DumperOptions;
 import org.yaml.snakeyaml.Yaml;
 import org.yaml.snakeyaml.nodes.Tag;
 
 import java.util.ArrayList;
-import java.util.Collection;
+import java.util.LinkedHashMap;
 import java.util.List;
+import java.util.Locale;
 import java.util.Map;
 import java.util.logging.Level;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
-import java.util.stream.Collectors;
 
 import static com.Acrobot.Breeze.Utils.StringUtil.getMinecraftCharWidth;
 import static com.Acrobot.Breeze.Utils.StringUtil.getMinecraftStringWidth;
@@ -52,6 +53,23 @@ public class MaterialUtil {
     // 15 dashes fit on one sign line with the default resource pack:
     public static final int MAXIMUM_SIGN_WIDTH = (short) getMinecraftStringWidth("---------------");
 
+    private static final Map<String, String> ABBREVIATIONS = StringUtil.map(
+            "Egg", "Eg",
+            "Spawn", "Spaw",
+            "Pottery", "Pot",
+            "Heartbreak", "Heartbr",
+            "Sherd", "Sher"
+    );
+
+    private static final Map<String, String> UNIDIRECTIONAL_ABBREVIATIONS = StringUtil.map(
+            "Endermite", "Endmite",
+            "Endmite", "Endmit",
+            "Wayfinder", "Wayfndr",
+            "Wayfndr", "Wf",
+            "Heartbr", "Hrtbr",
+            "Hrtbr", "Hrtb"
+    );
+
     private static final SimpleCache<String, Material> MATERIAL_CACHE = new SimpleCache<>(Properties.CACHE_SIZE);
 
     private static final Yaml YAML = new Yaml(new YamlBukkitConstructor(), new YamlRepresenter(), new DumperOptions());
@@ -100,24 +118,41 @@ public class MaterialUtil {
         if (!one.hasItemMeta() && !two.hasItemMeta()) {
             return true;
         }
-        Map<String, Object> oneSerMeta = one.getItemMeta().serialize();
-        Map<String, Object> twoSerMeta = two.getItemMeta().serialize();
+        ItemMeta oneMeta = one.getItemMeta();
+        ItemMeta twoMeta = two.getItemMeta();
+        // return true if both are null or same, false if only one is null
+        if (oneMeta == twoMeta || oneMeta == null || twoMeta == null) {
+            return oneMeta == twoMeta;
+        }
+        Map<String, Object> oneSerMeta = oneMeta.serialize();
+        Map<String, Object> twoSerMeta = twoMeta.serialize();
         if (oneSerMeta.equals(twoSerMeta)) {
             return true;
         }
 
         // Try to use same parsing as the YAML dumper in the ItemDatabase when generating the code as the last resort
         ItemStack oneDumped = YAML.loadAs(YAML.dump(one), ItemStack.class);
-        if (oneDumped.isSimilar(two) || oneDumped.getItemMeta().serialize().equals(twoSerMeta)) {
+        if (oneDumped.isSimilar(two)) {
+            return true;
+        }
+
+        ItemMeta oneDumpedMeta = oneDumped.getItemMeta();
+        if (oneDumpedMeta != null && oneDumpedMeta.serialize().equals(twoSerMeta)) {
             return true;
         }
 
         ItemStack twoDumped = YAML.loadAs(YAML.dump(two), ItemStack.class);
-        if (oneDumped.isSimilar(twoDumped) || oneDumped.getItemMeta().serialize().equals(twoDumped.getItemMeta().serialize())) {
+        if (oneDumped.isSimilar(twoDumped)) {
             return true;
         }
 
-        return false;
+        ItemMeta twoDumpedMeta = twoDumped.getItemMeta();
+        if (oneDumpedMeta != null && twoDumpedMeta != null && oneDumpedMeta.serialize().equals(twoDumpedMeta.serialize())) {
+            return true;
+        }
+
+        // return true if both are null or same, false otherwise
+        return oneDumpedMeta == twoDumpedMeta;
     }
 
     /**
@@ -127,7 +162,15 @@ public class MaterialUtil {
      * @return Material found
      */
     public static Material getMaterial(String name) {
-        String formatted = name.replaceAll("([a-z])([A-Z1-9])", "$1_$2").replace(' ', '_').toUpperCase();
+        String replacedName = name;
+        // revert unidirectional abbreviations
+        List<Map.Entry<String, String>> abbreviations = new ArrayList<>(UNIDIRECTIONAL_ABBREVIATIONS.entrySet());
+        for (int i = abbreviations.size() - 1; i >= 0; i--) {
+            Map.Entry<String, String> entry = abbreviations.get(i);
+            replacedName = replacedName.replaceAll(entry.getValue() + "(_|$)?", entry.getKey() + "$1");
+        }
+
+        String formatted = name.replaceAll("(?<!^)(?>\\s?)([A-Z1-9])", "_$1").replace(' ', '_').toUpperCase(Locale.ROOT);
 
         Material material = MATERIAL_CACHE.get(formatted);
         if (material != null) {
@@ -141,7 +184,7 @@ public class MaterialUtil {
             return material;
         }
 
-        material = new EnumParser<Material>().parse(name, Material.values());
+        material = new EnumParser<Material>().parse(replacedName, Material.values());
         if (material != null) {
             MATERIAL_CACHE.put(formatted, material);
         }
@@ -154,7 +197,9 @@ public class MaterialUtil {
      *
      * @param items The items to get the information from
      * @return The list, including the amount and names of the items
+     * @deprecated Use {@link ItemUtil#getItemList(ItemStack[])} instead!
      */
+    @Deprecated
     public static String getItemList(ItemStack[] items) {
         ItemStack[] mergedItems = InventoryUtil.mergeSimilarStacks(items);
 
@@ -169,6 +214,7 @@ public class MaterialUtil {
 
     /**
      * Returns item's name
+     * Use {@link ItemUtil#getName(ItemStack, int)} if you want to get name aliases too!
      *
      * @param itemStack ItemStack to name
      * @return ItemStack's name
@@ -192,6 +238,7 @@ public class MaterialUtil {
 
     /**
      * Returns item's name, just like on the sign
+     * Use {@link ItemUtil#getSignName(ItemStack)} if you want to get name aliases too!
      *
      * @param itemStack ItemStack to name
      * @return ItemStack's name
@@ -201,15 +248,15 @@ public class MaterialUtil {
     }
 
     /**
-     * Returns item's name, with a maximum width
+     * Returns item's name, with a maximum width.
+     * Use {@link ItemUtil#getName(ItemStack, int)} if you want to get name aliases too!
      *
      * @param itemStack ItemStack to name
      * @param maxWidth The max width that the name should have; 0 or below if it should be unlimited
      * @return ItemStack's name
      */
     public static String getName(ItemStack itemStack, int maxWidth) {
-        String alias = Odd.getAlias(itemStack);
-        String itemName = alias != null ? alias : itemStack.getType().toString();
+        String itemName = itemStack.getType().toString();
 
         String durability = "";
         if (itemStack.getDurability() != 0) {
@@ -222,23 +269,15 @@ public class MaterialUtil {
         }
 
         String code = StringUtil.capitalizeFirstLetter(itemName, '_');
-        int codeWidth = getMinecraftStringWidth(code + durability + metaData);
-        if (maxWidth > 0 && codeWidth > maxWidth) {
-            int exceeding = codeWidth - maxWidth;
-            code = getShortenedName(code, getMinecraftStringWidth(code) - exceeding);
+        if (maxWidth > 0) {
+            int codeWidth = getMinecraftStringWidth(code + durability + metaData);
+            if (codeWidth > maxWidth) {
+                int exceeding = codeWidth - maxWidth;
+                code = getShortenedName(code, getMinecraftStringWidth(code) - exceeding);
+            }
         }
 
-        code += durability + metaData;
-
-        ItemParseEvent parseEvent = new ItemParseEvent(code);
-        Bukkit.getPluginManager().callEvent(parseEvent);
-        ItemStack codeItem = parseEvent.getItem();
-        if (!equals(itemStack, codeItem)) {
-            throw new IllegalArgumentException("Cannot generate code for item " + itemStack + " with maximum length of " + maxWidth
-                    + " (code " + code + " results in item " + codeItem + ")");
-        }
-
-        return code;
+        return code + durability + metaData;
     }
 
     /**
@@ -249,17 +288,42 @@ public class MaterialUtil {
      * @return The name shortened to the max length
      */
     public static String getShortenedName(String itemName, int maxWidth) {
-        itemName = StringUtil.capitalizeFirstLetter(itemName.replace('_', ' '), ' ');
-        int width = getMinecraftStringWidth(itemName);
+        // Restore spaces in string that might be already be shortened
+        String name = itemName.replaceAll("([a-z])([A-Z1-9])", "$1 $2");
+        name = StringUtil.capitalizeFirstLetter(name.replace('_', ' '), ' ');
+        int width = getMinecraftStringWidth(name);
         if (width <= maxWidth) {
-            return itemName;
+            return name;
         }
-        String[] itemParts = itemName.split(" ");
-        itemName = String.join("", itemParts);
-        width = getMinecraftStringWidth(itemName);
+        String[] itemParts = name.split("[ \\-]");
+        String noSpaceName = String.join("", itemParts);
+        width = getMinecraftStringWidth(noSpaceName);
         if (width <= maxWidth) {
-            return itemName;
+            return noSpaceName;
         }
+
+        // Abbreviate some terms manually
+        for (Map.Entry<String, String> entry : ABBREVIATIONS.entrySet()) {
+            name = name.replaceAll(entry.getKey() + "( |$)", entry.getValue() + "$1");
+            itemParts = name.split("[ \\-]");
+            noSpaceName = String.join("", itemParts);
+            width = getMinecraftStringWidth(noSpaceName);
+            if (width <= maxWidth) {
+                return noSpaceName;
+            }
+        }
+
+        // Apply unidirectional abbreviations if it still doesn't work
+        for (Map.Entry<String, String> entry : UNIDIRECTIONAL_ABBREVIATIONS.entrySet()) {
+            name = name.replaceAll(entry.getKey() + "( |$)", entry.getValue() + "$1");
+            itemParts = name.split("[ \\-]");
+            noSpaceName = String.join("", itemParts);
+            width = getMinecraftStringWidth(noSpaceName);
+            if (width <= maxWidth) {
+                return noSpaceName;
+            }
+        }
+
         int exceeding = width - maxWidth;
         int shortestIndex = 0;
         int longestIndex = 0;
@@ -317,12 +381,6 @@ public class MaterialUtil {
      * @return ItemStack
      */
     public static ItemStack getItem(String itemName) {
-        ItemStack itemStack = Odd.getFromString(itemName);
-
-        if (itemStack != null) {
-            return itemStack;
-        }
-
         String[] split = itemName.split("[:\\-#]");
         for (int i = 0; i < split.length; i++) {
             split[i] = split[i].trim();
@@ -336,7 +394,7 @@ public class MaterialUtil {
             return null;
         }
 
-        itemStack = new ItemStack(material);
+        ItemStack itemStack = new ItemStack(material);
 
         ItemMeta meta = getMetadata(itemName);
 
@@ -393,17 +451,18 @@ public class MaterialUtil {
 
     private static class EnumParser<E extends Enum<E>> {
         private E parse(String name, E[] values) {
+            String formatted = name.replaceAll("(?<!^)(?>\\s?)([A-Z1-9])", "_$1").toUpperCase(Locale.ROOT).replace(' ', '_');
             try {
-                return E.valueOf(values[0].getDeclaringClass(), name.toUpperCase());
+                return E.valueOf(values[0].getDeclaringClass(), formatted);
             } catch (IllegalArgumentException exception) {
-                E currentEnum = null;
-                String[] typeParts = name.replaceAll("([a-z])([A-Z1-9])", "$1_$2").toUpperCase().split("[ _]");
+                List<E> possibleEnums = new ArrayList<>();
+                String[] typeParts = formatted.split("_");
                 int length = Short.MAX_VALUE;
                 for (E e : values) {
                     String enumName = e.name();
-                    if (enumName.length() < length && enumName.startsWith(name)) {
-                        length = (short) enumName.length();
-                        currentEnum = e;
+                    if (enumName.length() < length && enumName.startsWith(formatted)) {
+                        length = enumName.length();
+                        possibleEnums.add(e);
                     } else if (typeParts.length > 1) {
                         String[] nameParts = enumName.split("_");
                         if (typeParts.length == nameParts.length) {
@@ -415,13 +474,28 @@ public class MaterialUtil {
                                 }
                             }
                             if (matched) {
-                                currentEnum = e;
-                                break;
+                                possibleEnums.add(e);
                             }
                         }
                     }
                 }
-                return currentEnum;
+
+                if (possibleEnums.size() == 1) {
+                    return possibleEnums.get(0);
+                } else if (possibleEnums.size() > 1) {
+                    int formattedLength = formatted.length();
+                    int closestDeviation = Short.MAX_VALUE;
+                    E closestEnum = null;
+                    for (E possibleEnum : possibleEnums) {
+                        int deviation = possibleEnum.name().length() - formattedLength;
+                        if (deviation < closestDeviation) {
+                            closestDeviation = deviation;
+                            closestEnum = possibleEnum;
+                        }
+                    }
+                    return closestEnum;
+                }
+                return null;
             }
         }
     }
@@ -454,52 +528,6 @@ public class MaterialUtil {
         }
     }
 
-    public static class Odd {
-        private static boolean isInitialized = false;
-
-        /**
-         * Returns the item stack from OddItem plugin
-         *
-         * @param itemName Item name to parse
-         * @return itemStack that was parsed
-         */
-        public static ItemStack getFromString(String itemName) {
-            if (!isInitialized) {
-                return null;
-            }
-
-            String name = itemName.replace(':', ';');
-
-            try {
-                return OddItem.getItemStack(name);
-            } catch (Exception ex) {
-                return null;
-            }
-        }
-
-        public static String getAlias(ItemStack itemStack) {
-            if (!isInitialized) {
-                return null;
-            }
-
-            try {
-                Collection<String> aliases = OddItem.getAliases(itemStack);
-                if (!aliases.isEmpty()) {
-                    return aliases.iterator().next();
-                }
-            } catch (Exception ignored) {
-            }
-            return null;
-        }
-
-        /**
-         * Lets the class know that it's safe to use the OddItem methods now
-         */
-        public static void initialize() {
-            isInitialized = true;
-        }
-    }
-
     public static class Show {
         private static ShowItem showItem = null;
 
@@ -519,40 +547,61 @@ public class MaterialUtil {
          * @param message The raw message
          * @param stock   The items in stock
          */
-        public static boolean sendMessage(Player player, String message, ItemStack[] stock) {
+        public static boolean sendMessage(Player player, Messages.Message message, ItemStack[] stock, Map<String, String> replacementMap, String... replacements) {
+            return sendMessage(player, player.getName(), message, stock, replacementMap, replacements);
+        }
+
+        /**
+         * Send a message with hover info and icons
+         *
+         * @param player        The player to send the message to
+         * @param playerName    The name of the player in case he is offline and bungee messages are enabled
+         * @param message       The raw message
+         * @param stock         The items in stock
+         */
+        public static boolean sendMessage(Player player, String playerName, Messages.Message message, ItemStack[] stock, Map<String, String> replacementMap, String... replacements) {
+            return sendMessage(player, playerName, message, true, stock, replacementMap, replacements);
+        }
+
+        /**
+         * Send a message with hover info and icons
+         *
+         * @param player        The player to send the message to
+         * @param playerName    The name of the player in case he is offline and bungee messages are enabled
+         * @param message       The raw message
+         * @param showPrefix    If the prefix should show
+         * @param stock         The items in stock
+         */
+        public static boolean sendMessage(Player player, String playerName, Messages.Message message, boolean showPrefix, ItemStack[] stock, Map<String, String> replacementMap, String... replacements) {
             if (showItem == null) {
                 return false;
             }
 
-            List<String> itemJson = new ArrayList<>();
+            TextComponent.Builder itemComponent = Component.text();
             for (ItemStack item : InventoryUtil.mergeSimilarStacks(stock)) {
                 try {
-                    itemJson.add(showItem.getItemConverter().createComponent(item, Level.FINE).toJsonString(player));
+                    itemComponent.append(GsonComponentSerializer.gson().deserialize(showItem.getItemConverter().createComponent(item, Level.FINE).toJsonString(player)));
                 } catch (Exception e) {
                     ChestShop.getPlugin().getLogger().log(Level.WARNING, "Error while trying to send message '" + message + "' to player " + player.getName() + ": " + e.getMessage());
                     return false;
                 }
             }
 
-            String joinedItemJson = itemJson.stream().collect(Collectors.joining("," + new JSONObject(ImmutableMap.of("text", " ")).toJSONString() + ", "));
-
-            String prevColor = "";
-            List<String> parts = new ArrayList<>();
-            for (String s : message.split("%item")) {
-                parts.add(new JSONObject(ImmutableMap.of("text", prevColor + s)).toJSONString());
-                prevColor = ChatColor.getLastColors(s);
+            Map<String, String> newMap = new LinkedHashMap<>(replacementMap);
+            newMap.put("material", "item");
+            newMap.remove("item");
+            Component component = new Replacer()
+                    .placeholderSuffix("")
+                    .replace("item",itemComponent.build())
+                    .replaceIn(message.getComponent(player, showPrefix, newMap, replacements));
+            if (player != null) {
+                ChestShop.getAudiences().player(player).sendMessage(component);
+                return true;
+            } else if (playerName != null) {
+                ChestShop.sendBungeeMessage(playerName, component);
+                return true;
             }
 
-            String messageJsonString = String.join("," + joinedItemJson + ",", parts);
-
-            while (messageJsonString.startsWith(",")) {
-                messageJsonString = messageJsonString.substring(1);
-            }
-            while (messageJsonString.endsWith(",")) {
-                messageJsonString = messageJsonString.substring(0, messageJsonString.length() - 1);
-            }
-
-            showItem.tellRaw(player, messageJsonString);
             return true;
         }
     }

src/main/java/com/Acrobot/Breeze/Utils/NumberUtil.java

@@ -154,4 +154,14 @@ public class NumberUtil {
                 return Integer.toString(number);
         }
     }
+
+    /**
+     * Convert a long to an integer while not overflowing but returning Integer.MAX_VALUE
+     *
+     * @param number The long to convert
+     * @return The integer value or Integer.MAX_VALUE on overflow
+     */
+    public static int toInt(long number) {
+        return number > Integer.MAX_VALUE ? Integer.MAX_VALUE : (int) number;
+    }
 }

src/main/java/com/Acrobot/Breeze/Utils/PriceUtil.java

@@ -1,6 +1,7 @@
 package com.Acrobot.Breeze.Utils;
 
 import java.math.BigDecimal;
+import java.util.Locale;
 
 /**
  * @author Acrobot
@@ -23,8 +24,8 @@ public class PriceUtil {
      * @return exact price
      */
     public static BigDecimal getExact(String text, char indicator) {
-        String[] split = text.replace(" ", "").toLowerCase().split(":");
-        String character = String.valueOf(indicator).toLowerCase();
+        String[] split = text.replace(" ", "").toLowerCase(Locale.ROOT).split(":");
+        String character = String.valueOf(indicator).toLowerCase(Locale.ROOT);
 
         for (String part : split) {
             if (!part.startsWith(character) && !part.endsWith(character)) {
@@ -40,7 +41,7 @@ public class PriceUtil {
             try {
                 BigDecimal price = new BigDecimal(part);
 
-                if (price.compareTo(MAX) > 0 || price.compareTo(BigDecimal.ZERO) <= 0) {
+                if (price.compareTo(MAX) > 0 || price.compareTo(BigDecimal.ZERO) < 0) {
                     return NO_PRICE;
                 } else {
                     return price;

src/main/java/com/Acrobot/Breeze/Utils/QuantityUtil.java

@@ -0,0 +1,22 @@
+package com.Acrobot.Breeze.Utils;
+
+import java.util.regex.Pattern;
+
+/**
+ * @author bricefrisco
+ */
+public class QuantityUtil {
+    public static final Pattern QUANTITY_LINE_WITH_COUNTER_PATTERN = Pattern.compile("^Q [1-9][0-9]{0,4} : C [0-9]{1,5}$");
+
+    public static int parseQuantity(String quantityLine) throws IllegalArgumentException {
+        if (quantityLineContainsCounter(quantityLine)) {
+            return Integer.parseInt(quantityLine.split(" : ")[0].replace("Q ", ""));
+        }
+
+        return Integer.parseInt(quantityLine);
+    }
+
+    public static boolean quantityLineContainsCounter(String quantityLine) {
+        return QUANTITY_LINE_WITH_COUNTER_PATTERN.matcher(quantityLine).matches();
+    }
+}

src/main/java/com/Acrobot/Breeze/Utils/StringUtil.java

@@ -1,9 +1,13 @@
 package com.Acrobot.Breeze.Utils;
 
-import org.apache.commons.lang.WordUtils;
 import org.bukkit.ChatColor;
 
+import java.util.Arrays;
 import java.util.Collection;
+import java.util.LinkedHashMap;
+import java.util.Locale;
+import java.util.Map;
+import java.util.stream.Collectors;
 
 /**
  * @author Acrobot
@@ -18,9 +22,16 @@ public class StringUtil {
      * @return Reformatted string
      */
     public static String capitalizeFirstLetter(String string, char separator) {
-        char[] separators = new char[]{separator};
+        if (string == null || string.isEmpty()) {
+            return string;
+        }
 
-        return WordUtils.capitalizeFully(string, separators).replace(String.valueOf(separator), " ");
+        // Split into words
+        String[] words = string.toLowerCase(Locale.ROOT).split(String.valueOf(separator));
+        // Capitalize every word and return joined string
+        return Arrays.stream(words)
+                .map(word -> word.substring(0, 1).toUpperCase(Locale.ROOT) + word.substring(1))
+                .collect(Collectors.joining(" "));
     }
 
     /**
@@ -28,7 +39,7 @@ public class StringUtil {
      *
      * @param string String to reformat
      * @return Reformatted string
-     * @see com.Acrobot.Breeze.Utils.StringUtil#capitalizeFirstLetter(String, char)
+     * @see StringUtil#capitalizeFirstLetter(String, char)
      */
     public static String capitalizeFirstLetter(String string) {
         return capitalizeFirstLetter(string, ' ');
@@ -112,4 +123,52 @@ public class StringUtil {
         }
         return width;
     }
+
+    /**
+     * Strip whitespace from the front and back
+     * @param string The string to strip
+     * @return The string with all whitespace from front and back stripped; returns null if input is null
+     */
+    public static String strip(String string) {
+        if (string == null)
+            return null;
+        // The result stripped string
+        StringBuilder stripped = new StringBuilder();
+        // The current white space which only gets added once we find a non-whitespace character
+        StringBuilder cachedWhitespace = new StringBuilder();
+        // Check each code point (not characters to support UTF16 properly)
+        for (int codePoint : string.codePoints().toArray()) {
+            // Check if it's a whitespace, so we know if we should add it
+            if (!Character.isWhitespace(codePoint)) {
+                // Check if we have cached whitespace, if so append it first and reset the cache
+                if (cachedWhitespace.length() > 0) {
+                    stripped.append(cachedWhitespace);
+                    cachedWhitespace = new StringBuilder();
+                }
+                // Append current code point
+                stripped.appendCodePoint(codePoint);
+            } else if (stripped.length() > 0) {
+                // If we already have some non-whitespace content in the final stripped
+                // then cache the current whitespace as it wasn't at the start
+                cachedWhitespace.appendCodePoint(codePoint);
+            } // Otherwise, this was the start, and we don't need the cached whitespace
+        }
+        // Return the stripped string, without any whitespace from the end left
+        return stripped.toString();
+    }
+
+    /**
+     * Create a map from strings
+     * @param values The values to add
+     * @return The map
+     */
+    public static Map<String, String> map(String... values) {
+        Map<String, String> map = new LinkedHashMap<>();
+
+        for (int i = 0; i + 1 < values.length; i+=2) {
+            map.put(values[i], values[i + 1]);
+        }
+
+        return map;
+    }
 }

src/main/java/com/Acrobot/ChestShop/ChestShop.java

@@ -3,6 +3,7 @@ package com.Acrobot.ChestShop;
 import com.Acrobot.Breeze.Configuration.Configuration;
 import com.Acrobot.ChestShop.Commands.Give;
 import com.Acrobot.ChestShop.Commands.ItemInfo;
+import com.Acrobot.ChestShop.Commands.ShopInfo;
 import com.Acrobot.ChestShop.Commands.Toggle;
 import com.Acrobot.ChestShop.Commands.Version;
 import com.Acrobot.ChestShop.Commands.AccessToggle;
@@ -18,7 +19,12 @@ import com.Acrobot.ChestShop.Listeners.Economy.TaxModule;
 import com.Acrobot.ChestShop.Listeners.AuthMeChestShopListener;
 import com.Acrobot.ChestShop.Listeners.GarbageTextListener;
 import com.Acrobot.ChestShop.Listeners.Item.ItemMoveListener;
+import com.Acrobot.ChestShop.Listeners.Item.ItemStringListener;
 import com.Acrobot.ChestShop.Listeners.ItemInfoListener;
+import com.Acrobot.ChestShop.Listeners.Modules.ItemAliasModule;
+import com.Acrobot.ChestShop.Listeners.Modules.MetricsModule;
+import com.Acrobot.ChestShop.Listeners.Modules.StockCounterModule;
+import com.Acrobot.ChestShop.Listeners.ShopInfoListener;
 import com.Acrobot.ChestShop.Listeners.SignParseListener;
 import com.Acrobot.ChestShop.Listeners.Modules.DiscountModule;
 import com.Acrobot.ChestShop.Listeners.Modules.PriceRestrictionModule;
@@ -38,11 +44,18 @@ import com.Acrobot.ChestShop.Logging.FileFormatter;
 import com.Acrobot.ChestShop.Metadata.ItemDatabase;
 import com.Acrobot.ChestShop.Signs.RestrictedSign;
 import com.Acrobot.ChestShop.UUIDs.NameManager;
+import com.Acrobot.ChestShop.Updater.JenkinsBuildsNotifier;
 import com.Acrobot.ChestShop.Updater.Updater;
 
+import com.google.common.collect.ImmutableMap;
 import com.google.common.io.ByteArrayDataOutput;
 import com.google.common.io.ByteStreams;
 
+import net.kyori.adventure.platform.bukkit.BukkitAudiences;
+import net.kyori.adventure.text.Component;
+import net.kyori.adventure.text.serializer.gson.GsonComponentSerializer;
+import net.md_5.bungee.api.chat.BaseComponent;
+import net.md_5.bungee.chat.ComponentSerializer;
 import org.apache.logging.log4j.Level;
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Marker;
@@ -52,7 +65,15 @@ import org.apache.logging.log4j.core.config.LoggerConfig;
 import org.apache.logging.log4j.core.filter.AbstractFilter;
 import org.apache.logging.log4j.message.Message;
 
+import org.bstats.bukkit.Metrics;
+import org.bstats.charts.AdvancedBarChart;
+import org.bstats.charts.DrilldownPie;
+import org.bstats.charts.MultiLineChart;
+import org.bstats.charts.SimpleBarChart;
+import org.bstats.charts.SimplePie;
+import org.bstats.charts.SingleLineChart;
 import org.bukkit.Bukkit;
+import org.bukkit.Material;
 import org.bukkit.Server;
 import org.bukkit.command.CommandExecutor;
 import org.bukkit.command.PluginCommand;
@@ -66,9 +87,18 @@ import org.bukkit.plugin.java.JavaPlugin;
 import java.io.File;
 import java.io.IOException;
 import java.util.ArrayList;
+import java.util.LinkedHashMap;
 import java.util.List;
+import java.util.Locale;
+import java.util.Map;
+import java.util.UUID;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Executors;
+import java.util.concurrent.TimeUnit;
+import java.util.jar.JarFile;
 import java.util.logging.FileHandler;
 import java.util.logging.Logger;
+import java.util.stream.Collectors;
 
 /**
  * Main file of the plugin
@@ -79,11 +109,17 @@ public class ChestShop extends JavaPlugin {
     private static ChestShop plugin;
     private static Server server;
     private static PluginDescriptionFile description;
+    private static final ExecutorService executorService = Executors.newCachedThreadPool();
+
+    private static Metrics bStats;
+
+    private static BukkitAudiences audiences;
 
     private static File dataFolder;
     private static ItemDatabase itemDatabase;
 
     private static Logger logger;
+    private static Logger shopLogger;
     private FileHandler handler;
 
     private List<PluginCommand> commands = new ArrayList<>();
@@ -91,6 +127,8 @@ public class ChestShop extends JavaPlugin {
     public ChestShop() {
         dataFolder = getDataFolder();
         logger = getLogger();
+        shopLogger = Logger.getLogger("ChestShop Shops");
+        shopLogger.setParent(logger);
         description = getDescription();
         server = getServer();
         plugin = this;
@@ -103,13 +141,17 @@ public class ChestShop extends JavaPlugin {
 
     @Override
     public void onEnable() {
+        bStats = new Metrics(this, 1109);
+        audiences = BukkitAudiences.create(this);
         turnOffDatabaseLogging();
         if (!handleMigrations()) {
             return;
         }
 
         registerCommand("iteminfo", new ItemInfo(), Permission.ITEMINFO);
+        registerCommand("shopinfo", new ShopInfo(), Permission.SHOPINFO);
         registerCommand("csVersion", new Version(), Permission.ADMIN);
+        registerCommand("csMetrics", new com.Acrobot.ChestShop.Commands.Metrics(), Permission.ADMIN);
         registerCommand("csGive", new Give(), Permission.ADMIN);
         registerCommand("cstoggle", new Toggle(), Permission.NOTIFY_TOGGLE);
         registerCommand("csaccess", new AccessToggle(), Permission.ACCESS_TOGGLE);
@@ -127,38 +169,44 @@ public class ChestShop extends JavaPlugin {
 
         registerPluginMessagingChannels();
 
-        if (Properties.LOG_TO_FILE) {
-            File log = loadFile("ChestShop.log");
-
-            FileHandler handler = loadHandler(log.getAbsolutePath());
-            handler.setFormatter(new FileFormatter());
-
-            this.handler = handler;
-            logger.addHandler(handler);
-        }
-
-        if (!Properties.LOG_TO_CONSOLE) {
-            logger.setUseParentHandlers(false);
-        }
-
         startStatistics();
+        startBuildNotificatier();
         startUpdater();
     }
 
     private void registerCommand(String name, CommandExecutor executor, Permission permission) {
         PluginCommand command = getCommand(name);
-        command.setExecutor(executor);
-        command.setPermission(permission.toString());
-        commands.add(command);
+        if (command != null) {
+            command.setExecutor(executor);
+            command.setPermission(permission.toString());
+            commands.add(command);
+        }
     }
 
     public void loadConfig() {
-        Configuration.pairFileAndClass(loadFile("config.yml"), Properties.class);
-        Configuration.pairFileAndClass(loadFile("local.yml"), Messages.class);
+        Configuration.pairFileAndClass(loadFile("config.yml"), Properties.class, getBukkitLogger());
+
+        Messages.load();
 
         NameManager.load();
 
-        commands.forEach(c -> c.setPermissionMessage(Messages.prefix(Messages.ACCESS_DENIED)));
+        commands.forEach(c -> c.setPermissionMessage(Messages.ACCESS_DENIED.getTextWithPrefix(null)));
+
+        if (handler != null) {
+            shopLogger.removeHandler(handler);
+        }
+
+        if (Properties.LOG_TO_FILE) {
+            if (handler == null) {
+                File log = loadFile("ChestShop.log");
+
+                handler = loadHandler(log.getAbsolutePath());
+                handler.setFormatter(new FileFormatter());
+            }
+            shopLogger.addHandler(handler);
+        }
+
+        shopLogger.setUseParentHandlers(Properties.LOG_TO_CONSOLE);
     }
 
     private void turnOffDatabaseLogging() {
@@ -211,7 +259,7 @@ public class ChestShop extends JavaPlugin {
             try {
                 previousVersion.save(versionFile);
             } catch (IOException e) {
-                e.printStackTrace();
+                getLogger().log(java.util.logging.Level.SEVERE, "Unable to save new database version " + Migrations.CURRENT_DATABASE_VERSION, e);
             }
         }
 
@@ -228,7 +276,7 @@ public class ChestShop extends JavaPlugin {
             try {
                 previousVersion.save(versionFile);
             } catch (IOException e) {
-                e.printStackTrace();
+                getLogger().log(java.util.logging.Level.SEVERE, "Unable to save new database version " + newVersion, e);
             }
         }
         return true;
@@ -249,7 +297,7 @@ public class ChestShop extends JavaPlugin {
 
                 file.createNewFile();
             } catch (IOException e) {
-                e.printStackTrace();
+                getBukkitLogger().log(java.util.logging.Level.SEVERE, "Unable to load file " + file.getName(), e);
             }
         }
 
@@ -262,14 +310,17 @@ public class ChestShop extends JavaPlugin {
         try {
             handler = new FileHandler(path, true);
         } catch (IOException ex) {
-            ex.printStackTrace();
+            getBukkitLogger().log(java.util.logging.Level.SEVERE, "Unable to load handler " + path, ex);
         }
 
         return handler;
     }
 
     public void onDisable() {
-        getServer().getScheduler().cancelTasks(this);
+        executorService.shutdown();
+        try {
+            executorService.awaitTermination(15, TimeUnit.SECONDS);
+        } catch (InterruptedException ignored) {}
 
         Toggle.clearToggledPlayers();
 
@@ -283,6 +334,8 @@ public class ChestShop extends JavaPlugin {
     private void registerEvents() {
         registerEvent(new com.Acrobot.ChestShop.Plugins.ChestShop()); //Chest protection
 
+        registerEvent(new Dependencies());
+        
         registerEvent(new NameManager());
 
         registerPreShopCreationEvents();
@@ -305,7 +358,9 @@ public class ChestShop extends JavaPlugin {
         registerEvent(new PlayerTeleport());
 
         registerEvent(new SignParseListener());
+        registerEvent(new ItemStringListener());
         registerEvent(new ItemInfoListener());
+        registerEvent(new ShopInfoListener());
         registerEvent(new GarbageTextListener());
 
         Plugin authMe = getServer().getPluginManager().getPlugin("AuthMe");
@@ -355,6 +410,7 @@ public class ChestShop extends JavaPlugin {
             registerEvent(new AmountAndPriceChecker());
         }
 
+        registerEvent(new InvalidNameIgnorer());
         registerEvent(new CreativeModeIgnorer());
         registerEvent(new ErrorMessageSender());
         registerEvent(new PermissionChecker());
@@ -373,8 +429,11 @@ public class ChestShop extends JavaPlugin {
     }
 
     private void registerModules() {
+        registerEvent(new ItemAliasModule());
         registerEvent(new DiscountModule());
+        registerEvent(new MetricsModule());
         registerEvent(new PriceRestrictionModule());
+        registerEvent(new StockCounterModule());
 
         registerEconomicalModules();
     }
@@ -395,22 +454,119 @@ public class ChestShop extends JavaPlugin {
     }
 
     private void startStatistics() {
-        try {
-            new org.mcstats.Metrics(this).start();
-        } catch (IOException ex) {
-            ChestShop.getBukkitLogger().severe("There was an error while submitting MCStats statistics.");
-        }
-        new org.bstats.bukkit.MetricsLite(this);
+        try (JarFile jarFile = new JarFile(this.getFile())) {
+            String dist = jarFile.getManifest().getMainAttributes().getValue("Distribution-Type");
+            bStats.addCustomChart(new SimplePie("distributionType", () -> dist));
+        } catch (IOException ignored) {}
+
+        String serverVersion = getServer().getBukkitVersion().split("-")[0];
+        bStats.addCustomChart(createStaticDrilldownStat("versionMcSelf", serverVersion, getDescription().getVersion()));
+        bStats.addCustomChart(createStaticDrilldownStat("versionSelfMc", getDescription().getVersion(), serverVersion));
+
+        bStats.addCustomChart(createStaticDrilldownStat("serverTypeVersionSelf", getServer().getName(), getDescription().getVersion()));
+        bStats.addCustomChart(createStaticDrilldownStat("versionSelfServerType", getDescription().getVersion(), getServer().getName()));
+
+        bStats.addCustomChart(createStaticDrilldownStat("versionMcServerType", serverVersion, getServer().getName()));
+        bStats.addCustomChart(createStaticDrilldownStat("serverTypeVersionMc", getServer().getName(), serverVersion));
+
+        String javaVersion = System.getProperty("java.version");
+        bStats.addCustomChart(createStaticDrilldownStat("versionJavaSelf", javaVersion, getDescription().getVersion()));
+        bStats.addCustomChart(createStaticDrilldownStat("versionSelfJava", getDescription().getVersion(), javaVersion));
+
+        bStats.addCustomChart(createStaticDrilldownStat("versionJavaMc", javaVersion, serverVersion));
+        bStats.addCustomChart(createStaticDrilldownStat("versionMcJava", serverVersion, javaVersion));
+
+        bStats.addCustomChart(new SingleLineChart("shopAccounts", NameManager::getAccountCount));
+        bStats.addCustomChart(new MultiLineChart("transactionCount", () -> ImmutableMap.of(
+                "total", MetricsModule.getTotalTransactions(),
+                "buy", MetricsModule.getBuyTransactions(),
+                "sell", MetricsModule.getSellTransactions()
+        )));
+        bStats.addCustomChart(new MultiLineChart("itemCount", () -> ImmutableMap.of(
+                "total", MetricsModule.getTotalItemsCount(),
+                "buy", MetricsModule.getSoldItemsCount(),
+                "sell", MetricsModule.getBoughtItemsCount()
+        )));
+
+        bStats.addCustomChart(new SimplePie("includeSettingsInMetrics", () -> Properties.INCLUDE_SETTINGS_IN_METRICS ? "enabled" : "disabled"));
+        if (!Properties.INCLUDE_SETTINGS_IN_METRICS) return;
+
+        bStats.addCustomChart(new SimplePie("ensure-correct-playerid", () -> Properties.ENSURE_CORRECT_PLAYERID ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("allow-sign-chest-open", () -> Properties.ALLOW_SIGN_CHEST_OPEN ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("uses-server-economy-account", () -> !Properties.SERVER_ECONOMY_ACCOUNT.isEmpty() ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("uses-server-economy-account-uuid", () -> !Properties.SERVER_ECONOMY_ACCOUNT_UUID.equals(new UUID(0, 0)) ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("allow-partial-transactions", () -> Properties.ALLOW_PARTIAL_TRANSACTIONS ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("bungeecord-messages", () -> Properties.BUNGEECORD_MESSAGES ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("allow-multiple-shops-at-one-block", () -> Properties.ALLOW_MULTIPLE_SHOPS_AT_ONE_BLOCK ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("allow-partial-transactions", () -> Properties.ALLOW_PARTIAL_TRANSACTIONS ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("log-to-console", () -> Properties.LOG_TO_CONSOLE ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("log-to-file", () -> Properties.LOG_TO_FILE ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("auto-update", () -> !Properties.TURN_OFF_UPDATES ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("release-notifications", () -> !Properties.TURN_OFF_UPDATE_NOTIFIER ? "enabled" : "disabled"));
+        bStats.addCustomChart(new SimplePie("dev-build-notifications", () -> !Properties.TURN_OFF_DEV_UPDATE_NOTIFIER ? "enabled" : "disabled"));
+
+        bStats.addCustomChart(new AdvancedBarChart("pluginProperties", () -> {
+            Map<String, int[]> map = new LinkedHashMap<>();
+            map.put("ensure-correct-playerid", getChartArray(Properties.ENSURE_CORRECT_PLAYERID));
+            map.put("reverse-buttons", getChartArray(Properties.REVERSE_BUTTONS));
+            map.put("shift-sells-in-stacks", getChartArray(Properties.SHIFT_SELLS_IN_STACKS));
+            map.put("shift-sells-everything", getChartArray(Properties.SHIFT_SELLS_EVERYTHING));
+            map.put("allow-sign-chest-open", getChartArray(!Properties.ALLOW_SIGN_CHEST_OPEN));
+            map.put("sign-dying", getChartArray(!Properties.SIGN_DYING));
+            map.put("remove-empty-shops", getChartArray(!Properties.REMOVE_EMPTY_SHOPS));
+            map.put("remove-empty-chests", getChartArray(!Properties.REMOVE_EMPTY_CHESTS));
+            map.put("uses-server-economy-account", getChartArray(!Properties.SERVER_ECONOMY_ACCOUNT.isEmpty()));
+            map.put("uses-server-economy-account-uuid", getChartArray(!Properties.SERVER_ECONOMY_ACCOUNT_UUID.equals(new UUID(0, 0))));
+            map.put("allow-multiple-shops-at-one-block", getChartArray(Properties.ALLOW_MULTIPLE_SHOPS_AT_ONE_BLOCK));
+            map.put("allow-partial-transactions", getChartArray(Properties.ALLOW_PARTIAL_TRANSACTIONS));
+            map.put("bungeecord-messages", getChartArray(Properties.BUNGEECORD_MESSAGES));
+            map.put("log-to-console", getChartArray(Properties.LOG_TO_CONSOLE));
+            map.put("log-to-file", getChartArray(Properties.LOG_TO_FILE));
+            map.put("auto-update", getChartArray(!Properties.TURN_OFF_UPDATES));
+            map.put("release-notifications", getChartArray(!Properties.TURN_OFF_UPDATE_NOTIFIER));
+            map.put("dev-build-notifications", getChartArray(!Properties.TURN_OFF_DEV_UPDATE_NOTIFIER));
+            return map;
+        }));
+        bStats.addCustomChart(new SimpleBarChart("shopContainers",
+                () -> Properties.SHOP_CONTAINERS.stream().map(Material::name).collect(Collectors.toMap(k -> k, k -> 1))));
+    }
+
+    public static DrilldownPie createStaticDrilldownStat(String statId, String value1, String value2) {
+        final Map<String, Map<String, Integer>> map = ImmutableMap.of(value1, ImmutableMap.of(value2, 1));
+        return new DrilldownPie(statId, () -> map);
+    }
+
+    private int[] getChartArray(boolean value) {
+        return new int[]{!value ? 1 : 0, value ? 0 : 1};
     }
 
     private static final int PROJECT_BUKKITDEV_ID = 31263;
 
     private void startUpdater() {
         if (Properties.TURN_OFF_UPDATES) {
+            getLogger().info("Auto-updater is disabled. If you want the plugin to automatically download new releases then set 'TURN_OFF_UPDATES' to 'false' in your config.yml!");
+            if (!Properties.TURN_OFF_UPDATE_NOTIFIER) {
+                final Updater updater = new Updater(this, getPluginName().toLowerCase(Locale.ROOT), this.getFile(), Updater.UpdateType.NO_DOWNLOAD, true);
+                runInAsyncThread(() -> {
+                    if (updater.getResult() == Updater.UpdateResult.UPDATE_AVAILABLE) {
+                        getLogger().info("There is a new version available: " + updater.getLatestName() + ". You can download it from https://modrinth.com/plugin/" + getPluginName().toLowerCase(Locale.ROOT));
+                    }
+                });
+            }
             return;
         }
 
-        new Updater(this, PROJECT_BUKKITDEV_ID, this.getFile(), Updater.UpdateType.DEFAULT, true);
+        new Updater(this, getPluginName().toLowerCase(Locale.ROOT), this.getFile(), Updater.UpdateType.DEFAULT, true);
+    }
+
+    private static final String PROJECT_JENKINS_JOB_URL = "https://ci.minebench.de/job/ChestShop-3/";
+
+    private void startBuildNotificatier() {
+        if (Properties.TURN_OFF_DEV_UPDATE_NOTIFIER) {
+            return;
+        }
+
+        new JenkinsBuildsNotifier(this, PROJECT_JENKINS_JOB_URL);
     }
 
     ///////////////////////////////////////////////////////////////////////////////
@@ -423,10 +579,20 @@ public class ChestShop extends JavaPlugin {
         return dataFolder;
     }
 
+    public static Logger getShopLogger() {
+        return shopLogger;
+    }
+
     public static Logger getBukkitLogger() {
         return logger;
     }
 
+    public static void logDebug(String message) {
+        if (Properties.DEBUG) {
+            getBukkitLogger().info("[DEBUG] " + message);
+        }
+    }
+
     public static Server getBukkitServer() {
         return server;
     }
@@ -447,22 +613,51 @@ public class ChestShop extends JavaPlugin {
         return plugin;
     }
 
+    public static Metrics getMetrics() {
+        return bStats;
+    }
+
+    public static BukkitAudiences getAudiences() {
+        return audiences;
+    }
+
     public static void registerListener(Listener listener) {
         plugin.registerEvent(listener);
     }
 
-    public static void callEvent(Event event) {
+    public static <E extends Event> E callEvent(E event) {
         Bukkit.getPluginManager().callEvent(event);
+        return event;
+    }
+
+    public static void sendBungeeMessage(String playerName, Messages.Message message, Map<String, String> replacementMap, String... replacements) {
+        sendBungeeMessage(playerName, message.getComponent(null, true, replacementMap, replacements));
     }
 
     public static void sendBungeeMessage(String playerName, String message) {
+        sendBungeeMessage(playerName, "Message", message);
+    }
+
+    public static void sendBungeeMessage(String playerName, BaseComponent[] message) {
+        sendBungeeMessage(playerName, "MessageRaw", ComponentSerializer.toString(message));
+    }
+
+    public static void sendBungeeMessage(String playerName, Component message) {
+        sendBungeeMessage(playerName, "MessageRaw", GsonComponentSerializer.gson().serialize(message));
+    }
+
+    private static void sendBungeeMessage(String playerName, String channel, String message) {
         if (Properties.BUNGEECORD_MESSAGES && !Bukkit.getOnlinePlayers().isEmpty()) {
             ByteArrayDataOutput out = ByteStreams.newDataOutput();
-            out.writeUTF("Message");
+            out.writeUTF(channel);
             out.writeUTF(playerName);
             out.writeUTF(message);
 
             Bukkit.getOnlinePlayers().iterator().next().sendPluginMessage(plugin, "BungeeCord", out.toByteArray());
         }
     }
+
+    public static void runInAsyncThread(Runnable runnable) {
+        executorService.submit(runnable);
+    }
 }

src/main/java/com/Acrobot/ChestShop/Commands/AccessToggle.java

@@ -1,7 +1,7 @@
 package com.Acrobot.ChestShop.Commands;
 
 import com.Acrobot.ChestShop.Configuration.Messages;
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.OfflinePlayer;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandExecutor;
@@ -26,9 +26,9 @@ public class AccessToggle implements CommandExecutor {
         Player player = (Player) sender;
 
         if (setIgnoring(player, !isIgnoring(player))) {
-            player.sendMessage(Messages.prefix(Messages.TOGGLE_ACCESS_OFF));
+            Messages.TOGGLE_ACCESS_OFF.sendWithPrefix(player);
         } else {
-            player.sendMessage(Messages.prefix(Messages.TOGGLE_ACCESS_ON));
+            Messages.TOGGLE_ACCESS_ON.sendWithPrefix(player);
         }
 
         return true;
@@ -43,7 +43,7 @@ public class AccessToggle implements CommandExecutor {
     }
 
     public static boolean setIgnoring(Player player, boolean ignoring) {
-        Validate.notNull(player); // Make sure the player instance is not null, in case there are any errors in the code
+        Preconditions.checkNotNull(player); // Make sure the player instance is not null, in case there are any errors in the code
 
         if (ignoring) {
             toggledPlayers.add(player.getUniqueId());

src/main/java/com/Acrobot/ChestShop/Commands/Give.java

@@ -5,7 +5,7 @@ import com.Acrobot.Breeze.Utils.MaterialUtil;
 import com.Acrobot.Breeze.Utils.NumberUtil;
 import com.Acrobot.ChestShop.Configuration.Messages;
 import com.Acrobot.ChestShop.Events.ItemParseEvent;
-import com.Acrobot.ChestShop.Permission;
+import com.Acrobot.ChestShop.Utils.ItemUtil;
 import org.bukkit.Bukkit;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandExecutor;
@@ -13,8 +13,9 @@ import org.bukkit.command.CommandSender;
 import org.bukkit.entity.Player;
 import org.bukkit.inventory.ItemStack;
 
-import java.util.ArrayList;
+import java.util.HashSet;
 import java.util.List;
+import java.util.Set;
 
 /**
  * @author Acrobot
@@ -30,7 +31,7 @@ public class Give implements CommandExecutor {
         Player receiver = (sender instanceof Player ? (Player) sender : null);
         int quantity = 1;
 
-        List<Integer> disregardedIndexes = new ArrayList<Integer>();
+        Set<Integer> disregardedIndexes = new HashSet<>();
 
         if (args.length > 1) {
             for (int index = args.length - 1; index >= 0; --index) {
@@ -46,7 +47,7 @@ public class Give implements CommandExecutor {
             }
 
             for (int index = args.length - 1; index >= 0; --index) {
-                if (!NumberUtil.isInteger(args[index]) || Integer.parseInt(args[index]) < 0) {
+                if (disregardedIndexes.contains(index) || !NumberUtil.isInteger(args[index]) || Integer.parseInt(args[index]) < 0) {
                     continue;
                 }
 
@@ -58,28 +59,26 @@ public class Give implements CommandExecutor {
         }
 
         if (receiver == null) {
-            sender.sendMessage(Messages.prefix(Messages.PLAYER_NOT_FOUND));
+            Messages.PLAYER_NOT_FOUND.sendWithPrefix(sender);
             return true;
         }
 
         ItemStack item = getItem(args, disregardedIndexes);
 
         if (MaterialUtil.isEmpty(item)) {
-            sender.sendMessage(Messages.prefix(Messages.INCORRECT_ITEM_ID));
+            Messages.INCORRECT_ITEM_ID.sendWithPrefix(sender);
             return true;
         }
 
         item.setAmount(quantity);
         InventoryUtil.add(item, receiver.getInventory());
 
-        sender.sendMessage(Messages.prefix(Messages.ITEM_GIVEN
-                .replace("%item", MaterialUtil.getName(item))
-                .replace("%player", receiver.getName())));
+        Messages.ITEM_GIVEN.send(sender, "item", ItemUtil.getName(item), "player", receiver.getName());
 
         return true;
     }
 
-    private static ItemStack getItem(String[] arguments, List<Integer> disregardedElements) {
+    private static ItemStack getItem(String[] arguments, Set<Integer> disregardedElements) {
         StringBuilder builder = new StringBuilder(arguments.length * 5);
 
         for (int index = 0; index < arguments.length; ++index) {

src/main/java/com/Acrobot/ChestShop/Commands/ItemInfo.java

@@ -4,19 +4,25 @@ import com.Acrobot.Breeze.Utils.MaterialUtil;
 import com.Acrobot.Breeze.Utils.StringUtil;
 import com.Acrobot.ChestShop.ChestShop;
 import com.Acrobot.ChestShop.Configuration.Messages;
+import com.Acrobot.ChestShop.Configuration.Properties;
 import com.Acrobot.ChestShop.Events.ItemInfoEvent;
 import com.Acrobot.ChestShop.Events.ItemParseEvent;
+import com.Acrobot.ChestShop.Utils.ItemUtil;
+import com.google.common.collect.ImmutableMap;
 import org.bukkit.Bukkit;
 import org.bukkit.ChatColor;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandExecutor;
 import org.bukkit.command.CommandSender;
 import org.bukkit.entity.HumanEntity;
+import org.bukkit.entity.Player;
 import org.bukkit.inventory.ItemStack;
 
+import java.util.Map;
 import java.util.logging.Level;
 
 import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_shopname;
 
 /**
  * @author Acrobot
@@ -41,17 +47,11 @@ public class ItemInfo implements CommandExecutor {
             return false;
         }
 
-        sender.sendMessage(Messages.prefix(iteminfo));
-        try {
-            sender.sendMessage(ChatColor.WHITE + "Full Name: " + ChatColor.GRAY + MaterialUtil.getName(item));
-        } catch (IllegalArgumentException e) {
-            sender.sendMessage(ChatColor.RED + "Error while generating full name. Please contact an admin or take a look at the console/log!");
-            ChestShop.getPlugin().getLogger().log(Level.SEVERE, "Error while generating full item name", e);
-            return true;
-        }
+        iteminfo.send(sender);
+        if (!sendItemName(sender, item, Messages.iteminfo_fullname)) return true;
 
         try {
-            sender.sendMessage(ChatColor.WHITE + "Shop Sign: " + ChatColor.GRAY + MaterialUtil.getSignName(item));
+            iteminfo_shopname.send(sender, "item", ItemUtil.getSignName(item));
         } catch (IllegalArgumentException e) {
             sender.sendMessage(ChatColor.RED + "Error while generating shop sign name. Please contact an admin or take a look at the console/log!");
             ChestShop.getPlugin().getLogger().log(Level.SEVERE, "Error while generating shop sign item name", e);
@@ -64,23 +64,18 @@ public class ItemInfo implements CommandExecutor {
         return true;
     }
 
-    private static String getNameAndID(ItemStack item) {
-        return MaterialUtil.getName(item);
-    }
-
-    private static String getDurability(ItemStack item) {
-        if (item.getDurability() != 0) {
-            return ChatColor.DARK_GREEN + ":" + Integer.toString(item.getDurability());
-        } else {
-            return "";
+    public static boolean sendItemName(CommandSender sender, ItemStack item, Messages.Message message) {
+        try {
+            Map<String, String> replacementMap = ImmutableMap.of("item", ItemUtil.getName(item));
+            if (!Properties.SHOWITEM_MESSAGE || !(sender instanceof Player)
+                    || !MaterialUtil.Show.sendMessage((Player) sender, sender.getName(), message, false, new ItemStack[]{item}, replacementMap)) {
+                message.send(sender, replacementMap);
+            }
+        } catch (IllegalArgumentException e) {
+            sender.sendMessage(ChatColor.RED + "Error while generating full name. Please contact an admin or take a look at the console/log!");
+            ChestShop.getPlugin().getLogger().log(Level.SEVERE, "Error while generating full item name", e);
+            return false;
         }
-    }
-
-    private static String getMetadata(ItemStack item) {
-        if (!item.hasItemMeta()) {
-            return "";
-        }
-
-        return ChatColor.GOLD + "#" + MaterialUtil.Metadata.getItemCode(item);
+        return true;
     }
 }

src/main/java/com/Acrobot/ChestShop/Commands/Metrics.java

@@ -0,0 +1,26 @@
+package com.Acrobot.ChestShop.Commands;
+
+import com.Acrobot.ChestShop.Configuration.Messages;
+import com.Acrobot.ChestShop.Listeners.Modules.MetricsModule;
+import com.Acrobot.ChestShop.UUIDs.NameManager;
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandExecutor;
+import org.bukkit.command.CommandSender;
+
+/**
+ * @author Acrobot
+ */
+public class Metrics implements CommandExecutor {
+    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
+        Messages.METRICS.send(sender,
+                "accounts", String.valueOf(NameManager.getAccountCount()),
+                "totalTransactions", String.valueOf(MetricsModule.getTotalTransactions()),
+                "buyTransactions", String.valueOf(MetricsModule.getBuyTransactions()),
+                "sellTransactions", String.valueOf(MetricsModule.getSellTransactions()),
+                "totalItems", String.valueOf(MetricsModule.getTotalItemsCount()),
+                "boughtItems", String.valueOf(MetricsModule.getBoughtItemsCount()),
+                "soldItems", String.valueOf(MetricsModule.getSoldItemsCount())
+        );
+        return true;
+    }
+}

src/main/java/com/Acrobot/ChestShop/Commands/ShopInfo.java

@@ -0,0 +1,43 @@
+package com.Acrobot.ChestShop.Commands;
+
+import com.Acrobot.ChestShop.ChestShop;
+import com.Acrobot.ChestShop.Configuration.Messages;
+import com.Acrobot.ChestShop.Events.ShopInfoEvent;
+import com.Acrobot.ChestShop.Signs.ChestShopSign;
+import com.Acrobot.ChestShop.Utils.uBlock;
+import org.bukkit.ChatColor;
+import org.bukkit.block.Block;
+import org.bukkit.block.Sign;
+import org.bukkit.command.Command;
+import org.bukkit.command.CommandExecutor;
+import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
+
+/**
+ * @author Phoenix616
+ */
+public class ShopInfo implements CommandExecutor {
+    public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
+        if (sender instanceof Player) {
+            Block target = ((Player) sender).getTargetBlockExact(5);
+            if (target != null) {
+                Sign sign = null;
+                if (ChestShopSign.isValid(target)) {
+                    sign = (Sign) target.getState();
+                } else if (uBlock.couldBeShopContainer(target)) {
+                    sign = uBlock.getConnectedSign(target);
+                }
+
+                if (sign != null) {
+                    ShopInfoEvent event = new ShopInfoEvent((Player) sender, sign);
+                    ChestShop.callEvent(event);
+                } else {
+                    Messages.NO_SHOP_FOUND.sendWithPrefix(sender);
+                }
+            }
+        } else {
+            sender.sendMessage(ChatColor.RED + "Command must be run by a player!");
+        }
+        return true;
+    }
+}

src/main/java/com/Acrobot/ChestShop/Commands/Toggle.java

@@ -1,7 +1,7 @@
 package com.Acrobot.ChestShop.Commands;
 
 import com.Acrobot.ChestShop.Configuration.Messages;
-import org.apache.commons.lang.Validate;
+import com.google.common.base.Preconditions;
 import org.bukkit.Bukkit;
 import org.bukkit.OfflinePlayer;
 import org.bukkit.command.Command;
@@ -32,9 +32,9 @@ public class Toggle implements CommandExecutor {
         }
 
         if (setIgnoring(player, !isIgnoring(player))) {
-            player.sendMessage(Messages.prefix(Messages.TOGGLE_MESSAGES_OFF));
+            Messages.TOGGLE_MESSAGES_OFF.sendWithPrefix(player);
         } else {
-            player.sendMessage(Messages.prefix(Messages.TOGGLE_MESSAGES_ON));
+            Messages.TOGGLE_MESSAGES_ON.sendWithPrefix(player);
         }
 
         return true;
@@ -61,7 +61,7 @@ public class Toggle implements CommandExecutor {
     }
 
     public static boolean setIgnoring(Player player, boolean ignoring) {
-        Validate.notNull(player); // Make sure the player instance is not null, in case there are any errors in the code
+        Preconditions.checkNotNull(player); // Make sure the player instance is not null, in case there are any errors in the code
 
         if (ignoring) {
             toggledPlayers.add(player.getUniqueId());

src/main/java/com/Acrobot/ChestShop/Commands/Version.java

@@ -1,6 +1,7 @@
 package com.Acrobot.ChestShop.Commands;
 
 import com.Acrobot.ChestShop.ChestShop;
+import com.Acrobot.ChestShop.Events.ChestShopReloadEvent;
 import org.bukkit.ChatColor;
 import org.bukkit.command.Command;
 import org.bukkit.command.CommandExecutor;
@@ -12,7 +13,7 @@ import org.bukkit.command.CommandSender;
 public class Version implements CommandExecutor {
     public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
         if (args.length > 0 && args[0].equals("reload")) {
-            ChestShop.getPlugin().loadConfig();
+            ChestShop.callEvent(new ChestShopReloadEvent(sender));
 
             sender.sendMessage(ChatColor.DARK_GREEN + "The config was reloaded.");
             return true;

src/main/java/com/Acrobot/ChestShop/Configuration/Messages.java

@@ -1,93 +1,234 @@
 package com.Acrobot.ChestShop.Configuration;
 
-import com.Acrobot.Breeze.Configuration.Annotations.PrecededBySpace;
 import com.Acrobot.Breeze.Configuration.Configuration;
+import com.Acrobot.ChestShop.ChestShop;
+import de.themoep.minedown.adventure.MineDown;
+import de.themoep.utils.lang.bukkit.BukkitLanguageConfig;
+import de.themoep.utils.lang.bukkit.LanguageManager;
+import net.kyori.adventure.text.Component;
+import net.kyori.adventure.text.serializer.bungeecord.BungeeComponentSerializer;
+import net.kyori.adventure.text.serializer.legacy.LegacyComponentSerializer;
+import net.md_5.bungee.api.chat.BaseComponent;
+import org.bukkit.command.CommandSender;
+
+import java.io.File;
+import java.lang.reflect.Field;
+import java.lang.reflect.Modifier;
+import java.util.Collections;
+import java.util.Map;
+import java.util.logging.Level;
 
 /**
  * @author Acrobot
  */
 public class Messages {
-    public static String prefix = "&a[Shop] &r";
-    public static String iteminfo = "&aItem Information: &r";
+    public static Message prefix;
 
-    @PrecededBySpace
-    public static String ACCESS_DENIED = "You don't have permission to access that shop's storage container!";
+    public static Message shopinfo;
+    public static Message shopinfo_buy;
+    public static Message shopinfo_sell;
 
-    @PrecededBySpace
-    public static String NOT_ENOUGH_MONEY = "You don't have enough money!";
-    public static String NOT_ENOUGH_MONEY_SHOP = "Shop owner doesn't have enough money!";
+    public static Message iteminfo;
+    public static Message iteminfo_fullname;
+    public static Message iteminfo_shopname;
+    public static Message iteminfo_repaircost;
+    public static Message iteminfo_book;
+    public static Message iteminfo_book_generation;
+    public static Message iteminfo_leather_color;
+    public static Message iteminfo_bundle_items;
+    public static Message iteminfo_axolotl_variant;
+    public static Message iteminfo_recipes;
+    public static Message iteminfo_map_view;
+    public static Message iteminfo_map_location;
+    public static Message iteminfo_tropical_fish;
+    public static Message iteminfo_crossbow_projectiles;
+    public static Message iteminfo_crossbow_projectile;
+    public static Message iteminfo_lore;
 
-    @PrecededBySpace
-    public static String CLIENT_DEPOSIT_FAILED = "Money deposit to your account failed!";
-    public static String SHOP_DEPOSIT_FAILED = "Money deposit to shop owner failed!";
-    public static String NO_ECONOMY_ACCOUNT = "Economy account from shop owner doesn't exist!";
+    public static Message METRICS;
 
-    @PrecededBySpace
-    public static String NO_BUYING_HERE = "You can't buy here!";
-    public static String NO_SELLING_HERE = "You can't sell here!";
+    public static Message ACCESS_DENIED;
+    public static Message TRADE_DENIED;
+    public static Message TRADE_DENIED_ACCESS_PERMS;
+    public static Message TRADE_DENIED_CREATIVE_MODE;
 
-    @PrecededBySpace
-    public static String NOT_ENOUGH_SPACE_IN_INVENTORY = "You haven't got enough space in inventory!";
-    public static String NOT_ENOUGH_SPACE_IN_CHEST = "There isn't enough space in chest!";
-    public static String NOT_ENOUGH_ITEMS_TO_SELL = "You don't have enough items to sell!";
-    public static String NOT_ENOUGH_SPACE_IN_YOUR_SHOP = "%price %item&7 shop at &r%world/%x/%y/%z&7 is full! (%seller tried to sell)";
+    public static Message NOT_ENOUGH_MONEY;
+    public static Message NOT_ENOUGH_MONEY_SHOP;
 
-    @PrecededBySpace
-    public static String NOT_ENOUGH_STOCK = "This shop is out of stock.";
-    public static String NOT_ENOUGH_STOCK_IN_YOUR_SHOP = "%price %item&7 shop at &r%world/%x/%y/%z&7 is out of stock! (%buyer tried to buy)";
+    public static Message CLIENT_DEPOSIT_FAILED;
+    public static Message SHOP_DEPOSIT_FAILED;
+    public static Message NO_ECONOMY_ACCOUNT;
 
-    @PrecededBySpace
-    public static String YOU_BOUGHT_FROM_SHOP = "You bought %item from %owner for %price.";
-    public static String SOMEBODY_BOUGHT_FROM_YOUR_SHOP = "%buyer bought %item for %price from your shop at %world/%x/%y/%z.";
+    public static Message NO_BUYING_HERE;
+    public static Message NO_SELLING_HERE;
 
-    @PrecededBySpace
-    public static String YOU_SOLD_TO_SHOP = "You sold %item to %buyer for %price.";
-    public static String SOMEBODY_SOLD_TO_YOUR_SHOP = "%seller sold %item for %price to your shop at %world/%x/%y/%z.";
+    public static Message NOT_ENOUGH_SPACE_IN_INVENTORY;
+    public static Message NOT_ENOUGH_SPACE_IN_CHEST;
+    public static Message NOT_ENOUGH_ITEMS_TO_SELL;
+    public static Message NOT_ENOUGH_SPACE_IN_YOUR_SHOP;
 
-    @PrecededBySpace
-    public static String YOU_CANNOT_CREATE_SHOP = "You can't create this type of shop!";
-    public static String NO_CHEST_DETECTED = "Couldn't find a chest!";
-    public static String INVALID_SHOP_DETECTED = "The shop cannot be used!";
-    public static String CANNOT_ACCESS_THE_CHEST = "You don't have permissions to access this chest!";
+    public static Message NOT_ENOUGH_STOCK;
+    public static Message NOT_ENOUGH_STOCK_IN_YOUR_SHOP;
 
-    @PrecededBySpace
-    public static String SELL_PRICE_ABOVE_MAX = "Sell price is above maximum!";
-    public static String SELL_PRICE_BELOW_MIN ="Sell price is below minimum!";
-    public static String BUY_PRICE_ABOVE_MAX = "Buy price is above maximum!";
-    public static String BUY_PRICE_BELOW_MIN ="Buy price is below minimum!";
+    public static Message YOU_BOUGHT_FROM_SHOP;
+    public static Message SOMEBODY_BOUGHT_FROM_YOUR_SHOP;
 
-    @PrecededBySpace
-    public static String CLICK_TO_AUTOFILL_ITEM = "Click the sign with the item that this shop is for!";
-    public static String NO_ITEM_IN_HAND = "You don't have an item in your hand to autofill!";
+    public static Message YOU_SOLD_TO_SHOP;
+    public static Message SOMEBODY_SOLD_TO_YOUR_SHOP;
 
-    @PrecededBySpace
-    public static String PROTECTED_SHOP = "Successfully protected the shop with LWC!";
-    public static String SHOP_CREATED = "Shop successfully created!";
-    public static String SHOP_FEE_PAID = "You have been charged %amount";
-    public static String SHOP_REFUNDED = "You have been refunded %amount.";
-    public static String ITEM_GIVEN = "Given %item to %player.";
+    public static Message YOU_CANNOT_CREATE_SHOP;
+    public static Message NO_CHEST_DETECTED;
+    public static Message INVALID_SHOP_DETECTED;
+    public static Message INVALID_SHOP_PRICE;
+    public static Message INVALID_SHOP_QUANTITY;
+    public static Message CANNOT_ACCESS_THE_CHEST;
+    public static Message CANNOT_CHANGE_SIGN_BACKSIDE;
 
-    @PrecededBySpace
-    public static String RESTRICTED_SIGN_CREATED = "Sign successfully created!";
+    public static Message SELL_PRICE_HIGHER_THAN_BUY_PRICE;
+    public static Message SELL_PRICE_ABOVE_MAX;
+    public static Message SELL_PRICE_BELOW_MIN;
+    public static Message BUY_PRICE_ABOVE_MAX;
+    public static Message BUY_PRICE_BELOW_MIN;
 
-    @PrecededBySpace
-    public static String PLAYER_NOT_FOUND = "Player not found!";
-    public static String NO_PERMISSION = "You don't have permissions to do that!";
-    public static String INCORRECT_ITEM_ID = "You have specified an invalid item id!";
-    public static String NOT_ENOUGH_PROTECTIONS = "Could not create a protection!";
+    public static Message CLICK_TO_AUTOFILL_ITEM;
+    public static Message NO_ITEM_IN_HAND;
 
-    @PrecededBySpace
-    public static String CANNOT_CREATE_SHOP_HERE = "You can't create shop here!";
+    public static Message PROTECTED_SHOP;
+    public static Message PROTECTED_SHOP_SIGN;
+    public static Message SHOP_CREATED;
+    public static Message SHOP_FEE_PAID;
+    public static Message SHOP_REFUNDED;
+    public static Message ITEM_GIVEN;
 
-    @PrecededBySpace
-    public static String TOGGLE_MESSAGES_OFF = "You will no longer receive messages from your shop(s).";
-    public static String TOGGLE_MESSAGES_ON = "You will now receive messages from your shop(s).";
+    public static Message RESTRICTED_SIGN_CREATED;
 
-    @PrecededBySpace
-    public static String TOGGLE_ACCESS_ON = "You can no longer trade at shops that you have access to";
-    public static String TOGGLE_ACCESS_OFF = "You can now trade at shops that you have access to";
+    public static Message PLAYER_NOT_FOUND;
+    public static Message NO_PERMISSION;
+    public static Message INCORRECT_ITEM_ID;
+    public static Message INVALID_CLIENT_NAME;
+    public static Message NOT_ENOUGH_PROTECTIONS;
+    public static Message NO_SHOP_FOUND;
 
+    public static Message CANNOT_CREATE_SHOP_HERE;
+
+    public static Message TOGGLE_MESSAGES_OFF;
+    public static Message TOGGLE_MESSAGES_ON;
+
+    public static Message TOGGLE_ACCESS_ON;
+    public static Message TOGGLE_ACCESS_OFF;
+
+    @Deprecated
     public static String prefix(String message) {
-        return Configuration.getColoured(prefix + message);
+        return Configuration.getColoured(prefix.getLang(null) + message);
+    }
+
+    @Deprecated
+    public static String replace(String message, String... replacements) {
+        for (int i = 0; i + 1 < replacements.length; i+=2) {
+            message = message.replace("%" + replacements[i], replacements[i+1]);
+        }
+        return Configuration.getColoured(message);
+    }
+
+    private static LanguageManager manager;
+
+    public static void load() {
+        for (Field field : Messages.class.getFields()) {
+            if (!Modifier.isStatic(field.getModifiers())) {
+                continue;
+            }
+            try {
+                field.set(null, new Message(field.getName()));
+            } catch (IllegalAccessException e) {
+                ChestShop.getBukkitLogger().log(Level.SEVERE, "Error while setting Message " + field.getName() + "!", e);
+            }
+        }
+        manager = new LanguageManager(ChestShop.getPlugin(), Properties.DEFAULT_LANGUAGE);
+
+        if (manager.getDefaultConfig() == null) {
+            manager.setDefaultLocale("en");
+            ChestShop.getBukkitLogger().log(Level.WARNING, "There is no language file for your DEFAULT_LANGUAGE config setting of '" + Properties.DEFAULT_LANGUAGE + "' in your languages folder! Using default English as default until you have created one or changed the config option to another, existing language file.");
+        }
+
+        // Legacy locale.yml file
+        File legacyFile = new File(ChestShop.getPlugin().getDataFolder(), "local.yml");
+        if (legacyFile.exists()) {
+            ChestShop.getBukkitLogger().log(Level.INFO, "Found legacy local.yml. Loading it as 'legacy' language and using that for all messages.");
+            ChestShop.getBukkitLogger().log(Level.INFO, "As long as the legacy file is used automatic language switching based on the client settings will not be supported!");
+            ChestShop.getBukkitLogger().log(Level.INFO, "Import it into the corresponding language file and remove/rename the file if you don't want it anymore!");
+            manager.addConfig(new BukkitLanguageConfig(ChestShop.getPlugin(), "", legacyFile, "legacy", false));
+            manager.setDefaultLocale("legacy");
+            Properties.USE_CLIENT_LOCALE = false;
+        }
+
+        if (!Properties.USE_CLIENT_LOCALE) {
+            manager.setProvider(sender -> null);
+        }
+    }
+
+    public static class Message {
+        private String key;
+
+        public Message(String key) {
+            this.key = key;
+        }
+
+        public void sendWithPrefix(CommandSender sender, Map<String, String> replacementMap, String... replacements) {
+            ChestShop.getAudiences().sender(sender).sendMessage(getComponent(sender, true, replacementMap, replacements));
+        }
+
+        public void sendWithPrefix(CommandSender sender, Map<String, String> replacements) {
+            ChestShop.getAudiences().sender(sender).sendMessage(getComponent(sender, true, replacements));
+        }
+
+        public void sendWithPrefix(CommandSender sender, String... replacements) {
+            ChestShop.getAudiences().sender(sender).sendMessage(getComponent(sender, true, Collections.emptyMap(), replacements));
+        }
+
+        public void send(CommandSender sender, String... replacements) {
+            ChestShop.getAudiences().sender(sender).sendMessage(getComponent(sender, false, Collections.emptyMap(), replacements));
+        }
+
+        public void send(CommandSender sender, Map<String, String> replacements) {
+            ChestShop.getAudiences().sender(sender).sendMessage(getComponent(sender, false, replacements));
+        }
+
+        @Deprecated
+        public BaseComponent[] getComponents(CommandSender sender, boolean prefixSuffix, Map<String, String> replacementMap, String... replacements) {
+            return BungeeComponentSerializer.get().serialize(getComponent(sender, prefixSuffix, replacementMap, replacements));
+        }
+
+        public Component getComponent(CommandSender sender, boolean prefixSuffix, Map<String, String> replacementMap, String... replacements) {
+            MineDown mineDown = new MineDown("%prefix" + getLang(sender));
+            mineDown.placeholderSuffix("");
+            if (prefixSuffix) {
+                mineDown.replace("prefix", MineDown.parse(prefix.getLang(sender)));
+            } else {
+                mineDown.replace("prefix", "");
+            }
+            mineDown.replace(replacementMap);
+            mineDown.replace(replacements);
+            return mineDown.toComponent();
+        }
+
+        private String getLang(CommandSender sender) {
+            return manager.getConfig(sender).get(key);
+        }
+
+        public String getTextWithPrefix(CommandSender sender, Map<String, String> replacementMap, String... replacements) {
+            return LegacyComponentSerializer.legacySection().serialize(getComponent(sender, true, replacementMap, replacements));
+        }
+
+        public String getTextWithPrefix(CommandSender sender, String... replacements) {
+            return getTextWithPrefix(sender, Collections.emptyMap(), replacements);
+        }
+
+        public String getTextWithPrefix(CommandSender sender, Map<String, String> replacements) {
+            return getTextWithPrefix(sender, replacements, new String[0]);
+        }
+
+        public String getKey() {
+            return key;
+        }
     }
 }

src/main/java/com/Acrobot/ChestShop/Configuration/Properties.java

@@ -6,6 +6,7 @@ import com.Acrobot.Breeze.Configuration.Annotations.PrecededBySpace;
 import com.Acrobot.Breeze.Configuration.Configuration;
 import com.Acrobot.Breeze.Configuration.ValueParser;
 import com.Acrobot.ChestShop.ChestShop;
+import com.Acrobot.ChestShop.Security;
 import org.bukkit.Material;
 
 import java.math.BigDecimal;
@@ -13,7 +14,9 @@ import java.util.Arrays;
 import java.util.Collection;
 import java.util.EnumSet;
 import java.util.LinkedHashSet;
+import java.util.Locale;
 import java.util.Set;
+import java.util.UUID;
 import java.util.logging.Level;
 
 /**
@@ -22,25 +25,28 @@ import java.util.logging.Level;
 public class Properties {
 
     static {
-        Configuration.registerParser("StringSet", new ValueParser(){
-            public Object parseToJava(Object object) {
+        Configuration.registerParser("StringSet", new ValueParser() {
+            @Override
+            public <T> Object parseToJava(Class<T> type, Object object) {
                 if (object instanceof Collection) {
                     return new LinkedHashSet<>((Collection<String>) object);
                 }
                 return object;
             }
         });
-        Configuration.registerParser("MaterialSet", new ValueParser(){
-            public Object parseToJava(Object object) {
+        Configuration.registerParser("MaterialSet", new ValueParser() {
+            @Override
+            public <T> Object parseToJava(Class<T> type, Object object) {
                 if (object instanceof Collection) {
                     EnumSet<Material> set = EnumSet.noneOf(Material.class);
                     for (Object o : (Collection) object) {
                         if (o instanceof Material) {
                             set.add((Material) o);
                         } else if (o instanceof String) {
-                            try {
-                                set.add(Material.getMaterial(((String) o).toUpperCase()));
-                            } catch (IllegalArgumentException e) {
+                            Material m = Material.getMaterial(((String) o).toUpperCase(Locale.ROOT));
+                            if (m != null) {
+                                set.add(m);
+                            } else {
                                 ChestShop.getBukkitLogger().log(Level.WARNING, o + " is not a valid Material name in the config!");
                             }
                         }
@@ -50,7 +56,7 @@ public class Properties {
                 return object;
             }
         });
-        Configuration.registerParser("BigDecimal", new ValueParser(){
+        Configuration.registerParser("BigDecimal", new ValueParser() {
             @Override
             public String parseToYAML(Object object) {
                 if (object instanceof BigDecimal) {
@@ -59,7 +65,8 @@ public class Properties {
                 return super.parseToYAML(object);
             }
 
-            public Object parseToJava(Object object) {
+            @Override
+            public <T> Object parseToJava(Class<T> type, Object object) {
                 if (object instanceof Double) {
                     return BigDecimal.valueOf((Double) object);
                 } else if (object instanceof Long) {
@@ -70,15 +77,55 @@ public class Properties {
                 return object;
             }
         });
+        Configuration.registerParser("UUID", new ValueParser() {
+            @Override
+            public String parseToYAML(Object object) {
+                if (object instanceof UUID) {
+                    return object.toString();
+                }
+                return super.parseToYAML(object);
+            }
+
+            @Override
+            public <T> Object parseToJava(Class<T> type, Object object) {
+                if (object instanceof String) {
+                    return UUID.fromString((String) object);
+                }
+                return object;
+            }
+        });
     }
 
+    @ConfigurationComment("Should the plugin log some messages that are useful for debugging?")
+    public static boolean DEBUG = false;
+
+    @PrecededBySpace
     @ConfigurationComment("Do you want to turn off the automatic updates of ChestShop?")
-    public static boolean TURN_OFF_UPDATES = false;
+    public static boolean TURN_OFF_UPDATES = true;
+
+    @ConfigurationComment("Do you want to turn off the automatic notifications for releases?")
+    public static boolean TURN_OFF_UPDATE_NOTIFIER = false;
+
+    @ConfigurationComment("Do you want to turn off the automatic notifications for new development builds?")
+    public static boolean TURN_OFF_DEV_UPDATE_NOTIFIER = false;
+
+    @ConfigurationComment("Do you want to include some values of this config in the metrics? (This will not submit any sensitive data and helps in the development process)")
+    public static boolean INCLUDE_SETTINGS_IN_METRICS = true;
 
     @PrecededBySpace
     @ConfigurationComment("How large should the internal caches be?")
     public static int CACHE_SIZE = 1000;
 
+    @PrecededBySpace
+    @ConfigurationComment("The default language when the client's language can't be found.")
+    public static String DEFAULT_LANGUAGE = "en";
+
+    @ConfigurationComment("Should the plugin try to use a language file that matches the client's locale setting?")
+    public static boolean USE_CLIENT_LOCALE = true;
+
+    @ConfigurationComment("Should the plugin strip the colors from formatted price?")
+    public static boolean STRIP_PRICE_COLORS = false;
+
     @PrecededBySpace
     @ConfigurationComment("What containers are allowed to hold a shop? (Only blocks with inventories work!)")
     @Parser("MaterialSet")
@@ -91,10 +138,10 @@ public class Properties {
     @ConfigurationComment("(In 1/1000th of a second) How often can a player use the shop sign?")
     public static int SHOP_INTERACTION_INTERVAL = 250;
 
-    @ConfigurationComment("Do you want to allow using shops to people in creative mode?")
+    @ConfigurationComment("Do you want to block people in creative mode from using shops?")
     public static boolean IGNORE_CREATIVE_MODE = true;
 
-    @ConfigurationComment("Do you want to allow using shops to people who have access to it due to their permissions? (owners are always ignored)")
+    @ConfigurationComment("Do you want to block people who have access to a shop due to their permissions from using it? (owners are always ignored)")
     public static boolean IGNORE_ACCESS_PERMS = true;
 
     @ConfigurationComment("If true, people will buy with left-click and sell with right-click.")
@@ -103,11 +150,17 @@ public class Properties {
     @ConfigurationComment("If true, people will be able to buy/sell in 64 stacks while holding the crouch button.")
     public static boolean SHIFT_SELLS_IN_STACKS = false;
 
+    @ConfigurationComment("If true, people will be able to sell/buy everything available of the same type.")
+    public static boolean SHIFT_SELLS_EVERYTHING = false;
+
     @ConfigurationComment("What can you do by clicking shift with SHIFT_SELLS_IN_STACKS turned on? (ALL/BUY/SELL)")
     public static String SHIFT_ALLOWS = "ALL";
 
     @ConfigurationComment("Can shop's chest be opened by owner with right-clicking a shop's sign?")
-    public static boolean ALLOW_SIGN_CHEST_OPEN = true;
+    public static boolean ALLOW_SIGN_CHEST_OPEN = false;
+
+    @ConfigurationComment("If true and in 1.14+, the owner of a chest shop can click with a dye / ink sac to dye the sign.")
+    public static boolean SIGN_DYING = true;
 
     @ConfigurationComment("If true, when you left-click your own shop sign you won't open chest's inventory, but instead you will start destroying the sign.")
     public static boolean ALLOW_LEFT_CLICK_DESTROYING = true;
@@ -127,14 +180,20 @@ public class Properties {
     @ConfigurationComment("First line of your Admin Shop's sign should look like this:")
     public static String ADMIN_SHOP_NAME = "Admin Shop";
 
-    @ConfigurationComment("The economy account which Admin Shops should use and to which all taxes will go")
+    @ConfigurationComment("Make all admin shops be unlimited even if they have a shop container at the sign")
+    public static boolean FORCE_UNLIMITED_ADMIN_SHOP = false;
+
+    @ConfigurationComment("The name of the economy account which Admin Shops should use and to which all taxes will go")
     public static String SERVER_ECONOMY_ACCOUNT = "";
 
+    @ConfigurationComment("The uuid of the economy account for the Admin Shop. Useful for fake accounts as normally only accounts of players work")
+    public static UUID SERVER_ECONOMY_ACCOUNT_UUID = new UUID(0, 0);
+
     @ConfigurationComment("Percent of the price that should go to the server's account. (100 = 100 percent)")
-    public static int TAX_AMOUNT = 0;
+    public static double TAX_AMOUNT = 0;
 
     @ConfigurationComment("Percent of the price that should go to the server's account when buying from an Admin Shop.")
-    public static int SERVER_TAX_AMOUNT = 0;
+    public static double SERVER_TAX_AMOUNT = 0;
 
     @ConfigurationComment("Amount of money player must pay to create a shop")
     public static BigDecimal SHOP_CREATION_PRICE = BigDecimal.valueOf(0);
@@ -145,10 +204,22 @@ public class Properties {
     @ConfigurationComment("How many decimal places are allowed at a maximum for prices?")
     public static int PRICE_PRECISION = 2;
 
+    @ConfigurationComment("This makes sure that the UUIDs of player shop accounts match the server's online-mode setting. Disabling this might lead to issues with offline players and is therefore unsupported!")
+    public static boolean ENSURE_CORRECT_PLAYERID = true;
+
+    @ConfigurationComment("This regexp validates the name of the player. If the name doesn't match, the player will neither be able to create a valid shop sign, nor buy/sell from a shop.\n" +
+            "Note for Bedrock support: If you have Floodgate on your server, you should set this regexp to ^\\\\.?\\\\w+$ and ENSURE_CORRECT_PLAYERID to false\n" +
+            "If your Floodgate prefix is not a dot, then change the first . in the regexp (the one before the question mark) to whatever your prefix is.")
+    public static String VALID_PLAYERNAME_REGEXP = "^\\w+$";
+
     @PrecededBySpace
     @ConfigurationComment("Should we block shops that sell things for more than they buy? (This prevents newbies from creating shops that would be exploited)")
     public static boolean BLOCK_SHOPS_WITH_SELL_PRICE_HIGHER_THAN_BUY_PRICE = true;
 
+    @PrecededBySpace
+    @ConfigurationComment("Maximum amount of items that can be bought/sold at a shop. Default 3456 is a double chest of 64 stacks.")
+    public static int MAX_SHOP_AMOUNT = 3456;
+
     @PrecededBySpace
     @ConfigurationComment("Do you want to allow other players to build a shop on a block where there's one already?")
     public static boolean ALLOW_MULTIPLE_SHOPS_AT_ONE_BLOCK = false;
@@ -187,10 +258,10 @@ public class Properties {
     @ConfigurationComment("If true, plugin will log transactions in its own file")
     public static boolean LOG_TO_FILE = false;
 
-    @ConfigurationComment("Do you want ChestShop's messages to show up in console?")
+    @ConfigurationComment("Do you want ChestShop's transaction messages to show up in console?")
     public static boolean LOG_TO_CONSOLE = true;
 
-    @ConfigurationComment("Should all shop removals be logged to the console?")
+    @ConfigurationComment("Should all shop removals be logged?")
     public static boolean LOG_ALL_SHOP_REMOVALS = true;
 
     @PrecededBySpace
@@ -212,17 +283,29 @@ public class Properties {
     @ConfigurationComment("Do you want to disable the hopper protection, which prevents Hopper-Minecarts from taking items out of shops?")
     public static boolean TURN_OFF_HOPPER_PROTECTION = false;
 
+    @ConfigurationComment("Only allow users to buy/sell that have access to the sign's protection? (E.g. LWC protection)")
+    public static boolean CHECK_ACCESS_FOR_SHOP_USE = false;
+
     @ConfigurationComment("Do you want to protect shop chests with LWC?")
     public static boolean PROTECT_CHEST_WITH_LWC = false;
 
+    @ConfigurationComment("Of which type should the container protection be? Possible type: public, private, donate and on some LWC versions display")
+    public static Security.Type LWC_CHEST_PROTECTION_TYPE = Security.Type.PRIVATE;
+
     @ConfigurationComment("Do you want to protect shop signs with LWC?")
     public static boolean PROTECT_SIGN_WITH_LWC = false;
 
+    @ConfigurationComment("Of which type should the sign protection be? Possible type: public, private, donate and on some LWC versions display")
+    public static Security.Type LWC_SIGN_PROTECTION_TYPE = Security.Type.PRIVATE;
+
     @ConfigurationComment("Should the chest's LWC protection be removed once the shop sign is destroyed? ")
     public static boolean REMOVE_LWC_PROTECTION_AUTOMATICALLY = true;
 
+    @ConfigurationComment("Should LWC limits block shop creations?")
+    public static boolean LWC_LIMITS_BLOCK_CREATION = true;
+
     @PrecededBySpace
-    @ConfigurationComment("Do you want to only let people build inside regions?")
+    @ConfigurationComment("Do you want to only let people build inside WorldGuard regions?")
     public static boolean WORLDGUARD_INTEGRATION = false;
 
     @ConfigurationComment("Do you want to only let people build inside region flagged by doing /region regionName flag allow-shop allow?")
@@ -231,6 +314,14 @@ public class Properties {
     @ConfigurationComment("Do you want ChestShop to respect WorldGuard's chest protection?")
     public static boolean WORLDGUARD_USE_PROTECTION = false;
 
+    @PrecededBySpace
+    @ConfigurationComment("Do you want to only let people build inside GriefPrevention claims?")
+    public static boolean GRIEFPREVENTION_INTEGRATION = false;
+
+    @PrecededBySpace
+    @ConfigurationComment("Do you want to only let people build inside RedProtect regions?")
+    public static boolean REDPROTECT_INTEGRATION = false;
+
     @PrecededBySpace
     @ConfigurationComment("Do you want to deny shop access to unlogged users?")
     public static boolean AUTHME_HOOK = true;
@@ -245,4 +336,8 @@ public class Properties {
     @PrecededBySpace
     @ConfigurationComment("Add icons and make item names hoverable in transaction messages when ShowItem is installed?")
     public static boolean SHOWITEM_MESSAGE = true;
+
+    @PrecededBySpace
+    @ConfigurationComment("Add stock counter to quantity line?")
+    public static boolean USE_STOCK_COUNTER = false;
 }

src/main/java/com/Acrobot/ChestShop/Containers/AdminInventory.java

@@ -42,14 +42,13 @@ public class AdminInventory implements Inventory {
         maxStackSize = i;
     }
 
-    @Override
     public String getName() {
         return "Admin inventory";
     }
 
     @Override
     public ItemStack getItem(int i) {
-        if (content.length < i) {
+        if (content.length > i) {
             return content[i];
         }
         return null;
@@ -157,10 +156,12 @@ public class AdminInventory implements Inventory {
     public HashMap<Integer, ? extends ItemStack> all(ItemStack itemStack) {
         HashMap<Integer, ItemStack> items = new HashMap<Integer, ItemStack>();
 
-        ItemStack clone = itemStack.clone();
-        clone.setAmount(Integer.MAX_VALUE);
+        if (itemStack != null) {
+            ItemStack clone = itemStack.clone();
+            clone.setAmount(Integer.MAX_VALUE);
 
-        items.put(1, clone);
+            items.put(1, clone);
+        }
 
         return items;
     }
@@ -190,6 +191,10 @@ public class AdminInventory implements Inventory {
         return 0;
     }
 
+    public boolean isEmpty() {
+        return false;
+    }
+
     @Override
     public void remove(Material material) {
     }
@@ -206,12 +211,15 @@ public class AdminInventory implements Inventory {
     public void clear() {
     }
 
+    public int close() {
+        return 0;
+    }
+
     @Override
     public List<HumanEntity> getViewers() {
         return new ArrayList<>();
     }
 
-    @Override
     public String getTitle() {
         return "Admin inventory";
     }
@@ -226,6 +234,10 @@ public class AdminInventory implements Inventory {
         return null;
     }
 
+    public InventoryHolder getHolder(boolean useSnapshot) {
+        return null;
+    }
+
     @Override
     public ListIterator<ItemStack> iterator() {
         return Arrays.asList(content).listIterator();

src/main/java/com/Acrobot/ChestShop/Database/DaoCreator.java

@@ -4,8 +4,8 @@ import com.Acrobot.ChestShop.ChestShop;
 import com.j256.ormlite.dao.Dao;
 import com.j256.ormlite.dao.DaoManager;
 import com.j256.ormlite.dao.LruObjectCache;
-import com.j256.ormlite.db.SqliteDatabaseType;
 import com.j256.ormlite.jdbc.JdbcConnectionSource;
+import com.j256.ormlite.jdbc.db.SqliteDatabaseType;
 import com.j256.ormlite.support.ConnectionSource;
 import com.j256.ormlite.table.TableUtils;
 

src/main/java/com/Acrobot/ChestShop/Database/Migrations.java

@@ -28,8 +28,6 @@ public class Migrations {
     public static int migrate(int currentVersion) {
         if (currentVersion != CURRENT_DATABASE_VERSION) {
             ChestShop.getBukkitLogger().info("Updating database...");
-        } else {
-            return CURRENT_DATABASE_VERSION;
         }
 
         switch (currentVersion) {
@@ -67,7 +65,7 @@ public class Migrations {
             accounts.executeRaw("ALTER TABLE `accounts` ADD COLUMN lastSeenName VARCHAR");
             return true;
         } catch (SQLException e) {
-            e.printStackTrace();
+            ChestShop.getBukkitLogger().log(Level.SEVERE, "Error while migrating database to v2", e);
             return false;
         }
     }
@@ -113,7 +111,7 @@ public class Migrations {
                 }
                 try {
                     results.close();
-                } catch (IOException e1) {
+                } catch (Exception e1) {
                     ChestShop.getBukkitLogger().log(Level.WARNING, "Error while closing results! " + e.getMessage());
                 }
                 ChestShop.getBukkitLogger().log(Level.INFO, success + " accounts successfully migrated. " + error + " accounts failed to migrate!");
@@ -122,7 +120,7 @@ public class Migrations {
 
             return true;
         } catch (SQLException e) {
-            e.printStackTrace();
+            ChestShop.getBukkitLogger().log(Level.SEVERE, "Error while migrating database to v3", e);
             return false;
         }
     }
@@ -139,13 +137,13 @@ public class Migrations {
             try {
                 items.executeRawNoArgs("INSERT INTO `items` (id, code) SELECT id, code uuid FROM `items-old`");
             } catch (SQLException e) {
-                e.printStackTrace();
+                ChestShop.getBukkitLogger().log(Level.SEVERE, "Error while inserting items into new database while migrating to v4", e);
             }
             ChestShop.getBukkitLogger().log(Level.INFO, "Migration of items table finished in " + (System.currentTimeMillis() - start) / 1000.0 + "s!");
 
             return true;
         } catch (SQLException e) {
-            e.printStackTrace();
+            ChestShop.getBukkitLogger().log(Level.SEVERE, "Error while migrating database to v4", e);
             return false;
         }
     }

src/main/java/com/Acrobot/ChestShop/Dependencies.java

@@ -2,20 +2,33 @@ package com.Acrobot.ChestShop;
 
 import com.Acrobot.Breeze.Utils.MaterialUtil;
 import com.Acrobot.ChestShop.Configuration.Properties;
+import com.Acrobot.ChestShop.Listeners.Economy.EconomyAdapter;
 import com.Acrobot.ChestShop.Listeners.Economy.Plugins.ReserveListener;
 import com.Acrobot.ChestShop.Listeners.Economy.Plugins.VaultListener;
 import com.Acrobot.ChestShop.Plugins.*;
-import com.sk89q.worldguard.bukkit.WorldGuardPlugin;
+import com.google.common.collect.ImmutableMap;
+import org.bstats.charts.DrilldownPie;
 import org.bukkit.Bukkit;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.EventPriority;
 import org.bukkit.event.Listener;
+import org.bukkit.event.server.PluginEnableEvent;
 import org.bukkit.plugin.Plugin;
 import org.bukkit.plugin.PluginDescriptionFile;
 import org.bukkit.plugin.PluginManager;
 
+import java.util.AbstractMap;
+import java.util.HashMap;
+import java.util.Map;
+import java.util.logging.Level;
+import java.util.stream.Collectors;
+
 /**
  * @author Acrobot
  */
-public class Dependencies {
+public class Dependencies implements Listener {
+
+    private static final Map<String, String> versions = new HashMap<>();
 
     public static void initializePlugins() {
         PluginManager pluginManager = Bukkit.getPluginManager();
@@ -56,17 +69,31 @@ public class Dependencies {
             Plugin plugin = pluginManager.getPlugin(dependency);
 
             if (plugin != null && plugin.isEnabled()) {
-                loadPlugin(dependency, plugin);
+                try {
+                    loadPlugin(dependency, plugin);
+                } catch (Exception e) {
+                    plugin.getLogger().log(Level.WARNING, "Unable to hook into " + plugin.getName() + " " + plugin.getDescription().getVersion(), e);
+                }
             }
         }
 
-        return loadEconomy();
+        if (loadEconomy()) {
+            Map<String, Map<String, Integer>> map = versions.entrySet().stream()
+                    .map(e -> new AbstractMap.SimpleEntry<String, Map<String, Integer>>(e.getKey(), ImmutableMap.of(e.getValue(), 1)))
+                    .collect(Collectors.toMap(
+                            AbstractMap.SimpleEntry::getKey,
+                            AbstractMap.SimpleEntry::getValue
+                    ));
+            ChestShop.getMetrics().addCustomChart(new DrilldownPie("dependencies", () -> map));
+            return true;
+        }
+        return false;
     }
 
     private static boolean loadEconomy() {
         String plugin = "none";
 
-        Listener economy = null;
+        EconomyAdapter economy = null;
 
         if(Bukkit.getPluginManager().getPlugin("Reserve") != null) {
             plugin = "Reserve";
@@ -83,6 +110,9 @@ public class Dependencies {
             return false;
         }
 
+        ChestShop.getMetrics().addCustomChart(ChestShop.createStaticDrilldownStat("economyAdapter", plugin, Bukkit.getPluginManager().getPlugin(plugin).getDescription().getVersion()));
+        ChestShop.getMetrics().addCustomChart(ChestShop.createStaticDrilldownStat("economyPlugin", economy.getProviderInfo().getName(), economy.getProviderInfo().getVersion()));
+
         ChestShop.registerListener(economy);
         ChestShop.getBukkitLogger().info(plugin + " loaded!");
         return true;
@@ -107,12 +137,18 @@ public class Dependencies {
             case Lockette:
                 listener = new Lockette();
                 break;
+            case LockettePro:
+                listener = new LockettePro();
+                break;
             case Deadbolt:
                 listener = new Deadbolt();
                 break;
             case SimpleChestLock:
                 listener = SimpleChestLock.getSimpleChestLock(plugin);
                 break;
+            case BlockLocker:
+                listener = new BlockLocker();
+                break;
             case Residence:
                 if (plugin.getDescription().getVersion().startsWith("2")) {
                     ChestShop.getBukkitLogger().severe("You are using an old version of Residence! " +
@@ -126,7 +162,6 @@ public class Dependencies {
 
             //Terrain protection plugins
             case WorldGuard:
-                WorldGuardPlugin worldGuard = (WorldGuardPlugin) plugin;
                 boolean inUse = Properties.WORLDGUARD_USE_PROTECTION || Properties.WORLDGUARD_INTEGRATION;
 
                 if (!inUse) {
@@ -134,15 +169,29 @@ public class Dependencies {
                 }
 
                 if (Properties.WORLDGUARD_USE_PROTECTION) {
-                    ChestShop.registerListener(new WorldGuardProtection(worldGuard));
+                    ChestShop.registerListener(new WorldGuardProtection(plugin));
                 }
 
                 if (Properties.WORLDGUARD_INTEGRATION) {
-                    listener = new WorldGuardBuilding(worldGuard);
+                    listener = new WorldGuardBuilding(plugin);
                 }
 
                 break;
 
+            case GriefPrevention:
+                if (!Properties.GRIEFPREVENTION_INTEGRATION) {
+                    return;
+                }
+                listener = new GriefPrevenentionBuilding(plugin);
+                break;
+
+            case RedProtect:
+                if (!Properties.REDPROTECT_INTEGRATION) {
+                    return;
+                }
+                listener = new RedProtectBuilding(plugin);
+                break;
+
             //Other plugins
             case Heroes:
                 Heroes heroes = Heroes.getHeroes(plugin);
@@ -153,8 +202,8 @@ public class Dependencies {
 
                 listener = heroes;
                 break;
-            case OddItem:
-                MaterialUtil.Odd.initialize();
+            case ItemBridge:
+                listener = new ItemBridge();
                 break;
             case ShowItem:
                 MaterialUtil.Show.initialize(plugin);
@@ -166,22 +215,35 @@ public class Dependencies {
         }
 
         PluginDescriptionFile description = plugin.getDescription();
+        versions.put(description.getName(), description.getVersion());
         ChestShop.getBukkitLogger().info(description.getName() + " version " + description.getVersion() + " loaded.");
     }
 
     private static enum Dependency {
         LWC,
         Lockette,
+        LockettePro,
         Deadbolt,
         SimpleChestLock,
+        BlockLocker,
         Residence,
 
-        OddItem,
-
         WorldGuard,
+        GriefPrevention,
+        RedProtect,
 
         Heroes,
 
+        ItemBridge,
+
         ShowItem
     }
+    
+    @EventHandler(priority = EventPriority.MONITOR)
+    public void onEnable(PluginEnableEvent event) {
+        Plugin plugin = event.getPlugin();
+        if (ChestShop.getDependencies().contains(plugin.getName())) {
+            loadPlugin(plugin.getName(), plugin);
+        }
+    }
 }

src/main/java/com/Acrobot/ChestShop/Economy/Economy.java

@@ -41,7 +41,7 @@ public class Economy {
         CurrencyAddEvent event = new CurrencyAddEvent(BigDecimal.valueOf(amount), name, world);
         ChestShop.callEvent(event);
 
-        return event.isAdded();
+        return event.wasHandled();
     }
 
     /**
@@ -52,7 +52,7 @@ public class Economy {
         CurrencySubtractEvent event = new CurrencySubtractEvent(BigDecimal.valueOf(amount), name, world);
         ChestShop.callEvent(event);
 
-        return event.isSubtracted();
+        return event.wasHandled();
     }
 
     /**

src/main/java/com/Acrobot/ChestShop/Events/AccountAccessEvent.java

@@ -0,0 +1,57 @@
+package com.Acrobot.ChestShop.Events;
+
+import com.Acrobot.ChestShop.Database.Account;
+import org.bukkit.entity.Player;
+import org.bukkit.event.HandlerList;
+import org.bukkit.event.player.PlayerEvent;
+
+/**
+ * Represents an access request for a specific account.
+ */
+public class AccountAccessEvent extends PlayerEvent {
+    private static final HandlerList handlers = new HandlerList();
+    private final Account account;
+    private boolean canAccess = false;
+
+    public AccountAccessEvent(Player player, Account account) {
+        super(player);
+        this.account = account;
+    }
+
+    /**
+     * The account to check the access for
+     *
+     * @return The account
+     */
+    public Account getAccount() {
+        return account;
+    }
+
+    /**
+     * Whether or not the player can access the account.
+     *
+     * @return Whether or not the player can access the account
+     */
+    public boolean canAccess() {
+        return canAccess;
+    }
+
+    /**
+     * Set whether or not the player can access the account.
+     *
+     * @param canAccess Whether or not the player can access the account
+     */
+    public void setAccess(boolean canAccess) {
+        this.canAccess = canAccess;
+    }
+
+    @Override
+    public HandlerList getHandlers() {
+        return handlers;
+    }
+
+    public static HandlerList getHandlerList() {
+        return handlers;
+    }
+
+}

src/main/java/com/Acrobot/ChestShop/Events/AccountQueryEvent.java

@@ -11,6 +11,7 @@ public class AccountQueryEvent extends Event {
     private static final HandlerList handlers = new HandlerList();
     private final String name;
     private Account account = null;
+    private boolean searchOfflinePlayers = false;
 
     public AccountQueryEvent(String name) {
         this.name = name;
@@ -28,6 +29,23 @@ public class AccountQueryEvent extends Event {
         this.account = account;
     }
 
+    /**
+     * Get whether or not offline player data should be searched (too)
+     * @return Whether or not offline player data should be searched (too)
+     */
+    public boolean searchOfflinePlayers() {
+        return searchOfflinePlayers;
+    }
+
+    /**
+     * Set whether or not offline player data should be searched (too).
+     * This could lead to network lookups if the player by the name never joined the server!
+     * @param searchOfflinePlayers Whether or not offline player data should be searched (too)
+     */
+    public void searchOfflinePlayers(boolean searchOfflinePlayers) {
+        this.searchOfflinePlayers = searchOfflinePlayers;
+    }
+
     @Override
     public HandlerList getHandlers() {
         return handlers;

src/main/java/com/Acrobot/ChestShop/Events/ChestShopReloadEvent.java

@@ -0,0 +1,35 @@
+package com.Acrobot.ChestShop.Events;
+
+import org.bukkit.command.CommandSender;
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+
+/**
+ * Represents a plugin reload call
+ *
+ * @author Acrobot
+ */
+public class ChestShopReloadEvent extends Event {
+    private static final HandlerList handlers = new HandlerList();
+
+    private CommandSender sender;
+
+    public ChestShopReloadEvent(CommandSender sender) {
+        this.sender = sender;
+    }
+
+    /**
+     * @return CommandSender who initiated the call
+     */
+    public CommandSender getSender() {
+        return sender;
+    }
+
+    public HandlerList getHandlers() {
+        return handlers;
+    }
+
+    public static HandlerList getHandlerList() {
+        return handlers;
+    }
+}

src/main/java/com/Acrobot/ChestShop/Events/Economy/AccountCheckEvent.java

@@ -1,7 +1,6 @@
 package com.Acrobot.ChestShop.Events.Economy;
 
 import org.bukkit.World;
-import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
 import java.util.UUID;
@@ -11,10 +10,10 @@ import java.util.UUID;
  *
  * @author Acrobot
  */
-public class AccountCheckEvent extends Event {
+public class AccountCheckEvent extends EconomicEvent {
     private static final HandlerList handlers = new HandlerList();
 
-    boolean outcome = false;
+    private boolean outcome = false;
 
     private UUID account;
     private World world;

src/main/java/com/Acrobot/ChestShop/Events/Economy/CurrencyAddEvent.java

@@ -2,7 +2,6 @@ package com.Acrobot.ChestShop.Events.Economy;
 
 import org.bukkit.World;
 import org.bukkit.entity.Player;
-import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
 import java.math.BigDecimal;
@@ -15,11 +14,9 @@ import java.util.UUID;
  *
  * @author Acrobot
  */
-public class CurrencyAddEvent extends Event {
+public class CurrencyAddEvent extends EconomicEvent {
     private static final HandlerList handlers = new HandlerList();
 
-    boolean added = false;
-
     private BigDecimal amount;
     private UUID target;
     private World world;
@@ -70,18 +67,22 @@ public class CurrencyAddEvent extends Event {
 
     /**
      * @return Was the money already added to the account?
+     * @deprecated Use {@link #wasHandled()}
      */
+    @Deprecated
     public boolean isAdded() {
-        return added;
+        return wasHandled();
     }
 
     /**
      * Set if the money was added to the account
      *
      * @param added Was the money added?
+     * @deprecated Use {@link #setHandled(boolean)}
      */
+    @Deprecated
     public void setAdded(boolean added) {
-        this.added = added;
+        setHandled(added);
     }
 
     /**
@@ -98,6 +99,13 @@ public class CurrencyAddEvent extends Event {
         return target;
     }
 
+    /**
+     * @param target Account from which the currency is subtracted
+     */
+    public void setTarget(UUID target) {
+        this.target = target;
+    }
+
     public HandlerList getHandlers() {
         return handlers;
     }

src/main/java/com/Acrobot/ChestShop/Events/Economy/CurrencyAmountEvent.java

@@ -2,7 +2,6 @@ package com.Acrobot.ChestShop.Events.Economy;
 
 import org.bukkit.World;
 import org.bukkit.entity.Player;
-import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
 import java.math.BigDecimal;
@@ -13,7 +12,7 @@ import java.util.UUID;
  *
  * @author Acrobot
  */
-public class CurrencyAmountEvent extends Event {
+public class CurrencyAmountEvent extends EconomicEvent {
     private static final HandlerList handlers = new HandlerList();
 
     private BigDecimal amount = BigDecimal.ZERO;
@@ -77,6 +76,13 @@ public class CurrencyAmountEvent extends Event {
         return account;
     }
 
+    /**
+     * @param account Account that is checked
+     */
+    public void setAccount(UUID account) {
+        this.account = account;
+    }
+
     public HandlerList getHandlers() {
         return handlers;
     }

src/main/java/com/Acrobot/ChestShop/Events/Economy/CurrencyCheckEvent.java

@@ -2,7 +2,6 @@ package com.Acrobot.ChestShop.Events.Economy;
 
 import org.bukkit.World;
 import org.bukkit.entity.Player;
-import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
 import java.math.BigDecimal;
@@ -13,10 +12,10 @@ import java.util.UUID;
  *
  * @author Acrobot
  */
-public class CurrencyCheckEvent extends Event {
+public class CurrencyCheckEvent extends EconomicEvent {
     private static final HandlerList handlers = new HandlerList();
 
-    boolean outcome = false;
+    private boolean outcome = false;
 
     private BigDecimal amount;
     private UUID account;
@@ -59,6 +58,7 @@ public class CurrencyCheckEvent extends Event {
      * @return Amount of currency, as a double
      * @deprecated Use {@link #getAmount()} if possible
      */
+    @Deprecated
     public double getDoubleAmount() {
         return amount.doubleValue();
     }
@@ -67,7 +67,9 @@ public class CurrencyCheckEvent extends Event {
      * Sets the amount of currency transferred
      *
      * @param amount Amount to transfer
+     * @deprecated The amount should not be changed!
      */
+    @Deprecated
     public void setAmount(BigDecimal amount) {
         this.amount = amount;
     }
@@ -76,8 +78,9 @@ public class CurrencyCheckEvent extends Event {
      * Sets the amount of currency transferred
      *
      * @param amount Amount to transfer
-     * @deprecated Use {@link #setAmount(java.math.BigDecimal)} if possible
+     * @deprecated The amount should not be changed! Use {@link #setAmount(java.math.BigDecimal)} if possible
      */
+    @Deprecated
     public void setAmount(double amount) {
         this.amount = BigDecimal.valueOf(amount);
     }

src/main/java/com/Acrobot/ChestShop/Events/Economy/CurrencyFormatEvent.java

@@ -1,6 +1,5 @@
 package com.Acrobot.ChestShop.Events.Economy;
 
-import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
 import java.math.BigDecimal;
@@ -10,7 +9,7 @@ import java.math.BigDecimal;
  *
  * @author Acrobot
  */
-public class CurrencyFormatEvent extends Event {
+public class CurrencyFormatEvent extends EconomicEvent {
     private static final HandlerList handlers = new HandlerList();
 
     private final BigDecimal amount;

src/main/java/com/Acrobot/ChestShop/Events/Economy/CurrencyHoldEvent.java

@@ -2,7 +2,6 @@ package com.Acrobot.ChestShop.Events.Economy;
 
 import org.bukkit.World;
 import org.bukkit.entity.Player;
-import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
 import java.math.BigDecimal;
@@ -13,10 +12,10 @@ import java.util.UUID;
  *
  * @author Acrobot
  */
-public class CurrencyHoldEvent extends Event {
+public class CurrencyHoldEvent extends EconomicEvent {
     private static final HandlerList handlers = new HandlerList();
 
-    boolean canHold = true;
+    private boolean canHold = true;
 
     private BigDecimal amount;
     private UUID account;

src/main/java/com/Acrobot/ChestShop/Events/Economy/CurrencySubtractEvent.java

@@ -2,7 +2,6 @@ package com.Acrobot.ChestShop.Events.Economy;
 
 import org.bukkit.World;
 import org.bukkit.entity.Player;
-import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
 import java.math.BigDecimal;
@@ -15,11 +14,9 @@ import java.util.UUID;
  *
  * @author Acrobot
  */
-public class CurrencySubtractEvent extends Event {
+public class CurrencySubtractEvent extends EconomicEvent {
     private static final HandlerList handlers = new HandlerList();
 
-    boolean subtracted = false;
-
     private BigDecimal amount;
     private UUID target;
     private World world;
@@ -70,18 +67,22 @@ public class CurrencySubtractEvent extends Event {
 
     /**
      * @return Was the money already subtracted?
+     * @deprecated Use {@link #wasHandled()}
      */
+    @Deprecated
     public boolean isSubtracted() {
-        return subtracted;
+        return wasHandled();
     }
 
     /**
      * Set if the money was subtracted from the account
      *
      * @param subtracted Was the money subtracted?
+     * @deprecated Use {@link #setHandled(boolean)}
      */
+    @Deprecated
     public void setSubtracted(boolean subtracted) {
-        this.subtracted = subtracted;
+        setHandled(subtracted);
     }
 
     /**
@@ -98,6 +99,13 @@ public class CurrencySubtractEvent extends Event {
         return target;
     }
 
+    /**
+     * @param target Account from which the currency is subtracted
+     */
+    public void setTarget(UUID target) {
+        this.target = target;
+    }
+
     public HandlerList getHandlers() {
         return handlers;
     }

src/main/java/com/Acrobot/ChestShop/Events/Economy/CurrencyTransferEvent.java

@@ -1,8 +1,8 @@
 package com.Acrobot.ChestShop.Events.Economy;
 
+import com.Acrobot.ChestShop.Events.TransactionEvent;
 import org.bukkit.World;
 import org.bukkit.entity.Player;
-import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
 import java.math.BigDecimal;
@@ -13,7 +13,7 @@ import java.util.UUID;
  *
  * @author Acrobot
  */
-public class CurrencyTransferEvent extends Event {
+public class CurrencyTransferEvent extends EconomicEvent {
     private static final HandlerList handlers = new HandlerList();
 
     private BigDecimal amountSent;
@@ -24,28 +24,33 @@ public class CurrencyTransferEvent extends Event {
     private UUID partner;
 
     private Direction direction;
-    private boolean success = false;
+
+    private final TransactionEvent transactionEvent;
 
     public CurrencyTransferEvent(BigDecimal amount, Player initiator, UUID partner, Direction direction) {
         this(amount, amount, initiator, partner, direction);
     }
 
     public CurrencyTransferEvent(BigDecimal amountSent, BigDecimal amountReceived, Player initiator, UUID partner, Direction direction) {
+        this(amountSent, amountReceived, initiator, partner, direction, null);
+    }
+
+
+    public CurrencyTransferEvent(BigDecimal amount, Player initiator, UUID partner, Direction direction, TransactionEvent transactionEvent) {
+        this(amount, amount, initiator, partner, direction, transactionEvent);
+    }
+
+    public CurrencyTransferEvent(BigDecimal amountSent, BigDecimal amountReceived, Player initiator, UUID partner, Direction direction, TransactionEvent transactionEvent) {
         this.amountSent = amountSent;
         this.amountReceived = amountReceived;
         this.initiator = initiator;
 
         this.partner = partner;
         this.direction = direction;
+
+        this.transactionEvent = transactionEvent;
     }
 
-    /**
-     * @deprecated Use {{@link #CurrencyTransferEvent(BigDecimal, Player, UUID, Direction)}
-     */
-    @Deprecated
-    public CurrencyTransferEvent(double amount, Player initiator, UUID partner, Direction direction) {
-        this(BigDecimal.valueOf(amount), initiator, partner, direction);
-    }
 
     /**
      * @return Amount of currency sent
@@ -126,18 +131,22 @@ public class CurrencyTransferEvent extends Event {
 
     /**
      * @return If the currency has been successfully transferred
+     * @deprecated Use {@link #wasHandled()}
      */
+    @Deprecated
     public boolean hasBeenTransferred() {
-        return success;
+        return wasHandled();
     }
 
     /**
      * Sets the transaction's outcome
      *
      * @param success If the currency has been successfully transferred
+     * @deprecated Use {@link #setHandled(boolean)}
      */
+    @Deprecated
     public void setTransferred(boolean success) {
-        this.success = success;
+        setHandled(success);
     }
 
     /**
@@ -147,6 +156,15 @@ public class CurrencyTransferEvent extends Event {
         return direction;
     }
 
+    /**
+     * Gets the {@link TransactionEvent} associated with this currency transfer event.
+     *
+     * @return the transaction event.
+     */
+    public TransactionEvent getTransactionEvent() {
+        return transactionEvent;
+    }
+
     /**
      * Get the player who initiated this transaction
      *
@@ -163,6 +181,15 @@ public class CurrencyTransferEvent extends Event {
         return partner;
     }
 
+    /**
+     * Set the partner of the transaction
+     *
+     * @param partner the new partner of this transaction
+     */
+    public void setPartner(UUID partner) {
+        this.partner = partner;
+    }
+
     /**
      * @return The world in which the transaction occurs
      */

src/main/java/com/Acrobot/ChestShop/Events/Economy/EconomicEvent.java

@@ -0,0 +1,26 @@
+package com.Acrobot.ChestShop.Events.Economy;
+
+import org.bukkit.event.Event;
+
+public abstract class EconomicEvent extends Event {
+
+    private boolean handled = false;
+
+    /**
+     * Get whether or not this event was successfully handled by a listener
+     *
+     * @return Whether or not the amount was successfully handled
+     */
+    public boolean wasHandled() {
+        return handled;
+    }
+
+    /**
+     * Set whether or not this event was successfully handled by a listener
+     *
+     * @param handled Whether or not the amount was successfully handled
+     */
+    public void setHandled(boolean handled) {
+        this.handled = handled;
+    }
+}

src/main/java/com/Acrobot/ChestShop/Events/ItemStringQueryEvent.java

@@ -0,0 +1,72 @@
+package com.Acrobot.ChestShop.Events;
+
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+import org.bukkit.inventory.ItemStack;
+
+public class ItemStringQueryEvent extends Event {
+    private static final HandlerList handlers = new HandlerList();
+
+    private String itemString = null;
+    private final ItemStack item;
+    private final int maxWidth;
+
+    /**
+     * Query the item string representation of a certain item with a certain length
+     * @param item      The item to query the string for
+     */
+    public ItemStringQueryEvent(ItemStack item) {
+        this(item, 0);
+    }
+
+    /**
+     * Query the item string representation of a certain item with a certain length
+     * @param item      The item to query the string for
+     * @param maxWidth  The max width of the item string
+     */
+    public ItemStringQueryEvent(ItemStack item, int maxWidth) {
+        this.item = item;
+        this.maxWidth = maxWidth;
+    }
+
+    @Override
+    public HandlerList getHandlers() {
+        return handlers;
+    }
+
+    public static HandlerList getHandlerList() {
+        return handlers;
+    }
+
+    /**
+     * The item for which the string is queried
+     * @return The item
+     */
+    public ItemStack getItem() {
+        return item;
+    }
+
+    /**
+     * Get the item string that represents the item
+     * @return The item string that represents the item
+     */
+    public String getItemString() {
+        return itemString;
+    }
+
+    /**
+     * Set the item string that represents the item
+     * @param itemString  The item string that represents the item
+     */
+    public void setItemString(String itemString) {
+        this.itemString = itemString;
+    }
+
+    /**
+     * Get the max width that the result item string should have
+     * @return The max width of the result item string
+     */
+    public int getMaxWidth() {
+        return maxWidth;
+    }
+}

src/main/java/com/Acrobot/ChestShop/Events/PreShopCreationEvent.java

@@ -47,7 +47,11 @@ public class PreShopCreationEvent extends Event implements Cancellable {
      */
     @Override
     public void setCancelled(boolean cancel) {
-        outcome = CreationOutcome.OTHER;
+        if (cancel) {
+            outcome = CreationOutcome.OTHER;
+        } else {
+            outcome = CreationOutcome.SHOP_CREATED_SUCCESSFULLY;
+        }
     }
 
     /**
@@ -177,7 +181,7 @@ public class PreShopCreationEvent extends Event implements Cancellable {
         INVALID_PRICE,
         INVALID_QUANTITY,
 
-        ITEM_AUTOFILL,
+        ITEM_AUTOFILL(false),
 
         UNKNOWN_PLAYER,
 
@@ -198,8 +202,31 @@ public class PreShopCreationEvent extends Event implements Cancellable {
         /**
          * For plugin use
          */
-        OTHER,
+        OTHER(false),
+        /**
+         * Break the sign
+         */
+        OTHER_BREAK,
 
-        SHOP_CREATED_SUCCESSFULLY
+        SHOP_CREATED_SUCCESSFULLY(false);
+
+        private final boolean breakSign;
+
+        CreationOutcome() {
+            this.breakSign = true;
+        }
+
+        CreationOutcome(boolean breakSign) {
+            this.breakSign = breakSign;
+        }
+
+        /**
+         * Get whether or not this outcome should result in the shop sign getting broken
+         *
+         * @return Whether or not the shop sign gets broken
+         */
+        public boolean shouldBreakSign() {
+            return breakSign;
+        }
     }
 }

src/main/java/com/Acrobot/ChestShop/Events/PreTransactionEvent.java

@@ -149,26 +149,6 @@ public class PreTransactionEvent extends Event implements Cancellable {
         this.ownerAccount = ownerAccount;
     }
 
-    /**
-     * @return Shop's owner
-     * @deprecated Use {@link #getOwnerAccount}
-     */
-    @Deprecated
-    public OfflinePlayer getOwner() {
-        return Bukkit.getOfflinePlayer(ownerAccount.getUuid());
-    }
-
-    /**
-     * Sets the shop's owner
-     *
-     * @param owner Shop owner
-     * @deprecated Use {@link #setOwnerAccount(Account)}
-     */
-    @Deprecated
-    public void setOwner(OfflinePlayer owner) {
-        this.ownerAccount = NameManager.getOrCreateAccount(owner);
-    }
-
     /**
      * @return Owner's inventory
      */
@@ -217,7 +197,11 @@ public class PreTransactionEvent extends Event implements Cancellable {
 
     @Override
     public void setCancelled(boolean cancel) {
-        transactionOutcome = OTHER;
+        if (cancel) {
+            transactionOutcome = OTHER;
+        } else {
+            transactionOutcome = TRANSACTION_SUCCESFUL;
+        }
     }
 
     /**
@@ -263,6 +247,7 @@ public class PreTransactionEvent extends Event implements Cancellable {
         NOT_ENOUGH_STOCK_IN_INVENTORY,
 
         INVALID_SHOP,
+        INVALID_CLIENT_NAME,
 
         SPAM_CLICKING_PROTECTION,
         CREATIVE_MODE_PROTECTION,

src/main/java/com/Acrobot/ChestShop/Events/Protection/BuildPermissionEvent.java

@@ -2,22 +2,24 @@ package com.Acrobot.ChestShop.Events.Protection;
 
 import org.bukkit.Location;
 import org.bukkit.entity.Player;
+import org.bukkit.event.Cancellable;
 import org.bukkit.event.Event;
 import org.bukkit.event.HandlerList;
 
+import javax.annotation.Nullable;
+
 /**
  * @author Acrobot
  */
-public class BuildPermissionEvent extends Event {
+public class BuildPermissionEvent extends Event implements Cancellable {
     private static final HandlerList handlers = new HandlerList();
 
     private Player player;
     private Location chest, sign;
 
-    private int disallowed = 0;
-    private int received = 0;
+    private boolean allowed = true;
 
-    public BuildPermissionEvent(Player player, Location chest, Location sign) {
+    public BuildPermissionEvent(Player player, @Nullable Location chest, Location sign) {
         this.player = player;
         this.chest = chest;
         this.sign = sign;
@@ -27,7 +29,7 @@ public class BuildPermissionEvent extends Event {
         return player;
     }
 
-    public Location getChest() {
+    public @Nullable Location getChest() {
         return chest;
     }
 
@@ -36,24 +38,19 @@ public class BuildPermissionEvent extends Event {
     }
 
     public void allow() {
-        received++;
+        allowed = true;
     }
 
     public boolean isAllowed() {
-        return disallowed != received || received == 0;
+        return allowed;
     }
 
     public void allow(boolean yesOrNot) {
-        if (yesOrNot) {
-            allow();
-        } else {
-            disallow();
-        }
+        allowed = yesOrNot;
     }
 
     public void disallow() {
-        received++;
-        disallowed++;
+        allowed = false;
     }
 
     public HandlerList getHandlers() {
@@ -63,4 +60,14 @@ public class BuildPermissionEvent extends Event {
     public static HandlerList getHandlerList() {
         return handlers;
     }
+
+    @Override
+    public boolean isCancelled() {
+        return !isAllowed();
+    }
+
+    @Override
+    public void setCancelled(boolean cancel) {
+        allow(!cancel);
+    }
 }

src/main/java/com/Acrobot/ChestShop/Events/Protection/ProtectBlockEvent.java

@@ -1,5 +1,6 @@
 package com.Acrobot.ChestShop.Events.Protection;
 
+import com.Acrobot.ChestShop.Security;
 import org.bukkit.block.Block;
 import org.bukkit.entity.Player;
 import org.bukkit.event.Event;
@@ -16,7 +17,8 @@ public class ProtectBlockEvent extends Event {
 
     private final Player player;
     private final UUID protectionOwner;
-    private Block block;
+    private final Block block;
+    private final Security.Type type;
 
     boolean isProtected = false;
 
@@ -25,9 +27,14 @@ public class ProtectBlockEvent extends Event {
     }
 
     public ProtectBlockEvent(Block block, Player player, UUID protectionOwner) {
+        this(block, player, protectionOwner, Security.Type.PRIVATE);
+    }
+
+    public ProtectBlockEvent(Block block, Player player, UUID protectionOwner, Security.Type type) {
         this.block = block;
         this.player = player;
         this.protectionOwner = protectionOwner;
+        this.type = type;
     }
 
     public boolean isProtected() {
@@ -50,6 +57,10 @@ public class ProtectBlockEvent extends Event {
         return protectionOwner;
     }
 
+    public Security.Type getType() {
+        return type;
+    }
+
     public HandlerList getHandlers() {
         return handlers;
     }

src/main/java/com/Acrobot/ChestShop/Events/Protection/ProtectionCheckEvent.java

@@ -13,6 +13,7 @@ public class ProtectionCheckEvent extends Event {
 
     private Result result = Result.DEFAULT;
     private boolean ignoreBuiltInProtection = false;
+    private boolean checkManagement = true;
     private Block block;
     private Player player;
 
@@ -27,10 +28,21 @@ public class ProtectionCheckEvent extends Event {
         this.ignoreBuiltInProtection = ignoreBuiltInProtection;
     }
 
+    public ProtectionCheckEvent(Block block, Player player, boolean ignoreBuiltInProtection, boolean checkManagement) {
+        this.block = block;
+        this.player = player;
+        this.ignoreBuiltInProtection = ignoreBuiltInProtection;
+        this.checkManagement = checkManagement;
+    }
+
     public boolean isBuiltInProtectionIgnored() {
         return ignoreBuiltInProtection;
     }
 
+    public boolean checkCanManage() {
+        return checkManagement;
+    }
+
     public Result getResult() {
         return result;
     }

src/main/java/com/Acrobot/ChestShop/Events/ShopInfoEvent.java

@@ -0,0 +1,58 @@
+package com.Acrobot.ChestShop.Events;
+
+import org.bukkit.block.Sign;
+import org.bukkit.command.CommandSender;
+import org.bukkit.entity.Player;
+import org.bukkit.event.Cancellable;
+import org.bukkit.event.Event;
+import org.bukkit.event.HandlerList;
+
+/**
+ * Represents a /shopinfo call or middle click on a sign
+ *
+ * @author Phoenix616
+ */
+public class ShopInfoEvent extends Event implements Cancellable {
+    private static final HandlerList handlers = new HandlerList();
+
+    private Player sender;
+    private Sign sign;
+    private boolean cancelled = false;
+
+    public ShopInfoEvent(Player sender, Sign sign) {
+        this.sender = sender;
+        this.sign = sign;
+    }
+
+    /**
+     * @return The Player who initiated the call
+     */
+    public Player getSender() {
+        return sender;
+    }
+
+    /**
+     * @return The shop sign
+     */
+    public Sign getSign() {
+        return sign;
+    }
+
+    public HandlerList getHandlers() {
+        return handlers;
+    }
+
+    public static HandlerList getHandlerList() {
+        return handlers;
+    }
+
+    @Override
+    public boolean isCancelled() {
+        return cancelled;
+    }
+
+    @Override
+    public void setCancelled(boolean cancel) {
+        cancelled = cancel;
+    }
+}

src/main/java/com/Acrobot/ChestShop/Listeners/Block/BlockPlace.java

@@ -37,14 +37,15 @@ public class BlockPlace implements Listener {
         }
 
         if (!Security.canAccess(player, placed)) {
-            event.getPlayer().sendMessage(Messages.prefix(Messages.ACCESS_DENIED));
+            Messages.ACCESS_DENIED.sendWithPrefix(event.getPlayer());
             event.setCancelled(true);
+            return;
         }
 
         Block neighbor = uBlock.findNeighbor(placed);
 
         if (neighbor != null && !Security.canAccess(event.getPlayer(), neighbor)) {
-            event.getPlayer().sendMessage(Messages.prefix(Messages.ACCESS_DENIED));
+            Messages.ACCESS_DENIED.sendWithPrefix(event.getPlayer());
             event.setCancelled(true);
         }
 
@@ -65,10 +66,6 @@ public class BlockPlace implements Listener {
     public static void onHopperDropperPlace(BlockPlaceEvent event) {
         Block placed = event.getBlockPlaced();
 
-        if (placed.getType() != Material.HOPPER && placed.getType() != Material.DROPPER) {
-            return;
-        }
-
         List<BlockFace> searchDirections = new ArrayList<>();
         switch (placed.getType()) {
             case HOPPER:
@@ -78,6 +75,8 @@ public class BlockPlace implements Listener {
             case DROPPER:
                 searchDirections.add(((Directional) placed.getBlockData()).getFacing());
                 break;
+            default:
+                return;
         }
 
         for (BlockFace face : searchDirections) {
@@ -88,7 +87,7 @@ public class BlockPlace implements Listener {
             }
 
             if (!Security.canAccess(event.getPlayer(), relative)) {
-                event.getPlayer().sendMessage(Messages.prefix(Messages.ACCESS_DENIED));
+                Messages.ACCESS_DENIED.sendWithPrefix(event.getPlayer());
                 event.setCancelled(true);
                 return;
             }

src/main/java/com/Acrobot/ChestShop/Listeners/Block/Break/ChestBreak.java

@@ -1,5 +1,6 @@
 package com.Acrobot.ChestShop.Listeners.Block.Break;
 
+import com.Acrobot.ChestShop.Configuration.Messages;
 import com.Acrobot.ChestShop.Configuration.Properties;
 import com.Acrobot.ChestShop.Permission;
 import com.Acrobot.ChestShop.Signs.ChestShopSign;
@@ -10,6 +11,7 @@ import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
 import org.bukkit.event.block.BlockBreakEvent;
+import org.bukkit.event.entity.EntityChangeBlockEvent;
 import org.bukkit.event.entity.EntityExplodeEvent;
 
 /**
@@ -20,6 +22,7 @@ public class ChestBreak implements Listener {
     public static void onChestBreak(BlockBreakEvent event) {
         if (!canBeBroken(event.getBlock(), event.getPlayer())) {
             event.setCancelled(true);
+            Messages.ACCESS_DENIED.sendWithPrefix(event.getPlayer());
         }
     }
 
@@ -37,12 +40,19 @@ public class ChestBreak implements Listener {
         }
     }
 
+    @EventHandler(ignoreCancelled = true)
+    public static void onEntityChangeBlock(EntityChangeBlockEvent event) {
+        if (!canBeBroken(event.getBlock(), null)) {
+            event.setCancelled(true);
+        }
+    }
+
     private static boolean canBeBroken(Block block, Player breaker) {
         if (!uBlock.couldBeShopContainer(block) || !Properties.USE_BUILT_IN_PROTECTION) {
             return true;
         }
 
-        Sign shopSign = uBlock.findAnyNearbyShopSign(block);
+        Sign shopSign = uBlock.getConnectedSign(block);
         if (breaker != null) {
             return  ChestShopSign.hasPermission(breaker, Permission.OTHER_NAME_DESTROY, shopSign);
         }

src/main/java/com/Acrobot/ChestShop/Listeners/Block/Break/SignBreak.java

@@ -2,6 +2,7 @@ package com.Acrobot.ChestShop.Listeners.Block.Break;
 
 import com.Acrobot.Breeze.Utils.BlockUtil;
 import com.Acrobot.ChestShop.ChestShop;
+import com.Acrobot.ChestShop.Configuration.Messages;
 import com.Acrobot.ChestShop.Configuration.Properties;
 import com.Acrobot.ChestShop.Events.ShopDestroyedEvent;
 import com.Acrobot.ChestShop.Listeners.Block.Break.Attached.PhysicsBreak;
@@ -19,18 +20,20 @@ import org.bukkit.event.EventHandler;
 import org.bukkit.event.EventPriority;
 import org.bukkit.event.Listener;
 import org.bukkit.event.block.*;
+import org.bukkit.event.entity.EntityChangeBlockEvent;
 import org.bukkit.event.entity.EntityExplodeEvent;
 import org.bukkit.metadata.FixedMetadataValue;
 
+import java.lang.reflect.InvocationTargetException;
 import java.util.ArrayList;
 import java.util.Collections;
 import java.util.LinkedList;
 import java.util.List;
 
+import static com.Acrobot.Breeze.Utils.ImplementationAdapter.getState;
 import static com.Acrobot.Breeze.Utils.BlockUtil.getAttachedBlock;
 import static com.Acrobot.Breeze.Utils.BlockUtil.isSign;
 import static com.Acrobot.ChestShop.Permission.OTHER_NAME_DESTROY;
-import static com.Acrobot.ChestShop.Signs.ChestShopSign.NAME_LINE;
 
 /**
  * @author Acrobot
@@ -42,9 +45,9 @@ public class SignBreak implements Listener {
     public SignBreak() {
         try {
             Class.forName("com.destroystokyo.paper.event.block.BlockDestroyEvent");
-            ChestShop.getPlugin().registerEvent((Listener) Class.forName("com.Acrobot.ChestShop.Listeners.Block.Break.Attached.PaperBlockDestroy").newInstance());
+            ChestShop.getPlugin().registerEvent((Listener) Class.forName("com.Acrobot.ChestShop.Listeners.Block.Break.Attached.PaperBlockDestroy").getDeclaredConstructor().newInstance());
             ChestShop.getBukkitLogger().info("Using Paper's BlockDestroyEvent instead of the BlockPhysicsEvent!");
-        } catch (ClassNotFoundException | IllegalAccessException | InstantiationException e) {
+        } catch (ClassNotFoundException | IllegalAccessException | InstantiationException | NoSuchMethodException | InvocationTargetException e) {
             ChestShop.getPlugin().registerEvent(new PhysicsBreak());
         }
     }
@@ -54,11 +57,11 @@ public class SignBreak implements Listener {
             return;
         }
 
-        Sign sign = (Sign) block.getState();
+        Sign sign = (Sign) getState(block, false);
         Block attachedBlock = BlockUtil.getAttachedBlock(sign);
 
         if (attachedBlock.getType() == Material.AIR && ChestShopSign.isValid(sign)) {
-            sendShopDestroyedEvent(sign, block.hasMetadata(METADATA_NAME)
+            sendShopDestroyedEvent((Sign) block.getState(), block.hasMetadata(METADATA_NAME)
                     ? (Player) block.getMetadata(METADATA_NAME).get(0).value()
                     : null);
         }
@@ -68,6 +71,7 @@ public class SignBreak implements Listener {
     public static void onSignBreak(BlockBreakEvent event) {
         if (!canBlockBeBroken(event.getBlock(), event.getPlayer())) {
             event.setCancelled(true);
+            Messages.ACCESS_DENIED.sendWithPrefix(event.getPlayer());
             if (isSign(event.getBlock())) {
                 event.getBlock().getState().update();
             }
@@ -122,6 +126,13 @@ public class SignBreak implements Listener {
         }
     }
 
+    @EventHandler(ignoreCancelled = true)
+    public static void onEntityChangeBlock(EntityChangeBlockEvent event) {
+        if (!canBlockBeBroken(event.getBlock(), null)) {
+            event.setCancelled(true);
+        }
+    }
+
     public static boolean canBlockBeBroken(Block block, Player breaker) {
         List<Sign> attachedSigns = getAttachedSigns(block);
         List<Sign> brokenBlocks = new LinkedList<Sign>();
@@ -134,7 +145,7 @@ public class SignBreak implements Listener {
                 continue;
             }
 
-            if (Properties.TURN_OFF_SIGN_PROTECTION || canDestroyShop(breaker, sign.getLine(NAME_LINE))) {
+            if (Properties.TURN_OFF_SIGN_PROTECTION || canDestroyShop(breaker, ChestShopSign.getOwner(sign))) {
                 brokenBlocks.add(sign);
             } else {
                 canBeBroken = false;
@@ -157,11 +168,7 @@ public class SignBreak implements Listener {
     }
 
     public static void sendShopDestroyedEvent(Sign sign, Player player) {
-        Container connectedContainer = null;
-
-        if (!ChestShopSign.isAdminShop(sign)) {
-            connectedContainer = uBlock.findConnectedContainer(sign.getBlock());
-        }
+        Container connectedContainer = uBlock.findConnectedContainer(sign.getBlock());
 
         Event event = new ShopDestroyedEvent(player, sign, connectedContainer);
         ChestShop.callEvent(event);

src/main/java/com/Acrobot/ChestShop/Listeners/Block/SignCreate.java

@@ -3,26 +3,38 @@ package com.Acrobot.ChestShop.Listeners.Block;
 import com.Acrobot.Breeze.Utils.BlockUtil;
 import com.Acrobot.Breeze.Utils.StringUtil;
 import com.Acrobot.ChestShop.ChestShop;
-import com.Acrobot.ChestShop.Events.AccountQueryEvent;
+import com.Acrobot.ChestShop.Configuration.Messages;
 import com.Acrobot.ChestShop.Events.PreShopCreationEvent;
 import com.Acrobot.ChestShop.Events.ShopCreatedEvent;
+import com.Acrobot.ChestShop.Listeners.Block.Break.SignBreak;
 import com.Acrobot.ChestShop.Signs.ChestShopSign;
 import com.Acrobot.ChestShop.UUIDs.NameManager;
 import com.Acrobot.ChestShop.Utils.uBlock;
 import org.bukkit.block.Block;
 import org.bukkit.block.Sign;
+import org.bukkit.block.sign.Side;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
 import org.bukkit.event.block.SignChangeEvent;
 
 import static com.Acrobot.ChestShop.Permission.OTHER_NAME_DESTROY;
-import static com.Acrobot.ChestShop.Signs.ChestShopSign.NAME_LINE;
 
 /**
  * @author Acrobot
  */
 public class SignCreate implements Listener {
 
+    private static boolean HAS_SIGN_SIDES;
+
+    static {
+        try {
+            SignChangeEvent.class.getMethod("getSide");
+            HAS_SIGN_SIDES = true;
+        } catch (NoSuchMethodException e) {
+            HAS_SIGN_SIDES = false;
+        }
+    }
+
     @EventHandler(ignoreCancelled = true)
     public static void onSignChange(SignChangeEvent event) {
         Block signBlock = event.getBlock();
@@ -33,26 +45,47 @@ public class SignCreate implements Listener {
 
         Sign sign = (Sign) signBlock.getState();
 
-        if (ChestShopSign.isValid(sign) && !NameManager.canUseName(event.getPlayer(), OTHER_NAME_DESTROY, StringUtil.stripColourCodes(sign.getLine(NAME_LINE)))) {
+        if (HAS_SIGN_SIDES && event.getSide() != Side.FRONT) {
+            if (ChestShopSign.isValid(sign)) {
+                event.setCancelled(true);
+                Messages.CANNOT_CHANGE_SIGN_BACKSIDE.sendWithPrefix(event.getPlayer());
+            }
+            return;
+        }
+
+        if (ChestShopSign.isValid(event.getLines()) && !NameManager.canUseName(event.getPlayer(), OTHER_NAME_DESTROY, ChestShopSign.getOwner(event.getLines()))) {
             event.setCancelled(true);
             sign.update();
+            ChestShop.logDebug("Shop sign creation at " + sign.getLocation() + " by " + event.getPlayer().getName() + " was cancelled as they weren't able to create a shop for the account '" + ChestShopSign.getOwner(event.getLines()) + "'");
             return;
         }
 
         String[] lines = StringUtil.stripColourCodes(event.getLines());
 
         if (!ChestShopSign.isValidPreparedSign(lines)) {
+            // Check if a valid shop already existed previously
+            if (ChestShopSign.isValid(sign)) {
+                SignBreak.sendShopDestroyedEvent(sign, event.getPlayer());
+            }
             return;
         }
 
         PreShopCreationEvent preEvent = new PreShopCreationEvent(event.getPlayer(), sign, lines);
         ChestShop.callEvent(preEvent);
 
-        for (byte i = 0; i < event.getLines().length; ++i) {
+        if (preEvent.getOutcome().shouldBreakSign()) {
+            event.setCancelled(true);
+            signBlock.breakNaturally();
+            ChestShop.logDebug("Shop sign creation at " + sign.getLocation() + " by " + event.getPlayer().getName() + " was cancelled (creation outcome: " + preEvent.getOutcome() + ") and the sign broken");
+            return;
+        }
+
+        for (byte i = 0; i < preEvent.getSignLines().length && i < 4; ++i) {
             event.setLine(i, preEvent.getSignLine(i));
         }
 
         if (preEvent.isCancelled()) {
+            ChestShop.logDebug("Shop sign creation at " + sign.getLocation() + " by " + event.getPlayer().getName() + " was cancelled (creation outcome: " + preEvent.getOutcome() + ") and sign lines were set to " + String.join(", ", preEvent.getSignLines()));
             return;
         }
 

src/main/java/com/Acrobot/ChestShop/Listeners/Economy/EconomyAdapter.java

@@ -1,5 +1,6 @@
 package com.Acrobot.ChestShop.Listeners.Economy;
 
+import com.Acrobot.ChestShop.ChestShop;
 import com.Acrobot.ChestShop.Events.Economy.AccountCheckEvent;
 import com.Acrobot.ChestShop.Events.Economy.CurrencyAddEvent;
 import com.Acrobot.ChestShop.Events.Economy.CurrencyAmountEvent;
@@ -15,6 +16,8 @@ import java.math.BigDecimal;
 
 public abstract class EconomyAdapter implements Listener {
 
+    public abstract ProviderInfo getProviderInfo();
+
     public abstract void onAmountCheck(CurrencyAmountEvent event);
 
     public abstract void onCurrencyCheck(CurrencyCheckEvent event);
@@ -37,40 +40,57 @@ public abstract class EconomyAdapter implements Listener {
      * @param event The CurrencyTransferEvent to process
      */
     protected void processTransfer(CurrencyTransferEvent event) {
-        if (event.hasBeenTransferred()) {
+        if (event.wasHandled()) {
             return;
         }
 
         BigDecimal amountSent = event.getAmountSent();
         CurrencySubtractEvent currencySubtractEvent = new CurrencySubtractEvent(amountSent, event.getSender(), event.getWorld());
         if (!NameManager.isAdminShop(event.getSender())) {
-            onCurrencySubtraction(currencySubtractEvent);
+            ChestShop.callEvent(currencySubtractEvent);
         } else {
-            currencySubtractEvent.setSubtracted(true);
+            currencySubtractEvent.setHandled(true);
         }
 
-        if (!currencySubtractEvent.isSubtracted()) {
+        if (!currencySubtractEvent.wasHandled()) {
             return;
         }
 
         BigDecimal amountReceived = event.getAmountReceived().subtract(amountSent.subtract(currencySubtractEvent.getAmount()));
         CurrencyAddEvent currencyAddEvent = new CurrencyAddEvent(amountReceived, event.getReceiver(), event.getWorld());
         if (!NameManager.isAdminShop(event.getReceiver())) {
-            onCurrencyAdd(currencyAddEvent);
+            ChestShop.callEvent(currencyAddEvent);
         } else {
-            currencyAddEvent.setAdded(true);
+            currencyAddEvent.setHandled(true);
         }
 
-        if (currencyAddEvent.isAdded()) {
-            event.setTransferred(true);
+        if (currencyAddEvent.wasHandled()) {
+            event.setHandled(true);
         } else {
             CurrencyAddEvent currencyResetEvent = new CurrencyAddEvent(
                     currencySubtractEvent.getAmount(),
                     event.getSender(),
                     event.getWorld()
             );
-            onCurrencyAdd(currencyResetEvent);
+            ChestShop.callEvent(currencyResetEvent);
         }
     }
 
+    public static class ProviderInfo {
+        private final String name;
+        private final String version;
+
+        public ProviderInfo(String name, String version) {
+            this.name = name;
+            this.version = version;
+        }
+
+        public String getName() {
+            return name;
+        }
+
+        public String getVersion() {
+            return version;
+        }
+    }
 }

src/main/java/com/Acrobot/ChestShop/Listeners/Economy/Plugins/ReserveListener.java

@@ -1,5 +1,6 @@
 package com.Acrobot.ChestShop.Listeners.Economy.Plugins;
 
+import com.Acrobot.ChestShop.Configuration.Properties;
 import com.Acrobot.ChestShop.Events.Economy.AccountCheckEvent;
 import com.Acrobot.ChestShop.Events.Economy.CurrencyAddEvent;
 import com.Acrobot.ChestShop.Events.Economy.CurrencyAmountEvent;
@@ -12,6 +13,7 @@ import com.Acrobot.ChestShop.Listeners.Economy.EconomyAdapter;
 import net.tnemc.core.Reserve;
 import net.tnemc.core.economy.EconomyAPI;
 import org.bukkit.Bukkit;
+import org.bukkit.ChatColor;
 import org.bukkit.event.EventHandler;
 
 import javax.annotation.Nullable;
@@ -26,10 +28,15 @@ public class ReserveListener extends EconomyAdapter {
 
     private static @Nullable EconomyAPI economyAPI;
 
-    public ReserveListener(EconomyAPI api) {
+    public ReserveListener(@Nullable EconomyAPI api) {
         ReserveListener.economyAPI = api;
     }
 
+    @Override
+    public ProviderInfo getProviderInfo() {
+        return new ProviderInfo(economyAPI.name(), economyAPI.version());
+    }
+
     public static EconomyAPI getProvider() {
         return economyAPI;
     }
@@ -65,55 +72,60 @@ public class ReserveListener extends EconomyAdapter {
 
     @EventHandler
     public void onAmountCheck(CurrencyAmountEvent event) {
-        if (!provided() || !event.getAmount().equals(BigDecimal.ZERO)) {
+        if (!provided() || event.wasHandled() || !event.getAmount().equals(BigDecimal.ZERO)) {
             return;
         }
         event.setAmount(economyAPI.getHoldings(event.getAccount(), event.getWorld().getName()));
+        event.setHandled(true);
     }
 
     @EventHandler
     public void onCurrencyCheck(CurrencyCheckEvent event) {
-        if (!provided() || event.hasEnough()) {
+        if (!provided() || event.wasHandled() || event.hasEnough()) {
             return;
         }
         event.hasEnough(economyAPI.hasHoldings(event.getAccount(),
                 event.getAmount(),
                 event.getWorld().getName()));
+        event.setHandled(true);
     }
 
     @EventHandler
     public void onAccountCheck(AccountCheckEvent event) {
-        if (!provided() || event.hasAccount()) {
+        if (!provided() || event.wasHandled() || event.hasAccount()) {
             return;
         }
         event.hasAccount(economyAPI.hasAccount(event.getAccount()));
+        event.setHandled(true);
     }
 
     @EventHandler
     public void onCurrencyFormat(CurrencyFormatEvent event) {
-        if (!event.getFormattedAmount().isEmpty()) {
+        if ( event.wasHandled() || !event.getFormattedAmount().isEmpty()) {
             return;
         }
 
         if (provided()) {
-            event.setFormattedAmount(economyAPI.format(event.getAmount()));
+            String formatted = economyAPI.format(event.getAmount());
+            event.setFormattedAmount(Properties.STRIP_PRICE_COLORS ? ChatColor.stripColor(formatted) : formatted);
+            event.setHandled(true);
         }
     }
 
     @EventHandler
     public void onCurrencyAdd(CurrencyAddEvent event) {
-        if (!provided() || event.isAdded()) {
+        if (!provided() || event.wasHandled()) {
             return;
         }
-        event.setAdded(economyAPI.addHoldings(event.getTarget(), event.getAmount(), event.getWorld().getName()));
+        event.setHandled(economyAPI.addHoldings(event.getTarget(), event.getAmount(), event.getWorld().getName()));
     }
 
     @EventHandler
     public void onCurrencySubtraction(CurrencySubtractEvent event) {
-        if (!provided() || event.isSubtracted()) {
+        if (!provided() || event.wasHandled()) {
             return;
         }
-        event.setSubtracted(economyAPI.removeHoldings(event.getTarget(), event.getAmount(), event.getWorld().getName()));
+        event.setHandled(economyAPI.removeHoldings(event.getTarget(), event.getAmount(), event.getWorld().getName()));
     }
 
     @EventHandler
@@ -123,7 +135,7 @@ public class ReserveListener extends EconomyAdapter {
 
     @EventHandler
     public void onCurrencyHoldCheck(CurrencyHoldEvent event) {
-        if (event.getAccount() == null || !transactionCanFail()) {
+        if (!provided() || event.getAccount() == null || event.wasHandled() || !transactionCanFail() || event.canHold()) {
             return;
         }
 
@@ -134,5 +146,6 @@ public class ReserveListener extends EconomyAdapter {
         }
 
         event.canHold(economyAPI.canAddHoldings(event.getAccount(), event.getAmount(), world));
+        event.setHandled(true);
     }
 }

src/main/java/com/Acrobot/ChestShop/Listeners/Economy/Plugins/VaultListener.java

@@ -5,16 +5,19 @@ import java.util.logging.Level;
 
 import javax.annotation.Nullable;
 
+import com.Acrobot.ChestShop.Configuration.Properties;
 import com.Acrobot.ChestShop.Listeners.Economy.EconomyAdapter;
 import net.milkbowl.vault.economy.Economy;
 import net.milkbowl.vault.economy.EconomyResponse;
 
 import org.bukkit.Bukkit;
+import org.bukkit.ChatColor;
 import org.bukkit.OfflinePlayer;
 import org.bukkit.World;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.server.ServiceRegisterEvent;
 import org.bukkit.event.server.ServiceUnregisterEvent;
+import org.bukkit.plugin.Plugin;
 import org.bukkit.plugin.RegisteredServiceProvider;
 
 import com.Acrobot.ChestShop.ChestShop;
@@ -35,6 +38,7 @@ import com.Acrobot.ChestShop.Events.Economy.CurrencyTransferEvent;
 public class VaultListener extends EconomyAdapter {
     private RegisteredServiceProvider<Economy> rsp;
     private static Economy provider;
+    private Plugin providingPlugin;
 
     private VaultListener() {
         updateEconomyProvider();
@@ -45,6 +49,7 @@ public class VaultListener extends EconomyAdapter {
 
         if (rsp != null) {
             provider = rsp.getProvider();
+            providingPlugin = rsp.getPlugin();
             ChestShop.getBukkitLogger().log(Level.INFO, "Using " + provider.getName() + " as the Economy provider now.");
         }
     }
@@ -58,6 +63,11 @@ public class VaultListener extends EconomyAdapter {
         return true;
     }
 
+    @Override
+    public ProviderInfo getProviderInfo() {
+        return new ProviderInfo(provider.getName(), providingPlugin.getDescription().getVersion());
+    }
+
     public static Economy getProvider() { return provider; }
 
     public boolean transactionCanFail() {
@@ -99,7 +109,7 @@ public class VaultListener extends EconomyAdapter {
 
     @EventHandler
     public void onAmountCheck(CurrencyAmountEvent event) {
-        if (!checkSetup() || !event.getAmount().equals(BigDecimal.ZERO)) {
+        if (!checkSetup() || event.wasHandled() || !event.getAmount().equals(BigDecimal.ZERO)) {
             return;
         }
 
@@ -118,6 +128,7 @@ public class VaultListener extends EconomyAdapter {
             if (balance > Double.MAX_VALUE) {
                 balance = Double.MAX_VALUE;
             }
+            event.setHandled(true);
         } else {
             ChestShop.getBukkitLogger().log(Level.WARNING, "The server could not get the OfflinePlayer with the UUID " + event.getAccount() + " to check balance?");
         }
@@ -127,7 +138,7 @@ public class VaultListener extends EconomyAdapter {
 
     @EventHandler
     public void onCurrencyCheck(CurrencyCheckEvent event) {
-        if (!checkSetup() || event.hasEnough()) {
+        if (!checkSetup() || event.wasHandled() || event.hasEnough()) {
             return;
         }
 
@@ -136,8 +147,9 @@ public class VaultListener extends EconomyAdapter {
 
         try {
             event.hasEnough(lastSeen != null && provider.has(lastSeen, world.getName(), event.getAmount().doubleValue()));
+            event.setHandled(true);
         } catch (Exception e) {
-            ChestShop.getBukkitLogger().log(Level.WARNING, "Could not check if account of " + lastSeen.getUniqueId() + "/" + lastSeen.getName() + " has " + event.getDoubleAmount() + "." +
+            ChestShop.getBukkitLogger().log(Level.WARNING, "Could not check if account of " + lastSeen.getUniqueId() + "/" + lastSeen.getName() + " has " + event.getAmount() + "." +
                     "This is probably due to https://github.com/MilkBowl/Vault/issues/746 and has to be fixed in either Vault directly or your economy plugin." +
                     "If you are sure it's not this issue then please report the following error.", e);
         }
@@ -145,7 +157,7 @@ public class VaultListener extends EconomyAdapter {
 
     @EventHandler
     public void onAccountCheck(AccountCheckEvent event) {
-        if (!checkSetup() || event.hasAccount()) {
+        if (!checkSetup() || event.wasHandled() || event.hasAccount()) {
             return;
         }
 
@@ -155,6 +167,7 @@ public class VaultListener extends EconomyAdapter {
 
         try {
             event.hasAccount(lastSeen != null && provider.hasAccount(lastSeen, world.getName()));
+            event.setHandled(true);
         } catch (Exception e) {
             ChestShop.getBukkitLogger().log(Level.WARNING, "Could not check account balance of "+ lastSeen.getUniqueId() + "/" + lastSeen.getName() + "." +
                     "This is probably due to https://github.com/MilkBowl/Vault/issues/746 and has to be fixed in either Vault directly or your economy plugin." +
@@ -164,17 +177,18 @@ public class VaultListener extends EconomyAdapter {
 
     @EventHandler
     public void onCurrencyFormat(CurrencyFormatEvent event) {
-        if (!checkSetup() || !event.getFormattedAmount().isEmpty()) {
+        if (!checkSetup() || event.wasHandled() || !event.getFormattedAmount().isEmpty()) {
             return;
         }
 
-        String formatted = provider.format(event.getDoubleAmount());
-        event.setFormattedAmount(formatted);
+        String formatted = provider.format(event.getAmount().doubleValue());
+        event.setFormattedAmount(Properties.STRIP_PRICE_COLORS ? ChatColor.stripColor(formatted) : formatted);
+        event.setHandled(true);
     }
 
     @EventHandler
     public void onCurrencyAdd(CurrencyAddEvent event) {
-        if (!checkSetup() || event.isAdded()) {
+        if (!checkSetup() || event.wasHandled()) {
             return;
         }
 
@@ -185,7 +199,7 @@ public class VaultListener extends EconomyAdapter {
         if (lastSeen != null) {
             try {
                 EconomyResponse response = provider.depositPlayer(lastSeen, world.getName(), event.getAmount().doubleValue());
-                event.setAdded(response.type == EconomyResponse.ResponseType.SUCCESS);
+                event.setHandled(response.type == EconomyResponse.ResponseType.SUCCESS);
             } catch (Exception e) {
                 ChestShop.getBukkitLogger().log(Level.WARNING, "Could not add money to account of " + lastSeen.getUniqueId() + "/" + lastSeen.getName() + "." +
                         "This is probably due to https://github.com/MilkBowl/Vault/issues/746 and has to be fixed in either Vault directly or your economy plugin." +
@@ -198,7 +212,7 @@ public class VaultListener extends EconomyAdapter {
 
     @EventHandler
     public void onCurrencySubtraction(CurrencySubtractEvent event) {
-        if (!checkSetup() || event.isSubtracted()) {
+        if (!checkSetup() || event.wasHandled()) {
             return;
         }
 
@@ -209,7 +223,7 @@ public class VaultListener extends EconomyAdapter {
         if (lastSeen != null) {
             try {
                 EconomyResponse response = provider.withdrawPlayer(lastSeen, world.getName(), event.getAmount().doubleValue());
-                event.setSubtracted(response.type == EconomyResponse.ResponseType.SUCCESS);
+                event.setHandled(response.type == EconomyResponse.ResponseType.SUCCESS);
             } catch (Exception e) {
                 ChestShop.getBukkitLogger().log(Level.WARNING, "Could not add money to account of " + lastSeen.getUniqueId() + "/" + lastSeen.getName() + "." +
                         "This is probably due to https://github.com/MilkBowl/Vault/issues/746 and has to be fixed in either Vault directly or your economy plugin." +
@@ -230,7 +244,7 @@ public class VaultListener extends EconomyAdapter {
 
     @EventHandler
     public void onCurrencyHoldCheck(CurrencyHoldEvent event) {
-        if (!checkSetup() || event.getAccount() == null || !transactionCanFail()) {
+        if (!checkSetup() || event.wasHandled() || event.getAccount() == null || !transactionCanFail() || event.canHold()) {
             return;
         }
 
@@ -254,10 +268,12 @@ public class VaultListener extends EconomyAdapter {
 
             if (!response.transactionSuccess()) {
                 event.canHold(false);
+                event.setHandled(true);
                 return;
             }
 
             provider.withdrawPlayer(lastSeen, world, event.getAmount().doubleValue());
+            event.setHandled(true);
         } catch (Exception e) {
             ChestShop.getBukkitLogger().log(Level.WARNING, "Could not check if account of " + lastSeen.getUniqueId() + "/" + lastSeen.getName() + " can hold " + event.getAmount() + "." +
                     "This is probably due to https://github.com/MilkBowl/Vault/issues/746 and has to be fixed in either Vault directly or your economy plugin." +

src/main/java/com/Acrobot/ChestShop/Listeners/Economy/ServerAccountCorrector.java

@@ -1,6 +1,5 @@
 package com.Acrobot.ChestShop.Listeners.Economy;
 
-import com.Acrobot.ChestShop.ChestShop;
 import com.Acrobot.ChestShop.Database.Account;
 import com.Acrobot.ChestShop.Events.Economy.*;
 import com.Acrobot.ChestShop.UUIDs.NameManager;
@@ -27,15 +26,12 @@ public class ServerAccountCorrector implements Listener {
         Account account = NameManager.getServerEconomyAccount();
         target = account != null ? account.getUuid() : null;
 
-        event.setAdded(true);
         if (target == null) {
+            event.setHandled(true);
             return;
         }
 
-        CurrencyAddEvent currencyAddEvent = new CurrencyAddEvent(event.getAmount(), target, event.getWorld());
-        ChestShop.callEvent(currencyAddEvent);
-
-        event.setAdded(currencyAddEvent.isAdded());
+        event.setTarget(target);
     }
 
     @EventHandler(priority = EventPriority.LOWEST)
@@ -49,15 +45,12 @@ public class ServerAccountCorrector implements Listener {
         Account account = NameManager.getServerEconomyAccount();
         target = account != null ? account.getUuid() : null;
 
-        event.setSubtracted(true);
         if (target == null) {
+            event.setHandled(true);
             return;
         }
 
-        CurrencySubtractEvent currencySubtractEvent = new CurrencySubtractEvent(event.getAmount(), target, event.getWorld());
-        ChestShop.callEvent(currencySubtractEvent);
-
-        event.setSubtracted(currencySubtractEvent.isSubtracted());
+        event.setTarget(target);
     }
 
     @EventHandler(priority = EventPriority.LOWEST)
@@ -75,16 +68,7 @@ public class ServerAccountCorrector implements Listener {
             return;
         }
 
-        CurrencyTransferEvent currencyTransferEvent = new CurrencyTransferEvent(
-                event.getAmountSent(),
-                event.getAmountReceived(),
-                event.getInitiator(),
-                partner,
-                event.getDirection()
-        );
-        ChestShop.callEvent(currencyTransferEvent);
-        event.setTransferred(currencyTransferEvent.hasBeenTransferred());
-        return;
+        event.setPartner(partner);
     }
 
     @EventHandler(priority = EventPriority.LOWEST)
@@ -103,10 +87,7 @@ public class ServerAccountCorrector implements Listener {
             return;
         }
 
-        CurrencyCheckEvent currencyCheckEvent = new CurrencyCheckEvent(event.getAmount(), target, event.getWorld());
-        ChestShop.callEvent(currencyCheckEvent);
-
-        event.hasEnough(currencyCheckEvent.hasEnough());
+        event.setAccount(target);
     }
 
     @EventHandler(priority = EventPriority.LOWEST)
@@ -117,8 +98,15 @@ public class ServerAccountCorrector implements Listener {
             return;
         }
 
-        event.canHold(true);
-        event.setAccount(null);
+        Account account = NameManager.getServerEconomyAccount();
+        target = account != null ? account.getUuid() : null;
+
+        if (target == null) {
+            event.canHold(true);
+            return;
+        }
+
+        event.setAccount(target);
     }
 
     @EventHandler(priority = EventPriority.LOWEST)
@@ -137,9 +125,6 @@ public class ServerAccountCorrector implements Listener {
             return;
         }
 
-        CurrencyAmountEvent currencyAmountEvent = new CurrencyAmountEvent(target, event.getWorld());
-        ChestShop.callEvent(currencyAmountEvent);
-
-        event.setAmount(currencyAmountEvent.getAmount());
+        event.setAccount(target);
     }
 }

src/main/java/com/Acrobot/ChestShop/Listeners/Economy/TaxModule.java

@@ -4,8 +4,6 @@ import com.Acrobot.ChestShop.ChestShop;
 import com.Acrobot.ChestShop.Configuration.Properties;
 import com.Acrobot.ChestShop.Events.Economy.CurrencyAddEvent;
 import com.Acrobot.ChestShop.Events.Economy.CurrencyTransferEvent;
-import com.Acrobot.ChestShop.Events.PreTransactionEvent;
-import com.Acrobot.ChestShop.Events.TransactionEvent;
 import com.Acrobot.ChestShop.Permission;
 import com.Acrobot.ChestShop.UUIDs.NameManager;
 import org.bukkit.event.EventHandler;
@@ -19,9 +17,11 @@ import java.util.UUID;
  * @author Acrobot
  */
 public class TaxModule implements Listener {
+    private static final String TAX_RECEIVED_MESSAGE = "Applied a tax of %1$f percent (%2$.2f) to the received amount for a resulting price of %3$.2f";
+    private static final String TAX_SENT_MESSAGE = "Reduced buy price by tax of %1$f percent (%2$.2f) for a resulting price of %3$.2f as the buyer has the buy tax bypass permission";
 
-    private static float getTax(UUID partner) {
-        float taxAmount = NameManager.isAdminShop(partner) || NameManager.isServerEconomyAccount(partner)
+    private static double getTax(UUID partner) {
+        double taxAmount = NameManager.isAdminShop(partner) || NameManager.isServerEconomyAccount(partner)
                 ? Properties.SERVER_TAX_AMOUNT : Properties.TAX_AMOUNT;
 
         if (taxAmount == 0) {
@@ -31,17 +31,17 @@ public class TaxModule implements Listener {
         return taxAmount;
     }
 
-    private static BigDecimal getTaxAmount(BigDecimal price, float taxAmount) {
+    private static BigDecimal getTaxAmount(BigDecimal price, double taxAmount) {
         return price.multiply(BigDecimal.valueOf(taxAmount)).divide(BigDecimal.valueOf(100), Properties.PRICE_PRECISION, BigDecimal.ROUND_HALF_UP);
     }
 
     @EventHandler(priority = EventPriority.LOW)
     public static void onCurrencyTransfer(CurrencyTransferEvent event) {
-        if (event.hasBeenTransferred()) {
+        if (event.wasHandled()) {
             return;
         }
 
-        float taxAmount = getTax(event.getPartner());
+        double taxAmount = getTax(event.getPartner());
         if (taxAmount == 0) {
             return;
         }
@@ -49,17 +49,28 @@ public class TaxModule implements Listener {
         if (!Permission.has(event.getInitiator(), event.getDirection() == CurrencyTransferEvent.Direction.PARTNER ? Permission.NO_BUY_TAX : Permission.NO_SELL_TAX)) {
             if (!NameManager.isServerEconomyAccount(event.getReceiver())) {
                 BigDecimal tax = getTaxAmount(event.getAmountReceived(), taxAmount);
-                event.setAmountReceived(event.getAmountReceived().subtract(tax));
+                BigDecimal taxedAmount = event.getAmountReceived().subtract(tax);
+                event.setAmountReceived(taxedAmount);
                 if (NameManager.getServerEconomyAccount() != null) {
                     ChestShop.callEvent(new CurrencyAddEvent(
                             tax,
                             NameManager.getServerEconomyAccount().getUuid(),
                             event.getWorld()));
                 }
+                ChestShop.getShopLogger().info(String.format(TAX_RECEIVED_MESSAGE, taxAmount, tax, taxedAmount));
             }
         } else if (event.getDirection() == CurrencyTransferEvent.Direction.PARTNER && Permission.has(event.getInitiator(), Permission.NO_BUY_TAX)) {
-            event.setAmountSent(event.getAmountSent().subtract(getTaxAmount(event.getAmountSent(), taxAmount)));
-            event.setAmountReceived(event.getAmountReceived().subtract(getTaxAmount(event.getAmountReceived(), taxAmount)));
+            // Reduce paid amount as the buyer has permission to not pay taxes
+            BigDecimal taxSent = getTaxAmount(event.getAmountSent(), taxAmount);
+            BigDecimal taxedSentAmount = event.getAmountSent().subtract(taxSent);
+            event.setAmountSent(taxedSentAmount);
+            ChestShop.getShopLogger().info(String.format(TAX_SENT_MESSAGE, taxAmount, taxSent, taxedSentAmount));
+
+            // Reduce the amount that the seller receives anyways even though tax wasn't paid as that shouldn't make a difference for the seller
+            BigDecimal taxReceived = getTaxAmount(event.getAmountReceived(), taxAmount);
+            BigDecimal taxedReceivedAmount = event.getAmountReceived().subtract(taxReceived);
+            event.setAmountReceived(taxedReceivedAmount);
+            ChestShop.getShopLogger().info(String.format(TAX_RECEIVED_MESSAGE, taxAmount, taxReceived, taxedReceivedAmount));
         }
     }
-}
+}

src/main/java/com/Acrobot/ChestShop/Listeners/GarbageTextListener.java

@@ -16,15 +16,17 @@ public class GarbageTextListener implements Listener {
     public static void filterGarbage(SignChangeEvent event) {
         for (int i = 0; i < 4; ++i) {
             String line = event.getLine(i);
-            StringBuilder output = new StringBuilder(line.length());
+            if (line != null) {
+                StringBuilder output = new StringBuilder(line.length());
 
-            for (char character : line.toCharArray()) {
-                if (character < 0xF700 || character > 0xF747) {
-                    output.append(character);
+                for (char character : line.toCharArray()) {
+                    if (character < 0xF700 || character > 0xF747) {
+                        output.append(character);
+                    }
                 }
-            }
 
-            event.setLine(i, output.toString());
+                event.setLine(i, output.toString());
+            }
         }
     }
 }

src/main/java/com/Acrobot/ChestShop/Listeners/Item/ItemMoveListener.java

@@ -1,11 +1,17 @@
 package com.Acrobot.ChestShop.Listeners.Item;
 
+import com.Acrobot.ChestShop.Configuration.Properties;
+import com.Acrobot.ChestShop.Listeners.Modules.StockCounterModule;
 import com.Acrobot.ChestShop.Signs.ChestShopSign;
 import org.bukkit.block.BlockState;
+import org.bukkit.block.Hopper;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.EventPriority;
 import org.bukkit.event.Listener;
 import org.bukkit.event.inventory.InventoryMoveItemEvent;
+import org.bukkit.inventory.InventoryHolder;
+
+import static com.Acrobot.Breeze.Utils.ImplementationAdapter.getHolder;
 
 /**
  * @author Acrobot
@@ -14,14 +20,15 @@ public class ItemMoveListener implements Listener {
 
     @EventHandler(ignoreCancelled = true, priority = EventPriority.HIGHEST)
     public static void onItemMove(InventoryMoveItemEvent event) {
-        if (event.getSource() == null || event.getDestination().getHolder() instanceof BlockState) {
-            return;
-        }
+        InventoryHolder destinationHolder = getHolder(event.getDestination(), false);
+        InventoryHolder sourceHolder = getHolder(event.getSource(), false);
 
-        if (!ChestShopSign.isShopBlock(event.getSource().getHolder())) {
-            return;
+        if (!(destinationHolder instanceof BlockState) && ChestShopSign.isShopBlock(sourceHolder)) {
+            event.setCancelled(true);
+        } else if (Properties.USE_STOCK_COUNTER && ChestShopSign.isShopBlock(destinationHolder) && sourceHolder instanceof Hopper) {
+            StockCounterModule.updateCounterOnItemMoveEvent(event.getItem(), destinationHolder);
         }
-
-        event.setCancelled(true);
     }
+
+
 }

src/main/java/com/Acrobot/ChestShop/Listeners/Item/ItemStringListener.java

@@ -0,0 +1,18 @@
+package com.Acrobot.ChestShop.Listeners.Item;
+
+import com.Acrobot.Breeze.Utils.MaterialUtil;
+import com.Acrobot.ChestShop.Events.ItemStringQueryEvent;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.EventPriority;
+import org.bukkit.event.Listener;
+
+public class ItemStringListener implements Listener {
+
+    @EventHandler(priority = EventPriority.NORMAL)
+    public static void calculateItemString(ItemStringQueryEvent event) {
+        if (event.getItemString() == null) {
+            event.setItemString(MaterialUtil.getName(event.getItem(), event.getMaxWidth()));
+        }
+    }
+
+}

src/main/java/com/Acrobot/ChestShop/Listeners/ItemInfoListener.java

@@ -1,16 +1,30 @@
 package com.Acrobot.ChestShop.Listeners;
 
+import com.Acrobot.Breeze.Utils.StringUtil;
+import com.Acrobot.ChestShop.ChestShop;
+import com.Acrobot.ChestShop.Commands.ItemInfo;
 import com.Acrobot.ChestShop.Events.ItemInfoEvent;
 import org.bukkit.ChatColor;
-import org.bukkit.Material;
+import org.bukkit.Color;
+import org.bukkit.NamespacedKey;
 import org.bukkit.command.CommandSender;
 import org.bukkit.enchantments.Enchantment;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
 import org.bukkit.inventory.ItemStack;
+import org.bukkit.inventory.meta.AxolotlBucketMeta;
+import org.bukkit.inventory.meta.BookMeta;
+import org.bukkit.inventory.meta.BundleMeta;
+import org.bukkit.inventory.meta.CrossbowMeta;
 import org.bukkit.inventory.meta.EnchantmentStorageMeta;
 import org.bukkit.inventory.meta.ItemMeta;
-import org.bukkit.potion.Potion;
+import org.bukkit.inventory.meta.KnowledgeBookMeta;
+import org.bukkit.inventory.meta.LeatherArmorMeta;
+import org.bukkit.inventory.meta.MapMeta;
+import org.bukkit.inventory.meta.PotionMeta;
+import org.bukkit.inventory.meta.Repairable;
+import org.bukkit.inventory.meta.TropicalFishBucketMeta;
+import org.bukkit.map.MapView;
 import org.bukkit.potion.PotionEffect;
 
 import java.util.Map;
@@ -18,21 +32,99 @@ import java.util.Map;
 import static com.Acrobot.Breeze.Utils.NumberUtil.toRoman;
 import static com.Acrobot.Breeze.Utils.NumberUtil.toTime;
 import static com.Acrobot.Breeze.Utils.StringUtil.capitalizeFirstLetter;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_axolotl_variant;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_book;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_book_generation;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_bundle_items;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_crossbow_projectile;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_crossbow_projectiles;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_leather_color;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_lore;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_map_location;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_map_view;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_recipes;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_repaircost;
+import static com.Acrobot.ChestShop.Configuration.Messages.iteminfo_tropical_fish;
 
 /**
  * @author Acrobot
  */
 public class ItemInfoListener implements Listener {
+
+    // Register version dependent listeners
+    static {
+        try {
+            Class.forName("org.bukkit.inventory.meta.AxolotlBucketMeta");
+            ChestShop.registerListener(new Listener() {
+                @EventHandler
+                public void addAxolotlInfo(ItemInfoEvent event) {
+                    if (event.getItem().hasItemMeta()) {
+                        ItemMeta meta = event.getItem().getItemMeta();
+                        if (meta instanceof AxolotlBucketMeta) {
+                            iteminfo_axolotl_variant.send(event.getSender(), "variant", capitalizeFirstLetter(((AxolotlBucketMeta) meta).getVariant().name(), '_'));
+                        }
+                    }
+                }
+            });
+        } catch (ClassNotFoundException | NoClassDefFoundError ignored) {}
+        try {
+            Class.forName("org.bukkit.inventory.meta.BundleMeta");
+            ChestShop.registerListener(new Listener() {
+                @EventHandler
+                public void addAxolotlInfo(ItemInfoEvent event) {
+                    if (event.getItem().hasItemMeta()) {
+                        ItemMeta meta = event.getItem().getItemMeta();
+                        if (meta instanceof BundleMeta) {
+                            iteminfo_bundle_items.send(event.getSender(), "itemcount", String.valueOf(((BundleMeta) meta).getItems().size()));
+                        }
+                    }
+                }
+            });
+        } catch (ClassNotFoundException | NoClassDefFoundError ignored) {}
+        try {
+            Class.forName("org.bukkit.inventory.meta.CrossbowMeta");
+            ChestShop.registerListener(new Listener() {
+                @EventHandler
+                public void addCrossBowInfo(ItemInfoEvent event) {
+                    if (event.getItem().hasItemMeta()) {
+                        ItemMeta meta = event.getItem().getItemMeta();
+                        if (meta instanceof CrossbowMeta && ((CrossbowMeta) meta).hasChargedProjectiles()) {
+                            iteminfo_crossbow_projectiles.send(event.getSender());
+                            for (ItemStack chargedProjectile : ((CrossbowMeta) meta).getChargedProjectiles()) {
+                                ItemInfo.sendItemName(event.getSender(), chargedProjectile, iteminfo_crossbow_projectile);
+                                ChestShop.callEvent(new ItemInfoEvent(event.getSender(), chargedProjectile));
+                                event.getSender().sendMessage(ChatColor.GRAY + "---");
+                            }
+                        }
+                    }
+                }
+            });
+        } catch (ClassNotFoundException | NoClassDefFoundError ignored) {}
+    }
+
+    @EventHandler
+    public static void addRepairCost(ItemInfoEvent event) {
+        if (event.getItem().hasItemMeta()) {
+            ItemMeta meta = event.getItem().getItemMeta();
+            if (meta instanceof Repairable && ((Repairable) meta).getRepairCost() > 0) {
+                iteminfo_repaircost.send(event.getSender(), "cost", String.valueOf(((Repairable) meta).getRepairCost()));
+            }
+        }
+    }
+
     @EventHandler
     public static void addEnchantment(ItemInfoEvent event) {
         ItemStack item = event.getItem();
-        ItemMeta meta = item.getItemMeta();
         CommandSender sender = event.getSender();
 
         for (Map.Entry<Enchantment, Integer> enchantment : item.getEnchantments().entrySet()) {
             sender.sendMessage(ChatColor.AQUA + capitalizeFirstLetter(enchantment.getKey().getName(), '_') + ' ' + toRoman(enchantment.getValue()));
         }
 
+        if (!item.hasItemMeta()) {
+            return;
+        }
+        ItemMeta meta = item.getItemMeta();
         if (meta instanceof EnchantmentStorageMeta) {
             for (Map.Entry<Enchantment, Integer> enchantment : ((EnchantmentStorageMeta) meta).getStoredEnchants().entrySet()) {
                 sender.sendMessage(ChatColor.YELLOW + capitalizeFirstLetter(enchantment.getKey().getName(), '_') + ' ' + toRoman(enchantment.getValue()));
@@ -40,44 +132,133 @@ public class ItemInfoListener implements Listener {
         }
     }
 
+    @EventHandler
+    public static void addLeatherColor(ItemInfoEvent event) {
+        if (event.getItem().hasItemMeta()) {
+            ItemMeta meta = event.getItem().getItemMeta();
+            if (meta instanceof LeatherArmorMeta) {
+                Color color = ((LeatherArmorMeta) meta).getColor();
+                iteminfo_leather_color.send(event.getSender(),
+                        "colorred", String.valueOf(color.getRed()),
+                        "colorgreen", String.valueOf(color.getGreen()),
+                        "colorblue", String.valueOf(color.getBlue()),
+                        "colorhex", Integer.toHexString(color.asRGB())
+                );
+            }
+        }
+    }
+
+    @EventHandler
+    public static void addRecipes(ItemInfoEvent event) {
+        if (event.getItem().hasItemMeta()) {
+            ItemMeta meta = event.getItem().getItemMeta();
+            if (meta instanceof KnowledgeBookMeta && !((KnowledgeBookMeta) meta).getRecipes().isEmpty()) {
+                iteminfo_recipes.send(event.getSender());
+                for (NamespacedKey recipe : ((KnowledgeBookMeta) meta).getRecipes()) {
+                    event.getSender().sendMessage(ChatColor.GRAY + recipe.toString());
+                }
+            }
+        }
+    }
+
+    @EventHandler
+    public static void addTropicalFishInfo(ItemInfoEvent event) {
+        if (event.getItem().hasItemMeta()) {
+            ItemMeta meta = event.getItem().getItemMeta();
+            if (meta instanceof TropicalFishBucketMeta && ((TropicalFishBucketMeta) meta).hasVariant()) {
+                iteminfo_tropical_fish.send(event.getSender(),
+                        "pattern", capitalizeFirstLetter(((TropicalFishBucketMeta) meta).getPattern().name()),
+                        "patterncolor", capitalizeFirstLetter(((TropicalFishBucketMeta) meta).getPatternColor().name()),
+                        "bodycolor", capitalizeFirstLetter(((TropicalFishBucketMeta) meta).getBodyColor().name())
+                );
+            }
+        }
+    }
+
+    @EventHandler
+    public static void addMapInfo(ItemInfoEvent event) {
+        if (event.getItem().hasItemMeta()) {
+            ItemMeta meta = event.getItem().getItemMeta();
+            if (meta instanceof MapMeta) {
+                if (((MapMeta) meta).getMapView() != null) {
+                    MapView mapView = ((MapMeta) meta).getMapView();
+                    iteminfo_map_view.send(event.getSender(),
+                            "id", String.valueOf(mapView.getId()),
+                            "x", String.valueOf(mapView.getCenterX()),
+                            "z", String.valueOf(mapView.getCenterZ()),
+                            "world", mapView.getWorld() != null ? String.valueOf(mapView.getWorld().getName()) : "unknown",
+                            "scale", capitalizeFirstLetter(mapView.getScale().name(), '_'),
+                            "locked", String.valueOf(mapView.isLocked())
+                    );
+                }
+                if (((MapMeta) meta).hasLocationName()) {
+                    iteminfo_map_location.send(event.getSender(), "location", String.valueOf(((MapMeta) meta).getLocationName()));
+                }
+            }
+        }
+    }
+
     @EventHandler
     public static void addPotionInfo(ItemInfoEvent event) {
         ItemStack item = event.getItem();
-
-        if (item.getType() != Material.POTION || item.getDurability() == 0) {
+        if (!item.hasItemMeta()) {
             return;
         }
 
-        Potion potion;
-
-        try {
-            potion = Potion.fromItemStack(item);
-        } catch (IllegalArgumentException ex) {
+        ItemMeta meta = item.getItemMeta();
+        if (!(meta instanceof PotionMeta)) {
             return;
         }
 
-        StringBuilder message = new StringBuilder(50);
-
-        message.append(ChatColor.GRAY);
-
-        if (potion.getType() == null) {
-            return;
-        }
-
-        if (potion.isSplash()) {
-            message.append("Splash ");
-        }
-
-        message.append("Potion of ");
-        message.append(capitalizeFirstLetter(potion.getType().name(), '_')).append(' ');
-        message.append(toRoman(potion.getLevel()));
+        PotionMeta potionMeta = (PotionMeta) meta;
 
         CommandSender sender = event.getSender();
 
-        sender.sendMessage(message.toString());
+        if (potionMeta.getBasePotionType() != null) {
+            StringBuilder message = new StringBuilder(50);
 
-        for (PotionEffect effect : potion.getEffects()) {
-            sender.sendMessage(ChatColor.DARK_GRAY + capitalizeFirstLetter(effect.getType().getName(), '_') + ' ' + toTime(effect.getDuration() / 20));
+            message.append(ChatColor.GRAY);
+
+            message.append(capitalizeFirstLetter(item.getType().name(), '_')).append(" of ");
+            message.append(capitalizeFirstLetter(potionMeta.getBasePotionType().getKey().getKey(), '_')).append(' ');
+
+            sender.sendMessage(message.toString());
+        }
+
+        for (PotionEffect effect : potionMeta.getCustomEffects()) {
+            sender.sendMessage(ChatColor.DARK_GRAY + capitalizeFirstLetter(effect.getType().getKey().getKey(), '_')
+                    + ' ' + (effect.getAmplifier() + 1) + ' ' + toTime(effect.getDuration() / 20));
+        }
+    }
+
+    @EventHandler
+    public static void addBookInfo(ItemInfoEvent event) {
+        if (!event.getItem().hasItemMeta()) {
+            return;
+        }
+        ItemMeta meta = event.getItem().getItemMeta();
+        if (meta instanceof BookMeta) {
+            BookMeta book = (BookMeta) meta;
+            iteminfo_book.send(event.getSender(),
+                    "title", book.getTitle(),
+                    "author", book.getAuthor(),
+                    "pages", String.valueOf(book.getPageCount())
+            );
+            if (book.hasGeneration()) {
+                iteminfo_book_generation.send(event.getSender(),
+                        "generation", StringUtil.capitalizeFirstLetter(book.getGeneration().name(), '_')
+                );
+            }
+        }
+    }
+
+    @EventHandler
+    public static void addLoreInfo(ItemInfoEvent event) {
+        if (event.getItem().hasItemMeta()) {
+            ItemMeta meta = event.getItem().getItemMeta();
+            if (meta.hasLore()) {
+                iteminfo_lore.send(event.getSender(), "lore", String.join("\n", meta.getLore()));
+            }
         }
     }
 }

src/main/java/com/Acrobot/ChestShop/Listeners/Modules/DiscountModule.java

@@ -2,9 +2,10 @@ package com.Acrobot.ChestShop.Listeners.Modules;
 
 import com.Acrobot.Breeze.Utils.PriceUtil;
 import com.Acrobot.ChestShop.ChestShop;
-import com.Acrobot.ChestShop.Containers.AdminInventory;
+import com.Acrobot.ChestShop.Events.ChestShopReloadEvent;
 import com.Acrobot.ChestShop.Events.PreTransactionEvent;
 import com.Acrobot.ChestShop.Permission;
+import com.Acrobot.ChestShop.Signs.ChestShopSign;
 import com.Acrobot.ChestShop.UUIDs.NameManager;
 import org.bukkit.configuration.file.YamlConfiguration;
 import org.bukkit.entity.Player;
@@ -16,18 +17,23 @@ import java.io.IOException;
 import java.math.BigDecimal;
 import java.util.HashSet;
 import java.util.Set;
+import java.util.logging.Level;
 
 import static com.Acrobot.ChestShop.Events.TransactionEvent.TransactionType.BUY;
-import static com.Acrobot.ChestShop.Signs.ChestShopSign.PRICE_LINE;
 
 /**
  * @author Acrobot
  */
 public class DiscountModule implements Listener {
+    private static final String DISCOUNT_MESSAGE = "Applied a discount of %1$f percent for a resulting price of %2$.2f";
     private YamlConfiguration config;
     private Set<String> groupList = new HashSet<String>();
 
     public DiscountModule() {
+        load();
+    }
+
+    private void load() {
         config = YamlConfiguration.loadConfiguration(ChestShop.loadFile("discounts.yml"));
 
         config.options().header("This file is for discount management. You are able to do that:\n" +
@@ -39,12 +45,17 @@ public class DiscountModule implements Listener {
         try {
             config.save(ChestShop.loadFile("discounts.yml"));
         } catch (IOException e) {
-            e.printStackTrace();
+            ChestShop.getBukkitLogger().log(Level.SEVERE, "Error while loading discounts config", e);
         }
 
         groupList = config.getKeys(false);
     }
 
+    @EventHandler
+    public void onReload(ChestShopReloadEvent event) {
+        load();
+    }
+
     @EventHandler(priority = EventPriority.LOWEST, ignoreCancelled = true)
     public void onPreTransaction(PreTransactionEvent event) {
         if (event.getTransactionType() != BUY || !NameManager.isAdminShop(event.getOwnerAccount().getUuid())) {
@@ -53,13 +64,16 @@ public class DiscountModule implements Listener {
 
         Player client = event.getClient();
 
-        if (!PriceUtil.hasBuyPrice(event.getSign().getLine(PRICE_LINE))) {
+        if (!PriceUtil.hasBuyPrice(ChestShopSign.getPrice(event.getSign()))) {
             return;
         }
 
         for (String group : groupList) {
             if (Permission.has(client, Permission.DISCOUNT + group)) {
-                event.setExactPrice(event.getExactPrice().multiply(BigDecimal.valueOf(config.getDouble(group) / 100)));
+                double discount = config.getDouble(group);
+                BigDecimal discountedPrice = event.getExactPrice().multiply(BigDecimal.valueOf(discount / 100));
+                event.setExactPrice(discountedPrice);
+                ChestShop.getBukkitLogger().info(String.format(DISCOUNT_MESSAGE, discount, discountedPrice));
                 return;
             }
         }

src/main/java/com/Acrobot/ChestShop/Listeners/Modules/ItemAliasModule.java

@@ -0,0 +1,135 @@
+package com.Acrobot.ChestShop.Listeners.Modules;
+
+import com.Acrobot.Breeze.Utils.MaterialUtil;
+import com.Acrobot.ChestShop.ChestShop;
+import com.Acrobot.ChestShop.Events.ChestShopReloadEvent;
+import com.Acrobot.ChestShop.Events.ItemParseEvent;
+import com.Acrobot.ChestShop.Events.ItemStringQueryEvent;
+import com.google.common.collect.BiMap;
+import com.google.common.collect.HashBiMap;
+import org.bukkit.configuration.file.YamlConfiguration;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.EventPriority;
+import org.bukkit.event.Listener;
+
+import java.io.File;
+import java.io.IOException;
+import java.util.Locale;
+import java.util.Map;
+import java.util.logging.Level;
+
+import static com.Acrobot.Breeze.Utils.StringUtil.getMinecraftStringWidth;
+
+/**
+ * @author Acrobot
+ */
+public class ItemAliasModule implements Listener {
+    private YamlConfiguration configuration;
+    /**
+     * Map ChestShop item code -> alias
+     */
+    private BiMap<String, String> aliases;
+
+    public ItemAliasModule() {
+        load();
+    }
+
+    private void load() {
+        File file = new File(ChestShop.getFolder(), "itemAliases.yml");
+
+        configuration = YamlConfiguration.loadConfiguration(file);
+
+        configuration.options().header(
+                "This file specified optional aliases for certain item codes. (Use the full name from /iteminfo)"
+                        + "\nPlease note that these aliases should fit on a sign for it to work properly!"
+        );
+
+        if (!file.exists()) {
+            configuration.addDefault("Item String#3d", "My Cool Item");
+            configuration.addDefault("Other Material#Eg", "Some other Item");
+
+            try {
+                configuration.options().copyDefaults(true);
+                configuration.save(ChestShop.loadFile("itemAliases.yml"));
+            } catch (IOException e) {
+                ChestShop.getBukkitLogger().log(Level.SEVERE, "Error while saving item aliases config", e);
+            }
+        }
+
+        aliases = HashBiMap.create(configuration.getKeys(false).size());
+
+        for (String key : configuration.getKeys(false)) {
+            if (configuration.isString(key)) {
+                aliases.put(key, configuration.getString(key));
+            }
+        }
+    }
+
+    @EventHandler
+    public void onReload(ChestShopReloadEvent event) {
+        load();
+    }
+
+    @EventHandler(priority = EventPriority.LOW)
+    public void onItemParse(ItemParseEvent event) {
+        String code = aliases.inverse().get(event.getItemString());
+        if (code == null) {
+            String[] typeParts = event.getItemString().replaceAll("(?<!^)([A-Z1-9])", "_$1").toUpperCase(Locale.ROOT).split("[ _\\-]");
+            int length = Short.MAX_VALUE;
+            for (Map.Entry<String, String> entry : aliases.entrySet()) {
+                if (entry.getValue().length() < length && entry.getValue().toUpperCase(Locale.ROOT).startsWith(event.getItemString().toUpperCase(Locale.ROOT))) {
+                    length = (short) entry.getValue().length();
+                    code = entry.getKey();
+                } else if (typeParts.length > 1) {
+                    String[] nameParts = entry.getValue().toUpperCase(Locale.ROOT).split("[ _\\-]");
+                    if (typeParts.length == nameParts.length) {
+                        boolean matched = true;
+                        for (int i = 0; i < nameParts.length; i++) {
+                            if (!nameParts[i].startsWith(typeParts[i])) {
+                                matched = false;
+                                break;
+                            }
+                        }
+                        if (matched) {
+                            code = entry.getKey();
+                            break;
+                        }
+                    }
+                }
+            }
+        }
+        if (code != null) {
+            event.setItem(MaterialUtil.getItem(code));
+        }
+    }
+
+    @EventHandler(priority = EventPriority.HIGH)
+    public void onItemStringQuery(ItemStringQueryEvent event) {
+        if (event.getItemString() != null) {
+            String newCode = null;
+
+            if (aliases.containsKey(event.getItemString())) {
+                newCode = aliases.get(event.getItemString());
+            } else if (!event.getItemString().contains("#")) {
+                newCode = aliases.get(event.getItemString().toLowerCase(Locale.ROOT));
+            } else {
+                String[] parts = event.getItemString().split("#", 2);
+                String lowercaseCode = parts[0].toLowerCase(Locale.ROOT) + "#" + parts[1];
+                if (aliases.containsKey(lowercaseCode)) {
+                    newCode = aliases.get(lowercaseCode);
+                }
+            }
+
+            if (newCode != null) {
+                if (event.getMaxWidth() > 0) {
+                    int width = getMinecraftStringWidth(newCode);
+                    if (width > event.getMaxWidth()) {
+                        ChestShop.getBukkitLogger().warning("Can't use configured alias " + newCode + " as it's width (" + width + ") was wider than the allowed max width of " + event.getMaxWidth());
+                        return;
+                    }
+                }
+                event.setItemString(newCode);
+            }
+        }
+    }
+}

src/main/java/com/Acrobot/ChestShop/Listeners/Modules/MetricsModule.java

@@ -0,0 +1,92 @@
+package com.Acrobot.ChestShop.Listeners.Modules;
+
+import com.Acrobot.Breeze.Utils.NumberUtil;
+import com.Acrobot.ChestShop.Events.TransactionEvent;
+import org.bukkit.event.EventHandler;
+import org.bukkit.event.EventPriority;
+import org.bukkit.event.Listener;
+import org.bukkit.inventory.ItemStack;
+
+/**
+ * @author Acrobot
+ */
+public class MetricsModule implements Listener {
+
+    private static final long RESET_MINUTES = 30;
+
+    private static long lastReset = System.currentTimeMillis();
+
+    private static int buyTransactionsLast = -1;
+    private static int sellTransactionsLast = -1;
+    private static long buyTransactionsCurrent = 0;
+    private static long sellTransactionsCurrent = 0;
+
+    private static int boughtItemsLast = -1;
+    private static int soldItemsLast = -1;
+    private static long boughtItemsCurrent = 0;
+    private static long soldItemsCurrent = 0;
+
+    @EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true)
+    public static void onTransaction(final TransactionEvent event) {
+        checkReset();
+        switch (event.getTransactionType()) {
+            case BUY:
+                buyTransactionsCurrent++;
+                for (ItemStack itemStack : event.getStock()) {
+                    boughtItemsCurrent += itemStack.getAmount();
+                }
+                break;
+            case SELL:
+                sellTransactionsCurrent++;
+                for (ItemStack itemStack : event.getStock()) {
+                    soldItemsCurrent += itemStack.getAmount();
+                }
+                break;
+        }
+    }
+
+    public static int getBuyTransactions() {
+        checkReset();
+        return buyTransactionsLast > -1 ? buyTransactionsLast : NumberUtil.toInt(buyTransactionsCurrent);
+    }
+
+    public static int getSellTransactions() {
+        checkReset();
+        return sellTransactionsLast > -1 ? sellTransactionsLast : NumberUtil.toInt(sellTransactionsCurrent);
+    }
+
+    public static int getTotalTransactions() {
+        checkReset();
+        return getBuyTransactions() + getSellTransactions();
+    }
+
+    public static int getBoughtItemsCount() {
+        checkReset();
+        return boughtItemsLast > -1 ? boughtItemsLast : NumberUtil.toInt(boughtItemsCurrent);
+    }
+
+    public static int getSoldItemsCount() {
+        checkReset();
+        return soldItemsLast > -1 ? soldItemsLast : NumberUtil.toInt(soldItemsCurrent);
+    }
+
+    public static int getTotalItemsCount() {
+        checkReset();
+        return getBoughtItemsCount() + getSoldItemsCount();
+    }
+
+    private static void checkReset() {
+        if (lastReset + RESET_MINUTES * 60 * 1000 < System.currentTimeMillis()) {
+            lastReset = System.currentTimeMillis();
+            buyTransactionsLast = NumberUtil.toInt(buyTransactionsCurrent);
+            buyTransactionsCurrent = 0;
+            sellTransactionsLast = NumberUtil.toInt(sellTransactionsCurrent);
+            sellTransactionsCurrent = 0;
+            
+            boughtItemsLast = NumberUtil.toInt(boughtItemsCurrent);
+            boughtItemsCurrent = 0;
+            soldItemsLast = NumberUtil.toInt(soldItemsCurrent);
+            soldItemsCurrent = 0;
+        }
+    }
+}

src/main/java/com/Acrobot/ChestShop/Listeners/Modules/PriceRestrictionModule.java

@@ -2,12 +2,17 @@ package com.Acrobot.ChestShop.Listeners.Modules;
 
 import com.Acrobot.Breeze.Utils.PriceUtil;
 import com.Acrobot.ChestShop.ChestShop;
+import com.Acrobot.ChestShop.Configuration.Messages;
+import com.Acrobot.ChestShop.Events.ChestShopReloadEvent;
 import com.Acrobot.ChestShop.Events.ItemParseEvent;
 import com.Acrobot.ChestShop.Events.PreShopCreationEvent;
+import com.Acrobot.ChestShop.Permission;
+import com.Acrobot.ChestShop.Signs.ChestShopSign;
 import org.bukkit.Bukkit;
 import org.bukkit.Material;
 import org.bukkit.configuration.ConfigurationSection;
 import org.bukkit.configuration.file.YamlConfiguration;
+import org.bukkit.entity.Player;
 import org.bukkit.event.EventHandler;
 import org.bukkit.event.Listener;
 import org.bukkit.inventory.ItemStack;
@@ -15,15 +20,11 @@ import org.bukkit.inventory.ItemStack;
 import java.io.File;
 import java.io.IOException;
 import java.math.BigDecimal;
+import java.util.Locale;
 import java.util.logging.Level;
 
-import static com.Acrobot.ChestShop.Events.PreShopCreationEvent.CreationOutcome.BUY_PRICE_ABOVE_MAX;
-import static com.Acrobot.ChestShop.Events.PreShopCreationEvent.CreationOutcome.BUY_PRICE_BELOW_MIN;
-import static com.Acrobot.ChestShop.Events.PreShopCreationEvent.CreationOutcome.SELL_PRICE_ABOVE_MAX;
-import static com.Acrobot.ChestShop.Events.PreShopCreationEvent.CreationOutcome.SELL_PRICE_BELOW_MIN;
-import static com.Acrobot.ChestShop.Signs.ChestShopSign.ITEM_LINE;
-import static com.Acrobot.ChestShop.Signs.ChestShopSign.PRICE_LINE;
-import static com.Acrobot.ChestShop.Signs.ChestShopSign.QUANTITY_LINE;
+import static com.Acrobot.ChestShop.Events.PreShopCreationEvent.CreationOutcome.*;
+import static com.Acrobot.ChestShop.Permission.*;
 
 /**
  * @author Acrobot
@@ -33,6 +34,10 @@ public class PriceRestrictionModule implements Listener {
     private static final double INVALID_PATH = Double.MIN_VALUE;
 
     public PriceRestrictionModule() {
+        load();
+    }
+
+    private void load() {
         File file = new File(ChestShop.getFolder(), "priceLimits.yml");
 
         configuration = YamlConfiguration.loadConfiguration(file);
@@ -53,7 +58,7 @@ public class PriceRestrictionModule implements Listener {
                 configuration.options().copyDefaults(true);
                 configuration.save(ChestShop.loadFile("priceLimits.yml"));
             } catch (IOException e) {
-                e.printStackTrace();
+                ChestShop.getBukkitLogger().log(Level.SEVERE, "Error while loading priceLimits.yml", e);
             }
         } else if (!configuration.getBoolean("uses_materials")) {
             Material testMat = Material.matchMaterial("1");
@@ -68,7 +73,7 @@ public class PriceRestrictionModule implements Listener {
                     configuration.save(file);
                     ChestShop.getBukkitLogger().log(Level.INFO, "Conversion finished!");
                 } catch (IOException e) {
-                    e.printStackTrace();
+                    ChestShop.getBukkitLogger().log(Level.SEVERE, "Error while converting priceLimits.yml", e);
                 }
             } else {
                 ChestShop.getBukkitLogger().log(Level.WARNING, "Could not convert numeric IDs in priceLimits.yml to Material names!");
@@ -84,52 +89,71 @@ public class PriceRestrictionModule implements Listener {
             for (String typeId : section.getKeys(false)) {
                 Material material = Material.matchMaterial(typeId);
                 if (material != null) {
-                    configuration.set(sectionPath + "." + material.toString().toLowerCase(), configuration.get(sectionPath + "." + typeId));
+                    configuration.set(sectionPath + "." + material.toString().toLowerCase(Locale.ROOT), configuration.get(sectionPath + "." + typeId));
                     configuration.set(sectionPath + "." + typeId, null);
                 }
             }
         }
     }
 
+    @EventHandler
+    public void onReload(ChestShopReloadEvent event) {
+        load();
+    }
+
     @EventHandler
     public void onPreShopCreation(PreShopCreationEvent event) {
-        ItemParseEvent parseEvent = new ItemParseEvent(event.getSignLine(ITEM_LINE));
+        ItemParseEvent parseEvent = new ItemParseEvent(ChestShopSign.getItem(event.getSignLines()));
         Bukkit.getPluginManager().callEvent(parseEvent);
         ItemStack material = parseEvent.getItem();
+        Player player = event.getPlayer();
 
         if (material == null) {
             return;
         }
 
-        String itemType = material.getType().toString().toLowerCase();
+        String itemType = material.getType().toString().toLowerCase(Locale.ROOT);
         int amount;
         try {
-            amount = Integer.parseInt(event.getSignLine(QUANTITY_LINE));
+            amount = ChestShopSign.getQuantity(event.getSignLines());
         } catch (IllegalArgumentException e) {
             return;
         }
 
-        if (PriceUtil.hasBuyPrice(event.getSignLine(PRICE_LINE))) {
-            BigDecimal buyPrice = PriceUtil.getExactBuyPrice(event.getSignLine(PRICE_LINE));
+        String priceLine = ChestShopSign.getPrice(event.getSignLines());
+        if (PriceUtil.hasBuyPrice(priceLine)) {
+            BigDecimal buyPrice = PriceUtil.getExactBuyPrice(priceLine);
 
-            if (isValid("min.buy_price." + itemType) && buyPrice.compareTo(BigDecimal.valueOf(configuration.getDouble("min.buy_price." + itemType) * amount)) < 0) {
+            BigDecimal minBuyPrice = BigDecimal.valueOf(configuration.getDouble("min.buy_price." + itemType) * amount);
+            if (isValid("min.buy_price." + itemType) && buyPrice.compareTo(minBuyPrice) < 0
+                    && !Permission.has(player, NOLIMIT_MIN_BUY) && !Permission.has(player, NOLIMIT_MIN_BUY_ID + itemType)) {
                 event.setOutcome(BUY_PRICE_BELOW_MIN);
+                Messages.BUY_PRICE_BELOW_MIN.sendWithPrefix(player, "price", buyPrice.toPlainString(), "minprice", minBuyPrice.toPlainString());
             }
 
-            if (isValid("max.buy_price." + itemType) && buyPrice.compareTo(BigDecimal.valueOf(configuration.getDouble("max.buy_price." + itemType) * amount)) > 0) {
+            BigDecimal maxBuyPrice = BigDecimal.valueOf(configuration.getDouble("max.buy_price." + itemType) * amount);
+            if (isValid("max.buy_price." + itemType) && buyPrice.compareTo(maxBuyPrice) > 0
+                    && !Permission.has(player, NOLIMIT_MAX_BUY) && !Permission.has(player, NOLIMIT_MAX_BUY_ID + itemType)) {
                 event.setOutcome(BUY_PRICE_ABOVE_MAX);
+                Messages.BUY_PRICE_ABOVE_MAX.sendWithPrefix(player, "price", buyPrice.toPlainString(),  "maxprice", maxBuyPrice.toPlainString());
             }
         }
 
-        if (PriceUtil.hasSellPrice(event.getSignLine(PRICE_LINE))) {
-            BigDecimal sellPrice = PriceUtil.getExactSellPrice(event.getSignLine(PRICE_LINE));
+        if (PriceUtil.hasSellPrice(priceLine)) {
+            BigDecimal sellPrice = PriceUtil.getExactSellPrice(priceLine);
 
-            if (isValid("min.sell_price." + itemType) && sellPrice.compareTo(BigDecimal.valueOf(configuration.getDouble("min.sell_price." + itemType) * amount)) < 0) {
+            BigDecimal minSellPrice = BigDecimal.valueOf(configuration.getDouble("min.sell_price." + itemType) * amount);
+            if (isValid("min.sell_price." + itemType) && sellPrice.compareTo(minSellPrice) < 0
+                    && !Permission.has(player, NOLIMIT_MIN_SELL) && !Permission.has(player, NOLIMIT_MIN_SELL_ID + itemType)) {
                 event.setOutcome(SELL_PRICE_BELOW_MIN);
+                Messages.SELL_PRICE_BELOW_MIN.sendWithPrefix(player, "price", sellPrice.toPlainString(),  "minprice", minSellPrice.toPlainString());
             }
 
-            if (isValid("max.sell_price." + itemType) && sellPrice.compareTo(BigDecimal.valueOf(configuration.getDouble("max.sell_price." + itemType) * amount)) > 0) {
+            BigDecimal maxSellPrice = BigDecimal.valueOf(configuration.getDouble("max.sell_price." + itemType) * amount);
+            if (isValid("max.sell_price." + itemType) && sellPrice.compareTo(maxSellPrice) > 0
+                    && !Permission.has(player, NOLIMIT_MAX_SELL) && !Permission.has(player, NOLIMIT_MAX_SELL_ID + itemType)) {
                 event.setOutcome(SELL_PRICE_ABOVE_MAX);
+                Messages.SELL_PRICE_ABOVE_MAX.sendWithPrefix(player, "price", sellPrice.toPlainString(),  "maxprice", maxSellPrice.toPlainString());
             }
         }
     }