Array setters now throw exceptions only if they receive nil argumen…

…ts in place of an array-like object and they return an empty array for any other non-array-like object

dist/lamb.js

@@ -1,7 +1,7 @@
 /**
  * @overview lamb - A lightweight, and docile, JavaScript library to help embracing functional programming.
  * @author Andrea Scartabelli <andrea.scartabelli@gmail.com>
- * @version 0.23.0-rc.1
+ * @version 0.23.0-rc.2
  * @module lamb
  * @license MIT
  * @preserve
@@ -18,7 +18,7 @@
      * @category Core
      * @type String
      */
-    lamb._version =  "0.23.0-rc.1";
+    lamb._version =  "0.23.0-rc.2";
 
     // alias used as a placeholder argument for partial application
     var _ = lamb;
@@ -304,7 +304,7 @@
 
     function _setIndex (arrayLike, index, value, updater) {
         var result = slice(arrayLike);
-        var idx = _getNaturalIndex(index, arrayLike.length);
+        var idx = _getNaturalIndex(index, result.length);
 
         if (!isUndefined(idx)) {
             result[idx] = updater ? updater(arrayLike[idx]) : value;

package.json

@@ -31,7 +31,7 @@
     "coveralls": "gulp coverage && cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js && rm -rf ./coverage"
   },
   "tonicExample": "var _ = require('lamb');",
-  "version": "0.23.0-rc.1",
+  "version": "0.23.0-rc.2",
   "devDependencies": {
     "coveralls": "^2.11.8",
     "gulp": "^3.9.1",

src/accessors.js

@@ -9,7 +9,7 @@ function _isEnumerable (obj, key) {
 
 function _setIndex (arrayLike, index, value, updater) {
     var result = slice(arrayLike);
-    var idx = _getNaturalIndex(index, arrayLike.length);
+    var idx = _getNaturalIndex(index, result.length);
 
     if (!isUndefined(idx)) {
         result[idx] = updater ? updater(arrayLike[idx]) : value;

test/spec/accessorsSpec.js

@@ -125,6 +125,13 @@ describe("lamb.accessors", function () {
                 expect(lamb.setAt(0, 99)).toThrow();
                 expect(function () { lamb.setIndex(null, 0, 99); }).toThrow();
             });
+
+            it("should return an empty array if a non-array-like object is supplied", function () {
+                [/foo/, 1, function () {}, NaN, true, new Date()].forEach(function (v) {
+                    expect(lamb.setIndex(v, 2, 99)).toEqual([]);
+                    expect(lamb.setAt(2, 99)(v)).toEqual([]);
+                });
+            });
         });
 
         describe("updateIndex / updateAt", function () {
@@ -210,6 +217,13 @@ describe("lamb.accessors", function () {
                 expect(function () { lamb.updateIndex(null, 0, fn99); }).toThrow();
                 expect(lamb.updateAt(0, fn99)).toThrow();
             });
+
+            it("should return an empty array if a non-array-like object is supplied", function () {
+                [/foo/, 1, function () {}, NaN, true, new Date()].forEach(function (v) {
+                    expect(lamb.updateIndex(v, 2, fn99)).toEqual([]);
+                    expect(lamb.updateAt(2, fn99)(v)).toEqual([]);
+                });
+            });
         });
     });