Skip to content

Commit 0834ec0

Browse files
authored
Merge pull request #8699 from ProcessMaker/bugfix/FOUR-25963
FOUR-25963: Guard missing setting keys when rendering admin settings
2 parents c111eca + d4845b1 commit 0834ec0

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

resources/js/admin/settings/components/SettingText.vue

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,8 @@ export default {
150150
if (this.setting?.ui?.isNotEmpty) {
151151
return this.transformed !== "" && this.transformed !== null;
152152
}
153-
if (this.setting.key.startsWith("white_list.")) {
153+
const settingKey = this.setting?.key ?? "";
154+
if (settingKey.startsWith("white_list.")) {
154155
return this.validateURL(this.transformed);
155156
}
156157
return true;
@@ -190,7 +191,8 @@ export default {
190191
if (this.setting.ui?.isNotEmpty && (this.transformed === "" || this.transformed === null)) {
191192
return;
192193
}
193-
if (this.setting.key.startsWith("white_list.")) {
194+
const settingKey = this.setting?.key ?? "";
195+
if (settingKey.startsWith("white_list.")) {
194196
if (!this.validateURL(this.transformed)) {
195197
return;
196198
}

0 commit comments

Comments
 (0)