Lint metadata directory

.eslintignore

@@ -13,10 +13,6 @@ apis/client/profile/visibility/lib/bootstrap-switch.js
 ## We want to lint all those files                                  ##
 ## Please remove the line on this file if you are changing the file ##
 ######################################################################
-metadata/client/edit/autoform.js
-metadata/collection/index.js
-metadata/collection/permissions.js
-metadata/collection/server/publication.js
 monitoring/client/autoform.js
 monitoring/client/monitoring.js
 monitoring/collection/index.js

apis/server/methods/delete.js

@@ -4,7 +4,7 @@ import { Mongo } from 'meteor/mongo';
 
 import Apis from '/apis/collection';
 import ApiBacklogItems from '/backlog/collection';
-import { ApiMetadata } from '/metadata/collection';
+import ApiMetadata from '/metadata/collection';
 import DocumentationFiles from '/documentation/collection';
 import Feedback from '/feedback/collection';
 import { MonitoringSettings, MonitoringData } from '/monitoring/collection';

metadata/client/edit/edit.js

@@ -1,6 +1,6 @@
 import { Template } from 'meteor/templating';
 import { TAPi18n } from 'meteor/tap:i18n';
-import { ApiMetadata } from '../../collection';
+import ApiMetadata from '../../collection';
 
 Template.editApiMetadata.helpers({
   apiMetadataCollection () {

metadata/client/view/metadata.js

@@ -1,7 +1,7 @@
 import { Template } from 'meteor/templating';
 
 import formatDate from '/core/helper_functions/format_date';
-import { ApiMetadata } from '../../collection';
+import ApiMetadata from '../../collection';
 
 Template.viewApiMetadata.onCreated(function () {
   // Get reference to template instance

metadata/collection/index.js

@@ -1,3 +1,5 @@
-const ApiMetadata = new Mongo.Collection("apiMetadata");
+import { Mongo } from 'meteor/mongo';
 
-export { ApiMetadata };
+const ApiMetadata = new Mongo.Collection('apiMetadata');
+
+export default ApiMetadata;

metadata/collection/permissions.js

@@ -1,4 +1,4 @@
-import { ApiMetadata } from '/metadata/collection';
+import ApiMetadata from '/metadata/collection';
 import Apis from '/apis/collection';
 
 ApiMetadata.allow({
@@ -8,17 +8,17 @@ ApiMetadata.allow({
     // Make sure there is only one document per API Backend ID
     // TODO: refactor ApiMetadata schema to use 'apiId' field
     if (ApiMetadata.find({ apiBackendId: apiId }).count() !== 0) {
+      // eslint-disable-next-line no-console
       console.log('no insert allowed');
       return false;
-    } else {
+    }
       // Find related API Backend, select only "managerIds" field
-      const api = Apis.findOne(apiId, { fields: { managerIds: 1 } });
+    const api = Apis.findOne(apiId, { fields: { managerIds: 1 } });
 
       // Check if current user can edit API Backend
-      const userCanEdit = api.currentUserCanEdit();
+    const userCanEdit = api.currentUserCanEdit();
 
-      return userCanEdit;
-    }
+    return userCanEdit;
   },
   update (userId, metadata) {
     // Get API Backend ID

metadata/collection/schema.js

@@ -1,6 +1,6 @@
 import { SimpleSchema } from 'meteor/aldeed:simple-schema';
 import { TAPi18n } from 'meteor/tap:i18n';
-import { ApiMetadata } from '/metadata/collection';
+import ApiMetadata from '/metadata/collection';
 
 ApiMetadata.schema = new SimpleSchema({
   // TODO: migrate to use 'apiId' instead of 'apiBackendId'

metadata/collection/server/publication.js

@@ -1,6 +1,12 @@
-import { ApiMetadata } from '/metadata/collection/';
+import { check } from 'meteor/check';
+import { Meteor } from 'meteor/meteor';
+
+import ApiMetadata from '/metadata/collection/';
+
+Meteor.publish('apiMetadata', (apiId) => {
+  // Make sure apiId is a String
+  check(apiId, String);
 
-Meteor.publish('apiMetadata', function (apiId) {
   // Get metadata document for API Backend
   // TODO: migrate ApiMetadata schema to use 'apiId' instead of 'apiBackendId'
   const apiMetadata = ApiMetadata.find({ apiBackendId: apiId });