Remove duplication by relying on Capitano's new required option feature

This commit is contained in:
Juan Cruz Viotti 2015-01-08 12:11:10 -03:00
parent dc75481880
commit d2b0772673
5 changed files with 2 additions and 11 deletions

View File

@ -7,9 +7,6 @@ errors = require('../errors/errors')
permissions = require('../permissions/permissions')
exports.list = permissions.user (params, options) ->
if not options.application?
errors.handle(new Error('You have to specify an application'))
resin.models.device.getAllByApplication options.application, (error, devices) ->
errors.handle(error) if error?

View File

@ -11,9 +11,6 @@ isSystemVariable = (environmentVariable) ->
SYSTEM_VAR_REGEX.test(environmentVariable.name)
exports.list = permissions.user (params, options) ->
if not options.application?
errors.handle(new Error('You have to specify an application'))
resin.models.environmentVariables.getAllByApplication options.application, (error, environmentVariables) ->
errors.handle(error) if error?
@ -28,9 +25,6 @@ exports.remove = permissions.user (params, options) ->
, errors.handle
exports.add = permissions.user (params, options) ->
if not options.application?
errors.handle(new Error('You have to specify an application'))
if not params.value?
params.value = process.env[params.key]

View File

@ -4,7 +4,6 @@ path = require('path')
mkdirp = require('mkdirp')
ProgressBar = require('progress')
resin = require('resin-sdk')
connection = require('../connection/connection')
log = require('../log/log')
permissions = require('../permissions/permissions')
errors = require('../errors/errors')

View File

@ -58,6 +58,7 @@ applicationOption =
parameter: 'application'
description: 'application id'
alias: [ 'a', 'app' ]
required: 'You have to specify an application'
deviceOption =
signature: 'device'

View File

@ -47,7 +47,7 @@
},
"dependencies": {
"async": "~0.9.0",
"capitano": "~1.0.4",
"capitano": "~1.1.0",
"cliff": "~0.1.9",
"coffee-script": "~1.8.0",
"conf.js": "^0.1.1",