yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Slider property editor: Fix for preset value handling of enableRange (#20772)

Browse Source 
Fix config value access in UmbSliderPropertyValuePreset

Updated the `UmbSliderPropertyValuePreset` class to ensure the `.value` property is accessed for configuration items. This change improves the accuracy of retrieving `enableRange`, `min`, `max`, and `step` values, addressing potential bugs in value processing.

Co-authored-by: Luuk Peters <Luuk.Peters@proudnerds.com>
This commit is contained in:
Luuk Peters
2025-11-09 22:38:16 +01:00
committed by GitHub
parent aae316e17e
commit 04918ec3d2
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/Umbraco.Web.UI.Client/src/packages/property-editors/slider/slider-property-value-preset.ts
View File

@@ -6,16 +6,16 @@ export class UmbSliderPropertyValuePreset
implements UmbPropertyValuePreset<UmbSliderPropertyEditorUiValue, UmbPropertyEditorConfig>
{
async processValue(value: undefined | UmbSliderPropertyEditorUiValue, config: UmbPropertyEditorConfig) {
const enableRange = Boolean(config.find((x) => x.alias === 'enableRange') ?? false);
const enableRange = Boolean(config.find((x) => x.alias === 'enableRange')?.value ?? false);
/*
const min = Number(config.find((x) => x.alias === 'minVal') ?? 0);
const max = Number(config.find((x) => x.alias === 'maxVal') ?? 100);
const min = Number(config.find((x) => x.alias === 'minVal')?.value ?? 0);
const max = Number(config.find((x) => x.alias === 'maxVal')?.value ?? 100);
const minVerified = isNaN(min) ? undefined : min;
const maxVerified = isNaN(max) ? undefined : max;
*/
const step = (config.find((x) => x.alias === 'step') as number | undefined) ?? 0;
const step = Number(config.find((x) => x.alias === 'step')?.value ?? 0);
const stepVerified = step > 0 ? step : 1;
const initValueMin = Number(config.find((x) => x.alias === 'initVal1')?.value) || 0;