Skip to content

Commit 6439860

Browse files
Merge pull request #491 from Accenture/bugfix/490-builddefinition-does-not-rename-extracted-sql-ssjs-files
#490 builddefinition does not rename extracted sql ssjs files
2 parents aec8598 + cae7862 commit 6439860

File tree

3 files changed

+11
-5
lines changed

3 files changed

+11
-5
lines changed

.vscode/settings.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@
3131
"editor.defaultFormatter": "esbenp.prettier-vscode"
3232
},
3333
"[markdown]": {
34-
"editor.defaultFormatter": "yzhang.markdown-all-in-one"
34+
"editor.defaultFormatter": "esbenp.prettier-vscode"
3535
},
3636
"[typescript]": {
3737
"editor.defaultFormatter": "esbenp.prettier-vscode"

lib/metadataTypes/Query.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -260,17 +260,20 @@ class Query extends MetadataType {
260260
const targetDirArr = Array.isArray(targetDir) ? targetDir : [targetDir];
261261
const nestedFilePaths = [];
262262

263+
// keep old name if creating templates, otherwise use new name
264+
const fileName = mode === 'definition' ? metadata[this.definition.keyField] : templateName;
265+
263266
for (const targetDir of targetDirArr) {
264267
File.writeToFile(
265268
[targetDir, this.definition.type],
266-
templateName + '.' + this.definition.type + '-meta',
269+
fileName + '.' + this.definition.type + '-meta',
267270
'sql',
268271
code
269272
);
270273
nestedFilePaths.push([
271274
targetDir,
272275
this.definition.type,
273-
templateName + '.' + this.definition.type + '-meta.sql',
276+
fileName + '.' + this.definition.type + '-meta.sql',
274277
]);
275278
}
276279
return nestedFilePaths;

lib/metadataTypes/Script.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -234,17 +234,20 @@ class Script extends MetadataType {
234234
const targetDirArr = Array.isArray(targetDir) ? targetDir : [targetDir];
235235
const nestedFilePaths = [];
236236

237+
// keep old name if creating templates, otherwise use new name
238+
const fileName = mode === 'definition' ? metadata[this.definition.keyField] : templateName;
239+
237240
for (const targetDir of targetDirArr) {
238241
File.writeToFile(
239242
[targetDir, this.definition.type],
240-
templateName + '.' + this.definition.type + '-meta',
243+
fileName + '.' + this.definition.type + '-meta',
241244
'ssjs',
242245
code
243246
);
244247
nestedFilePaths.push([
245248
targetDir,
246249
this.definition.type,
247-
templateName + '.' + this.definition.type + '-meta.ssjs',
250+
fileName + '.' + this.definition.type + '-meta.ssjs',
248251
]);
249252
}
250253
return nestedFilePaths;

0 commit comments

Comments
 (0)