fixed lodash#5818

lpabinash · Feb 14, 2024 · 112945a · 112945a
1 parent c7c70a7
commit 112945a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 8 deletions.
diff --git a/src/.internal/baseSum.ts b/src/.internal/baseSum.ts
@@ -7,15 +7,15 @@
  * @returns {number} Returns the sum.
  */
 function baseSum(array, iteratee) {
-  let result
+  let result;
 
   for (const value of array) {
-    const current = iteratee(value)
-    if (current !== undefined) {
-      result = result === undefined ? current : (result + current)
+    const current = iteratee(value);
+    if (typeof current === 'number' && !isNaN(current)) {
+      result = result === undefined ? current : (result + current);
     }
   }
-  return result
+  return result;
 }
 
-export default baseSum
+export default baseSum;
diff --git a/src/sumBy.ts b/src/sumBy.ts
@@ -18,7 +18,8 @@ import baseSum from './.internal/baseSum.js';
  * // => 20
  */
 function sumBy(array, iteratee) {
-    return array != null && array.length ? baseSum(array, iteratee) : 0;
+    const result = array != null && array.length ? baseSum(array, iteratee) : 0;
+    return typeof result === 'number' && !isNaN(result) ? result : NaN;
 }
 
-export default sumBy;
+export default sumBy;