Skip to content

Commit 74b0a68

Browse files
Fix on replace rules (#417)
* Fix on replace rules * Fixed default setting --------- Co-authored-by: Niels de Jong <[email protected]>
1 parent adccd37 commit 74b0a68

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

src/chart/ChartUtils.ts

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -182,6 +182,7 @@ export function replaceDashboardParameters(str, parameters) {
182182
}
183183
let rx = /`.([^`]*)`/g;
184184
let regexSquareBrackets = /\[(.*?)\]/g;
185+
let rxSimple = /\$neodash_\w*/g;
185186

186187
/**
187188
* Define function to access elements in an array/object type dashboard parameter.
@@ -205,15 +206,15 @@ export function replaceDashboardParameters(str, parameters) {
205206
return RenderSubValue(val);
206207
};
207208

208-
let newString = str.replace(rx, parameterElementReplacer);
209+
const parameterSimpleReplacer = (_) => {
210+
let param = _.replace(`$`, '').trim();
211+
let val = parameters?.[param] || null;
212+
let type = getRecordType(val);
213+
let valueRender = type === 'string' ? val : RenderSubValue(val);
214+
return valueRender;
215+
};
209216

210-
Object.keys(parameters).forEach((key) => {
211-
let valueRender = RenderSubValue(parameters[key]);
212-
newString = newString.replaceAll(`$${key} `, `${valueRender} `);
213-
newString = newString.replaceAll(`$${key},`, `${valueRender},`);
214-
newString = newString.replaceAll(`$${key}.`, `${valueRender}.`);
215-
newString = newString.replaceAll(`$${key})`, `${valueRender})`);
216-
});
217+
let newString = str.replace(rx, parameterElementReplacer).replace(rxSimple, parameterSimpleReplacer);
217218

218219
return newString;
219220
}

src/config/ReportConfig.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1237,7 +1237,7 @@ export const REPORT_TYPES = {
12371237
label: 'Replace global parameters in Markdown',
12381238
type: SELECTION_TYPES.LIST,
12391239
values: [true, false],
1240-
default: false,
1240+
default: true,
12411241
},
12421242
fullscreenEnabled: {
12431243
label: 'Fullscreen enabled',

0 commit comments

Comments
 (0)