Use error handler. Closes #35

.gitignore

@@ -7,3 +7,4 @@ tmp/**
 docs**
 .idea
 node_modules
+testee.log

readme.md

@@ -442,9 +442,9 @@ Append the dForm object to each selected element. If the element is of the same
 a `type : 'form'` on a `<form>`) or if no type has been given run the subscribers and
 add the attributes on the current element. Optionally use a converter with a given name.
 
-**$(form).dform(url \[, success\])** *{String}* *{Function}*<br />
-Load a JSON form definition using GET from a given URL and execute a success handler when it returns.
-The handler gets the data passed and has `this` refer to the form element.
+**$(form).dform(url \[, success\], \[, error\])** *{String}* *{Function}* *{Function}*<br />
+Load a JSON form definition using GET from a given URL and execute a success handler when it returns
+or an error handler if the request faiuls. The handler gets the data passed and has `this` refer to the form element.
 
 **$(form).dform('run', options)** *{Object}*<br />
 Run all subscribers from a given dForm object on the selected element(s).

src/dform.js

@@ -30,7 +30,7 @@
 		 * @param {Object} object The object to use
 		 * @return {Array} An array containing all properties in the object
 		 */
-			keyset = function (object) {
+		keyset = function (object) {
 			return $.map(object, function (val, key) {
 				return key;
 			});
@@ -44,7 +44,7 @@
 		 * @return {Object} A new object containing only the properties
 		 * with names given in keys
 		 */
-			withKeys = function (object, keys) {
+		withKeys = function (object, keys) {
 			var result = {};
 			each(keys, function (index, value) {
 				if (object[value]) {
@@ -80,7 +80,7 @@
 		 * @param {String} type The type of the current element as in the registered types
 		 * @return {Object} The jQuery object
 		 */
-			runSubscription = function (name, options, type) {
+		runSubscription = function (name, options, type) {
 			if ($.dform.hasSubscription(name)) {
 				this.each(function () {
 					var element = $(this);
@@ -98,7 +98,7 @@
 		 * @param {Object} options The options to use
 		 * @return {Object} The jQuery element this function has been called on
 		 */
-			runAll = function (options) {
+		runAll = function (options) {
 			var type = options.type, self = this;
 			// Run preprocessing subscribers
 			this.dform('run', '[pre]', options, type);
@@ -278,18 +278,18 @@
 				 */
 				ajax : function (params, success, error) {
 					var options = {
+						success: function (data) {
+							self.dform(data);
+							if(success) {
+								success.call(self, data);
+							}
+						},
 						error : error,
 						url : params
 					}, self = this;
 					if (typeof params !== 'string') {
 						$.extend(options, params);
 					}
-					options.success = function (data) {
-						self.dform(data);
-						if(success) {
-							success.call(self, data);
-						}
-					}
 					$.ajax(options);
 				},
 				/**

test/test.js

@@ -61,7 +61,10 @@ $(document).ready(function () {
 		$('<div>').dform('test.json', function(data) {
 			equal(data.html, 'The test', 'Data passed to success callback');
 			equal(this.html(), 'The test', 'Form got created');
-			start();
+			$('<div>').dform('missing.json', function() {}, function() {
+				ok(true, 'Error handler called');
+				start();
+			});
 		});
 	})
 });