correct GroupAddUnit return value, fixes #1009

Frotty · Frotty · commit be796b24a24c · 2021-07-11T13:35:33.000+02:00
diff --git a/de.peeeq.wurstscript/src/main/java/de/peeeq/wurstio/jassinterpreter/providers/GroupProvider.java b/de.peeeq.wurstscript/src/main/java/de/peeeq/wurstio/jassinterpreter/providers/GroupProvider.java
@@ -26,6 +26,9 @@ public void GroupClear(IlConstHandle group) {
 
     public ILconstBool GroupAddUnit(IlConstHandle group, IlConstHandle unit) {
         LinkedHashSet<IlConstHandle> groupList = (LinkedHashSet<IlConstHandle>) group.getObj();
+        if (groupList.contains(unit)) {
+            return ILconstBool.FALSE;
+        }
         groupList.add(unit);
         return ILconstBool.TRUE;
     }

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,9 @@ public void GroupClear(IlConstHandle group) {`
`26`	`26`
`27`	`27`	`public ILconstBool GroupAddUnit(IlConstHandle group, IlConstHandle unit) {`
`28`	`28`	`LinkedHashSet<IlConstHandle> groupList = (LinkedHashSet<IlConstHandle>) group.getObj();`
	`29`	`+ if (groupList.contains(unit)) {`
	`30`	`+ return ILconstBool.FALSE;`
	`31`	`+ }`
`29`	`32`	`groupList.add(unit);`
`30`	`33`	`return ILconstBool.TRUE;`
`31`	`34`	`}`