Merge branch 'dev/feature' into fix/durability-error

src/main/java/ch/njol/skript/classes/registry/RegistrySerializer.java

@@ -43,27 +43,27 @@ public RegistrySerializer(Registry<R> registry) {
 	@Override
 	public @NotNull Fields serialize(R o) {
 		Fields fields = new Fields();
-		fields.putPrimitive("name", o.getKey().toString());
+		fields.putObject("name", o.getKey().toString());
 		return fields;
 	}
 
 	@Override
-	protected R deserialize(Fields fields) {
-		try {
-			String name = fields.getAndRemovePrimitive("name", String.class);
-			NamespacedKey namespacedKey;
-			if (!name.contains(":")) {
-				// Old variables
-				namespacedKey = NamespacedKey.minecraft(name);
-			} else {
-				namespacedKey = NamespacedKey.fromString(name);
-			}
-			if (namespacedKey == null)
-				return null;
-			return registry.get(namespacedKey);
-		} catch (StreamCorruptedException e) {
-			return null;
+	protected R deserialize(Fields fields) throws StreamCorruptedException {
+		String name = fields.getAndRemoveObject("name", String.class);
+		assert name != null;
+		NamespacedKey namespacedKey;
+		if (!name.contains(":")) {
+			// Old variables
+			namespacedKey = NamespacedKey.minecraft(name);
+		} else {
+			namespacedKey = NamespacedKey.fromString(name);
 		}
+		if (namespacedKey == null)
+			throw new StreamCorruptedException("Invalid namespacedkey: " + name);
+		R object = registry.get(namespacedKey);
+		if (object == null)
+			throw new StreamCorruptedException("Invalid object from registry: " + namespacedKey);
+		return object;
 	}
 
 	@Override

src/test/skript/tests/misc/registry.sk

@@ -0,0 +1,8 @@
+test "registry" when minecraft version is "1.14":
+
+	# Test namespaced keys
+	assert curse of vanishing = minecraft:vanishing_curse with "'curse of vanishing' enchant should match namespace key"
+
+	# Test serialization
+	set {test::enchantment} to minecraft:sharpness
+	assert {test::enchantment} = sharpness with "variable should have been set to sharpness enchantment"