Skip to content

Commit 3e2c4d9

Browse files
Lint metadata directory
1 parent 78ebb6d commit 3e2c4d9

File tree

3 files changed

+7
-2
lines changed

3 files changed

+7
-2
lines changed

.eslintignore

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,6 @@ apis/client/profile/visibility/lib/bootstrap-switch.js
1313
## We want to lint all those files ##
1414
## Please remove the line on this file if you are changing the file ##
1515
######################################################################
16-
metadata/collection/permissions.js
17-
metadata/collection/server/publication.js
1816
monitoring/client/autoform.js
1917
monitoring/client/monitoring.js
2018
monitoring/collection/index.js

metadata/collection/permissions.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ ApiMetadata.allow({
88
// Make sure there is only one document per API Backend ID
99
// TODO: refactor ApiMetadata schema to use 'apiId' field
1010
if (ApiMetadata.find({ apiBackendId: apiId }).count() !== 0) {
11+
// eslint-disable-next-line no-console
1112
console.log('no insert allowed');
1213
return false;
1314
}

metadata/collection/server/publication.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,12 @@
1+
import { check } from 'meteor/check';
2+
import { Meteor } from 'meteor/meteor';
3+
14
import ApiMetadata from '/metadata/collection/';
25

36
Meteor.publish('apiMetadata', (apiId) => {
7+
// Make sure apiId is a String
8+
check(apiId, String);
9+
410
// Get metadata document for API Backend
511
// TODO: migrate ApiMetadata schema to use 'apiId' instead of 'apiBackendId'
612
const apiMetadata = ApiMetadata.find({ apiBackendId: apiId });

0 commit comments

Comments
 (0)