@abaplint/core 2.113.85 → 2.113.86

package/build/src/rules/sql_value_conversion.js

@@ -18,12 +18,12 @@ class SQLValueConversion {
             key: "sql_value_conversion",
             title: "Implicit SQL Value Conversion",
             shortDescription: `Ensure types match when selecting from database`,
-            extendedInformation: `
-* Integer to CHAR conversion
-* Integer to NUMC conversion
-* NUMC to Integer conversion
-* CHAR to Integer conversion
-* Source field longer than database field, CHAR -> CHAR
+            extendedInformation: `
+* Integer to CHAR conversion
+* Integer to NUMC conversion
+* NUMC to Integer conversion
+* CHAR to Integer conversion
+* Source field longer than database field, CHAR -> CHAR
 * Source field longer than database field, NUMC -> NUMC`,
             tags: [],
         };

package/build/src/rules/start_at_tab.js

@@ -20,7 +20,7 @@ class StartAtTab extends _abap_rule_1.ABAPRule {
             key: "start_at_tab",
             title: "Start at tab",
             shortDescription: `Checks that statements start at tabstops.`,
-            extendedInformation: `Reports max 100 issues per file
+            extendedInformation: `Reports max 100 issues per file
 https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#indent-and-snap-to-tab`,
             tags: [_irule_1.RuleTag.Whitespace, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
             badExample: ` WRITE a.`,

package/build/src/rules/strict_sql.js

@@ -22,12 +22,12 @@ class StrictSQL extends _abap_rule_1.ABAPRule {
             key: "strict_sql",
             title: "Strict SQL",
             shortDescription: `Strict SQL`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapinto_clause.htm
-
-https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abenopensql_strict_mode_750.htm
-
-Also see separate rule sql_escape_host_variables
-
+            extendedInformation: `https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-US/abapinto_clause.htm
+
+https://help.sap.com/doc/abapdocu_751_index_htm/7.51/en-us/abenopensql_strict_mode_750.htm
+
+Also see separate rule sql_escape_host_variables
+
 Activates from v750 and up`,
             tags: [_irule_1.RuleTag.Upport, _irule_1.RuleTag.Quickfix],
             badExample: `SELECT * FROM ztabl INTO TABLE @rt_content WHERE type = @iv_type ORDER BY PRIMARY KEY.`,

package/build/src/rules/sy_modification.js

@@ -19,11 +19,11 @@ class SyModification extends _abap_rule_1.ABAPRule {
             key: "sy_modification",
             title: "Modification of SY fields",
             shortDescription: `Finds modification of sy fields`,
-            extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abensystem_fields.htm
-
+            extendedInformation: `https://help.sap.com/doc/abapdocu_750_index_htm/7.50/en-US/abensystem_fields.htm
+
 Changes to SY-TVAR* fields are not reported`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `sy-uname = 2.
+            badExample: `sy-uname = 2.
 sy = sy.`,
         };
     }

package/build/src/rules/tabl_enhancement_category.js

@@ -17,8 +17,8 @@ class TABLEnhancementCategory {
             key: "tabl_enhancement_category",
             title: "TABL enhancement category must be set",
             shortDescription: `Checks that tables do not have the enhancement category 'not classified'.`,
-            extendedInformation: `SAP note 3063227 changes the default to 'Cannot be enhanced'.
-
+            extendedInformation: `SAP note 3063227 changes the default to 'Cannot be enhanced'.
+
 You may use standard report RS_DDIC_CLASSIFICATION_FINAL for adjustment.`,
             tags: [],
         };

package/build/src/rules/tables_declared_locally.js

@@ -22,8 +22,8 @@ class TablesDeclaredLocally extends _abap_rule_1.ABAPRule {
             shortDescription: `TABLES are always global, so declare them globally`,
             extendedInformation: `https://help.sap.com/doc/abapdocu_752_index_htm/7.52/en-us/abaptables.htm`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `FORM foo.
-  TABLES t100.
+            badExample: `FORM foo.
+  TABLES t100.
 ENDFORM.`,
             goodExample: `TABLES t000.`,
         };

package/build/src/rules/type_form_parameters.js

@@ -21,9 +21,9 @@ class TypeFormParameters extends _abap_rule_1.ABAPRule {
             title: "Type FORM parameters",
             shortDescription: `Checks for untyped FORM parameters`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `FORM foo USING bar.
+            badExample: `FORM foo USING bar.
 ENDFORM.`,
-            goodExample: `FORM foo USING bar TYPE string.
+            goodExample: `FORM foo USING bar TYPE string.
 ENDFORM.`,
         };
     }

package/build/src/rules/unnecessary_pragma.js

@@ -28,38 +28,38 @@ class UnnecessaryPragma extends _abap_rule_1.ABAPRule {
             key: "unnecessary_pragma",
             title: "Unnecessary Pragma",
             shortDescription: `Finds pragmas which can be removed`,
-            extendedInformation: `* NO_HANDLER with handler
-
-* NEEDED without definition
-
-* NO_TEXT without texts
-
-* SUBRC_OK where sy-subrc is checked
-
+            extendedInformation: `* NO_HANDLER with handler
+
+* NEEDED without definition
+
+* NO_TEXT without texts
+
+* SUBRC_OK where sy-subrc is checked
+
 NO_HANDLER inside macros are not checked`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `TRY.
-    ...
-  CATCH zcx_abapgit_exception ##NO_HANDLER.
-    RETURN. " it has a handler
-ENDTRY.
-MESSAGE w125(zbar) WITH c_foo INTO message ##NEEDED ##NO_TEXT.
-SELECT SINGLE * FROM tadir INTO @DATA(sdfs) ##SUBRC_OK.
-IF sy-subrc <> 0.
+            badExample: `TRY.
+    ...
+  CATCH zcx_abapgit_exception ##NO_HANDLER.
+    RETURN. " it has a handler
+ENDTRY.
+MESSAGE w125(zbar) WITH c_foo INTO message ##NEEDED ##NO_TEXT.
+SELECT SINGLE * FROM tadir INTO @DATA(sdfs) ##SUBRC_OK.
+IF sy-subrc <> 0.
 ENDIF.`,
-            goodExample: `TRY.
-    ...
-  CATCH zcx_abapgit_exception.
-    RETURN.
-ENDTRY.
-MESSAGE w125(zbar) WITH c_foo INTO message.
-SELECT SINGLE * FROM tadir INTO @DATA(sdfs).
-IF sy-subrc <> 0.
-ENDIF.
-
-DATA: BEGIN OF blah ##NEEDED,
-        test1 TYPE string,
-        test2 TYPE string,
+            goodExample: `TRY.
+    ...
+  CATCH zcx_abapgit_exception.
+    RETURN.
+ENDTRY.
+MESSAGE w125(zbar) WITH c_foo INTO message.
+SELECT SINGLE * FROM tadir INTO @DATA(sdfs).
+IF sy-subrc <> 0.
+ENDIF.
+
+DATA: BEGIN OF blah ##NEEDED,
+        test1 TYPE string,
+        test2 TYPE string,
       END OF blah.`,
         };
     }

package/build/src/rules/unnecessary_return.js

@@ -28,18 +28,18 @@ class UnnecessaryReturn extends _abap_rule_1.ABAPRule {
             shortDescription: `Finds unnecessary RETURN statements`,
             extendedInformation: `Finds unnecessary RETURN statements`,
             tags: [_irule_1.RuleTag.SingleFile, _irule_1.RuleTag.Quickfix],
-            badExample: `FORM hello1.
-  WRITE 'world'.
-  RETURN.
-ENDFORM.
-
-FORM foo.
-  IF 1 = 2.
-    RETURN.
-  ENDIF.
+            badExample: `FORM hello1.
+  WRITE 'world'.
+  RETURN.
+ENDFORM.
+
+FORM foo.
+  IF 1 = 2.
+    RETURN.
+  ENDIF.
 ENDFORM.`,
-            goodExample: `FORM hello2.
-  WRITE 'world'.
+            goodExample: `FORM hello2.
+  WRITE 'world'.
 ENDFORM.`,
         };
     }

package/build/src/rules/unused_macros.js

@@ -26,13 +26,13 @@ class UnusedMacros {
             title: "Unused macros",
             shortDescription: `Checks for unused macro definitions definitions`,
             tags: [_irule_1.RuleTag.Quickfix],
-            badExample: `DEFINE foobar1.
-  WRITE 'hello'.
+            badExample: `DEFINE foobar1.
+  WRITE 'hello'.
 END-OF-DEFINITION.`,
-            goodExample: `DEFINE foobar2.
-  WRITE 'hello'.
-END-OF-DEFINITION.
-
+            goodExample: `DEFINE foobar2.
+  WRITE 'hello'.
+END-OF-DEFINITION.
+
 foobar2.`,
         };
     }

package/build/src/rules/unused_methods.js

@@ -60,18 +60,18 @@ class UnusedMethods {
             key: "unused_methods",
             title: "Unused methods",
             shortDescription: `Checks for unused methods`,
-            extendedInformation: `Checks private and protected methods.
-
-Unused methods are not reported if the object contains parser or syntax errors.
-Quick fixes only appears for private methods or projected methods where the class doesnt have any subclasses.
-
-Skips:
-* methods FOR TESTING
-* methods SETUP + TEARDOWN + CLASS_SETUP + CLASS_TEARDOWN in testclasses
-* class_constructor + constructor methods
-* event handlers
-* methods that are redefined
-* INCLUDEs
+            extendedInformation: `Checks private and protected methods.
+
+Unused methods are not reported if the object contains parser or syntax errors.
+Quick fixes only appears for private methods or projected methods where the class doesnt have any subclasses.
+
+Skips:
+* methods FOR TESTING
+* methods SETUP + TEARDOWN + CLASS_SETUP + CLASS_TEARDOWN in testclasses
+* class_constructor + constructor methods
+* event handlers
+* methods that are redefined
+* INCLUDEs
 `,
             tags: [_irule_1.RuleTag.Quickfix],
             pragma: "##CALLED",

package/build/src/rules/unused_variables.js

@@ -66,23 +66,23 @@ class UnusedVariables {
             key: "unused_variables",
             title: "Unused variables",
             shortDescription: `Checks for unused variables and constants`,
-            extendedInformation: `Skips event parameters.
-
-Note that this currently does not work if the source code uses macros.
-
-Unused variables are not reported if the object contains parser or syntax errors.
-
+            extendedInformation: `Skips event parameters.
+
+Note that this currently does not work if the source code uses macros.
+
+Unused variables are not reported if the object contains parser or syntax errors.
+
 Errors found in INCLUDES are reported for the main program.`,
             tags: [_irule_1.RuleTag.Quickfix],
             pragma: "##NEEDED",
             pseudoComment: "EC NEEDED",
-            badExample: `DATA: BEGIN OF blah1,
-      test  TYPE string,
-      test2 TYPE string,
+            badExample: `DATA: BEGIN OF blah1,
+      test  TYPE string,
+      test2 TYPE string,
     END OF blah1.`,
-            goodExample: `DATA: BEGIN OF blah2 ##NEEDED,
-      test  TYPE string,
-      test2 TYPE string,
+            goodExample: `DATA: BEGIN OF blah2 ##NEEDED,
+      test  TYPE string,
+      test2 TYPE string,
     END OF blah2.`,
         };
     }

package/build/src/rules/use_bool_expression.js

@@ -27,15 +27,15 @@ class UseBoolExpression extends _abap_rule_1.ABAPRule {
             shortDescription: `Use boolean expression, xsdbool from 740sp08 and up, boolc from 702 and up`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-xsdbool-to-set-boolean-variables`,
             tags: [_irule_1.RuleTag.Upport, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.Quickfix, _irule_1.RuleTag.SingleFile],
-            badExample: `IF line IS INITIAL.
-  has_entries = abap_false.
-ELSE.
-  has_entries = abap_true.
-ENDIF.
-
+            badExample: `IF line IS INITIAL.
+  has_entries = abap_false.
+ELSE.
+  has_entries = abap_true.
+ENDIF.
+
 DATA(fsdf) = COND #( WHEN foo <> bar THEN abap_true ELSE abap_false ).`,
-            goodExample: `DATA(has_entries) = xsdbool( line IS NOT INITIAL ).
-
+            goodExample: `DATA(has_entries) = xsdbool( line IS NOT INITIAL ).
+
 DATA(fsdf) = xsdbool( foo <> bar ).`,
         };
     }

package/build/src/rules/use_class_based_exceptions.js

@@ -22,15 +22,15 @@ class UseClassBasedExceptions extends _abap_rule_1.ABAPRule {
             shortDescription: `Use class based exceptions, checks interface and class definitions`,
             extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#use-class-based-exceptions`,
             tags: [_irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
-            badExample: `INTERFACE lif.
-  METHODS load_data
-    EXCEPTIONS
-      invalid_parameter.
+            badExample: `INTERFACE lif.
+  METHODS load_data
+    EXCEPTIONS
+      invalid_parameter.
 ENDINTERFACE.`,
-            goodExample: `INTERFACE lif.
-  METHODS load_data
-    RAISING
-      cx_something.
+            goodExample: `INTERFACE lif.
+  METHODS load_data
+    RAISING
+      cx_something.
 ENDINTERFACE.`,
         };
     }

package/build/src/rules/use_line_exists.js

@@ -22,15 +22,15 @@ class UseLineExists extends _abap_rule_1.ABAPRule {
             key: "use_line_exists",
             title: "Use line_exists",
             shortDescription: `Use line_exists, from 740sp02 and up`,
-            extendedInformation: `
-https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-line_exists-to-read-table-or-loop-at
-
+            extendedInformation: `
+https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-line_exists-to-read-table-or-loop-at
+
 Not reported if the READ TABLE statement contains BINARY SEARCH.`,
             tags: [_irule_1.RuleTag.Upport, _irule_1.RuleTag.Styleguide, _irule_1.RuleTag.SingleFile],
-            badExample: `READ TABLE my_table TRANSPORTING NO FIELDS WITH KEY key = 'A'.
-IF sy-subrc = 0.
+            badExample: `READ TABLE my_table TRANSPORTING NO FIELDS WITH KEY key = 'A'.
+IF sy-subrc = 0.
 ENDIF.`,
-            goodExample: `IF line_exists( my_table[ key = 'A' ] ).
+            goodExample: `IF line_exists( my_table[ key = 'A' ] ).
 ENDIF.`,
         };
     }

package/build/src/rules/use_new.js

@@ -23,10 +23,10 @@ class UseNew extends _abap_rule_1.ABAPRule {
             key: "use_new",
             title: "Use NEW",
             shortDescription: `Checks for deprecated CREATE OBJECT statements.`,
-            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-new-to-create-object
-
-If the target variable is referenced in the CREATE OBJECT statement, no errors are issued
-
+            extendedInformation: `https://github.com/SAP/styleguides/blob/main/clean-abap/CleanABAP.md#prefer-new-to-create-object
+
+If the target variable is referenced in the CREATE OBJECT statement, no errors are issued
+
 Applicable from v740sp02 and up`,
             badExample: `CREATE OBJECT ref.`,
             goodExample: `ref = NEW #( ).`,

package/build/src/rules/when_others_last.js

@@ -21,13 +21,13 @@ class WhenOthersLast extends _abap_rule_1.ABAPRule {
             title: "WHEN OTHERS last",
             shortDescription: `Checks that WHEN OTHERS is placed the last within a CASE statement.`,
             tags: [_irule_1.RuleTag.SingleFile],
-            badExample: `CASE bar.
-  WHEN OTHERS.
-  WHEN 2.
+            badExample: `CASE bar.
+  WHEN OTHERS.
+  WHEN 2.
 ENDCASE.`,
-            goodExample: `CASE bar.
-  WHEN 2.
-  WHEN OTHERS.
+            goodExample: `CASE bar.
+  WHEN 2.
+  WHEN OTHERS.
 ENDCASE.`,
         };
     }

package/package.json

@@ -1,70 +1,70 @@
-{
-    "name": "@abaplint/core",
-    "version": "2.113.85",
-    "description": "abaplint - Core API",
-    "main": "build/src/index.js",
-    "typings": "build/abaplint.d.ts",
-    "funding": "https://github.com/sponsors/larshp",
-    "scripts": {
-        "lint": "eslint src/**/*.ts test/**/*.ts",
-        "lint:fix": "eslint src/**/*.ts test/**/*.ts --fix",
-        "compile": "tsc && sh scripts/version.sh",
-        "test": "npm run compile && mocha --timeout 1000 && npm run lint && npm run schema && api-extractor run",
-        "test:only": "npm run compile && mocha --timeout 1000000",
-        "test:parallel": "npm run compile && mocha --timeout 1000 --parallel --reporter dot",
-        "coverage": "npm run compile && c8 mocha && c8 report --reporter=html",
-        "lexer_performance": "tsc && curl -o lexer_performance.abap https://raw.githubusercontent.com/abapGit/build/main/zabapgit_standalone.prog.abap && node build/adhoc/lexer_performance.js",
-        "parser_performance": "tsc && curl -o lexer_performance.abap https://raw.githubusercontent.com/abapGit/build/main/zabapgit_standalone.prog.abap && node build/adhoc/parser_performance.js",
-        "syntax_performance": "tsc && curl -o lexer_performance.abap https://raw.githubusercontent.com/abapGit/build/main/zabapgit_standalone.prog.abap && node build/adhoc/syntax_performance.js",
-        "typed_array": "tsc && node build/adhoc/typed_array.js",
-        "schema": "node scripts/schema.js > scripts/schema.ts && ts-json-schema-generator --tsconfig tsconfig_schema.json --jsDoc extended --path scripts/schema.ts > scripts/schema.json && node scripts/schema_post.js",
-        "publish:minor": "npm --no-git-tag-version version minor && rm -rf build && npm install && npm run test && npm publish --access public",
-        "publish:patch": "npm --no-git-tag-version version patch && rm -rf build && npm install && npm run test && npm publish --access public"
-    },
-    "mocha": {
-        "recursive": true,
-        "reporter": "progress",
-        "spec": "build/test/**/*.js",
-        "require": "source-map-support/register"
-    },
-    "c8": {
-        "include": [
-            "build/src/**/*.js"
-        ]
-    },
-    "repository": {
-        "type": "git",
-        "url": "git+https://github.com/abaplint/abaplint.git"
-    },
-    "engines": {
-        "node": ">=12.0.0"
-    },
-    "keywords": [
-        "ABAP",
-        "lint"
-    ],
-    "author": "Lars Hvam Petersen",
-    "license": "MIT",
-    "bugs": {
-        "url": "https://github.com/abaplint/abaplint/issues"
-    },
-    "homepage": "https://abaplint.org",
-    "devDependencies": {
-        "@microsoft/api-extractor": "^7.48.1",
-        "@types/chai": "^4.3.20",
-        "@types/mocha": "^10.0.10",
-        "@types/node": "^22.10.2",
-        "chai": "^4.5.0",
-        "eslint": "^9.17.0",
-        "mocha": "^11.0.1",
-        "c8": "^10.1.3",
-        "source-map-support": "^0.5.21",
-        "ts-json-schema-generator": "^2.3.0",
-        "typescript": "^5.7.2"
-    },
-    "dependencies": {
-        "fast-xml-parser": "^4.5.1",
-        "json5": "^2.2.3",
-        "vscode-languageserver-types": "^3.17.5"
-    }
-}
+{
+    "name": "@abaplint/core",
+    "version": "2.113.86",
+    "description": "abaplint - Core API",
+    "main": "build/src/index.js",
+    "typings": "build/abaplint.d.ts",
+    "funding": "https://github.com/sponsors/larshp",
+    "scripts": {
+        "lint": "eslint src/**/*.ts test/**/*.ts",
+        "lint:fix": "eslint src/**/*.ts test/**/*.ts --fix",
+        "compile": "tsc && sh scripts/version.sh",
+        "test": "npm run compile && mocha --timeout 1000 && npm run lint && npm run schema && api-extractor run",
+        "test:only": "npm run compile && mocha --timeout 1000000",
+        "test:parallel": "npm run compile && mocha --timeout 1000 --parallel --reporter dot",
+        "coverage": "npm run compile && c8 mocha && c8 report --reporter=html",
+        "lexer_performance": "tsc && curl -o lexer_performance.abap https://raw.githubusercontent.com/abapGit/build/main/zabapgit_standalone.prog.abap && node build/adhoc/lexer_performance.js",
+        "parser_performance": "tsc && curl -o lexer_performance.abap https://raw.githubusercontent.com/abapGit/build/main/zabapgit_standalone.prog.abap && node build/adhoc/parser_performance.js",
+        "syntax_performance": "tsc && curl -o lexer_performance.abap https://raw.githubusercontent.com/abapGit/build/main/zabapgit_standalone.prog.abap && node build/adhoc/syntax_performance.js",
+        "typed_array": "tsc && node build/adhoc/typed_array.js",
+        "schema": "node scripts/schema.js > scripts/schema.ts && ts-json-schema-generator --tsconfig tsconfig_schema.json --jsDoc extended --path scripts/schema.ts > scripts/schema.json && node scripts/schema_post.js",
+        "publish:minor": "npm --no-git-tag-version version minor && rm -rf build && npm install && npm run test && npm publish --access public",
+        "publish:patch": "npm --no-git-tag-version version patch && rm -rf build && npm install && npm run test && npm publish --access public"
+    },
+    "mocha": {
+        "recursive": true,
+        "reporter": "progress",
+        "spec": "build/test/**/*.js",
+        "require": "source-map-support/register"
+    },
+    "c8": {
+        "include": [
+            "build/src/**/*.js"
+        ]
+    },
+    "repository": {
+        "type": "git",
+        "url": "git+https://github.com/abaplint/abaplint.git"
+    },
+    "engines": {
+        "node": ">=12.0.0"
+    },
+    "keywords": [
+        "ABAP",
+        "lint"
+    ],
+    "author": "Lars Hvam Petersen",
+    "license": "MIT",
+    "bugs": {
+        "url": "https://github.com/abaplint/abaplint/issues"
+    },
+    "homepage": "https://abaplint.org",
+    "devDependencies": {
+        "@microsoft/api-extractor": "^7.48.1",
+        "@types/chai": "^4.3.20",
+        "@types/mocha": "^10.0.10",
+        "@types/node": "^22.10.5",
+        "chai": "^4.5.0",
+        "eslint": "^9.17.0",
+        "mocha": "^11.0.1",
+        "c8": "^10.1.3",
+        "source-map-support": "^0.5.21",
+        "ts-json-schema-generator": "^2.3.0",
+        "typescript": "^5.7.2"
+    },
+    "dependencies": {
+        "fast-xml-parser": "^4.5.1",
+        "json5": "^2.2.3",
+        "vscode-languageserver-types": "^3.17.5"
+    }
+}