Skip to content

Commit 276f19c

Browse files
committed
Merge remote-tracking branch 'origin/fixevent'
2 parents 9552ca2 + afd0441 commit 276f19c

1 file changed

Lines changed: 6 additions & 5 deletions

File tree

internal/Trigger.js

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -120,9 +120,7 @@ const triggers = {
120120
*/
121121
export async function runSaveTrigger(option, optionValue, event = {}) {
122122
// create object in case event is empty
123-
if (!event) {
124-
event = {};
125-
}
123+
event = event || {};
126124

127125
if (option === undefined) {
128126
console.info("run all save triggers");
@@ -164,11 +162,11 @@ export async function runSaveTrigger(option, optionValue, event = {}) {
164162
* @param {string} option
165163
* @param {Object} optionValue
166164
* @param {Array} saveTriggerValues value returned by potentially run safe triggers
167-
* @param {Event} event the event (input or change) that triggered saving
165+
* @param {Event} [event] the event (input or change) that triggered saving
168166
* @returns {Promise}
169167
* @see {@link overrideSave}
170168
*/
171-
export async function runOverrideSave(option, optionValue, saveTriggerValues, event) {
169+
export async function runOverrideSave(option, optionValue, saveTriggerValues, event = {}) {
172170
// run all registered triggers for that option
173171
const allRegisteredOverrides = triggers.overrideSave.filter((trigger) => trigger.option === option);
174172
if (allRegisteredOverrides.length === 0) {
@@ -177,6 +175,9 @@ export async function runOverrideSave(option, optionValue, saveTriggerValues, ev
177175

178176
console.info("runOverrideSave:", `${allRegisteredOverrides.length}x`, option, optionValue, saveTriggerValues, event);
179177

178+
// default event parameter to empty object
179+
event = event || {};
180+
180181
let lastPromise = Promise.resolve({
181182
option,
182183
optionValue,

0 commit comments

Comments
 (0)