Skip to content

Commit f81e591

Browse files
author
Martii
committed
Post fix typo
* Show the actual model name in stderr * Elaborated more on line notes Applies to OpenUserJS#126 and trounces profoundly upon OpenUserJS#262 (comment)OpenUserJS#262 (comment)
1 parent 09f174c commit f81e591

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

libs/remove.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -105,12 +105,12 @@ function remove(aModel, aContent, aUser, aReason, aAutomated, aCallback) {
105105
aContent.remove(function (aErr) {
106106
if (aErr) {
107107
console.error('Failed to remove', aModel.modelName);
108-
aCallback(aErr);
108+
aCallback(aErr); // NOTE: Same as `true` but specific e.g. stop all removal(s)
109109
return;
110110
}
111111

112112
if (aModel.modelName === 'User') {
113-
aCallback(true); // NOTE: Stop any series removals
113+
aCallback(true); // NOTE: Stop any series removals and done
114114
} else {
115115
aCallback(null); // NOTE: Continue any series and non-User single removals
116116
}
@@ -128,7 +128,7 @@ exports.remove = function (aModel, aContent, aUser, aReason, aCallback) {
128128
if (aModel.modelName !== 'User') {
129129
remove(aModel, aContent, aUser, aReason, false, function (aErr) {
130130
if (aErr) {
131-
console.warn('Failed to remove User\n', aErr);
131+
console.warn('Failed to remove', aModel.modelName, '\n', aErr);
132132
aCallback(false);
133133
return;
134134
}

0 commit comments

Comments
 (0)