isml-linter 5.40.2 → 5.40.4

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## [5.40.4] - 2023-01-15
+
+### Fixed
+ - Invalid ">" character detection;
+ - Invalid element issue length;
+ - "no-inline-style" rule;
+ 
+## [5.40.3] - 2022-09-12
+
+### Fixed
+ - [Issue #35][issue#35] - Manually set bin file linebreak to LF, as previous version didn't fix the issue;
 
 ## [5.40.2] - 2022-09-05
 
@@ -1023,6 +1034,8 @@
 ### Added
 - Linter is published;
 
+[5.40.4]: https://github.com/FabiowQuixada/isml-linter/compare/v5.40.3...v5.40.4
+[5.40.3]: https://github.com/FabiowQuixada/isml-linter/compare/v5.40.2...v5.40.3
 [5.40.2]: https://github.com/FabiowQuixada/isml-linter/compare/v5.40.1...v5.40.2
 [5.40.1]: https://github.com/FabiowQuixada/isml-linter/compare/v5.40.0...v5.40.1
 [5.40.0]: https://github.com/FabiowQuixada/isml-linter/compare/v5.39.4...v5.40.0

package/bin/isml-linter.js

@@ -1,32 +1,32 @@
-#!/usr/bin/env node
-
-require('../src/util/NativeExtensionUtils');
-
-const CommandLineUtils = require('../src/util/CommandLineUtils');
-const ConfigUtils      = require('../src/util/ConfigUtils');
-
-try {
-    const commandObj = CommandLineUtils.parseCommand();
-
-    if (!commandObj) {
-        process.exit(0);
-    }
-
-    if (commandObj.options.indexOf('--init') >= 0) {
-        ConfigUtils.init();
-        process.exit(0);
-    }
-
-    const IsmlLinter       = require('../src/publicApi');
-    const filePatternArray = commandObj.files;
-    const exitCode         = IsmlLinter.build(filePatternArray);
-
-    if (commandObj.options.indexOf('--build') >= 0) {
-        process.exit(exitCode);
-    }
-
-} catch (e) {
-    const ConsoleUtils = require('../src/util/ConsoleUtils');
-    ConsoleUtils.printExceptionMsg(e.stack || e);
-    process.exit(1);
-}
+#!/usr/bin/env node
+
+require('../src/util/NativeExtensionUtils');
+
+const CommandLineUtils = require('../src/util/CommandLineUtils');
+const ConfigUtils      = require('../src/util/ConfigUtils');
+
+try {
+    const commandObj = CommandLineUtils.parseCommand();
+
+    if (!commandObj) {
+        process.exit(0);
+    }
+
+    if (commandObj.options.indexOf('--init') >= 0) {
+        ConfigUtils.init();
+        process.exit(0);
+    }
+
+    const IsmlLinter       = require('../src/publicApi');
+    const filePatternArray = commandObj.files;
+    const exitCode         = IsmlLinter.build(filePatternArray);
+
+    if (commandObj.options.indexOf('--build') >= 0) {
+        process.exit(exitCode);
+    }
+
+} catch (e) {
+    const ConsoleUtils = require('../src/util/ConsoleUtils');
+    ConsoleUtils.printExceptionMsg(e.stack || e);
+    process.exit(1);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "isml-linter",
-  "version": "5.40.2",
+  "version": "5.40.4",
   "author": "Fabiow Quixadá <ftquixada@gmail.com>",
   "license": "MIT",
   "main": "src/publicApi.js",

package/src/isml_tree/ParseUtils.js

@@ -140,6 +140,14 @@ const parseNextElement = state => {
             newElement.globalPos,
             state.templatePath
         );
+    } else if (newElement.value.trim() === '>') {
+        throw ExceptionUtils.invalidCharacterError(
+            '>',
+            newElement.lineNumber,
+            newElement.globalPos,
+            1,
+            state.templatePath
+        );
     }
 
     return newElement;

package/src/rules/line_by_line/no-inline-style.js

@@ -2,7 +2,7 @@ const SingleLineRulePrototype = require('../prototypes/SingleLineRulePrototype')
 
 const ruleId         = require('path').basename(__filename).slice(0, -3);
 const description    = 'Avoid using inline style';
-const occurrenceText = 'style="';
+const occurrenceText = ' style="';
 
 const Rule = Object.create(SingleLineRulePrototype);
 
@@ -11,7 +11,7 @@ Rule.init(ruleId, description);
 Rule.isBroken = function(line) { return line.indexOf('<isprint') === -1 && line.indexOf(occurrenceText) >= 0; };
 
 Rule.getColumnNumber = function(line) {
-    return Math.max(line.indexOf(occurrenceText), 0) + 1;
+    return Math.max(line.indexOf(occurrenceText), 0) + 2;
 };
 
 Rule.getFirstOccurrence = function(line) {
@@ -23,8 +23,8 @@ Rule.getFirstOccurrence = function(line) {
         const matchPos = line.indexOf(occurrenceText);
 
         result = {
-            globalPos : matchPos,
-            length    : occurrenceText.length - 2
+            globalPos : matchPos + 1,
+            length    : occurrenceText.length - 3
         };
     }
 

package/src/util/ExceptionUtils.js

@@ -105,7 +105,7 @@ const invalidNestedIsifError = (tagType, lineNumber, globalPos, templatePath) =>
         message      : `An error occurred while parsing element "<${tagType}>" in line ${lineNumber}. Try moving the closing character ">" of the "<${tagType}>" element to outside of the "<isif>" condition.`,
         templatePath : templatePath,
         globalPos,
-        length       : tagType.length,
+        length       : tagType.length + 1,
         lineNumber   : lineNumber,
         isCustom     : true,
         type         : types.INVALID_NESTED_ISIF