From d150dc7d026e89e042bc134b03fd69fd3e2023aa Mon Sep 17 00:00:00 2001 From: sgorsh Date: Sun, 9 Apr 2017 14:00:34 +0200 Subject: [PATCH] Use getOwnPropertyNames to check for empty object. Added hasOwnProperty check to for..in loop. --- src/lib/utils/is-object-empty.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/lib/utils/is-object-empty.js b/src/lib/utils/is-object-empty.js index c8c765e72..695c3d248 100644 --- a/src/lib/utils/is-object-empty.js +++ b/src/lib/utils/is-object-empty.js @@ -1,11 +1,12 @@ export default function isObjectEmpty(obj) { - if (Object.keys) { - return (Object.keys(obj).length === 0); + if (Object.getOwnPropertyNames) { + return (Object.getOwnPropertyNames(obj).length === 0); } else { var k; for (k in obj) { - // even if its not own property I'd still call it non-empty - return false; + if (obj.hasOwnProperty(k)) { + return false; + } } return true; } -- 2.47.2