@abaplint/cli 2.112.6 → 2.112.8

package/build/cli.js

@@ -22098,9 +22098,9 @@ class ObjectOriented {
         return ignore;
     }
     // returns list of interfaces implemented
-    fromInterfaces(classDefinition, skip) {
+    fromInterfaces(definition, skip) {
         const ignore = [];
-        for (const i of classDefinition.getImplementing()) {
+        for (const i of definition.getImplementing()) {
             ignore.push(...this.fromInterfaceByName(i.name, ignore.concat(skip || [])));
         }
         return ignore;
@@ -23734,7 +23734,7 @@ class BasicTypes {
             const attr = token.getStr();
             const c = new _object_oriented_1.ObjectOriented(this.input.scope).searchConstantName(obj, attr);
             if (c instanceof class_constant_1.ClassConstant) {
-                this.input.scope.addReference(firstToken, obj, _reference_1.ReferenceType.ObjectOrientedReference, this.input.filename);
+                this.input.scope.addReference(firstToken, obj, _reference_1.ReferenceType.ObjectOrientedReference, this.input.filename, { ooName: obj.getName() });
                 this.input.scope.addReference(token, c, _reference_1.ReferenceType.DataReadReference, this.input.filename);
                 const val = c.getValue();
                 if (typeof val === "string") {
@@ -38730,6 +38730,7 @@ const _scope_type_1 = __webpack_require__(/*! ../5_syntax/_scope_type */ "./node
 const event_definition_1 = __webpack_require__(/*! ./event_definition */ "./node_modules/@abaplint/core/build/src/abap/types/event_definition.js");
 const method_definitions_1 = __webpack_require__(/*! ./method_definitions */ "./node_modules/@abaplint/core/build/src/abap/types/method_definitions.js");
 const _reference_1 = __webpack_require__(/*! ../5_syntax/_reference */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_reference.js");
+const _object_oriented_1 = __webpack_require__(/*! ../5_syntax/_object_oriented */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/_object_oriented.js");
 class InterfaceDefinition extends _identifier_1.Identifier {
     constructor(node, input) {
         if (!(node.get() instanceof Structures.Interface)) {
@@ -38795,32 +38796,12 @@ class InterfaceDefinition extends _identifier_1.Identifier {
     }
     parse(input, node) {
         this.checkInterfacesExists(input, node);
+        const helper = new _object_oriented_1.ObjectOriented(input.scope);
+        helper.fromInterfaces(this);
         // todo, proper sequencing, the statements should be processed line by line
         this.attributes = new class_attributes_1.Attributes(node, input);
         this.typeDefinitions = this.attributes.getTypes();
         this.aliases = this.attributes.getAliases();
-        // todo, cleanup aliases, vs "object_oriented.ts" vs "class_implementation.ts"
-        // this adds the aliased types to scope?
-        /*
-        for (const a of this.aliases) {
-          const [objName, fieldName] = a.getComponent().split("~");
-          const idef = input.scope.findInterfaceDefinition(objName);
-          if (idef) {
-            const foundType = idef.getTypeDefinitions().getByName(fieldName);
-            if (foundType) {
-              input.scope.addTypeNamed(a.getName(), foundType);
-            } else {
-              const foundField = idef.getAttributes().findByName(fieldName);
-              if (foundField && foundField instanceof ClassConstant) {
-                const token = new TokenIdentifier(a.getStart(), a.getName());
-                const id = new TypedIdentifier(token, input.filename, foundField.getType());
-                const constant = new ClassConstant(id, Visibility.Public, foundField.getValue());
-                input.scope.addIdentifier(constant);
-              }
-            }
-          }
-        }
-        */
         const events = node.findAllStatements(Statements.Events);
         for (const e of events) {
             this.events.push(new event_definition_1.EventDefinition(e, visibility_1.Visibility.Public, input));
@@ -39684,6 +39665,9 @@ class CDSParser {
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
             res = combi_1.Combi.run(new Expressions.CDSDefineCustom(), tokens, version_1.defaultVersion);
         }
+        if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
+            res = combi_1.Combi.run(new Expressions.CDSExtendView(), tokens, version_1.defaultVersion);
+        }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
             return undefined;
         }
@@ -40154,6 +40138,31 @@ exports.CDSElement = CDSElement;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_extend_view.js":
+/*!**********************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/cds/expressions/cds_extend_view.js ***!
+  \**********************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.CDSExtendView = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/cds/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+const cds_name_1 = __webpack_require__(/*! ./cds_name */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_name.js");
+class CDSExtendView extends combi_1.Expression {
+    getRunnable() {
+        const namedot = (0, combi_1.seq)(cds_name_1.CDSName, (0, combi_1.opt)((0, combi_1.seq)(".", cds_name_1.CDSName)), (0, combi_1.opt)(_1.CDSAs));
+        const valueNested = (0, combi_1.seq)("{", namedot, (0, combi_1.star)((0, combi_1.seq)(",", namedot)), "}");
+        return (0, combi_1.seq)((0, combi_1.star)(_1.CDSAnnotation), (0, combi_1.str)("EXTEND VIEW"), (0, combi_1.opt)((0, combi_1.str)("ENTITY")), cds_name_1.CDSName, (0, combi_1.str)("WITH"), (0, combi_1.opt)(cds_name_1.CDSName), valueNested, (0, combi_1.opt)(";"));
+    }
+}
+exports.CDSExtendView = CDSExtendView;
+//# sourceMappingURL=cds_extend_view.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_function.js":
 /*!*******************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/cds/expressions/cds_function.js ***!
@@ -40222,6 +40231,28 @@ exports.CDSGroupBy = CDSGroupBy;
 
 /***/ }),
 
+/***/ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_having.js":
+/*!*****************************************************************************!*\
+  !*** ./node_modules/@abaplint/core/build/src/cds/expressions/cds_having.js ***!
+  \*****************************************************************************/
+/***/ ((__unused_webpack_module, exports, __webpack_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.CDSHaving = void 0;
+const _1 = __webpack_require__(/*! . */ "./node_modules/@abaplint/core/build/src/cds/expressions/index.js");
+const combi_1 = __webpack_require__(/*! ../../abap/2_statements/combi */ "./node_modules/@abaplint/core/build/src/abap/2_statements/combi.js");
+class CDSHaving extends combi_1.Expression {
+    getRunnable() {
+        return (0, combi_1.seq)("HAVING", _1.CDSCondition);
+    }
+}
+exports.CDSHaving = CDSHaving;
+//# sourceMappingURL=cds_having.js.map
+
+/***/ }),
+
 /***/ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_integer.js":
 /*!******************************************************************************!*\
   !*** ./node_modules/@abaplint/core/build/src/cds/expressions/cds_integer.js ***!
@@ -40400,7 +40431,7 @@ class CDSSelect extends combi_1.Expression {
         const fields = (0, combi_1.opt)((0, combi_1.seq)((0, combi_1.star)((0, combi_1.seq)(_1.CDSElement, ",")), _1.CDSElement));
         const distinct = (0, combi_1.str)("DISTINCT");
         const elements = (0, combi_1.seq)((0, combi_1.str)("{"), (0, combi_1.plus)(_1.CDSElement), (0, combi_1.star)((0, combi_1.seq)(",", _1.CDSElement)), (0, combi_1.str)("}"));
-        return (0, combi_1.seq)("SELECT", (0, combi_1.opt)(distinct), (0, combi_1.opt)(fields), "FROM", _1.CDSSource, (0, combi_1.opt)(_1.CDSParametersSelect), (0, combi_1.opt)(_1.CDSAs), (0, combi_1.star)(cds_join_1.CDSJoin), (0, combi_1.star)(_1.CDSComposition), (0, combi_1.star)(cds_association_1.CDSAssociation), (0, combi_1.star)(_1.CDSComposition), (0, combi_1.opt)(elements), (0, combi_1.opt)(_1.CDSGroupBy), (0, combi_1.opt)(_1.CDSWhere), (0, combi_1.opt)((0, combi_1.seq)("UNION", (0, combi_1.opt)("ALL"), CDSSelect)));
+        return (0, combi_1.seq)("SELECT", (0, combi_1.opt)(distinct), (0, combi_1.opt)(fields), "FROM", _1.CDSSource, (0, combi_1.opt)(_1.CDSParametersSelect), (0, combi_1.opt)(_1.CDSAs), (0, combi_1.star)(cds_join_1.CDSJoin), (0, combi_1.star)(_1.CDSComposition), (0, combi_1.star)(cds_association_1.CDSAssociation), (0, combi_1.star)(_1.CDSComposition), (0, combi_1.opt)(elements), (0, combi_1.opt)(_1.CDSWhere), (0, combi_1.opt)(_1.CDSGroupBy), (0, combi_1.opt)(_1.CDSHaving), (0, combi_1.opt)((0, combi_1.seq)("UNION", (0, combi_1.opt)("ALL"), CDSSelect)));
     }
 }
 exports.CDSSelect = CDSSelect;
@@ -40562,14 +40593,16 @@ __exportStar(__webpack_require__(/*! ./cds_define_custom */ "./node_modules/@aba
 __exportStar(__webpack_require__(/*! ./cds_define_projection */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_define_projection.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_define_view */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_define_view.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_element */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_element.js"), exports);
+__exportStar(__webpack_require__(/*! ./cds_extend_view */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_extend_view.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_function */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_function.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_group_by */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_group_by.js"), exports);
-__exportStar(__webpack_require__(/*! ./cds_provider_contract */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_provider_contract.js"), exports);
+__exportStar(__webpack_require__(/*! ./cds_having */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_having.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_integer */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_integer.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_join */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_join.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_name */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_name.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_parameters_select */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_parameters_select.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_parameters */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_parameters.js"), exports);
+__exportStar(__webpack_require__(/*! ./cds_provider_contract */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_provider_contract.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_relation */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_relation.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_select */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_select.js"), exports);
 __exportStar(__webpack_require__(/*! ./cds_source */ "./node_modules/@abaplint/core/build/src/cds/expressions/cds_source.js"), exports);
@@ -52350,7 +52383,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.112.6";
+        return "2.112.8";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -56440,7 +56473,7 @@ class CyclicOO {
         return {
             key: "cyclic_oo",
             title: "Cyclic OO",
-            shortDescription: `Finds cyclic OO references`,
+            shortDescription: `Finds cyclic/circular OO references`,
             extendedInformation: `Runs for global INTF + CLAS objects
 
 Objects must be without syntax errors for this rule to take effect
@@ -76419,6 +76452,7 @@ Builder.prototype.j2x = function(jObj, level) {
       //repeated nodes
       const arrLen = jObj[key].length;
       let listTagVal = "";
+      let listTagAttr = "";
       for (let j = 0; j < arrLen; j++) {
         const item = jObj[key][j];
         if (typeof item === 'undefined') {
@@ -76428,17 +76462,27 @@ Builder.prototype.j2x = function(jObj, level) {
           else val += this.indentate(level) + '<' + key + '/' + this.tagEndChar;
           // val += this.indentate(level) + '<' + key + '/' + this.tagEndChar;
         } else if (typeof item === 'object') {
-          if(this.options.oneListGroup ){
-            listTagVal += this.j2x(item, level + 1).val;
+          if(this.options.oneListGroup){
+            const result = this.j2x(item, level + 1);
+            listTagVal += result.val;
+            if (this.options.attributesGroupName && item.hasOwnProperty(this.options.attributesGroupName)) {
+              listTagAttr += result.attrStr
+            }
           }else{
             listTagVal += this.processTextOrObjNode(item, key, level)
           }
         } else {
-          listTagVal += this.buildTextValNode(item, key, '', level);
+          if (this.options.oneListGroup) {
+            let textValue = this.options.tagValueProcessor(key, item);
+            textValue = this.replaceEntitiesValue(textValue);
+            listTagVal += textValue;
+          } else {
+            listTagVal += this.buildTextValNode(item, key, '', level);
+          }
         }
       }
       if(this.options.oneListGroup){
-        listTagVal = this.buildObjectNode(listTagVal, key, '', level);
+        listTagVal = this.buildObjectNode(listTagVal, key, listTagAttr, level);
       }
       val += listTagVal;
     } else {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.112.6",
+    "version": "2.112.8",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,23 +38,23 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.112.6",
+        "@abaplint/core": "^2.112.8",
         "@types/chai": "^4.3.16",
         "@types/glob": "^8.1.0",
         "@types/minimist": "^1.2.5",
         "@types/mocha": "^10.0.7",
-        "@types/node": "^20.14.11",
+        "@types/node": "^22.0.0",
         "@types/progress": "^2.0.7",
-        "chai": "^4.4.1",
+        "chai": "^4.5.0",
         "chalk": "^5.3.0",
         "eslint": "^8.57.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
-        "memfs": "^4.9.3",
+        "memfs": "=4.9.3",
         "minimist": "^1.2.8",
         "mocha": "^10.7.0",
         "progress": "^2.0.3",
-        "typescript": "^5.5.3",
+        "typescript": "^5.5.4",
         "webpack": "^5.93.0",
         "webpack-cli": "^5.1.4",
         "xml-js": "^1.6.11"