Replace the Visual Effect system

.github/CODEOWNERS

@@ -70,3 +70,6 @@
 
 # EntityData
 /src/main/java/ch/njol/skript/entity @Absolutionism @skriptlang/core-developers
+
+# Particles/Effects
+/src/main/java/org/skriptlang/skript/bukkit/particles @sovdeeth @skriptlang/core-developers

src/main/java/ch/njol/skript/Skript.java

@@ -101,6 +101,7 @@
 import org.skriptlang.skript.bukkit.itemcomponents.ItemComponentModule;
 import org.skriptlang.skript.bukkit.log.runtime.BukkitRuntimeErrorConsumer;
 import org.skriptlang.skript.bukkit.loottables.LootTableModule;
+import org.skriptlang.skript.bukkit.particles.ParticleModule;
 import org.skriptlang.skript.bukkit.registration.BukkitRegistryKeys;
 import org.skriptlang.skript.bukkit.registration.BukkitSyntaxInfos;
 import org.skriptlang.skript.bukkit.tags.TagModule;
@@ -444,7 +445,7 @@ public void onEnable() {
 					if (!aliasesFolder.mkdirs())
 						throw new IOException("Could not create the directory " + aliasesFolder);
 				}
-				
+
 				f = new ZipFile(getFile());
 				for (ZipEntry e : new EnumerationIterable<ZipEntry>(f.entries())) {
 					if (e.isDirectory())
@@ -599,7 +600,8 @@ public void onEnable() {
 					new DamageSourceModule(),
 					new ItemComponentModule(),
 					new BrewingModule(),
-					new CommonModule()
+					new CommonModule(),
+					new ParticleModule()
 				);
 		} catch (final Exception e) {
 			exception(e, "Could not load required .class files: " + e.getLocalizedMessage());

src/main/java/ch/njol/skript/classes/EnumParser.java

@@ -20,7 +20,7 @@ public class EnumParser<E extends Enum<E>> extends PatternedParser<E> implements
 	private final Class<E> enumClass;
 	private final String languageNode;
 	private String[] names;
-	private final Map<String, E> parseMap = new HashMap<>();
+	protected final Map<String, E> parseMap = new HashMap<>();
 	private String[] patterns;
 
 	/**
@@ -65,11 +65,17 @@ void refresh() {
 				String first = strippedOption.getFirst();
 				Integer second = strippedOption.getSecond();
 
+				NonNullPair<String, String> singlePlural = Noun.getPlural(first);
+				String single = singlePlural.getFirst();
+				String plural = singlePlural.getSecond();
+
 				if (names[ordinal] == null) { // Add to name array if needed
-					names[ordinal] = first;
+					names[ordinal] = single;
 				}
 
-				parseMap.put(first, constant);
+				parseMap.put(single, constant);
+				if (!plural.isEmpty())
+					parseMap.put(plural, constant);
 				if (second != -1) { // There is a gender present
 					parseMap.put(Noun.getArticleWithSpace(second, Language.F_INDEFINITE_ARTICLE) + first, constant);
 				}

src/main/java/ch/njol/skript/classes/data/SkriptClasses.java

@@ -18,8 +18,6 @@
 import ch.njol.skript.localization.RegexMessage;
 import ch.njol.skript.registrations.Classes;
 import ch.njol.skript.util.*;
-import ch.njol.skript.util.visual.VisualEffect;
-import ch.njol.skript.util.visual.VisualEffects;
 import ch.njol.yggdrasil.Fields;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
@@ -475,35 +473,6 @@ public String toVariableNameString(final Experience xp) {
 				})
 				.serializer(new YggdrasilSerializer<>()));
 
-		Classes.registerClass(new ClassInfo<>(VisualEffect.class, "visualeffect")
-				.name("Visual Effect")
-				.description("A visible effect, e.g. particles.")
-				.examples("show wolf hearts on the clicked wolf",
-						"play mob spawner flames at the targeted block to the player")
-				.usage(VisualEffects.getAllNames())
-				.since("2.1")
-				.user("(visual|particle) effects?")
-				.after("itemtype")
-				.parser(new Parser<VisualEffect>() {
-					@Override
-					@Nullable
-					public VisualEffect parse(String s, ParseContext context) {
-						return VisualEffects.parse(s);
-					}
-
-					@Override
-					public String toString(VisualEffect e, int flags) {
-						return e.toString(flags);
-					}
-
-					@Override
-					public String toVariableNameString(VisualEffect e) {
-						return e.toString();
-					}
-
-				})
-				.serializer(new YggdrasilSerializer<>()));
-
 		Classes.registerClass(new ClassInfo<>(GameruleValue.class, "gamerulevalue")
 				.user("gamerule values?")
 				.name("Gamerule Value")

src/main/java/ch/njol/skript/entity/SimpleEntityData.java

@@ -252,6 +252,7 @@ private static void addSuperEntity(String codeName, Class<? extends Entity> enti
 		addSuperEntity("mob", Mob.class);
 		addSuperEntity("creature", Creature.class);
 		addSuperEntity("animal", Animals.class);
+		addSuperEntity("tameable", Tameable.class);
 		addSuperEntity("fish", Fish.class);
 		addSuperEntity("golem", Golem.class);
 		addSuperEntity("projectile", Projectile.class);

src/main/java/ch/njol/skript/expressions/ExprVelocity.java

@@ -1,68 +1,100 @@
 package ch.njol.skript.expressions;
 
-import org.bukkit.entity.Entity;
-import org.bukkit.event.Event;
-import org.bukkit.util.Vector;
-import org.jetbrains.annotations.Nullable;
-
 import ch.njol.skript.classes.Changer.ChangeMode;
 import ch.njol.skript.doc.Description;
-import ch.njol.skript.doc.Examples;
+import ch.njol.skript.doc.Example;
 import ch.njol.skript.doc.Name;
 import ch.njol.skript.doc.Since;
 import ch.njol.skript.expressions.base.SimplePropertyExpression;
 import ch.njol.util.coll.CollectionUtils;
+import org.bukkit.entity.Entity;
+import org.bukkit.event.Event;
+import org.bukkit.util.Vector;
+import org.jetbrains.annotations.Nullable;
+import org.joml.Vector3d;
+import org.skriptlang.skript.bukkit.particles.particleeffects.DirectionalEffect;
 
-/**
- * @author Sashie
- */
-@Name("Vectors - Velocity")
-@Description("Gets or changes velocity of an entity.")
-@Examples({"set player's velocity to {_v}"})
+// TODO: replace with type property expression
+@Name("Velocity")
+@Description({
+	"Gets or changes velocity of an entity or particle.",
+	"Setting the velocity of a particle will remove its random dispersion and force it to be a single particle."
+})
+@Example("set player's velocity to {_v}")
+@Example("set the velocity of {_particle} to vector(0, 1, 0)")
+@Example("""
+	if the vector length of the player's velocity is greater than 5:
+	    send "You're moving fast!" to the player
+	""")
 @Since("2.2-dev31")
-public class ExprVelocity extends SimplePropertyExpression<Entity, Vector> {
+public class ExprVelocity extends SimplePropertyExpression<Object, Vector> {
 
 	static {
-		register(ExprVelocity.class, Vector.class, "velocit(y|ies)", "entities");
+		register(ExprVelocity.class, Vector.class, "velocit(y|ies)", "entities/directionalparticles");
 	}
 
 	@Override
-	@Nullable
-	public Vector convert(Entity e) {
-		return e.getVelocity();
+	public @Nullable Vector convert(Object object) {
+		if (object instanceof Entity entity)
+			return entity.getVelocity();
+		if (object instanceof DirectionalEffect particleEffect && particleEffect.hasVelocity())
+			return Vector.fromJOML(particleEffect.velocity());
+		return null;
 	}
 
 	@Override
-	@Nullable
-	@SuppressWarnings("null")
-	public Class<?>[] acceptChange(ChangeMode mode) {
+	public Class<?> @Nullable [] acceptChange(ChangeMode mode) {
 		if ((mode == ChangeMode.ADD || mode == ChangeMode.REMOVE || mode == ChangeMode.SET || mode == ChangeMode.DELETE || mode == ChangeMode.RESET))
 			return CollectionUtils.array(Vector.class);
 		return null;
 	}
 
 	@Override
-	@SuppressWarnings("null")
-	public void change(Event e, @Nullable Object[] delta, ChangeMode mode) {
+	public void change(Event event, Object @Nullable [] delta, ChangeMode mode) {
 		assert mode == ChangeMode.DELETE || mode == ChangeMode.RESET || delta != null;
-		for (final Entity entity : getExpr().getArray(e)) {
-			if (entity == null)
-				return;
-			switch (mode) {
-				case ADD:
-					entity.setVelocity(entity.getVelocity().add((Vector) delta[0]));
-					break;
-				case REMOVE:
-					entity.setVelocity(entity.getVelocity().subtract((Vector) delta[0]));
-					break;
-				case REMOVE_ALL:
-					break;
-				case DELETE:
-				case RESET:
-					entity.setVelocity(new Vector());
-					break;	
-				case SET:
-					entity.setVelocity((Vector) delta[0]);
+		for (Object object : getExpr().getArray(event)) {
+			// entities
+			if (object instanceof Entity entity) {
+				switch (mode) {
+					case ADD:
+						entity.setVelocity(entity.getVelocity().add((Vector) delta[0]));
+						break;
+					case REMOVE:
+						entity.setVelocity(entity.getVelocity().subtract((Vector) delta[0]));
+						break;
+					case REMOVE_ALL:
+						break;
+					case DELETE:
+					case RESET:
+						entity.setVelocity(new Vector());
+						break;
+					case SET:
+						entity.setVelocity((Vector) delta[0]);
+				}
+			// particles (don't allow add/remove if no velocity is set)
+			} else if (object instanceof DirectionalEffect particleEffect) {
+				switch (mode) {
+					case ADD:
+						if (!particleEffect.hasVelocity())
+							continue;
+						particleEffect.velocity(particleEffect.velocity().add(((Vector) delta[0]).toVector3d()));
+						break;
+					case REMOVE:
+						if (!particleEffect.hasVelocity())
+							continue;
+						particleEffect.velocity(particleEffect.velocity().sub(((Vector) delta[0]).toVector3d()));
+						break;
+					case REMOVE_ALL:
+						break;
+					case DELETE:
+					case RESET:
+						if (!particleEffect.hasVelocity())
+							continue;
+						particleEffect.velocity(new Vector3d());
+						break;
+					case SET:
+						particleEffect.velocity(((Vector) delta[0]).toVector3d());
+				}
 			}
 		}
 	}