From 94b090850389621172292e9c946388ca080e1ccb Mon Sep 17 00:00:00 2001 From: cjihrig Date: Thu, 8 Oct 2020 21:30:15 -0400 Subject: [PATCH] fs: simplify validateRmOptions() error handling PR-URL: https://github.com/nodejs/node/pull/35567 Reviewed-By: Jiawen Geng Reviewed-By: Luigi Pinca Reviewed-By: Benjamin Gruenbaum Reviewed-By: Rich Trott Reviewed-By: Anto Aravinth --- lib/internal/fs/utils.js | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/lib/internal/fs/utils.js b/lib/internal/fs/utils.js index ff2033ed32..658f764ee2 100644 --- a/lib/internal/fs/utils.js +++ b/lib/internal/fs/utils.js @@ -684,17 +684,13 @@ const validateRmOptions = hideStackFrames((path, options, callback) => { ); lazyLoadFs().stat(path, (err, stats) => { - if (err && err.code === 'ENOENT') { - if (options.force) { + if (err) { + if (options.force && err.code === 'ENOENT') { return callback(null, options); } return callback(err, options); } - if (err) { - return callback(err); - } - if (stats.isDirectory() && !options.recursive) { return callback(new ERR_FS_EISDIR({ code: 'EISDIR',