Spearhead
/
joyent-portal
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

fix(cp-frontend): handle manifests without env varibales

This commit is contained in:
Sérgio Ramos 2017-08-22 14:37:28 +01:00
parent ccf1fd6f9a
commit b5ea1580e5
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
packages/cp-frontend/src/containers/manifest/edit-or-create.js
View File

@ -11,6 +11,7 @@ import uniq from 'lodash.uniq';
import find from 'lodash.find';
import { safeLoad } from 'js-yaml';
import uuid from 'uuid/v4';
import forceArray from 'force-array';
import DeploymentGroupBySlugQuery from '@graphql/DeploymentGroupBySlug.gql';
import DeploymentGroupCreateMutation from '@graphql/DeploymentGroupCreate.gql';
@ -138,6 +139,7 @@ class DeploymentGroupEditOrCreate extends Component {
);
return filenames
.filter(Boolean)
.filter(filename => !find(currentFiles, ['name', filename]))
.map(this.getDefaultFile)
.concat(currentFiles);
@ -169,9 +171,9 @@ class DeploymentGroupEditOrCreate extends Component {
return environment;
}
const names = manifest
.match(INTERPOLATE_REGEX)
.map(name => name.replace(/^\$/, ''));
const names = forceArray(manifest.match(INTERPOLATE_REGEX)).map(name =>
name.replace(/^\$/, '')
);
const vars = uniq(names).map(name => `\n${name}=`).join('');