Fixed #860 - abstract static function in module was called instead of using the real implementation

Author: peq

de.peeeq.wurstscript/src/main/java/de/peeeq/wurstscript/attributes/OverriddenFunctions.java

@@ -49,7 +49,7 @@ private static FunctionDefinition getRealFuncDef(FuncDef f, WScope scope) {
                     if (nl.getLevel() == c.attrLevel()
                             && nl.getDef() instanceof FunctionDefinition
                             && nl instanceof FuncLink
-                            && WurstValidator.canOverride((FuncLink) nl, fNameLink)
+                            && WurstValidator.canOverride((FuncLink) nl, fNameLink, true)
                             ) {
                         return ((FuncLink) nl).getDef().attrRealFuncDef();
                     }

de.peeeq.wurstscript/src/main/java/de/peeeq/wurstscript/translation/imtranslation/ImTranslator.java

@@ -1084,7 +1084,7 @@ public Map<ClassDef, FuncDef> getClassesWithImplementation(Collection<ClassDef>
                         FuncLink funcLink = (FuncLink) nameLink;
                         FuncDef f = (FuncDef) funcLink.getDef();
                         // check if function f overrides func
-                        if (WurstValidator.canOverride(funcLink, funcNameLink)) {
+                        if (WurstValidator.canOverride(funcLink, funcNameLink, false)) {
                             result.put(c, f);
                         }
                     }

de.peeeq.wurstscript/src/main/java/de/peeeq/wurstscript/validation/WurstValidator.java

@@ -2063,8 +2063,8 @@ private void checkCodeArrays(TypeExprArray e) {
     /**
      * checks if func1 can override func2
      */
-    public static boolean canOverride(FuncLink func1, FuncLink func2) {
-        return checkOverride(func1, func2, false) == null;
+    public static boolean canOverride(FuncLink func1, FuncLink func2, boolean allowStaticOverride) {
+        return checkOverride(func1, func2, allowStaticOverride) == null;
     }
 
     /**

de.peeeq.wurstscript/src/test/java/tests/wurstscript/tests/ModuleTests.java

@@ -294,6 +294,25 @@ public void overrideStatic() {
         );
     }
 
+    @Test
+    public void overrideStaticAndActuallyDoStuff() {
+        testAssertOkLines(true,
+            "package Test",
+            "native testSuccess()",
+            "module Test",
+            "    abstract static function foo() returns int",
+            "    static function bar() returns int",
+            "        return foo() * foo()",
+            "class A",
+            "    use Test",
+            "    override static function foo() returns int",
+            "        return 3",
+            "init",
+            "    if A.bar() == 9",
+            "        testSuccess()"
+        );
+    }
+
     @Test
     public void localInModuleConstructor() {
         testAssertOkLines(false,