Skip to content

Commit 6264636

Browse files
committed
fix: append suffix to all filenames
1 parent 7a93d13 commit 6264636

File tree

9 files changed

+13
-11
lines changed

9 files changed

+13
-11
lines changed

src/ditamap/base-ditamap.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,8 @@ import { Ditamap } from './ditamap';
1010
export class BaseDitamap extends Ditamap {
1111
public constructor(topics: string[]) {
1212
// Set the data of topic and filenames
13-
super(Ditamap.getFileName(), {
14-
namespaceDitamapFiles: topics.sort().map((topic) => `${topic}/cli_reference_${topic}.ditamap`),
13+
super(Ditamap.file('cli_reference', 'ditamap'), {
14+
namespaceDitamapFiles: topics.sort().map((topic) => Ditamap.file(`${topic}/cli_reference_${topic}`, 'ditamap')),
1515
});
1616
}
1717

src/ditamap/cli-reference-topic.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import { Ditamap } from './ditamap';
1010

1111
export class CLIReferenceTopic extends Ditamap {
1212
public constructor(topic: string, longDescription: string) {
13-
const filename = `cli_reference_${topic}.xml`;
13+
const filename = Ditamap.file(`cli_reference_${topic}`, 'xml');
1414
// Set the data of topic and filenames
1515
super(filename, {
1616
topic,

src/ditamap/cli-reference.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { Ditamap } from './ditamap';
99

1010
export class CLIReference extends Ditamap {
1111
public constructor() {
12-
const filename = 'cli_reference.xml';
12+
const filename = Ditamap.file('cli_reference', 'xml');
1313

1414
super(filename, {
1515
cliVersion: Ditamap.cliVersion,

src/ditamap/command.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export type CommandHelpInfo = {
3030
export class Command extends Ditamap {
3131
public constructor(topic: string, subtopic: string, command: Dictionary, commandMeta: JsonMap = {}) {
3232
const commandWithUnderscores = ensureString(command.id).replace(/:/g, '_');
33-
const filename = `cli_reference_${commandWithUnderscores}.xml`;
33+
const filename = Ditamap.file(`cli_reference_${commandWithUnderscores}`, 'xml');
3434

3535
super(filename, {});
3636

src/ditamap/ditamap.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,8 @@ export abstract class Ditamap {
5252
Ditamap._suffix = suffix;
5353
}
5454

55-
public static getFileName(): string {
56-
return Ditamap.suffix ? `cli_reference_${Ditamap.suffix}.ditamap` : 'cli_reference.ditamap';
55+
public static file(name: string, ext: string): string {
56+
return Ditamap.suffix ? `${name}_${Ditamap.suffix}.${ext}` : `${name}.${ext}`;
5757
}
5858

5959
private static _suffix: string;

src/ditamap/main-topic-intro.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ import { Ditamap } from './ditamap';
1212

1313
export class MainTopicIntro extends Ditamap {
1414
public constructor(topic: string, subtopic: string, subTopicMeta: JsonMap) {
15-
const filename = `cli_reference_${topic}_${subtopic}.xml`;
15+
const filename = Ditamap.file(`cli_reference_${topic}_${subtopic}`, 'xml');
1616

1717
let trailblazerCommunityUrl;
1818
let trailblazerCommunityName;

src/ditamap/topic-commands.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ import { Ditamap } from './ditamap';
1111

1212
export class TopicCommands extends Ditamap {
1313
public constructor(topic: string, topicMeta: JsonMap) {
14-
const filename = `cli_reference_${topic}_commands.xml`;
14+
const filename = Ditamap.file(`cli_reference_${topic}_commands`, 'xml');
1515
// Set the data of topic and filenames
1616
super(filename, topicMeta);
1717

src/ditamap/topic-ditamap.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ import { Ditamap } from './ditamap';
1010

1111
export class TopicDitamap extends Ditamap {
1212
public constructor(topic: string, commandIds: string[]) {
13-
const filename = `cli_reference_${topic}.ditamap`;
13+
const filename = Ditamap.file(`cli_reference_${topic}`, 'ditamap');
1414
// Set the data of topic and filenames
1515
const commands = commandIds.sort().map((c) => ({ command: c.replace(/:/g, '_') }));
1616
super(filename, { topic, commands });

src/utils.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,12 +11,14 @@ import { EOL } from 'os';
1111
import { join } from 'path';
1212
import { Dictionary, isObject } from '@salesforce/ts-types';
1313
import { fs } from '@salesforce/core';
14+
import { Ditamap } from './ditamap/ditamap';
1415

1516
export const events = new EventEmitter();
1617

1718
export async function copyStaticFile(outputDir: string, fileDir: string, fileName: string): Promise<void> {
1819
const source = join(fileDir, fileName);
19-
const dest = join(outputDir, fileName);
20+
const [name, ext] = fileName.split('.');
21+
const dest = join(outputDir, Ditamap.file(name, ext));
2022
await fs.mkdirp(outputDir);
2123
copyFileSync(source, dest);
2224
}

0 commit comments

Comments
 (0)