chore: linter the entire project using the new rules

5 files changed, 23 insertions, 21 deletions

diff --git a/src/api/utils/Log.js b/src/api/utils/Log.js
index 8b38c2b..99d11e4 100644
--- a/src/api/utils/Log.js
+++ b/src/api/utils/Log.js
@@ -27,12 +27,6 @@ class Log {
 		else console.log(chalk.gray(args)); // eslint-disable-line no-console
 	}
 
-	/*
-	static dump(args) {
-		dump(args);
-	}
-	*/
-
 	static checkIfArrayOrObject(thing) {
 		if (typeof thing === typeof [] || typeof thing === typeof {}) return true;
 		return false;
diff --git a/src/api/utils/ThumbUtil.js b/src/api/utils/ThumbUtil.js
index f8c73e7..98ba5c0 100644
--- a/src/api/utils/ThumbUtil.js
+++ b/src/api/utils/ThumbUtil.js
@@ -22,7 +22,9 @@ class ThumbUtil {
 		const output = `${filename.slice(0, -ext.length)}.png`;
 		const previewOutput = `${filename.slice(0, -ext.length)}.webm`;
 
+		// eslint-disable-next-line max-len
 		if (ThumbUtil.imageExtensions.includes(ext)) return ThumbUtil.generateThumbnailForImage(filename, output);
+		// eslint-disable-next-line max-len
 		if (ThumbUtil.videoExtensions.includes(ext)) return ThumbUtil.generateThumbnailForVideo(filename, previewOutput);
 		return null;
 	}
@@ -75,17 +77,21 @@ class ThumbUtil {
 	}
 
 	static getFileThumbnail(filename) {
-		// TODO: refactor so we don't do the same compare multiple times (poor cpu cycles)
 		if (!filename) return null;
 		const ext = path.extname(filename).toLowerCase();
-		if (!ThumbUtil.imageExtensions.includes(ext) && !ThumbUtil.videoExtensions.includes(ext)) return null;
-		if (ThumbUtil.imageExtensions.includes(ext)) return { thumb: `${filename.slice(0, -ext.length)}.png` };
-		if (ThumbUtil.videoExtensions.includes(ext)) {
+
+		const isImage = ThumbUtil.imageExtensions.includes(ext);
+		const isVideo = ThumbUtil.videoExtensions.includes(ext);
+
+		if (isImage) return { thumb: `${filename.slice(0, -ext.length)}.png` };
+		if (isVideo) {
 			return {
 				thumb: `${filename.slice(0, -ext.length)}.png`,
 				preview: `${filename.slice(0, -ext.length)}.webm`,
 			};
 		}
+
+		return null;
 	}
 
 	static async removeThumbs({ thumb, preview }) {
diff --git a/src/api/utils/Util.js b/src/api/utils/Util.js
index ab59c95..905948a 100644
--- a/src/api/utils/Util.js
+++ b/src/api/utils/Util.js
@@ -52,7 +52,7 @@ class Util {
 
 	static getUniqueFilename(name) {
 		const retry = (i = 0) => {
-			const filename =				randomstring.generate({
+			const filename = randomstring.generate({
 				length: parseInt(process.env.GENERATED_FILENAME_LENGTH, 10),
 				capitalization: 'lowercase',
 			}) + path.extname(name).toLowerCase();
@@ -81,7 +81,7 @@ class Util {
 			/*
 				It's funny but if you do i++ the asignment never gets done resulting in an infinite loop
 			*/
-			if (i < 5) return retry(++i);
+			if (i < 5) return retry(i + 1);
 			log.error('Couldnt allocate identifier for album');
 			return null;
 		};
diff --git a/src/api/utils/videoPreview/FragmentPreview.js b/src/api/utils/videoPreview/FragmentPreview.js
index 8815392..bf623c1 100644
--- a/src/api/utils/videoPreview/FragmentPreview.js
+++ b/src/api/utils/videoPreview/FragmentPreview.js
@@ -1,3 +1,4 @@
+/* eslint-disable no-bitwise */
 const ffmpeg = require('fluent-ffmpeg');
 const probe = require('ffmpeg-probe');
 
@@ -24,7 +25,7 @@ const getStartTime = (vDuration, fDuration, ignoreBeforePercent, ignoreAfterPerc
 	return getRandomInt(ignoreBeforePercent * safeVDuration, ignoreAfterPercent * safeVDuration);
 };
 
-module.exports = async opts => {
+module.exports = async (opts) => {
 	const {
 		log = noop,
 
@@ -37,7 +38,7 @@ module.exports = async opts => {
 
 		fragmentDurationSecond = 3,
 		ignoreBeforePercent = 0.25,
-		ignoreAfterPercent = 0.75
+		ignoreAfterPercent = 0.75,
 	} = opts;
 
 	const info = await probe(input);
@@ -77,7 +78,7 @@ module.exports = async opts => {
 			.outputOptions([`-t ${fragmentDurationSecond}`])
 			.noAudio()
 			.output(output)
-			.on('start', cmd => log && log({ cmd }))
+			.on('start', (cmd) => log && log({ cmd }))
 			.on('end', resolve)
 			.on('error', reject)
 			.run();
diff --git a/src/api/utils/videoPreview/FrameIntervalPreview.js b/src/api/utils/videoPreview/FrameIntervalPreview.js
index 75f6d2b..8c5f1c3 100644
--- a/src/api/utils/videoPreview/FrameIntervalPreview.js
+++ b/src/api/utils/videoPreview/FrameIntervalPreview.js
@@ -1,9 +1,10 @@
+/* eslint-disable no-bitwise */
 const ffmpeg = require('fluent-ffmpeg');
 const probe = require('ffmpeg-probe');
 
 const noop = () => {};
 
-module.exports = async opts => {
+module.exports = async (opts) => {
 	const {
 		log = noop,
 
@@ -15,13 +16,13 @@ module.exports = async opts => {
 		output,
 
 		numFrames,
-		numFramesPercent = 0.05
+		numFramesPercent = 0.05,
 	} = opts;
 
 	const info = await probe(input);
 	// const numFramesTotal = parseInt(info.streams[0].nb_frames, 10);
 	const { avg_frame_rate: avgFrameRate, duration } = info.streams[0];
-	const [frames, time] = avgFrameRate.split('/').map(e => parseInt(e, 10));
+	const [frames, time] = avgFrameRate.split('/').map((e) => parseInt(e, 10));
 
 	const numFramesTotal = (frames / time) * duration;
 
@@ -31,7 +32,7 @@ module.exports = async opts => {
 
 	const result = {
 		output,
-		numFrames: numFramesToCapture
+		numFrames: numFramesToCapture,
 	};
 
 	await new Promise((resolve, reject) => {
@@ -62,9 +63,9 @@ module.exports = async opts => {
 			.noAudio()
 			.outputFormat('webm')
 			.output(output)
-			.on('start', cmd => log && log({ cmd }))
+			.on('start', (cmd) => log && log({ cmd }))
 			.on('end', () => resolve())
-			.on('error', err => reject(err))
+			.on('error', (err) => reject(err))
 			.run();
 	});