Merge pull request #59 from aliasadidev/replace-xml-2-js-package

fix lint errors

Author: aliasadidev

src/modules/js2xml.ts

@@ -1,7 +1,7 @@
-import { helper } from "./xml-helper.module";
+import { Helper } from "./xml-helper.module";
 
 var currentElement: any, currentElementName: any;
-var h = new helper();
+var h = new Helper();
 
 function validateOptions(userOptions: any) {
   var options = h.copyOptions(userOptions);
@@ -98,7 +98,9 @@ function writeInstruction(instruction: any, options: any, depth: any) {
     return '<?' + instructionName + writeAttributes(instruction[key][options.attributesKey], options, depth) + '?>';
   } else {
     var instructionValue = instruction[key] ? instruction[key] : '';
-    if ('instructionFn' in options) instructionValue = options.instructionFn(instructionValue, key, currentElementName, currentElement);
+    if ('instructionFn' in options) {
+      instructionValue = options.instructionFn(instructionValue, key, currentElementName, currentElement);
+    }
     return '<?' + instructionName + (instructionValue ? ' ' + instructionValue : '') + '?>';
   }
 }
@@ -116,7 +118,9 @@ function writeDoctype(doctype: any, options: any) {
 }
 
 function writeText(text: any, options: any) {
-  if (options.ignoreText) return '';
+  if (options.ignoreText) {
+    return '';
+  }
   text = '' + text; // ensure Number and Boolean are converted to String
   text = text.replace(/&amp;/g, '&'); // desanitize to avoid double sanitization
   text = text.replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;');

src/modules/xml-helper.module.ts

@@ -1,4 +1,4 @@
-export class helper {
+export class Helper {
   public copyOptions(options: any) {
     var key, copy: any = {};
     for (key in options) {

src/modules/xml2js.ts

@@ -1,12 +1,12 @@
-import { helper } from "./xml-helper.module";
+import { Helper } from "./xml-helper.module";
 var sax = require('sax');
 
 var options: any;
 var pureJsParser = true;
 
 var currentElement: any;
 
-let h = new helper();
+let h = new Helper();
 function validateOptions(userOptions: any) {
   options = h.copyOptions(userOptions);
   h.ensureFlagExists('ignoreDeclaration', options);
@@ -142,11 +142,15 @@ function manipulateAttributes(attributes: any) {
     var key;
     for (key in attributes) {
       if (attributes.hasOwnProperty(key)) {
-        if (options.trim) attributes[key] = attributes[key].trim();
+        if (options.trim) {
+          attributes[key] = attributes[key].trim();
+        }
         if (options.nativeTypeAttributes) {
           attributes[key] = nativeType(attributes[key]);
         }
-        if ('attributeValueFn' in options) attributes[key] = options.attributeValueFn(attributes[key], key, currentElement);
+        if ('attributeValueFn' in options) {
+          attributes[key] = options.attributeValueFn(attributes[key], key, currentElement);
+        }
         if ('attributeNameFn' in options) {
           var temp = attributes[key];
           delete attributes[key];
@@ -199,7 +203,7 @@ function onInstruction(instruction: any) {
 
 function onStartElement(name: any, attributes: any) {
   var element: any;
-  var isSelfClosing = name?.isSelfClosing === true
+  var isSelfClosing = name?.isSelfClosing === true;
   if (typeof name === 'object') {
     attributes = name.attributes;
     name = name.name;