commit bb0093d45bf5044c7aa49b4716b9ffe75df12ff1 parent f7de119be91599da91a3c9ef7e0bf2ba29a4c2f5 Author: Sergej Orlov <wladimirych@gmail.com> Date: Tue, 3 Dec 2019 12:45:17 +0100 lint: fixed unused variables Diffstat:
19 files changed, 21 insertions(+), 24 deletions(-)
diff --git a/.eslintrc b/.eslintrc @@ -10,6 +10,7 @@ "sourceType": "module" }, "rules": { + "no-unused-vars": ["error", {"argsIgnorePattern": "_unused(_.+)?"}], "globals": { "NODE_ENV": true, "RELEASE_VER": true diff --git a/src/lib/leaflet.control.coordinates/index.js b/src/lib/leaflet.control.coordinates/index.js @@ -54,9 +54,7 @@ L.Control.Coordinates = L.Control.extend({ return this.format().wrapperClass; }, this); - this.formatCode.subscribe((_) => { - this.saveStateToStorage(); - }, this); + this.formatCode.subscribe(this.saveStateToStorage, this); }, onAdd: function(map) { @@ -160,7 +158,7 @@ L.Control.Coordinates = L.Control.extend({ return !!this._isEnabled; }, - onClick: function(e) { + onClick: function() { this.setEnabled(!this.isEnabled()); this.saveStateToStorage(); } diff --git a/src/lib/leaflet.control.elevation-profile/index.js b/src/lib/leaflet.control.elevation-profile/index.js @@ -113,7 +113,7 @@ var DragEvents = L.Class.extend({ includes: L.Mixin.Events, initialize: function(eventsSource, eventsTarget, options) { - options = L.setOptions(this, options); + L.setOptions(this, options); if (eventsTarget) { this.eventsTarget = eventsTarget; } else { diff --git a/src/lib/leaflet.control.jnx/hash-state.js b/src/lib/leaflet.control.jnx/hash-state.js @@ -21,7 +21,7 @@ L.Control.JNX.include({ unserializeState: function(values) { - function validateFloat(value, min, max) { + function validateFloat(value) { value = parseFloat(value); if (isNaN(value)) { throw new Error('INVALID VALUE'); diff --git a/src/lib/leaflet.control.layers.configure/index.js b/src/lib/leaflet.control.layers.configure/index.js @@ -429,7 +429,7 @@ function enableConfig(control, {layers, customLayersOrder}) { L.DomEvent.stop(e); this.showCustomLayerForm([ {caption: 'Save', callback: (fieldValues) => this.onCustomLayerChangeClicked(layer, fieldValues)}, - {caption: 'Delete', callback: (fieldValues) => this.onCustomLayerDeletelClicked(layer)}, + {caption: 'Delete', callback: () => this.onCustomLayerDeletelClicked(layer)}, {caption: 'Cancel', callback: () => this.onCustomLayerCancelClicked()} ], layer.fieldValues ); diff --git a/src/lib/leaflet.control.locate/index.js b/src/lib/leaflet.control.locate/index.js @@ -92,7 +92,7 @@ const PositionMarker = L.LayerGroup.extend({ this._updatePrecisionState(); }, - _onZoom: function(e) { + _onZoom: function() { this._updatePrecisionState(); } diff --git a/src/lib/leaflet.control.panoramas/index.js b/src/lib/leaflet.control.panoramas/index.js @@ -278,7 +278,7 @@ L.Control.Panoramas = L.Control.extend({ this.notifyChanged(); }, - onPanoramaCloseClick: function(e) { + onPanoramaCloseClick: function() { this.hidePanoViewer(); }, diff --git a/src/lib/leaflet.control.panoramas/lib/wikimedia/coverage-layer.js b/src/lib/leaflet.control.panoramas/lib/wikimedia/coverage-layer.js @@ -134,7 +134,7 @@ const WikimediaVectorCoverage = L.GridLayer.extend({ delete tile._adjustment; }, - drawTile: function(canvas, coords) { + drawTile: function(canvas, _unused_coords) { if (!this._map) { return; } diff --git a/src/lib/leaflet.control.printPages/decorations.js b/src/lib/leaflet.control.printPages/decorations.js @@ -33,7 +33,7 @@ class PrintStaticLayer { }; } - _drawRaster(canvas, printOptions) { + _drawRaster(_unused_canvas, _unused_printOptions) { return; } } diff --git a/src/lib/leaflet.control.printPages/pageFeature.js b/src/lib/leaflet.control.printPages/pageFeature.js @@ -93,7 +93,7 @@ const PageFeature = L.Marker.extend({ return L.point(...this.paperSize); }, - rotate: function(e) { + rotate: function() { this.paperSize = [this.paperSize[1], this.paperSize[0]]; this.updateView(); } diff --git a/src/lib/leaflet.control.track-list/lib/parsers/zip.js b/src/lib/leaflet.control.track-list/lib/parsers/zip.js @@ -3,7 +3,7 @@ import jsInflate from './jsInflate'; import {decode866} from './codePages'; import parseGeoFile from '../parseGeoFile'; -function parseZip(txt, name) { +function parseZip(txt, _unused_name) { try { var unzipper = new JSUnzip(txt); } catch (e) { diff --git a/src/lib/leaflet.control.track-list/lib/services/nakarte/index.js b/src/lib/leaflet.control.track-list/lib/services/nakarte/index.js @@ -89,7 +89,7 @@ class NakarteUrl { .filter(([name, ]) => paramNames.includes(name)); } - isOurUrl(url) { + isOurUrl() { return this._params.length > 0; } diff --git a/src/lib/leaflet.control.track-list/track-list.js b/src/lib/leaflet.control.track-list/track-list.js @@ -511,9 +511,8 @@ L.Control.TrackList = L.Control.extend({ const key = btoa(hashDigest).replace(/\//g, '_').replace(/\+/g, '-').replace(/=/g, ''); const url = window.location + '&nktl=' + key; copyToClipboard(url, mouseEvent); - fetch(`${config.tracksStorageServer}/track/${key}`, {method: 'POST', data: serialized}).then((xhr) => { - }, - (e) => { + fetch(`${config.tracksStorageServer}/track/${key}`, {method: 'POST', data: serialized}).then( + null, (e) => { let message = e.message || e; if (e.xhr.status === 413) { message = 'track is too big'; diff --git a/src/lib/leaflet.fixes/fixWorldCopyJump.js b/src/lib/leaflet.fixes/fixWorldCopyJump.js @@ -117,7 +117,7 @@ function fixVectorMarkerWorldJump() { this._lastResetLongitude = this.getCenter().lng; }); - this.on('move', (e) => { + this.on('move', () => { const lng = this.getCenter().lng; if (this._lastResetLongitude === null) { this._lastResetLongitude = lng; diff --git a/src/lib/leaflet.hashState/Leaflet.Control.Layers.js b/src/lib/leaflet.hashState/Leaflet.Control.Layers.js @@ -7,7 +7,7 @@ L.Control.Layers.include({ stateChangeEvents: ['baselayerchange', 'overlayadd', 'overlayremove'], stateChangeEventsSource: '_map', - serializeState: function(e) { + serializeState: function() { const keys = []; this._map.eachLayer((layer) => { diff --git a/src/lib/leaflet.layer.rasterize/TileLayer.js b/src/lib/leaflet.layer.rasterize/TileLayer.js @@ -3,7 +3,7 @@ import urlViaCorsProxy from '~/lib/CORSProxy'; import {imgFromDataString} from './imgFromDataString'; const GridLayerGrabMixin = { - tileImagePromiseFromCoords: function(coords) { + tileImagePromiseFromCoords: function(_unused_coords) { throw new Error('Method not implemented'); }, diff --git a/src/lib/notifications/index.js b/src/lib/notifications/index.js @@ -1,7 +1,7 @@ import alertify from 'alertify.js'; import './style.css'; -function notify(message, level) { +function notify(message) { alertify.alert(message); } diff --git a/src/lib/tiled-data-loader/index.js b/src/lib/tiled-data-loader/index.js @@ -24,7 +24,7 @@ class TiledDataLoader { return Object.assign({}, layerTileCoords); } - makeRequestData(dataTileCoords) { + makeRequestData(_unused_dataTileCoords) { throw new Error('Not implemented'); // return { // url, @@ -32,7 +32,7 @@ class TiledDataLoader { // } } - processResponse(xhr, originalDataTileCoords) { + processResponse(_unused_xhr, _unused_originalDataTileCoords) { throw new Error('Not implemented'); // return { // tileData, diff --git a/webpack/paths.js b/webpack/paths.js @@ -1,5 +1,4 @@ const path = require('path'); -const fs = require('fs'); function resolveApp(relativePath) { return path.resolve(__dirname, '..', relativePath);