@abaplint/core 2.95.32 → 2.95.34

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

@@ -287,17 +287,24 @@ class BasicTypes {
         }
         const typeTableKeys = node.findAllExpressions(expressions_1.TypeTableKey);
         const firstKey = typeTableKeys[0];
+        const isNamed = (firstKey === null || firstKey === void 0 ? void 0 : firstKey.findDirectExpression(expressions_1.Field)) !== undefined;
         const primaryKey = {
             name: "primary_key",
             type: type,
-            isUnique: (firstKey === null || firstKey === void 0 ? void 0 : firstKey.concatTokens().toUpperCase().includes("WITH UNIQUE ")) === true,
+            isUnique: isNamed ? false : (firstKey === null || firstKey === void 0 ? void 0 : firstKey.concatTokens().toUpperCase().includes("WITH UNIQUE ")) === true,
             keyFields: [],
         };
-        for (const k of (firstKey === null || firstKey === void 0 ? void 0 : firstKey.findDirectExpressions(expressions_1.FieldSub)) || []) {
-            primaryKey.keyFields.push(k.concatTokens().toUpperCase());
+        let start = 1;
+        if (isNamed === false) {
+            for (const k of (firstKey === null || firstKey === void 0 ? void 0 : firstKey.findDirectExpressions(expressions_1.FieldSub)) || []) {
+                primaryKey.keyFields.push(k.concatTokens().toUpperCase());
+            }
+        }
+        else {
+            start = 0;
         }
         const secondaryKeys = [];
-        for (let i = 1; i < typeTableKeys.length; i++) {
+        for (let i = start; i < typeTableKeys.length; i++) {
             const row = typeTableKeys[i];
             const name = (_b = row.findDirectExpression(expressions_1.Field)) === null || _b === void 0 ? void 0 : _b.concatTokens();
             if (name === undefined) {

package/build/src/abap/types/basic/data_reference_type.js

@@ -14,6 +14,10 @@ class DataReference extends _abstract_type_1.AbstractType {
         return this.type;
     }
     toABAP() {
+        const type = this.type.toABAP();
+        if (type.includes(" TABLE OF ")) {
+            return ""; // hmm, should this return undefined?
+        }
         return "REF TO " + this.type.toABAP();
     }
     isGeneric() {

package/build/src/abap/types/basic/table_type.js

@@ -12,9 +12,13 @@ var TableAccessType;
 })(TableAccessType = exports.TableAccessType || (exports.TableAccessType = {}));
 class TableType extends _abstract_type_1.AbstractType {
     constructor(rowType, options, qualifiedName) {
+        var _a;
         super({ qualifiedName: qualifiedName });
         this.rowType = rowType;
         this.options = options;
+        if (((_a = options.primaryKey) === null || _a === void 0 ? void 0 : _a.type) === TableAccessType.standard && options.primaryKey.isUnique === true) {
+            throw new Error("STANDARD tables cannot have UNIQUE key");
+        }
     }
     getOptions() {
         return this.options;

package/build/src/objects/gateway_vocabulary_annotation.js

@@ -9,7 +9,7 @@ class GatewayVocabularyAnnotation extends _abstract_object_1.AbstractObject {
     getAllowedNaming() {
         return {
             maxLength: 36,
-            allowNamespace: false,
+            allowNamespace: true,
         };
     }
     getDescription() {

package/build/src/registry.js

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

package/build/src/rules/downport.js

@@ -1545,7 +1545,7 @@ ${indentation}    output = ${topTarget}.`;
         const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
         return issue_1.Issue.atToken(lowFile, high.getFirstToken(), "Outline LOOP input", this.getMetadata().key, this.conf.severity, fix);
     }
-    outlineLoopTarget(node, lowFile, _highSyntax) {
+    outlineLoopTarget(node, lowFile, highSyntax) {
         var _a, _b, _c, _d, _e, _f, _g;
         // also allows outlining of voided types
         if (!(node.get() instanceof Statements.Loop)) {
@@ -1556,16 +1556,19 @@ ${indentation}    output = ${topTarget}.`;
             return undefined;
         }
         const concat = node.concatTokens().toUpperCase();
-        if (concat.includes(" REFERENCE INTO ")
-            || concat.includes(" GROUP BY ")
-            || concat.startsWith("LOOP AT GROUP ")) {
+        if (concat.includes(" GROUP BY ") || concat.startsWith("LOOP AT GROUP ")) {
             return undefined;
         }
+        const isReference = concat.includes(" REFERENCE INTO ");
         const indentation = " ".repeat(node.getFirstToken().getStart().getCol() - 1);
         const dataTarget = (_c = (_b = node.findDirectExpression(Expressions.LoopTarget)) === null || _b === void 0 ? void 0 : _b.findDirectExpression(Expressions.Target)) === null || _c === void 0 ? void 0 : _c.findDirectExpression(Expressions.InlineData);
         if (dataTarget) {
             const targetName = ((_d = dataTarget.findDirectExpression(Expressions.TargetField)) === null || _d === void 0 ? void 0 : _d.concatTokens()) || "DOWNPORT_ERROR";
-            const code = `DATA ${targetName} LIKE LINE OF ${sourceName}.\n${indentation}`;
+            let code = `DATA ${targetName} LIKE LINE OF ${sourceName}.\n${indentation}`;
+            if (isReference) {
+                const likeName = this.uniqueName(node.getFirstToken().getStart(), lowFile.getFilename(), highSyntax);
+                code = `DATA ${likeName} LIKE LINE OF ${sourceName}.\n${indentation}DATA ${targetName} LIKE REF TO ${likeName}.\n${indentation}`;
+            }
             const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, node.getFirstToken().getStart(), code);
             const fix2 = edit_helper_1.EditHelper.replaceRange(lowFile, dataTarget.getFirstToken().getStart(), dataTarget.getLastToken().getEnd(), targetName);
             const fix = edit_helper_1.EditHelper.merge(fix2, fix1);
@@ -2173,10 +2176,15 @@ ${indentation}    output = ${topTarget}.`;
             else if (found.getType() instanceof basic_1.VoidType && found.getType().getQualifiedName() === undefined) {
                 continue;
             }
-            let type = found.getType().getQualifiedName() ? (_b = found.getType().getQualifiedName()) === null || _b === void 0 ? void 0 : _b.toLowerCase() : found.getType().toABAP();
+            let type = found.getType().getQualifiedName()
+                ? (_b = found.getType().getQualifiedName()) === null || _b === void 0 ? void 0 : _b.toLowerCase()
+                : found.getType().toABAP();
             if (found.getType() instanceof basic_1.ObjectReferenceType) {
                 type = found.getType().toABAP();
             }
+            if (type === "") {
+                continue;
+            }
             const code = `DATA ${name} TYPE ${type}.\n` +
                 " ".repeat(node.getFirstToken().getStart().getCol() - 1);
             const fix1 = edit_helper_1.EditHelper.insertAt(lowFile, node.getFirstToken().getStart(), code);

package/package.json

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