flatten now throws an exception only for nil values, converts ar…

…ray-likes to arrays and treats every other value as an empty array

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.28.0-alpha.14
+ * @version 0.28.0-alpha.15
  * @module lamb
  * @license MIT
  * @preserve
@@ -18,7 +18,7 @@
      * @category Core
      * @type String
      */
-    lamb._version =  "0.28.0-alpha.14";
+    lamb._version =  "0.28.0-alpha.15";
 
     // alias used as a placeholder argument for partial application
     var _ = lamb;
@@ -1919,7 +1919,7 @@
      * @returns {Array}
      */
     function flatten (array) {
-        return _flatten(array, []);
+    	return Array.isArray(array) ? _flatten(array, []) : slice(array);
     }
 
     /**

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.28.0-alpha.14",
+  "version": "0.28.0-alpha.15",
   "devDependencies": {
     "coveralls": "^2.11.9",
     "gulp": "^3.9.1",

src/array.js

@@ -284,7 +284,7 @@ function flatMapWith (iteratee, iterateeContext) {
  * @returns {Array}
  */
 function flatten (array) {
-    return _flatten(array, []);
+	return Array.isArray(array) ? _flatten(array, []) : slice(array);
 }
 
 /**

test/spec/arraySpec.js

@@ -398,9 +398,24 @@ describe("lamb.array", function () {
             expect(lamb.flatten(input)).toEqual(["a", "b", {"c" : ["d"]}]);
         });
 
+		it("should return an array copy of the source object if supplied with an array-like", function () {
+			expect(lamb.flatten("foo")).toEqual(["f", "o", "o"]);
+		});
+
         it("should throw an exception if called without arguments", function () {
             expect(lamb.flatten).toThrow();
         });
+
+        it("should throw an exception if supplied with `null` or `undefined` instead of an array-like", function () {
+            expect(function () { lamb.flatten(null); }).toThrow();
+            expect(function () { lamb.flatten(void 0); }).toThrow();
+        });
+
+        it("should treat every other value as an empty array", function () {
+            [{}, /foo/, 1, function () {}, NaN, true, new Date()].forEach(function (value) {
+                expect(lamb.flatten(value)).toEqual([]);
+            });
+        });
     });
 
     describe("init", function () {