@abaplint/core 2.97.8 → 2.97.10

package/build/abaplint.d.ts

@@ -1286,7 +1286,7 @@ export declare class CurrentScope {
     findTypePoolConstant(name: string | undefined): TypedIdentifier | undefined;
     findTypePoolType(name: string): TypedIdentifier | undefined;
     /** Lookup interface in local and global scope */
-    findInterfaceDefinition(name: string): IInterfaceDefinition | undefined;
+    findInterfaceDefinition(name: string | undefined): IInterfaceDefinition | undefined;
     findFormDefinition(name: string): IFormDefinition | undefined;
     findType(name: string | undefined): TypedIdentifier | undefined;
     findExtraLikeType(name: string | undefined): TypedIdentifier | undefined;

package/build/src/abap/5_syntax/_current_scope.js

@@ -251,6 +251,9 @@ class CurrentScope {
     /** Lookup interface in local and global scope */
     findInterfaceDefinition(name) {
         var _a;
+        if (name === undefined) {
+            return undefined;
+        }
         const ilocal = (_a = this.current) === null || _a === void 0 ? void 0 : _a.findInterfaceDefinition(name);
         if (ilocal) {
             return ilocal;

package/build/src/abap/5_syntax/_type_utils.js

@@ -190,14 +190,32 @@ class TypeUtils {
         // todo
         return true;
     }
+    structureContainsString(structure) {
+        for (const c of structure.getComponents()) {
+            if (c.type instanceof basic_1.StringType) {
+                return true;
+            }
+        }
+        return false;
+    }
     isAssignableStrict(source, target) {
         var _a;
+        /*
+            console.dir(source);
+            console.dir(target);
+        */
         if (source instanceof basic_1.CharacterType && target instanceof basic_1.CharacterType) {
             if (((_a = source.getAbstractTypeData()) === null || _a === void 0 ? void 0 : _a.derivedFromConstant) === true) {
                 return source.getLength() <= target.getLength();
             }
             return source.getLength() === target.getLength();
         }
+        else if (source instanceof basic_1.StringType && target instanceof basic_1.StructureType) {
+            if (this.structureContainsString(target)) {
+                return false;
+            }
+            return true;
+        }
         return this.isAssignable(source, target);
     }
     isAssignable(source, target) {
@@ -209,12 +227,14 @@ class TypeUtils {
             if (target.isWithHeader()) {
                 return this.isAssignable(source, target.getRowType());
             }
-            if (source instanceof basic_1.TableType
-                || source instanceof basic_1.VoidType
+            if (source instanceof basic_1.VoidType
                 || source instanceof basic_1.AnyType
                 || source instanceof basic_1.UnknownType) {
                 return true;
             }
+            else if (source instanceof basic_1.TableType) {
+                return this.isAssignableStrict(source.getRowType(), target.getRowType());
+            }
             return false;
         }
         else if (target instanceof basic_1.ObjectReferenceType && source instanceof basic_1.ObjectReferenceType) {
@@ -244,12 +264,17 @@ class TypeUtils {
             if (source instanceof basic_1.TableType && source.isWithHeader()) {
                 return this.isAssignable(source.getRowType(), target);
             }
-            else if (source instanceof basic_1.StructureType
-                || source instanceof basic_1.VoidType
+            else if (source instanceof basic_1.VoidType
                 || source instanceof basic_1.AnyType
                 || source instanceof basic_1.UnknownType) {
                 return true;
             }
+            else if (source instanceof basic_1.StructureType) {
+                if (this.structureContainsString(target) && !this.structureContainsString(source)) {
+                    return false;
+                }
+                return true;
+            }
             else if (target.containsVoid() === true) {
                 return true;
             }

package/build/src/abap/5_syntax/statements/create_object.js

@@ -59,6 +59,11 @@ class CreateObject {
                     if (id instanceof types_1.InterfaceDefinition && type === undefined) {
                         throw new Error("Interface reference, cannot be instantiated");
                     }
+                    else if (found instanceof basic_1.ObjectReferenceType
+                        && type === undefined
+                        && scope.findInterfaceDefinition(found.getQualifiedName())) {
+                        throw new Error("Interface reference, cannot be instantiated");
+                    }
                     else if (id instanceof types_1.ClassDefinition && cdef === undefined) {
                         cdef = id;
                     }

package/build/src/registry.js

@@ -63,7 +63,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.97.8";
+        return "2.97.10";
     }
     getDDICReferences() {
         return this.references;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.97.8",
+    "version": "2.97.10",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",