Skip to content

Commit 8e5d0ac

Browse files
committed
πŸ‘¨β€πŸ’» Upgrade codebase with using .includes() instead of .indexOf() for strings
1 parent 56576c0 commit 8e5d0ac

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

β€Žcore.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,7 @@ export default class FilesCollectionCore extends EventEmitter {
134134
* @returns {Object}
135135
*/
136136
_getExt(fileName) {
137-
if (!!~fileName.indexOf('.')) {
137+
if (fileName.includes('.')) {
138138
const extension = (fileName.split('.').pop().split('?')[0] || '').toLowerCase();
139139
return { ext: extension, extension, extensionWithDot: `.${extension}` };
140140
}

β€Žlib.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ for (let i = 0; i < _helpers.length; i++) {
123123
*/
124124
const fixJSONParse = function(obj) {
125125
for (let key in obj) {
126-
if (helpers.isString(obj[key]) && !!~obj[key].indexOf('=--JSON-DATE--=')) {
126+
if (helpers.isString(obj[key]) && obj[key].includes('=--JSON-DATE--=')) {
127127
obj[key] = obj[key].replace('=--JSON-DATE--=', '');
128128
obj[key] = new Date(parseInt(obj[key]));
129129
} else if (helpers.isObject(obj[key])) {
@@ -134,7 +134,7 @@ const fixJSONParse = function(obj) {
134134
v = obj[key][i];
135135
if (helpers.isObject(v)) {
136136
obj[key][i] = fixJSONParse(v);
137-
} else if (helpers.isString(v) && !!~v.indexOf('=--JSON-DATE--=')) {
137+
} else if (helpers.isString(v) && v.includes('=--JSON-DATE--=')) {
138138
v = v.replace('=--JSON-DATE--=', '');
139139
obj[key][i] = new Date(parseInt(v));
140140
}

β€Župload.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -157,11 +157,11 @@ export class UploadInstance extends EventEmitter {
157157
throw new Meteor.Error(400, '"fileName" must me specified for base64 upload!');
158158
}
159159

160-
if (!!~this.config.file.indexOf('data:')) {
160+
if (this.config.file.includes('data:')) {
161161
this.config.file = this.config.file.replace('data:', '');
162162
}
163163

164-
if (!!~this.config.file.indexOf(',')) {
164+
if (this.config.file.includes(',')) {
165165
const _file = this.config.file.split(',');
166166
this.fileData = {
167167
size: Math.floor(((_file[1].replace(/\=/g, '')).length / 4) * 3),

0 commit comments

Comments
Β (0)