Merge pull request #3441 from apinf/feature/0.55.0

Feature/0.55.0

Author: matleppa

.travis.yml

@@ -1,6 +1,4 @@
-sudo: required
-dist: trusty
-group: deprecated-2017Q4
+sudo: false
 
 language: node_js
 node_js:
@@ -30,8 +28,11 @@ services:
   - docker
 
 before_install:
+  - pwd
+  - ls -la
   # Install meteor locally on CI
-  - if [ ! -e "$HOME/.meteor/meteor" ]; then cat $HOME/.travis_install_meteor | sed s/--progress-bar/-sL/g | /bin/sh; fi
+  - if [ ! -e "$HOME/.meteor/meteor" ]; then cat .travis_install_meteor | sed s/--progress-bar/-sL/g | /bin/sh; fi
+
 
 before_script:
   - yarn

apinf_packages/about/client/about.html

@@ -33,7 +33,7 @@ <h3>
               Apinf
             </dt>
             <dd>
-              0.54.0
+              0.55.0
             </dd>
             <dt>
               API Umbrella

apinf_packages/api_catalog/client/api_catalog.js

@@ -25,6 +25,9 @@ import 'locale-compare-polyfill';
 // Npm packages imports
 import _ from 'lodash';
 
+// APInf imports
+import localisedSorting from '/apinf_packages/core/helper_functions/string_utils';
+
 Template.apiCatalog.onCreated(function () {
   // Get reference to template instance
   const instance = this;
@@ -213,15 +216,14 @@ Template.apiCatalog.helpers({
   apis () {
     // Get apis collection via Pagination
     const apis = Template.instance().pagination.getPage();
-    // Get the language
-    const language = TAPi18n.getLanguage();
+
     // Get the sort via Pagination
     const sort = Template.instance().pagination.sort();
     // When sorted by name
     if (sort.name) {
       // use custom sort function with i18n support
       apis.sort((a, b) => {
-        return a.name.localeCompare(b.name, language) * sort.name;
+        return localisedSorting(a.name, b.name, sort.name);
       });
     }
     return apis;

apinf_packages/api_docs/client/manage/manage.js

@@ -152,13 +152,18 @@ Template.manageApiDocumentationModal.events({
     Modal.hide('manageApiDocumentationModal');
   },
   'click #cancel-button': function (event, templateInstance) {
-    // Get fileId value
-    const fileId = Session.get('fileId');
+    const apiDocs = ApiDocs.findOne();
 
-    // If file is uploaded and a user clicks on "Cancel" button
-    if (fileId) {
-      // Remove uploaded file from collection
-      templateInstance.removeDocumentationFile(fileId);
+    // Make sure File isn't uploaded
+    if (!apiDocs || !apiDocs.fileId) {
+      // Get fileId value
+      const fileId = Session.get('fileId');
+
+      // If file is uploaded and a user clicks on "Cancel" button
+      if (fileId) {
+        // Remove uploaded file from collection
+        templateInstance.removeDocumentationFile(fileId);
+      }
     }
   },
   'click #add-link': function () {

apinf_packages/api_docs/client/swagger_ui/content/content.js

@@ -146,6 +146,7 @@ Template.swaggerUiContent.onRendered(function () {
     const ui = SwaggerUIBundle({
       url: api.documentationUrl(),
       dom_id: '#swagger-ui',
+      defaultModelsExpandDepth: 0,
       // Provides custom configs for Swagger
       configs: this.configs,
       presets: [

apinf_packages/api_docs/client/view/documentation.html

@@ -27,11 +27,13 @@
                 {{_ "apiDocumentation_buttonText_closeEditor" }}
               </button>
             {{ else }}
-              <button
-                class="btn btn-xs btn-info editor">
-                <i class="fa fa-file-text-o"></i>&nbsp;
-                {{_ "apiDocumentation_buttonText_openEditor" }}
-              </button>
+              {{# if developmentFeatures }}
+                <button
+                  class="btn btn-xs btn-info editor">
+                  <i class="fa fa-file-text-o"></i>&nbsp;
+                  {{_ "apiDocumentation_buttonText_openEditor" }}
+                </button>
+              {{/ if }}
             {{/ if }}
             <button
               id="manage-api-documentation"

apinf_packages/api_docs/client/view/documentation.js

@@ -52,6 +52,8 @@ Template.apiDocumentation.onCreated(function () {
 
   // Subscribe to code generator settings
   instance.subscribe('singleSetting', 'sdkCodeGenerator');
+  // Subscribe to development Features settings
+  instance.subscribe('singleSetting', 'developmentFeatures');
 });
 
 Template.apiDocumentation.onRendered(() => {

apinf_packages/apis/client/authorized_user/form/autoform.js

@@ -0,0 +1,41 @@
+/* Copyright 2017 Apinf Oy
+This file is covered by the EUPL license.
+You may obtain a copy of the licence at
+https://joinup.ec.europa.eu/community/eupl/og_page/european-union-public-licence-eupl-v11 */
+
+// Meteor contributed packages imports
+import { AutoForm } from 'meteor/aldeed:autoform';
+import { TAPi18n } from 'meteor/tap:i18n';
+import { sAlert } from 'meteor/juliancwirko:s-alert';
+
+AutoForm.hooks({
+  authorizedUserForm: {
+    onSuccess () {
+      // Get success message translation
+      const message = TAPi18n.__('authorizedUserForm_success_message');
+
+      // Alert user of success
+      sAlert.success(message);
+    },
+    onError (formType, error) {
+      // Get error type string from error object
+      const errorType = error.error;
+
+      if (errorType === 'user-not-registered') {
+        // Get error message translation
+        const message = TAPi18n.__('authorizedUserForm_errorText_userNotRegistered');
+
+        // Display error
+        sAlert.error(message, { timeout: 'none' });
+      }
+
+      if (errorType === 'user-already-exist') {
+        // Get error message translation
+        const message = TAPi18n.__('authorizedUserForm_errorText_alreadyAuthorized');
+
+        // Display error
+        sAlert.error(message, { timeout: 'none' });
+      }
+    },
+  },
+});

apinf_packages/apis_authorization/client/form/form.html renamed to apinf_packages/apis/client/authorized_user/form/form.html

@@ -10,10 +10,16 @@ <h4>
   {{# autoForm
     schema=emailSchema
     id="authorizedUserForm"
-    class="form-inline" }}
+    class="form-inline"
+    type="method"
+    meteormethod="addAuthorizedUser"
+  }}
     <fieldset>
-      {{> afQuickField name='email' }}
-      {{> afQuickField name='apiId' value=apiId type="hidden" }}
+      <p>
+        {{_ "apiUserAuthorizationForm_helpText_usernameEmail" }}
+      </p>
+      {{> afQuickField name='user' }}
+      {{> afQuickField name='apiId' value=api._id type="hidden" }}
     </fieldset>
 
     <button type="submit" class="btn btn-primary">

apinf_packages/apis_authorization/client/form/form.js renamed to apinf_packages/apis/client/authorized_user/form/form.js

@@ -10,20 +10,7 @@ import { Template } from 'meteor/templating';
 import emailSchema from './schema';
 
 Template.apiUserAuthorizationForm.helpers({
-  apiId () {
-    // Get API ID
-    const apiId = Template.instance().data.api._id;
-
-    return apiId;
-  },
   emailSchema () {
     return emailSchema;
   },
 });
-
-Template.apiUserAuthorizationForm.events({
-  'submit #authorizedUserForm': function (event) {
-    // Prevent form from reloading page
-    event.preventDefault();
-  },
-});