From a285b3104e17b3503f50eb1964862ac9090ec35f Mon Sep 17 00:00:00 2001
From: simon <Anise5-humid5-Allow-5Lulu-Honk>
Date: Sun, 14 Jan 2018 13:33:40 -0500
Subject: [PATCH] Moved Vault Door code to NPC Plugin. Fixed code styling.
 Fixed compilation errors

---
 .../net/simon987/npcplugin}/VaultDoor.java    | 18 ++--
 .../net/simon987/npcplugin}/VaultPlugin.java  |  2 +-
 Plugin Vault/pom.xml                          | 31 ------
 .../src/main/resources/plugin.properties      |  3 -
 .../simon987/server/crypto/AutokeyCypher.java | 30 +++---
 .../simon987/server/crypto/CaesarCypher.java  |  8 +-
 .../server/crypto/CryptoProvider.java         | 12 +--
 .../net/simon987/server/crypto/Cypher.java    |  4 +-
 .../net/simon987/server/crypto/NoCypher.java  | 39 ++++----
 .../server/crypto/RandomStringGenerator.java  | 14 ++-
 .../crypto/ShiftSubstitutionCypher.java       | 97 +++++++++++--------
 .../server/crypto/VigenereCypher.java         | 20 ++--
 12 files changed, 134 insertions(+), 144 deletions(-)
 rename {Plugin Vault/src/main/java/net/simon987/vaultplugin => Plugin NPC/src/main/java/net/simon987/npcplugin}/VaultDoor.java (82%)
 rename {Plugin Vault/src/main/java/net/simon987/vaultplugin => Plugin NPC/src/main/java/net/simon987/npcplugin}/VaultPlugin.java (90%)
 delete mode 100644 Plugin Vault/pom.xml
 delete mode 100644 Plugin Vault/src/main/resources/plugin.properties

diff --git a/Plugin Vault/src/main/java/net/simon987/vaultplugin/VaultDoor.java b/Plugin NPC/src/main/java/net/simon987/npcplugin/VaultDoor.java
similarity index 82%
rename from Plugin Vault/src/main/java/net/simon987/vaultplugin/VaultDoor.java
rename to Plugin NPC/src/main/java/net/simon987/npcplugin/VaultDoor.java
index c75378c..e5afa6f 100644
--- a/Plugin Vault/src/main/java/net/simon987/vaultplugin/VaultDoor.java	
+++ b/Plugin NPC/src/main/java/net/simon987/npcplugin/VaultDoor.java	
@@ -1,12 +1,13 @@
-package net.simon987.vaultplugin;
+package net.simon987.npcplugin;
 
 import com.mongodb.BasicDBObject;
+import net.simon987.server.GameServer;
+import net.simon987.server.crypto.RandomStringGenerator;
 import net.simon987.server.game.Enterable;
 import net.simon987.server.game.GameObject;
 import net.simon987.server.game.Programmable;
 import net.simon987.server.game.Updatable;
 import net.simon987.server.logging.LogManager;
-import net.simon987.server.crypto.CryptoProvider;
 
 import java.util.Arrays;
 
@@ -20,7 +21,7 @@ public class VaultDoor extends GameObject implements Programmable, Enterable, Up
      */
     private char[] password;
 
-    private RandomString random_string_generator;
+    private RandomStringGenerator randomStringGenerator;
 
     /**
      * Whether or not the vault door is opened
@@ -34,13 +35,12 @@ public class VaultDoor extends GameObject implements Programmable, Enterable, Up
     private int OPEN_TIME = GameServer.INSTANCE.getConfig().getInt("vault_door_open_time");
 
     private int openedTimer = 0;
-    private int cypher_id;
+    private int cypherId;
 
-    public VaultDoor(int cypher_id){
-        this.cypher_id = cypher_id;
-        this.random_string_generator = new RandomStringGenerator(PASSWORD_LENGTH);
+    public VaultDoor(int cypherId) {
+        this.cypherId = cypherId;
+        this.randomStringGenerator = new RandomStringGenerator();
 
-        password = GameServer.INSTANCE.getConfig().getRandomPassword();
     }
 
 
@@ -49,7 +49,7 @@ public class VaultDoor extends GameObject implements Programmable, Enterable, Up
         if (open){
             if (openedTimer <= 0) {
                 //Door was open for OPEN_TIME, close it and regen password
-                password = GameServer.INSTANCE.getConfig().getRandomPassword();
+                //password = GameServer.INSTANCE.getConfig().getRandomPassword();
                 open = false;
                 openedTimer = 0;
                 LogManager.LOGGER.fine("Closed Vault door ID: " + getObjectId());
diff --git a/Plugin Vault/src/main/java/net/simon987/vaultplugin/VaultPlugin.java b/Plugin NPC/src/main/java/net/simon987/npcplugin/VaultPlugin.java
similarity index 90%
rename from Plugin Vault/src/main/java/net/simon987/vaultplugin/VaultPlugin.java
rename to Plugin NPC/src/main/java/net/simon987/npcplugin/VaultPlugin.java
index d44405a..652967c 100644
--- a/Plugin Vault/src/main/java/net/simon987/vaultplugin/VaultPlugin.java	
+++ b/Plugin NPC/src/main/java/net/simon987/npcplugin/VaultPlugin.java	
@@ -1,4 +1,4 @@
-package net.simon987.vaultplugin;
+package net.simon987.npcplugin;
 
 import net.simon987.server.ServerConfiguration;
 import net.simon987.server.logging.LogManager;
diff --git a/Plugin Vault/pom.xml b/Plugin Vault/pom.xml
deleted file mode 100644
index 8c85358..0000000
--- a/Plugin Vault/pom.xml	
+++ /dev/null
@@ -1,31 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<project xmlns="http://maven.apache.org/POM/4.0.0"
-         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
-    <modelVersion>4.0.0</modelVersion>
-
-    <parent>
-        <groupId>net.simon987.server</groupId>
-        <artifactId>server_root</artifactId>
-        <version>1.2a</version>
-    </parent>
-
-    <groupId>net.simon987.pluginplant</groupId>
-    <artifactId>plugin-vault</artifactId>
-    <version>1.4a</version>
-
-
-    <dependencies>
-        <dependency>
-            <groupId>com.googlecode.json-simple</groupId>
-            <artifactId>json-simple</artifactId>
-            <version>1.1.1</version>
-        </dependency>
-
-        <dependency>
-            <groupId>net.simon987.server</groupId>
-            <artifactId>server</artifactId>
-            <version>1.2a</version>
-        </dependency>
-    </dependencies>
-</project>
\ No newline at end of file
diff --git a/Plugin Vault/src/main/resources/plugin.properties b/Plugin Vault/src/main/resources/plugin.properties
deleted file mode 100644
index 72e6b22..0000000
--- a/Plugin Vault/src/main/resources/plugin.properties	
+++ /dev/null
@@ -1,3 +0,0 @@
-classpath=net.simon987.vaultplugin.VaultPlugin
-name=Vault Plugin
-version=1.0
\ No newline at end of file
diff --git a/Server/src/main/java/net/simon987/server/crypto/AutokeyCypher.java b/Server/src/main/java/net/simon987/server/crypto/AutokeyCypher.java
index 707d3a0..ede4ee3 100644
--- a/Server/src/main/java/net/simon987/server/crypto/AutokeyCypher.java
+++ b/Server/src/main/java/net/simon987/server/crypto/AutokeyCypher.java
@@ -10,22 +10,24 @@ public class AutokeyCypher extends ShiftSubstitutionCypher {
         super();
     }
 
-    @override
-    protected char encryptionShiftAt(int position, char[] plaintext, char[] key, char[] partial_cyphertext){
-        if (i<key.length){
-            return key[i];
-        } else {
-            return plaintext[i-key.length];
-        }
+    @Override
+    protected char encryptionShiftAt(int position, char[] plaintext, char[] key, char[] partialCypherText) {
+//        if (i < key.length){
+//            return key[i];
+//        } else {
+//            return plaintext[i - key.length];
+//        }
+        return 0;
     }
 
-    @override
-    protected char decryptionShiftAt(int position, char[] cyphertext, char[] key, char[] partial_plaintext){
-        if (i<key.length){
-            return key[i];
-        } else {
-            return partial_plaintext[i-key.length];
-        }
+    @Override
+    protected char decryptionShiftAt(int position, char[] cypherText, char[] key, char[] partialPlainText) {
+//        if (i < key.length){
+//            return key[i];
+//        } else {
+//            return partialPlainText[i - key.length];
+//        }
+        return 0;
     }
 	
 }
\ No newline at end of file
diff --git a/Server/src/main/java/net/simon987/server/crypto/CaesarCypher.java b/Server/src/main/java/net/simon987/server/crypto/CaesarCypher.java
index 9370bc5..fa6b448 100644
--- a/Server/src/main/java/net/simon987/server/crypto/CaesarCypher.java
+++ b/Server/src/main/java/net/simon987/server/crypto/CaesarCypher.java
@@ -13,16 +13,16 @@ public class CaesarCypher extends ShiftSubstitutionCypher {
     /**
      * Uses the first character of the key as the shift, and ignores the rest.
      */
-    @override
-    protected char encryptionShiftAt(int position, char[] plaintext, char[] key, char[] partial_cyphertext){
+    @Override
+    protected char encryptionShiftAt(int position, char[] plaintext, char[] key, char[] partialCypherText) {
         return key[0];
     }
 
     /**
      * Uses the first character of the key as the shift, and ignores the rest.
      */
-    @override
-    protected char decryptionShiftAt(int position, char[] cyphertext, char[] key, char[] partial_plaintext){
+    @Override
+    protected char decryptionShiftAt(int position, char[] cypherText, char[] key, char[] partialPlainText) {
         return key[0];
     }
 
diff --git a/Server/src/main/java/net/simon987/server/crypto/CryptoProvider.java b/Server/src/main/java/net/simon987/server/crypto/CryptoProvider.java
index 7850e65..777ca2f 100644
--- a/Server/src/main/java/net/simon987/server/crypto/CryptoProvider.java
+++ b/Server/src/main/java/net/simon987/server/crypto/CryptoProvider.java
@@ -7,14 +7,14 @@ public class CryptoProvider{
 	public static final int VIGENERE_CYPHER = 0x0002;
 	public static final int AUTOKEY_CYPHER = 0x0003;
 
-    public static final int PASSWORD_LENGTH = 8; // Same as CubotComPort.MESSAGE_LENGTH
+	public static final int PASSWORD_LENGTH = 8; //Same as CubotComPort.MESSAGE_LENGTH
 
-    private charset;
-	private password_generator;
+	private String charset;
+	private RandomStringGenerator passwordGenerator;
 
 	public CryptoProvider(String charset){
 		this.charset = charset;
-		this.password_generator = new RandomStringGenerator(PASSWORD_LENGTH,charset);
+		this.passwordGenerator = new RandomStringGenerator(PASSWORD_LENGTH, charset);
 	}
 
 	public CryptoProvider(){
@@ -36,8 +36,4 @@ public class CryptoProvider{
 		}
 	}
 
-	public static String getRandomPassword(){
-		return this.password_generator.nextString().toCharArray();
-	}
-
 }
\ No newline at end of file
diff --git a/Server/src/main/java/net/simon987/server/crypto/Cypher.java b/Server/src/main/java/net/simon987/server/crypto/Cypher.java
index 97c8776..96fdc47 100644
--- a/Server/src/main/java/net/simon987/server/crypto/Cypher.java
+++ b/Server/src/main/java/net/simon987/server/crypto/Cypher.java
@@ -2,9 +2,9 @@ package net.simon987.server.crypto;
 
 interface Cypher {
 
-	public char[] encrypt(char[] plaintext, char[] key);
+	public char[] encrypt(char[] plainText, char[] key) throws CryptoException;
 
-	public char[] decrypt(char[] cyphertext, char[] key);
+	public char[] decrypt(char[] cypherText, char[] key) throws CryptoException;
 
 	public String textCharset();
 	
diff --git a/Server/src/main/java/net/simon987/server/crypto/NoCypher.java b/Server/src/main/java/net/simon987/server/crypto/NoCypher.java
index c48912a..34d511f 100644
--- a/Server/src/main/java/net/simon987/server/crypto/NoCypher.java
+++ b/Server/src/main/java/net/simon987/server/crypto/NoCypher.java
@@ -1,6 +1,6 @@
 package net.simon987.server.crypto;
 
-public abstract class NoCypher implements Cypher {
+public class NoCypher implements Cypher {
 
 	private String charset;
 
@@ -8,32 +8,35 @@ public abstract class NoCypher implements Cypher {
 		this.charset = charset;
 	}
 
-	public NoCypher(){
-		this(RandomStringGenerator.alphanum);
-	}
+    public NoCypher() {
+        this(RandomStringGenerator.ALPHANUMERIC_CHARSET);
+    }
 
-	public char[] encrypt(char[] plaintext, char[] key){
-        char[] cyphertext = new char[plaintext.length];
-        for (int i = 0; i< plaintext.length; i++){
-        	char p = plaintext[i];
+    public char[] encrypt(char[] plainText, char[] key) throws InvalidCharsetException {
+
+        char[] cypherText = new char[plainText.length];
+        for (int i = 0; i < plainText.length; i++) {
+            char p = plainText[i];
             int p_ind = charset.indexOf(p);
-            if (p_ind == -1){
-                throw InvalidCharsetException("Plaintext contains invalid character: "+p);
+            if (p_ind == -1) {
+                throw new InvalidCharsetException("Plaintext contains invalid character: " + p);
             }
-            cyphertext[i] = p;
+            cypherText[i] = p;
         }
-        return cyphertext;
-	}
+        return cypherText;
+    }
 
 
+    public char[] decrypt(char[] cypherText, char[] key) throws InvalidCharsetException {
 
-	public char[] decrypt(char[] cyphertext, char[] key){
-        char[] plaintext = new char[cyphertext.length];
-        for (int i = 0; i< cyphertext.length; i++){
-        	char c = cyphertext[i];
+        char[] plaintext = new char[cypherText.length];
+
+        for (int i = 0; i < cypherText.length; i++) {
+
+            char c = cypherText[i];
             int c_ind = charset.indexOf(c);
             if (c_ind == -1){
-                throw InvalidCharsetException("Cyphertext contains invalid character: "+c);
+                throw new InvalidCharsetException("Cyphertext contains invalid character: " + c);
             }
             plaintext[i] = c;
         }
diff --git a/Server/src/main/java/net/simon987/server/crypto/RandomStringGenerator.java b/Server/src/main/java/net/simon987/server/crypto/RandomStringGenerator.java
index 47e807c..f474d5b 100644
--- a/Server/src/main/java/net/simon987/server/crypto/RandomStringGenerator.java
+++ b/Server/src/main/java/net/simon987/server/crypto/RandomStringGenerator.java
@@ -35,8 +35,12 @@ public class RandomStringGenerator {
     private final char[] buf;
 
     public RandomStringGenerator(int length, Random random, String charset) {
-        if (length < 1) throw new IllegalArgumentException();
-        if (charset.length() < 2) throw new IllegalArgumentException();
+
+        if (length < 1) {
+            throw new IllegalArgumentException();
+        } else if (charset.length() < 2) {
+            throw new IllegalArgumentException();
+        }
         this.random = Objects.requireNonNull(random);
         this.charset = charset.toCharArray();
         this.buf = new char[length];
@@ -46,14 +50,14 @@ public class RandomStringGenerator {
      * Create an alphanumeric string generator.
      */
     public RandomStringGenerator(int length, Random random) {
-        this(length, random, alphanum);
+        this(length, random, ALPHANUMERIC_CHARSET);
     }
 
     /**
      * Create an alphanumeric string generator using the given charset.
      */
     public RandomStringGenerator(int length, String charset) {
-        this(length, new SecureRandom(), ALPHANUMERIC_CHARSET);
+        this(length, new SecureRandom(), charset);
     }
 
 
@@ -61,7 +65,7 @@ public class RandomStringGenerator {
      * Create an alphanumeric strings from a secure generator.
      */
     public RandomStringGenerator(int length) {
-        this(length, new SecureRandom(),ALPHANUMERIC_CHARSET);
+        this(length, new SecureRandom(), ALPHANUMERIC_CHARSET);
     }
 
     /**
diff --git a/Server/src/main/java/net/simon987/server/crypto/ShiftSubstitutionCypher.java b/Server/src/main/java/net/simon987/server/crypto/ShiftSubstitutionCypher.java
index 97440a3..cb9b023 100644
--- a/Server/src/main/java/net/simon987/server/crypto/ShiftSubstitutionCypher.java
+++ b/Server/src/main/java/net/simon987/server/crypto/ShiftSubstitutionCypher.java
@@ -9,55 +9,70 @@ public abstract class ShiftSubstitutionCypher implements Cypher {
 	}
 
 	public ShiftSubstitutionCypher(){
-		this(RandomStringGenerator.alphanum);
-	}
+        this(RandomStringGenerator.ALPHANUMERIC_CHARSET);
+    }
 
-    protected abstract char encryptionShiftAt(int position, char[] plaintext, char[] key, char[] partial_cyphertext);
-    protected abstract char decryptionShiftAt(int position, char[] cyphertext, char[] key, char[] partial_plaintext);
+    protected abstract char encryptionShiftAt(int position, char[] plaintext, char[] key, char[] partialCypherText);
 
-	public char[] encrypt(char[] plaintext, char[] key){
-        if (key.length==0){
-            throw InvalidKeyException("Key is empty");
+    protected abstract char decryptionShiftAt(int position, char[] cypherText, char[] key, char[] partialPlainText);
+
+    public char[] encrypt(char[] plainText, char[] key) throws CryptoException {
+
+        if (key.length == 0) {
+            throw new InvalidKeyException("Key is empty");
         }
-        int charset_length = charset.length();
-        char[] cyphertext = new char[plaintext.length];
-        for (int i = 0; i< plaintext.length; i++){
-            char p = plaintext[i];
-            char k = encryptionShiftAt(i,plaintext,key,cyphertext);
+
+        char[] cypherText = new char[plainText.length];
+
+        for (int i = 0; i < plainText.length; i++) {
+
+            char p = plainText[i];
+            char k = encryptionShiftAt(i, plainText, key, cypherText);
             int p_ind = charset.indexOf(p);
-            if (p_ind == -1){
-                throw InvalidCharsetException("Plaintext contains invalid character: "+p);
-            }
-            int k_ind = charset.indexOf(k);
-            if (k_ind == -1){
-                throw InvalidCharsetException("Key contains invalid character: "+k); 
-            }
-            int c_int = (p_ind+k_ind)%charset_length;
-            char c = charset.charAt(c_int);
-            cyphertext[i] = c;
-        }
-        return cyphertext;
-	}
 
-	public char[] decrypt(char[] cyphertext, char[] key){
-        if (key.length==0){
-            throw InvalidKeyException("Key is empty");
-        }
-        int charset_length = charset.length();
-        char[] plaintext = new char[cyphertext.length];
-        for (int i = 0; i< cyphertext.length; i++){
-            char c = cyphertext[i];
-            char k = decryptionShiftAt(i,cyphertext,key,plaintext);
-            int c_ind = charset.indexOf(c);
-            if (c_ind == -1){
-                throw InvalidCharsetException("Cyphertext contains invalid character: "+c);
+            if (p_ind == -1) {
+                throw new InvalidCharsetException("Plaintext contains invalid character: " + p);
             }
+
             int k_ind = charset.indexOf(k);
-            if (k_ind == -1){
-                throw InvalidCharsetException("Password contains invalid character: "+k); 
+
+            if (k_ind == -1) {
+                throw new InvalidCharsetException("Key contains invalid character: " + k);
             }
-            int p_int = (c_ind-k_ind)%charset_length;
-            char p = charset.charAt(p_int);
+
+            int c_int = (p_ind + k_ind) % charset.length();
+            char c = charset.charAt(c_int);
+            cypherText[i] = c;
+        }
+        return cypherText;
+    }
+
+    public char[] decrypt(char[] cypherText, char[] key) throws CryptoException {
+
+        if (key.length == 0) {
+            throw new InvalidKeyException("Key is empty");
+        }
+
+        char[] plaintext = new char[cypherText.length];
+
+        for (int i = 0; i < cypherText.length; i++) {
+
+            char c = cypherText[i];
+            char k = decryptionShiftAt(i, cypherText, key, plaintext);
+            int cInd = charset.indexOf(c);
+
+            if (cInd == -1) {
+                throw new InvalidCharsetException("CypherText contains invalid character: " + c);
+            }
+
+            int kInd = charset.indexOf(k);
+
+            if (kInd == -1) {
+                throw new InvalidCharsetException("Password contains invalid character: " + k);
+            }
+
+            int pInt = (cInd - kInd) % charset.length();
+            char p = charset.charAt(pInt);
             plaintext[i] = p;
         }
         return plaintext;
diff --git a/Server/src/main/java/net/simon987/server/crypto/VigenereCypher.java b/Server/src/main/java/net/simon987/server/crypto/VigenereCypher.java
index e417dae..77a2879 100644
--- a/Server/src/main/java/net/simon987/server/crypto/VigenereCypher.java
+++ b/Server/src/main/java/net/simon987/server/crypto/VigenereCypher.java
@@ -10,16 +10,20 @@ public class VigenereCypher extends ShiftSubstitutionCypher {
 		super();
 	}
 
-    @override
-    protected char encryptionShiftAt(int position, char[] plaintext, char[] key, char[] partial_cyphertext){
-        int j = i % key.length;
-        return key[j];
+    @Override
+    protected char encryptionShiftAt(int position, char[] plaintext, char[] key, char[] partialCypherText) {
+//        int j = i % key.length;
+//        return key[j];
+
+        return 0;
     }
 
-    @override
-    protected char decryptionShiftAt(int position, char[] cyphertext, char[] key, char[] partial_plaintext){
-        int j = i % key.length;
-        return key[j];
+    @Override
+    protected char decryptionShiftAt(int position, char[] cypherText, char[] key, char[] partialPlainText) {
+//        int j = i % key.length;
+//        return key[j];
+
+        return 0;
     }
     
 }
\ No newline at end of file