Skip to content

Commit b1e6045

Browse files
committed
Replace deprecated "exists" function with "stat"
1 parent 6c7da6e commit b1e6045

File tree

1 file changed

+36
-28
lines changed

1 file changed

+36
-28
lines changed

src/services/helper/filesystem.ts

Lines changed: 36 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
1-
import { exists, readFile, unlink, readdir } from 'fs';
1+
import { readFile, unlink, readdir, stat } from 'fs';
22
import { resolve } from 'path';
33

4+
import { logger } from '@config/logger';
5+
46
/**
57
* FilesystemService
68
*
@@ -18,16 +20,18 @@ export class FilesystemService {
1820
*/
1921
public static readFolder(path: string): Promise<string[]> {
2022
return new Promise((resolve, reject) => {
21-
exists(path, (exists: boolean) => {
22-
if (exists) {
23-
return readdir(path, (err, files) => {
24-
if (err) {
25-
reject(err);
26-
}
27-
resolve(files);
28-
});
23+
stat(path, (err: NodeJS.ErrnoException | null) => {
24+
if (err) {
25+
logger.error(`${path} does not exists!`);
26+
reject(err);
2927
}
30-
reject('Folder does not exist!');
28+
29+
return readdir(path, (err, files) => {
30+
if (err) {
31+
reject(err);
32+
}
33+
resolve(files);
34+
});
3135
});
3236
});
3337
}
@@ -41,16 +45,18 @@ export class FilesystemService {
4145
*/
4246
public static readFile(path: string, encoding: string = 'utf8'): Promise<string> {
4347
return new Promise((resolve, reject) => {
44-
exists(path, (exists: boolean) => {
45-
if (exists) {
46-
return readFile(path, encoding, (err, data) => {
47-
if (err) {
48-
reject(err);
49-
}
50-
resolve(data);
51-
});
48+
stat(path, (err: NodeJS.ErrnoException | null) => {
49+
if (err) {
50+
logger.error(`${path} does not exists!`);
51+
reject(err);
5252
}
53-
reject('File does not exist!');
53+
54+
return readFile(path, encoding, (err, data) => {
55+
if (err) {
56+
reject(err);
57+
}
58+
resolve(data);
59+
});
5460
});
5561
});
5662
}
@@ -63,16 +69,18 @@ export class FilesystemService {
6369
*/
6470
public static deleteFile(path: string): Promise<string | undefined> {
6571
return new Promise((resolve, reject) => {
66-
exists(path, (exists: boolean) => {
67-
if (exists) {
68-
return unlink(path, (err) => {
69-
if (err) {
70-
reject(err);
71-
}
72-
resolve();
73-
});
72+
stat(path, (err: NodeJS.ErrnoException | null) => {
73+
if (err) {
74+
logger.error(`${path} does not exists!`);
75+
reject(err);
7476
}
75-
reject('File does not exist!');
77+
78+
return unlink(path, (err) => {
79+
if (err) {
80+
reject(err);
81+
}
82+
resolve();
83+
});
7684
});
7785
});
7886
}

0 commit comments

Comments
 (0)