updated swagger-js for https://github.com/wordnik/swagger-js/issues/151

dist/lib/swagger-client.js

@@ -1104,6 +1104,7 @@ var Property = function(name, obj, required) {
   this.description = obj.description;
   this.obj = obj;
   this.optional = true;
+  this.default = obj.default || null;
   this.example = obj.example || null;
 }
 
@@ -1136,6 +1137,8 @@ Property.prototype.sampleValue = function(isArray, ignoredModels) {
   }
   else if(this.example)
     output = this.example;
+  else if(this.default)
+    output = this.default;
   else if(type === 'date-time')
     output = new Date().toISOString();
   else if(type === 'string')

dist/lib/swagger.js

@@ -1,5 +1,7 @@
 // swagger.js
-// version 2.0.34
+// version 2.0.41
+
+(function () {
 
   var __bind = function (fn, me) {
     return function () {
@@ -7,7 +9,7 @@ var __bind = function(fn, me){
     };
   };
 
-log = function(){
+  var log = function () {
     log.history = log.history || [];
     log.history.push(arguments);
     if (this.console) {
@@ -15,6 +17,16 @@ log = function(){
     }
   };
 
+  // if you want to apply conditional formatting of parameter values
+  var parameterMacro = function (value) {
+    return value;
+  }
+
+  // if you want to apply conditional formatting of model property values
+  var modelPropertyMacro = function (value) {
+    return value;
+  }
+
   if (!Array.prototype.indexOf) {
     Array.prototype.indexOf = function (obj, start) {
       for (var i = (start || 0), j = this.length; i < j; i++) {
@@ -79,8 +91,8 @@ Object.keys = Object.keys || (function () {
     };
   })();
 
-
   var SwaggerApi = function (url, options) {
+    this.isBuilt = false;
     this.url = null;
     this.debug = false;
     this.basePath = null;
@@ -88,6 +100,8 @@ var SwaggerApi = function(url, options) {
     this.authorizationScheme = null;
     this.info = null;
     this.useJQuery = false;
+    this.modelsArray = [];
+    this.isValid;
 
     options = (options || {});
     if (url)
@@ -109,11 +123,15 @@ var SwaggerApi = function(url, options) {
 
     this.failure = options.failure != null ? options.failure : function () { };
     this.progress = options.progress != null ? options.progress : function () { };
-  if (options.success != null)
+    if (options.success != null) {
       this.build();
+      this.isBuilt = true;
+    }
   }
 
   SwaggerApi.prototype.build = function () {
+    if (this.isBuilt)
+      return this;
     var _this = this;
     this.progress('fetching resource list: ' + this.url);
     var obj = {
@@ -121,7 +139,7 @@ SwaggerApi.prototype.build = function() {
       url: this.url,
       method: "get",
       headers: {
-      accept: "application/json"
+        accept: "application/json,application/json;charset=\"utf-8\",*/*"
       },
       on: {
         error: function (response) {
@@ -158,6 +176,7 @@ SwaggerApi.prototype.buildFromSpec = function(response) {
     }
     this.apis = {};
     this.apisArray = [];
+    this.consumes = response.consumes;
     this.produces = response.produces;
     this.authSchemes = response.authorizations;
     if (response.info != null) {
@@ -175,28 +194,29 @@ SwaggerApi.prototype.buildFromSpec = function(response) {
         }
       }
     }
-  if (response.basePath) {
+    if (response.basePath)
       this.basePath = response.basePath;
-  } else if (this.url.indexOf('?') > 0) {
+    else if (this.url.indexOf('?') > 0)
       this.basePath = this.url.substring(0, this.url.lastIndexOf('?'));
-  } else {
+    else
       this.basePath = this.url;
-  }
+
     if (isApi) {
       var newName = response.resourcePath.replace(/\//g, '');
       this.resourcePath = response.resourcePath;
-    res = new SwaggerResource(response, this);
+      var res = new SwaggerResource(response, this);
       this.apis[newName] = res;
       this.apisArray.push(res);
     } else {
       var k;
       for (k = 0; k < response.apis.length; k++) {
         var resource = response.apis[k];
-      res = new SwaggerResource(resource, this);
+        var res = new SwaggerResource(resource, this);
         this.apis[res.name] = res;
         this.apisArray.push(res);
       }
     }
+    this.isValid = true;
     if (this.success) {
       this.success();
     }
@@ -239,11 +259,12 @@ SwaggerApi.prototype.buildFrom1_1Spec = function(response) {
     } else {
       for (k = 0; k < response.apis.length; k++) {
         resource = response.apis[k];
-      res = new SwaggerResource(resource, this);
+        var res = new SwaggerResource(resource, this);
         this.apis[res.name] = res;
         this.apisArray.push(res);
       }
     }
+    this.isValid = true;
     if (this.success) {
       this.success();
     }
@@ -276,7 +297,6 @@ SwaggerApi.prototype.fail = function(message) {
 
   SwaggerApi.prototype.setConsolidatedModels = function () {
     var model, modelName, resource, resource_name, _i, _len, _ref, _ref1, _results;
-  this.modelsArray = [];
     this.models = {};
     _ref = this.apis;
     for (resource_name in _ref) {
@@ -321,8 +341,8 @@ var SwaggerResource = function(resourceObj, api) {
     var _this = this;
     this.api = api;
     this.api = this.api;
-  produces = [];
+    var consumes = (this.consumes | []);
-  consumes = [];
+    var produces = (this.produces | []);
     this.path = this.api.resourcePath != null ? this.api.resourcePath : resourceObj.path;
     this.description = resourceObj.description;
 
@@ -348,12 +368,12 @@ var SwaggerResource = function(resourceObj, api) {
         this.url = this.api.basePath + this.path.replace('{format}', 'json');
       }
       this.api.progress('fetching resource ' + this.name + ': ' + this.url);
-    obj = {
+      var obj = {
         url: this.url,
         method: "get",
         useJQuery: this.useJQuery,
         headers: {
-        accept: "application/json"
+          accept: "application/json,application/json;charset=\"utf-8\",*/*"
         },
         on: {
           response: function (resp) {
@@ -430,7 +450,7 @@ SwaggerResource.prototype.addModels = function(models) {
       }
       var output = [];
       for (var i = 0; i < this.modelsArray.length; i++) {
-      model = this.modelsArray[i];
+        var model = this.modelsArray[i];
         output.push(model.setReferencedModels(this.models));
       }
       return output;
@@ -439,9 +459,9 @@ SwaggerResource.prototype.addModels = function(models) {
 
   SwaggerResource.prototype.addOperations = function (resource_path, ops, consumes, produces) {
     if (ops) {
-    output = [];
+      var output = [];
       for (var i = 0; i < ops.length; i++) {
-      o = ops[i];
+        var o = ops[i];
         consumes = this.consumes;
         produces = this.produces;
         if (o.consumes != null)
@@ -453,7 +473,7 @@ SwaggerResource.prototype.addOperations = function(resource_path, ops, consumes,
           produces = o.produces;
         else
           produces = this.produces;
-      type = (o.type||o.responseClass);
+        var type = (o.type || o.responseClass);
 
         if (type === "array") {
           ref = null;
@@ -461,8 +481,8 @@ SwaggerResource.prototype.addOperations = function(resource_path, ops, consumes,
             ref = o.items["type"] || o.items["$ref"];
           type = "array[" + ref + "]";
         }
-      responseMessages = o.responseMessages;
+        var responseMessages = o.responseMessages;
-      method = o.method;
+        var method = o.method;
         if (o.httpMethod) {
           method = o.httpMethod;
         }
@@ -478,7 +498,7 @@ SwaggerResource.prototype.addOperations = function(resource_path, ops, consumes,
           }
         }
         o.nickname = this.sanitize(o.nickname);
-      op = new SwaggerOperation(o.nickname, resource_path, method, o.parameters, o.summary, o.notes, type, responseMessages, this, consumes, produces, o.authorizations);
+        var op = new SwaggerOperation(o.nickname, resource_path, method, o.parameters, o.summary, o.notes, type, responseMessages, this, consumes, produces, o.authorizations);
         this.operations[op.nickname] = op;
         output.push(this.operationsArray.push(op));
       }
@@ -524,7 +544,7 @@ var SwaggerModel = function(modelName, obj) {
           }
         }
       }
-    prop = new SwaggerModelProperty(propertyName, obj.properties[propertyName]);
+      var prop = new SwaggerModelProperty(propertyName, obj.properties[propertyName]);
       this.properties.push(prop);
     }
   }
@@ -547,7 +567,7 @@ SwaggerModel.prototype.setReferencedModels = function(allModels) {
   SwaggerModel.prototype.getMockSignature = function (modelsToIgnore) {
     var propertiesStr = [];
     for (var i = 0; i < this.properties.length; i++) {
-    prop = this.properties[i];
+      var prop = this.properties[i];
       propertiesStr.push(prop.toString());
     }
 
@@ -562,7 +582,7 @@ SwaggerModel.prototype.getMockSignature = function(modelsToIgnore) {
     modelsToIgnore.push(this.name);
 
     for (var i = 0; i < this.properties.length; i++) {
-    prop = this.properties[i];
+      var prop = this.properties[i];
       if ((prop.refModel != null) && modelsToIgnore.indexOf(prop.refModel.name) === -1) {
         returnVal = returnVal + ('<br>' + prop.refModel.getMockSignature(modelsToIgnore));
       }
@@ -579,7 +599,7 @@ SwaggerModel.prototype.createJSONSample = function(modelsToIgnore) {
       var modelsToIgnore = (modelsToIgnore || [])
       modelsToIgnore.push(this.name);
       for (var i = 0; i < this.properties.length; i++) {
-      prop = this.properties[i];
+        var prop = this.properties[i];
         result[prop.name] = prop.getSampleValue(modelsToIgnore);
       }
       modelsToIgnore.pop(this.name);
@@ -593,6 +613,7 @@ var SwaggerModelProperty = function(name, obj) {
     this.isCollection = this.dataType && (this.dataType.toLowerCase() === 'array' || this.dataType.toLowerCase() === 'list' || this.dataType.toLowerCase() === 'set');
     this.descr = obj.description;
     this.required = obj.required;
+    this.defaultValue = modelPropertyMacro(obj.defaultValue);
     if (obj.items != null) {
       if (obj.items.type != null) {
         this.refDataType = obj.items.type;
@@ -620,7 +641,7 @@ var SwaggerModelProperty = function(name, obj) {
 
   SwaggerModelProperty.prototype.getSampleValue = function (modelsToIgnore) {
     var result;
-  if ((this.refModel != null) && (modelsToIgnore.indexOf(prop.refModel.name) === -1)) {
+    if ((this.refModel != null) && (modelsToIgnore.indexOf(this.refModel.name) === -1)) {
       result = this.refModel.createJSONSample(modelsToIgnore);
     } else {
       if (this.isCollection) {
@@ -638,7 +659,9 @@ SwaggerModelProperty.prototype.getSampleValue = function(modelsToIgnore) {
 
   SwaggerModelProperty.prototype.toSampleValue = function (value) {
     var result;
-  if (value === "integer") {
+    if ((typeof this.defaultValue !== 'undefined') && this.defaultValue !== null) {
+      result = this.defaultValue;
+    } else if (value === "integer") {
       result = 0;
     } else if (value === "boolean") {
       result = false;
@@ -686,8 +709,10 @@ var SwaggerOperation = function(nickname, path, method, parameters, summary, not
     this.authorizations = authorizations;
     this["do"] = __bind(this["do"], this);
 
-  if (errors.length > 0)
+    if (errors.length > 0) {
+      console.error('SwaggerOperation errors', errors, arguments);
       this.resource.api.fail(errors);
+    }
 
     this.path = this.path.replace('{format}', 'json');
     this.method = this.method.toLowerCase();
@@ -768,6 +793,7 @@ var SwaggerOperation = function(nickname, path, method, parameters, summary, not
           }
         }
       }
+      param.defaultValue = parameterMacro(param.defaultValue);
     }
     this.resource[this.nickname] = function (args, callback, error) {
       return _this["do"](args, callback, error);
@@ -888,7 +914,7 @@ SwaggerOperation.prototype["do"] = function(args, opts, callback, error) {
     if (possibleParams) {
       var key;
       for (key in possibleParams) {
-      value = possibleParams[key];
+        var value = possibleParams[key];
         if (args[value.name]) {
           params[value.name] = args[value.name];
         }
@@ -935,7 +961,7 @@ SwaggerOperation.prototype.urlify = function(args) {
       if (param.paramType === 'path') {
         if (args[param.name]) {
           // apply path params and remove from args
-        var reg = new RegExp('\{' + param.name + '[^\}]*\}', 'gi');
+          var reg = new RegExp('\\{\\s*?' + param.name + '.*?\\}(?=\\s*?(\\/|$))', 'gi');
           url = url.replace(reg, this.encodePathParam(args[param.name]));
           delete args[param.name];
         }
@@ -949,12 +975,25 @@ SwaggerOperation.prototype.urlify = function(args) {
       var param = params[i];
       if (param.paramType === 'query') {
         if (args[param.name] !== undefined) {
+          var value = args[param.name];
           if (queryParams !== '')
-          queryParams += "&";
+            queryParams += '&';
+          if (Array.isArray(value)) {
+            var j;
+            var output = '';
+            for(j = 0; j < value.length; j++) {
+              if(j > 0)
+                output += ',';
+              output += encodeURIComponent(value[j]);
+            }
+            queryParams += encodeURIComponent(param.name) + '=' + output;
+          }
+          else {
             queryParams += encodeURIComponent(param.name) + '=' + encodeURIComponent(args[param.name]);            
           }
         }
       }
+    }
     if ((queryParams != null) && queryParams.length > 0)
       url += '?' + queryParams;
     return url;
@@ -1104,18 +1143,19 @@ var SwaggerRequest = function(type, url, params, opts, successCallback, errorCal
     var body = params.body;
 
     // encode the body for form submits
-  if (headers["Accept"] && headers["Accept"].indexOf("application/x-www-form-urlencoded") === 0) {
+    if (headers["Content-Type"]) {
-    var fields = {};
-    var possibleParams = {};
       var values = {};
-    var key;
+      var i;
-    for(key in formParams){
+      var operationParams = this.operation.parameters;
-      var param = formParams[key];
+      for (i = 0; i < operationParams.length; i++) {
+        var param = operationParams[i];
+        if (param.paramType === "form")
           values[param.name] = param;
       }
 
+      if (headers["Content-Type"].indexOf("application/x-www-form-urlencoded") === 0) {
         var encoded = "";
-    var key;
+        var key, value;
         for (key in values) {
           value = this.params[key];
           if (typeof value !== 'undefined') {
@@ -1126,7 +1166,27 @@ var SwaggerRequest = function(type, url, params, opts, successCallback, errorCal
         }
         body = encoded;
       }
+      else if (headers["Content-Type"].indexOf("multipart/form-data") === 0) {
+        // encode the body for form submits
+        var data = "";
+        var boundary = "----SwaggerFormBoundary" + Date.now();
+        var key, value;
+        for (key in values) {
+          value = this.params[key];
+          if (typeof value !== 'undefined') {
+            data += '--' + boundary + '\n';
+            data += 'Content-Disposition: form-data; name="' + key + '"';
+            data += '\n\n';
+            data += value + "\n";
+          }
+        }
+        data += "--" + boundary + "--\n";
+        headers["Content-Type"] = "multipart/form-data; boundary=" + boundary;
+        body = data;
+      }
+    }
 
+    var obj;
     if (!((this.headers != null) && (this.headers.mock != null))) {
       obj = {
         url: this.url,
@@ -1155,7 +1215,7 @@ var SwaggerRequest = function(type, url, params, opts, successCallback, errorCal
       } else {
         e = exports;
       }
-    status = e.authorizations.apply(obj, this.operation.authorizations);
+      var status = e.authorizations.apply(obj, this.operation.authorizations);
       if (opts.mock == null) {
         if (status !== false) {
           new SwaggerHttp().execute(obj);
@@ -1166,12 +1226,13 @@ var SwaggerRequest = function(type, url, params, opts, successCallback, errorCal
         return obj;
       }
     }
+    return obj;
   };
 
   SwaggerRequest.prototype.setHeaders = function (params, operation) {
     // default type
     var accepts = "application/json";
-  var contentType = null;
+    var consumes = "application/json";
 
     var allDefinedParams = this.operation.parameters;
     var definedFormParams = [];
@@ -1198,25 +1259,25 @@ SwaggerRequest.prototype.setHeaders = function(params, operation) {
     // if there's a body, need to set the accepts header via requestContentType
     if (body && (this.type === "POST" || this.type === "PUT" || this.type === "PATCH" || this.type === "DELETE")) {
       if (this.opts.requestContentType)
-      accepts = this.opts.requestContentType;
+        consumes = this.opts.requestContentType;
     } else {
       // if any form params, content type must be set
       if (definedFormParams.length > 0) {
         if (definedFileParams.length > 0)
-        accepts = "multipart/form-data";
+          consumes = "multipart/form-data";
         else
-        accepts = "application/x-www-form-urlencoded";
+          consumes = "application/x-www-form-urlencoded";
       }
-    else if (this.type == "DELETE")
+      else if (this.type === "DELETE")
         body = "{}";
       else if (this.type != "DELETE")
         accepts = null;
     }
 
-  if (contentType && this.operation.consumes) {
+    if (consumes && this.operation.consumes) {
-    if (this.operation.consumes.indexOf(contentType) === -1) {
+      if (this.operation.consumes.indexOf(consumes) === -1) {
-      log("server doesn't consume " + contentType + ", try " + JSON.stringify(this.operation.consumes));
+        log("server doesn't consume " + consumes + ", try " + JSON.stringify(this.operation.consumes));
-      contentType = this.operation.consumes[0];
+        consumes = this.operation.consumes[0];
       }
     }
 
@@ -1231,33 +1292,11 @@ SwaggerRequest.prototype.setHeaders = function(params, operation) {
         accepts = this.operation.produces[0];
       }
     }
-  if (contentType && contentType.indexOf("application/x-www-form-urlencoded") === 0) {
-    var fields = {};
-    var possibleParams = {};
-    var values = {};
-    var key;
-    for(key in formParams){
-      var param = formParams[key];
-      values[param.name] = param;
-    }
 
-    var encoded = "";
+    if ((consumes && body !== "") || (consumes === "application/x-www-form-urlencoded"))
-    var key;
+      headers["Content-Type"] = consumes;
-    for(key in values) {
-      value = this.params[key];
-      if(typeof value !== 'undefined'){
-        if(encoded !== "")
-          encoded += "&";
-        encoded += encodeURIComponent(key) + '=' + encodeURIComponent(value);
-      }
-    }
-    body = encoded;
-  }
-  if ((contentType && body !== "") || (contentType === "application/x-www-form-urlencoded"))
-    headers["Content-Type"] = contentType;
     if (accepts)
       headers["Accept"] = accepts;
-
     return headers;
   }
 
@@ -1463,11 +1502,36 @@ ShredHttpClient.prototype.execute = function(obj) {
       return out;
     };
 
-  res = {
+    // Transform an error into a usable response-like object
+    var transformError = function (error) {
+      var out = {
+        // Default to a status of 0 - The client will treat this as a generic permissions sort of error
+        status: 0,
+        data: error.message || error
+      };
+
+      if (error.code) {
+        out.obj = error;
+
+        if (error.code === 'ENOTFOUND' || error.code === 'ECONNREFUSED') {
+          // We can tell the client that this should be treated as a missing resource and not as a permissions thing
+          out.status = 404;
+        }
+      }
+
+      return out;
+    };
+
+    var res = {
       error: function (response) {
         if (obj)
           return cb.error(transform(response));
       },
+      // Catch the Shred error raised when the request errors as it is made (i.e. No Response is coming)
+      request_error: function (err) {
+        if (obj)
+          return cb.error(transformError(err));
+      },
       redirect: function (response) {
         if (obj)
           return cb.redirect(transform(response));
@@ -1505,7 +1569,7 @@ SwaggerAuthorizations.prototype.remove = function(name) {
 
   SwaggerAuthorizations.prototype.apply = function (obj, authorizations) {
     var status = null;
-  var key;
+    var key, value, result;
 
     // if the "authorizations" key is undefined, or has an empty array, add all keys
     if (typeof authorizations === 'undefined' || Object.keys(authorizations).length == 0) {
@@ -1535,10 +1599,11 @@ SwaggerAuthorizations.prototype.apply = function(obj, authorizations) {
   /**
    * ApiKeyAuthorization allows a query param or header to be injected
    */
-var ApiKeyAuthorization = function(name, value, type) {
+  var ApiKeyAuthorization = function (name, value, type, delimiter) {
     this.name = name;
     this.value = value;
     this.type = type;
+    this.delimiter = delimiter;
   };
 
   ApiKeyAuthorization.prototype.apply = function (obj, authorizations) {
@@ -1549,6 +1614,11 @@ ApiKeyAuthorization.prototype.apply = function(obj, authorizations) {
         obj.url = obj.url + "?" + this.name + "=" + this.value;
       return true;
     } else if (this.type === "header") {
+      if (typeof obj.headers[this.name] !== 'undefined') {
+        if (typeof this.delimiter !== 'undefined')
+          obj.headers[this.name] = obj.headers[this.name] + this.delimiter + this.value;
+      }
+      else
         obj.headers[this.name] = this.value;
       return true;
     }
@@ -1603,3 +1673,6 @@ e.SwaggerModel = SwaggerModel;
   e.SwaggerModelProperty = SwaggerModelProperty;
   e.SwaggerResource = SwaggerResource;
   e.SwaggerApi = SwaggerApi;
+  e.log = log;
+
+})();

lib/swagger-client.js

@@ -1104,6 +1104,7 @@ var Property = function(name, obj, required) {
   this.description = obj.description;
   this.obj = obj;
   this.optional = true;
+  this.default = obj.default || null;
   this.example = obj.example || null;
 }
 
@@ -1136,6 +1137,8 @@ Property.prototype.sampleValue = function(isArray, ignoredModels) {
   }
   else if(this.example)
     output = this.example;
+  else if(this.default)
+    output = this.default;
   else if(type === 'date-time')
     output = new Date().toISOString();
   else if(type === 'string')