Mirrors/lodash
1
0
Fork 0
mirror of https://github.com/whoisclebs/lodash.git synced 2026-01-29 14:37:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

Make _.isEmpty work as expected on buffers.

Browse Source
This commit is contained in:
Federico De Giuli
2016-03-22 18:48:11 -04:00
committed by John-David Dalton
parent 583b03ba5f
commit 271f64ee3f
2 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
lodash.js
View File

@@ -10173,8 +10173,8 @@
/**
* Checks if `value` is an empty object or collection. A value is considered
* empty if it's an `arguments` object, array, string, or jQuery-like collection
* with a length of `0` or has no own enumerable string keyed properties.
* empty if it's an `arguments` object, array, buffer, string, or jQuery-like
* collection with a length of `0` or has no own enumerable string keyed properties.
*
* @static
* @memberOf _
@@ -10201,8 +10201,8 @@
*/
function isEmpty(value) {
if (isArrayLike(value) &&
(isArray(value) || isString(value) ||
isFunction(value.splice) || isArguments(value))) {
(isArray(value) || isString(value) || isFunction(value.splice) ||
isArguments(value) || isBuffer(value))) {
return !value.length;
}
for (var key in value) {

10
test/test.js
View File

@@ -8658,7 +8658,7 @@
var args = arguments;
QUnit.test('should return `true` for empty values', function(assert) {
assert.expect(8);
assert.expect(10);
var expected = lodashStable.map(empties, alwaysTrue),
actual = lodashStable.map(empties, _.isEmpty);
@@ -8672,6 +8672,14 @@
assert.strictEqual(_.isEmpty(/x/), true);
assert.strictEqual(_.isEmpty(symbol), true);
assert.strictEqual(_.isEmpty(), true);
if (Buffer) {
assert.strictEqual(_.isEmpty(new Buffer(0)), true);
assert.strictEqual(_.isEmpty(new Buffer(1)), false);
}
else {
skipAssert(assert, 2);
}
});
QUnit.test('should return `false` for non-empty values', function(assert) {