Fix code style

Replace unary increments/decrements to num += 1 and num -= 1 and add missing new lines for better code readability. Issue: #4708
handsontable · Feb 15, 2018 · 6b6b3a2 · 6b6b3a2
1 parent 6df6ef0
commit 6b6b3a2
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 10 deletions.
diff --git a/src/3rdparty/walkontable/src/border.js b/src/3rdparty/walkontable/src/border.js
@@ -326,7 +326,7 @@ class Border {
 
     const rowsCount = this.wot.wtTable.getRenderedRowsCount();
 
-    for (let i = 0; i < rowsCount; i++) {
+    for (let i = 0; i < rowsCount; i += 1) {
       let s = this.wot.wtTable.rowFilter.renderedToSource(i);
 
       if (s >= corners[0] && s <= corners[2]) {
@@ -335,7 +335,7 @@ class Border {
       }
     }
 
-    for (let i = rowsCount - 1; i >= 0; i--) {
+    for (let i = rowsCount - 1; i >= 0; i -= 1) {
       let s = this.wot.wtTable.rowFilter.renderedToSource(i);
 
       if (s >= corners[0] && s <= corners[2]) {
@@ -346,7 +346,7 @@ class Border {
 
     const columnsCount = this.wot.wtTable.getRenderedColumnsCount();
 
-    for (let i = 0; i < columnsCount; i++) {
+    for (let i = 0; i < columnsCount; i += 1) {
       let s = this.wot.wtTable.columnFilter.renderedToSource(i);
 
       if (s >= corners[1] && s <= corners[3]) {
@@ -355,7 +355,7 @@ class Border {
       }
     }
 
-    for (let i = columnsCount - 1; i >= 0; i--) {
+    for (let i = columnsCount - 1; i >= 0; i -= 1) {
       let s = this.wot.wtTable.columnFilter.renderedToSource(i);
 
       if (s >= corners[1] && s <= corners[3]) {
@@ -374,12 +374,11 @@ class Border {
     const fromOffset = offset(fromTD);
     const toOffset = isMultiple ? offset(toTD) : fromOffset;
     const containerOffset = offset(this.wot.wtTable.TABLE);
-
     const minTop = fromOffset.top;
     const minLeft = fromOffset.left;
+
     let height = toOffset.top + outerHeight(toTD) - minTop;
     let width = toOffset.left + outerWidth(toTD) - minLeft;
-
     let top = minTop - containerOffset.top - 1;
     let left = minLeft - containerOffset.left - 1;
     let style = getComputedStyle(fromTD);

diff --git a/src/3rdparty/walkontable/src/selection.js b/src/3rdparty/walkontable/src/selection.js
@@ -160,7 +160,7 @@ class Selection {
       let index = previousIndex >= 0 ? previousIndex : layerLevelOwner;
       let className = baseClassName;
 
-      index--;
+      index -= 1;
 
       const previousClassName = index === 0 ? baseClassName : `${baseClassName}-${index}`;
 

diff --git a/src/helpers/array.js b/src/helpers/array.js
@@ -89,6 +89,7 @@ export function arrayFilter(array, predicate) {
   if (!Array.isArray(array)) {
     iterable = Array.from(array);
   }
+
   const length = iterable.length;
   const result = [];
   let resIndex = -1;

diff --git a/src/plugins/contextMenu/menu.js b/src/plugins/contextMenu/menu.js
@@ -667,7 +667,7 @@ class Menu {
    * @param {Object} preventScrolling Object with `value` property where its value change will be observed.
    * @param {Number} selectionLayerLevel The number which indicates what selection layer is currently modified.
    */
-  onAfterSelection(r, c, r2, c2, preventScrolling, selectionLayer) {
+  onAfterSelection(r, c, r2, c2, preventScrolling) {
     if (this.keyEvent === false) {
       preventScrolling.value = true;
     }

diff --git a/src/selection/selection.js b/src/selection/selection.js
@@ -243,6 +243,7 @@ class Selection {
         }
       }
     }
+
     if (this.highlight.isEnabledFor(HEADER_TYPE)) {
       if (this.settings.selectionMode === 'single') {
         headerHighlight.add(cellRange.highlight);

diff --git a/src/utils/keyStateObserver.js b/src/utils/keyStateObserver.js
@@ -28,15 +28,15 @@ function startObserving() {
     });
   }
 
-  refCount++;
+  refCount += 1;
 }
 
 /**
  * Stops observing keyboard keys states and clear all previously saved states.
  */
 function stopObserving() {
   if (refCount > 0) {
-    refCount--;
+    refCount -= 1;
   }
 
   if (refCount === 0) {