@kineticdata/react 5.1.0-rc.1 → 5.1.1

package/lib/apis/core/attributeDefinitions.js

@@ -32,11 +32,13 @@ var validateOptions = function validateOptions(functionName, requiredOptions, op
 };
 
 var buildEndpoint = function buildEndpoint(_ref) {
-  var attributeType = _ref.attributeType,
-      kappSlug = _ref.kappSlug,
-      attributeName = _ref.attributeName;
+  var kappSlug = _ref.kappSlug,
+      at = _ref.attributeType,
+      an = _ref.attributeName;
+  var attributeType = encodeURIComponent(at);
+  var attributeName = encodeURIComponent(an);
   var basePath = kappSlug ? "".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/").concat(attributeType) : "".concat(bundle.apiLocation(), "/").concat(attributeType);
-  return attributeName ? "".concat(basePath, "/").concat(attributeName) : basePath;
+  return an ? "".concat(basePath, "/").concat(attributeName) : basePath;
 };
 
 export var fetchAttributeDefinitions = function fetchAttributeDefinitions() {
@@ -70,12 +72,12 @@ export var createAttributeDefinition = function createAttributeDefinition() {
   var kappSlug = options.kappSlug,
       attributeType = options.attributeType,
       attributeDefinition = options.attributeDefinition;
-  validateOptions('createAttributeDefinition', ['attributeType', 'attributeDefinition'], options); // For Creates, we don't append the name to the basePath (it goes in the body)
-  // so not using the buildEndpoint function
+  validateOptions('createAttributeDefinition', ['attributeType', 'attributeDefinition'], options); // The API returns the singular name of the attribute type, so we remove the "s"
 
-  var basePath = kappSlug ? "".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/").concat(attributeType) : "".concat(bundle.apiLocation(), "/").concat(attributeType); // The API returns the singular name of the attribute type, so we remove the "s"
-
-  return axios.post(basePath, attributeDefinition, {
+  return axios.post(buildEndpoint({
+    kappSlug: kappSlug,
+    attributeType: attributeType
+  }), attributeDefinition, {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {

package/lib/apis/core/attributeDefinitions.test.js

@@ -325,7 +325,7 @@ describe('attributeDefinitions api', function () {
 
             case 3:
               result = _context6.sent;
-              expect(axios.put.mock.calls).toEqual([['space/app/api/v1/spaceAttributeDefinitions/Test Attribute', {
+              expect(axios.put.mock.calls).toEqual([['space/app/api/v1/spaceAttributeDefinitions/Test%20Attribute', {
                 name: 'Test Attribute',
                 description: 'Test Attr Desc',
                 allowsMultiple: true

package/lib/apis/core/authentication.js

@@ -4,7 +4,7 @@ import _asyncToGenerator from "@babel/runtime/helpers/esm/asyncToGenerator";
 import _objectSpread from "@babel/runtime/helpers/esm/objectSpread2";
 import axios from 'axios';
 import { bundle } from '../../helpers';
-import { fetchProfile } from '../../apis';
+import { fetchProfile } from './profile';
 import { handleErrors } from '../http';
 export var login = function login(_ref) {
   var username = _ref.username,
@@ -20,7 +20,7 @@ export var logoutDirect = function logoutDirect() {
   return axios.get("".concat(bundle.spaceLocation(), "/app/logout"));
 };
 var checkedOrigin = process.env.REACT_APP_API_HOST ? process.env.REACT_APP_API_HOST : typeof window !== 'undefined' ? window.location.origin : null;
-var clientId = process.env.REACT_APP_OAUTH_CLIENT_ID || 'system';
+var clientId = process.env.REACT_APP_OAUTH_CLIENT_ID ? process.env.REACT_APP_OAUTH_CLIENT_ID : process.env.NODE_ENV === 'production' ? 'system' : 'system-dev';
 export var retrieveJwt = function retrieveJwt() {
   return new Promise(function (resolve) {
     var iframe = document.createElement('iframe');
@@ -52,9 +52,17 @@ export var singleSignOn = function singleSignOn(spaceSlug, dimensions) {
     var options = _objectSpread(_objectSpread({}, dimensions), getPopupPosition(window, dimensions));
 
     var endpoint = bundle.spaceLocation() + '/app/saml/login/alias/' + spaceSlug;
-    var popup = window.open(endpoint, target, stringifyOptions(options)); // Create an event handler that closes the popup window if we focus the
+    var popup = window.open(endpoint, target, stringifyOptions(options));
+
+    if (!popup) {
+      resolve({
+        error: 'Enterprise Sign In popup was blocked by the browser.'
+      });
+      return;
+    } // Create an event handler that closes the popup window if we focus the
     // parent window
 
+
     var windowFocusHandler = function windowFocusHandler() {
       popup.close();
       window.removeEventListener('focus', windowFocusHandler);
@@ -80,7 +88,7 @@ export var singleSignOn = function singleSignOn(spaceSlug, dimensions) {
                 }
 
                 resolve({
-                  error: 'Single Sign-on cancelled'
+                  error: 'Enterprise Sign In was cancelled.'
                 });
                 _context.next = 12;
                 break;
@@ -107,7 +115,7 @@ export var singleSignOn = function singleSignOn(spaceSlug, dimensions) {
                 } else {
                   popup.close();
                   resolve({
-                    error: 'Single Sign-on timed out'
+                    error: 'Enterprise Sign In timed out.'
                   });
                 }
 

package/lib/apis/core/backgroundJobs.js

@@ -9,8 +9,9 @@ var backgroundJobPath = function backgroundJobPath() {
       kappSlug = _ref.kappSlug,
       job = _ref.job;
 
+  var j = encodeURIComponent(job);
   var basePath = !formSlug && !kappSlug ? "".concat(bundle.apiLocation(), "/backgroundJobs") : !formSlug && kappSlug ? "".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/backgroundJobs") : formSlug && !kappSlug ? "".concat(bundle.apiLocation(), "/forms/").concat(formSlug, "/backgroundJobs") : "".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/forms/").concat(formSlug, "/backgroundJobs");
-  return basePath + (job ? "/".concat(job) : '');
+  return basePath + (job ? "/".concat(j) : '');
 };
 
 export var fetchBackgroundJobs = function fetchBackgroundJobs() {

package/lib/apis/core/bridgeModelAttributeMappings.js

@@ -22,7 +22,7 @@ var _apiGroup = apiGroup({
       var modelName = _ref2.modelName,
           mappingName = _ref2.mappingName,
           attributeName = _ref2.attributeName;
-      return "/models/".concat(modelName, "/mappings/").concat(mappingName, "/attributes/").concat(attributeName);
+      return "/models/".concat(modelName, "/mappings/").concat(mappingName, "/attributes/").concat(encodeURIComponent(attributeName));
     },
     transform: function transform(response) {
       return {

package/lib/apis/core/bridgeModelAttributes.js

@@ -1,10 +1,19 @@
 import axios from 'axios';
 import { bundle } from '../../helpers';
 import { handleErrors, headerBuilder, paramBuilder, validateOptions } from '../http';
+
+var buildEndpoint = function buildEndpoint(_ref) {
+  var modelName = _ref.modelName,
+      attributeName = _ref.attributeName;
+  var an = encodeURIComponent(attributeName);
+  var mn = encodeURIComponent(modelName);
+  return attributeName ? "".concat(bundle.apiLocation(), "/models/").concat(mn, "/attributes") : "".concat(bundle.apiLocation(), "/models/").concat(mn, "/attributes/").concat(an);
+};
+
 export var fetchBridgeModelAttributes = function fetchBridgeModelAttributes() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   validateOptions('fetchBridgeModelAttributes', ['modelName'], options);
-  return axios.get("".concat(bundle.apiLocation(), "/models/").concat(options.modelName, "/attributes"), {
+  return axios.get(buildEndpoint(options), {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -16,9 +25,7 @@ export var fetchBridgeModelAttributes = function fetchBridgeModelAttributes() {
 export var fetchBridgeModelAttribute = function fetchBridgeModelAttribute() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   validateOptions('fetchBridgeModelAttribute', ['modelName', 'attributeName'], options);
-  var modelName = options.modelName,
-      attributeName = options.attributeName;
-  return axios.get("".concat(bundle.apiLocation(), "/models/").concat(modelName, "/attributes/").concat(attributeName), {
+  return axios.get(buildEndpoint(options), {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -30,7 +37,7 @@ export var fetchBridgeModelAttribute = function fetchBridgeModelAttribute() {
 export var createBridgeModelAttribute = function createBridgeModelAttribute() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   validateOptions('createBridgeModelAttribute', ['modelName', 'bridgeModelAttribute'], options);
-  return axios.post("".concat(bundle.apiLocation(), "/models/").concat(options.modelName, "/attributes"), options.bridgeModelAttribute, {
+  return axios.post(buildEndpoint(options), options.bridgeModelAttribute, {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -42,10 +49,8 @@ export var createBridgeModelAttribute = function createBridgeModelAttribute() {
 export var updateBridgeModelAttribute = function updateBridgeModelAttribute() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   validateOptions('updateBridgeModelAttribute', ['modelName', 'attributeName', 'bridgeModelAttribute'], options);
-  var modelName = options.modelName,
-      attributeName = options.attributeName,
-      bridgeModelAttribute = options.bridgeModelAttribute;
-  return axios.put("".concat(bundle.apiLocation(), "/models/").concat(modelName, "/attributes/").concat(attributeName), bridgeModelAttribute, {
+  var bridgeModelAttribute = options.bridgeModelAttribute;
+  return axios.put(buildEndpoint(options), bridgeModelAttribute, {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -57,9 +62,7 @@ export var updateBridgeModelAttribute = function updateBridgeModelAttribute() {
 export var deleteBridgeModelAttribute = function deleteBridgeModelAttribute() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   validateOptions('deleteBridgeModelAttribute', ['modelName', 'attributeName'], options);
-  var modelName = options.modelName,
-      attributeName = options.attributeName;
-  return axios["delete"]("".concat(bundle.apiLocation(), "/models/").concat(modelName, "/attributes/").concat(attributeName), {
+  return axios["delete"](buildEndpoint(options), {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {

package/lib/apis/core/bridgeModelMappings.js

@@ -20,7 +20,7 @@ var _apiGroup = apiGroup({
     url: function url(_ref2) {
       var modelName = _ref2.modelName,
           mappingName = _ref2.mappingName;
-      return "/models/".concat(modelName, "/mappings/").concat(mappingName);
+      return "/models/".concat(encodeURIComponent(modelName), "/mappings/").concat(encodeURIComponent(mappingName));
     },
     transform: function transform(response) {
       return {

package/lib/apis/core/bridgeModelQualificationMappings.js

@@ -8,7 +8,7 @@ var _apiGroup = apiGroup({
     url: function url(_ref) {
       var modelName = _ref.modelName,
           mappingName = _ref.mappingName;
-      return "/models/".concat(modelName, "/mappings/").concat(mappingName, "/qualifications");
+      return "/models/".concat(encodeURIComponent(modelName), "/mappings/").concat(encodeURIComponent(mappingName), "/qualifications");
     },
     transform: function transform(response) {
       return {
@@ -22,7 +22,7 @@ var _apiGroup = apiGroup({
       var modelName = _ref2.modelName,
           mappingName = _ref2.mappingName,
           qualificationName = _ref2.qualificationName;
-      return "/models/".concat(modelName, "/mappings/").concat(mappingName, "/qualifications/").concat(qualificationName);
+      return "/models/".concat(encodeURIComponent(modelName), "/mappings/").concat(encodeURIComponent(mappingName), "/qualifications/").concat(encodeURIComponent(qualificationName));
     },
     transform: function transform(response) {
       return {

package/lib/apis/core/bridgeModelQualifications.js

@@ -7,7 +7,7 @@ var _apiGroup = apiGroup({
     requiredOptions: ['modelName'],
     url: function url(_ref) {
       var modelName = _ref.modelName;
-      return "/models/".concat(modelName, "/qualifications");
+      return "/models/".concat(encodeURIComponent(modelName), "/qualifications");
     },
     transform: function transform(response) {
       return {
@@ -20,7 +20,7 @@ var _apiGroup = apiGroup({
     url: function url(_ref2) {
       var modelName = _ref2.modelName,
           qualificationName = _ref2.qualificationName;
-      return "/models/".concat(modelName, "/qualifications/").concat(qualificationName);
+      return "/models/".concat(encodeURIComponent(modelName), "/qualifications/").concat(encodeURIComponent(qualificationName));
     },
     transform: function transform(response) {
       return {

package/lib/apis/core/bridgeModels.js

@@ -2,9 +2,16 @@ import _objectSpread from "@babel/runtime/helpers/esm/objectSpread2";
 import axios from 'axios';
 import { bundle } from '../../helpers';
 import { handleErrors, headerBuilder, paramBuilder, validateOptions } from '../http';
+
+var buildEndpoint = function buildEndpoint(_ref) {
+  var modelName = _ref.modelName;
+  var mn = encodeURIComponent(modelName);
+  return modelName ? "".concat(bundle.apiLocation(), "/models/").concat(mn) : "".concat(bundle.apiLocation(), "/models");
+};
+
 export var fetchBridgeModels = function fetchBridgeModels() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
-  return axios.get("".concat(bundle.apiLocation(), "/models"), {
+  return axios.get(buildEndpoint(options), {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -16,7 +23,7 @@ export var fetchBridgeModels = function fetchBridgeModels() {
 export var fetchBridgeModel = function fetchBridgeModel() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   validateOptions('fetchBridgeModel', ['modelName'], options);
-  return axios.get("".concat(bundle.apiLocation(), "/models/").concat(options.modelName), {
+  return axios.get(buildEndpoint(options), {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -29,8 +36,7 @@ var TEST_METHODS = ['retrieve', 'search', 'count'];
 export var testBridgeModel = function testBridgeModel() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   validateOptions('testBridgeModel', ['modelName', 'qualificationName', 'method'], options);
-  var modelName = options.modelName,
-      qualificationName = options.qualificationName,
+  var qualificationName = options.qualificationName,
       _options$attributes = options.attributes,
       attributes = _options$attributes === void 0 ? [] : _options$attributes;
   var method = TEST_METHODS.includes(options.method) ? options.method : 'retrieve';
@@ -38,7 +44,7 @@ export var testBridgeModel = function testBridgeModel() {
     params["parameters[".concat(parameter.name, "]")] = parameter.value;
     return params;
   }, {});
-  return axios.post("".concat(bundle.spaceLocation(), "/app/models/").concat(modelName, "/qualifications/").concat(qualificationName, "/").concat(method), null, {
+  return axios.post("".concat(buildEndpoint(options), "/qualifications/").concat(encodeURIComponent(qualificationName), "/").concat(encodeURIComponent(method)), null, {
     params: _objectSpread(_objectSpread({}, paramBuilder(options)), {}, {
       attributes: attributes.join(',')
     }, parameters),
@@ -48,7 +54,7 @@ export var testBridgeModel = function testBridgeModel() {
 export var createBridgeModel = function createBridgeModel() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   validateOptions('createBridgeModel', ['bridgeModel'], options);
-  return axios.post("".concat(bundle.apiLocation(), "/models"), options.bridgeModel, {
+  return axios.post(buildEndpoint(options), options.bridgeModel, {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -60,7 +66,7 @@ export var createBridgeModel = function createBridgeModel() {
 export var updateBridgeModel = function updateBridgeModel() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   validateOptions('updateBridgeModel', ['modelName', 'bridgeModel'], options);
-  return axios.put("".concat(bundle.apiLocation(), "/models/").concat(options.modelName), options.bridgeModel, {
+  return axios.put(buildEndpoint(options), options.bridgeModel, {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -72,7 +78,7 @@ export var updateBridgeModel = function updateBridgeModel() {
 export var deleteBridgeModel = function deleteBridgeModel() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
   validateOptions('deleteBridgeModel', ['modelName'], options);
-  return axios["delete"]("".concat(bundle.apiLocation(), "/models/").concat(options.modelName), {
+  return axios["delete"](buildEndpoint(options), {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {

package/lib/apis/core/bridgedresources.js

@@ -31,10 +31,11 @@ export var bridgedResourceUrl = function bridgedResourceUrl(options) {
 
   if (!bridgedResourceName) {
     throw new Error('Property "bridgedResourceName" is required.');
-  } // build the url
+  }
 
+  var brn = encodeURIComponent(options.bridgedResourceName); // build the url
 
-  var url = datastore ? "".concat(bundle.spaceLocation(), "/app/datastore/forms/").concat(options.formSlug, "/bridgedResources/").concat(encodeURIComponent(options.bridgedResourceName)) : "".concat(bundle.spaceLocation(), "/").concat(kappSlug, "/").concat(options.formSlug, "/bridgedResources/").concat(encodeURIComponent(options.bridgedResourceName)); // append any attributes if they were specified
+  var url = datastore ? "".concat(bundle.spaceLocation(), "/app/datastore/forms/").concat(formSlug, "/bridgedResources/").concat(brn) : "".concat(bundle.spaceLocation(), "/").concat(kappSlug, "/").concat(formSlug, "/bridgedResources/").concat(brn); // append any attributes if they were specified
 
   if (counting) {
     url += '/count';

package/lib/apis/core/fileResources.js

@@ -0,0 +1,36 @@
+import { apiGroup } from '../http';
+
+var _apiGroup = apiGroup({
+  name: 'FileResource',
+  dataOption: 'fileResource',
+  plural: {
+    requiredOptions: [],
+    url: function url() {
+      return '/fileResources';
+    },
+    transform: function transform(response) {
+      return {
+        fileResources: response.data.fileResources
+      };
+    }
+  },
+  singular: {
+    requiredOptions: ['fileResourceSlug'],
+    url: function url(_ref) {
+      var fileResourceSlug = _ref.fileResourceSlug;
+      return "/fileResources/".concat(fileResourceSlug);
+    },
+    transform: function transform(response) {
+      return {
+        fileResource: response.data.fileResource
+      };
+    }
+  }
+}),
+    fetchFileResources = _apiGroup.fetchFileResources,
+    fetchFileResource = _apiGroup.fetchFileResource,
+    createFileResource = _apiGroup.createFileResource,
+    updateFileResource = _apiGroup.updateFileResource,
+    deleteFileResource = _apiGroup.deleteFileResource;
+
+export { fetchFileResources, fetchFileResource, createFileResource, updateFileResource, deleteFileResource };

package/lib/apis/core/filestores.js

@@ -0,0 +1,40 @@
+import { apiGroup } from '../http';
+
+var _apiGroup = apiGroup({
+  name: 'Filestore',
+  dataOption: 'filestore',
+  plural: {
+    requiredOptions: [],
+    url: function url(_ref) {
+      var _ref$agentSlug = _ref.agentSlug,
+          agentSlug = _ref$agentSlug === void 0 ? 'system' : _ref$agentSlug;
+      return "/app/components/agents/".concat(agentSlug, "/app/api/v1/filestores");
+    },
+    transform: function transform(response) {
+      return {
+        filestores: response.data.filestores
+      };
+    }
+  },
+  singular: {
+    requiredOptions: ['filestoreSlug'],
+    url: function url(_ref2) {
+      var _ref2$agentSlug = _ref2.agentSlug,
+          agentSlug = _ref2$agentSlug === void 0 ? 'system' : _ref2$agentSlug,
+          filestoreSlug = _ref2.filestoreSlug;
+      return "/app/components/agents/".concat(agentSlug, "/app/api/v1/filestores/").concat(filestoreSlug);
+    },
+    transform: function transform(response) {
+      return {
+        filestore: response.data.filestore
+      };
+    }
+  }
+}),
+    fetchFilestores = _apiGroup.fetchFilestores,
+    fetchFilestore = _apiGroup.fetchFilestore,
+    createFilestore = _apiGroup.createFilestore,
+    updateFilestore = _apiGroup.updateFilestore,
+    deleteFilestore = _apiGroup.deleteFilestore;
+
+export { fetchFilestores, fetchFilestore, createFilestore, updateFilestore, deleteFilestore };

package/lib/apis/core/formTypes.js

@@ -1,12 +1,18 @@
 import axios from 'axios';
 import { bundle } from '../../helpers';
 import { handleErrors, headerBuilder, paramBuilder } from '../http';
+
+var buildEndpoint = function buildEndpoint(_ref) {
+  var _ref$kappSlug = _ref.kappSlug,
+      kappSlug = _ref$kappSlug === void 0 ? bundle.kappSlug() : _ref$kappSlug,
+      name = _ref.name;
+  return name ? "".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/formTypes/").concat(encodeURIComponent(name)) : "".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/formTypes");
+};
+
 export var fetchFormTypes = function fetchFormTypes() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
-  var _options$kappSlug = options.kappSlug,
-      kappSlug = _options$kappSlug === void 0 ? bundle.kappSlug() : _options$kappSlug; // Build URL and fetch the form types.
-
-  return axios.get("".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/formTypes"), {
+  // Build URL and fetch the form types.
+  return axios.get(buildEndpoint(options), {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -17,16 +23,14 @@ export var fetchFormTypes = function fetchFormTypes() {
 };
 export var fetchFormType = function fetchFormType() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
-  var _options$kappSlug2 = options.kappSlug,
-      kappSlug = _options$kappSlug2 === void 0 ? bundle.kappSlug() : _options$kappSlug2,
-      name = options.name;
+  var name = options.name;
 
   if (!name) {
     throw new Error('fetchFormType failed! The option "name" is required.');
   } // Build URL and fetch the form type.
 
 
-  return axios.get("".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/formTypes/").concat(name), {
+  return axios.get(buildEndpoint(options), {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -37,16 +41,14 @@ export var fetchFormType = function fetchFormType() {
 };
 export var createFormType = function createFormType() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
-  var _options$kappSlug3 = options.kappSlug,
-      kappSlug = _options$kappSlug3 === void 0 ? bundle.kappSlug() : _options$kappSlug3,
-      formType = options.formType;
+  var formType = options.formType;
 
   if (!formType) {
     throw new Error('createFormType failed! The option "formType" is required.');
   } // Build URL and create the form type.
 
 
-  return axios.post("".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/formTypes"), formType, {
+  return axios.post(buildEndpoint(options), formType, {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -57,9 +59,7 @@ export var createFormType = function createFormType() {
 };
 export var updateFormType = function updateFormType() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
-  var _options$kappSlug4 = options.kappSlug,
-      kappSlug = _options$kappSlug4 === void 0 ? bundle.kappSlug() : _options$kappSlug4,
-      formType = options.formType,
+  var formType = options.formType,
       name = options.name;
 
   if (!formType) {
@@ -71,7 +71,7 @@ export var updateFormType = function updateFormType() {
   } // Build URL and update the form type.
 
 
-  return axios.put("".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/formTypes/").concat(name), formType, {
+  return axios.put(buildEndpoint(options), formType, {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -82,11 +82,8 @@ export var updateFormType = function updateFormType() {
 };
 export var deleteFormType = function deleteFormType() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
-  var _options$kappSlug5 = options.kappSlug,
-      kappSlug = _options$kappSlug5 === void 0 ? bundle.kappSlug() : _options$kappSlug5,
-      name = options.name; // Build URL and delete the form type.
-
-  return axios["delete"]("".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/formTypes/").concat(name), {
+  // Build URL and delete the form type.
+  return axios["delete"](buildEndpoint(options), {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {

package/lib/apis/core/oauthClients.js

@@ -2,9 +2,15 @@ import _objectSpread from "@babel/runtime/helpers/esm/objectSpread2";
 import axios from 'axios';
 import { bundle } from '../../helpers';
 import { handleErrors, paramBuilder, headerBuilder } from '../http';
+
+var buildEndpoint = function buildEndpoint(_ref) {
+  var clientId = _ref.clientId;
+  return clientId ? "".concat(bundle.apiLocation(), "/oauthClients/").concat(encodeURIComponent(clientId)) : "".concat(bundle.apiLocation(), "/oauthClients");
+};
+
 export var fetchOAuthClients = function fetchOAuthClients() {
   var options = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
-  return axios.get("".concat(bundle.apiLocation(), "/oauthClients"), {
+  return axios.get(buildEndpoint(options), {
     params: _objectSpread({}, paramBuilder(options)),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -21,7 +27,7 @@ export var fetchOAuthClient = function fetchOAuthClient() {
     throw new Error('fetchOAuthClient failed! The option "clientId" is required.');
   }
 
-  return axios.get("".concat(bundle.apiLocation(), "/oauthClients/").concat(clientId), {
+  return axios.get(buildEndpoint(options), {
     params: _objectSpread({}, paramBuilder(options)),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -44,7 +50,7 @@ export var updateOAuthClient = function updateOAuthClient() {
   } // Build URL and fetch the space.
 
 
-  return axios.put("".concat(bundle.apiLocation(), "/oauthClients/").concat(clientId), client, {
+  return axios.put(buildEndpoint(options), client, {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -62,7 +68,7 @@ export var createOAuthClient = function createOAuthClient() {
   } // Build URL and fetch the space.
 
 
-  return axios.post("".concat(bundle.apiLocation(), "/oauthClients"), client, {
+  return axios.post(buildEndpoint(options), client, {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {
@@ -80,7 +86,7 @@ export var deleteOAuthClient = function deleteOAuthClient() {
   } // Build URL and fetch the space.
 
 
-  return axios["delete"]("".concat(bundle.apiLocation(), "/oauthClients/").concat(clientId), {
+  return axios["delete"](buildEndpoint(options), {
     params: paramBuilder(options),
     headers: headerBuilder(options)
   }).then(function (response) {

package/lib/apis/core/securityPolicyDefinitions.js

@@ -25,7 +25,7 @@ var buildEndpoint = function buildEndpoint(_ref) {
   var kappSlug = _ref.kappSlug,
       securityPolicyName = _ref.securityPolicyName;
   var basePath = kappSlug ? "".concat(bundle.apiLocation(), "/kapps/").concat(kappSlug, "/securityPolicyDefinitions") : "".concat(bundle.apiLocation(), "/securityPolicyDefinitions");
-  return securityPolicyName ? "".concat(basePath, "/").concat(securityPolicyName) : basePath;
+  return securityPolicyName ? "".concat(basePath, "/").concat(encodeURIComponent(securityPolicyName)) : basePath;
 };
 
 export var fetchSecurityPolicyDefinitions = function fetchSecurityPolicyDefinitions() {

package/lib/apis/core/securityPolicyDefinitions.test.js

@@ -31,7 +31,7 @@ describe('securityPolicyDefinitions api', function () {
                   securityPolicyDefinitions: [{
                     message: 'Must be an administrator.',
                     name: 'Admins',
-                    rule: "/* \n * Space admins are allowed access regardless of the the result of security \n * policies.  Returning 'false' denies anyone but a space access. \n */\nfalse",
+                    rule: "/* \n * Space admins are allowed access regardless of the result of security \n * policies.  Returning 'false' denies anyone but a space access. \n */\nfalse",
                     type: 'Space'
                   }, {
                     message: 'Must be authenticated.',
@@ -61,7 +61,7 @@ describe('securityPolicyDefinitions api', function () {
                 securityPolicyDefinitions: [{
                   message: 'Must be an administrator.',
                   name: 'Admins',
-                  rule: "/* \n * Space admins are allowed access regardless of the the result of security \n * policies.  Returning 'false' denies anyone but a space access. \n */\nfalse",
+                  rule: "/* \n * Space admins are allowed access regardless of the result of security \n * policies.  Returning 'false' denies anyone but a space access. \n */\nfalse",
                   type: 'Space'
                 }, {
                   message: 'Must be authenticated.',
@@ -95,7 +95,7 @@ describe('securityPolicyDefinitions api', function () {
                   securityPolicyDefinitions: [{
                     message: 'Must be an administrator.',
                     name: 'Admins',
-                    rule: "/* \n * Space admins are allowed access regardless of the the result of security \n * policies.  Returning 'false' denies anyone but a space access. \n */\nfalse",
+                    rule: "/* \n * Space admins are allowed access regardless of the result of security \n * policies.  Returning 'false' denies anyone but a space access. \n */\nfalse",
                     type: 'Kapp'
                   }, {
                     message: 'Must be authenticated.',
@@ -132,7 +132,7 @@ describe('securityPolicyDefinitions api', function () {
                 securityPolicyDefinitions: [{
                   message: 'Must be an administrator.',
                   name: 'Admins',
-                  rule: "/* \n * Space admins are allowed access regardless of the the result of security \n * policies.  Returning 'false' denies anyone but a space access. \n */\nfalse",
+                  rule: "/* \n * Space admins are allowed access regardless of the result of security \n * policies.  Returning 'false' denies anyone but a space access. \n */\nfalse",
                   type: 'Kapp'
                 }, {
                   message: 'Must be authenticated.',