fix: sanitizeDisplayNameOrToast setDisplayNameError to onDisplayNameError

since it can be useState or redux
pull/3083/head
William Grant 1 year ago
parent 1e16be1be0
commit 7b634df2b5

@ -4,7 +4,7 @@ import { sanitizeSessionUsername } from '../../../session/utils/String';
export function sanitizeDisplayNameOrToast( export function sanitizeDisplayNameOrToast(
displayName: string, displayName: string,
// can be a useState or redux function // can be a useState or redux function
setDisplayNameError: (error: string | undefined) => any, onDisplayNameError: (error: string | undefined) => any,
dispatch?: Dispatch dispatch?: Dispatch
) { ) {
try { try {
@ -12,17 +12,17 @@ export function sanitizeDisplayNameOrToast(
const trimName = sanitizedName.trim(); const trimName = sanitizedName.trim();
const errorString = !trimName ? window.i18n('displayNameEmpty') : undefined; const errorString = !trimName ? window.i18n('displayNameEmpty') : undefined;
if (dispatch) { if (dispatch) {
dispatch(setDisplayNameError(errorString)); dispatch(onDisplayNameError(errorString));
} else { } else {
setDisplayNameError(errorString); onDisplayNameError(errorString); // this is is either calling dispatch in the caller or just `setDisplayNameError`
} }
return sanitizedName; return sanitizedName;
} catch (e) { } catch (e) {
if (dispatch) { if (dispatch) {
dispatch(setDisplayNameError(window.i18n('displayNameErrorDescriptionShorter'))); dispatch(onDisplayNameError(window.i18n('displayNameErrorDescriptionShorter')));
} else { } else {
setDisplayNameError(window.i18n('displayNameErrorDescriptionShorter')); onDisplayNameError(window.i18n('displayNameErrorDescriptionShorter'));
} }
return displayName; return displayName;
} }

Loading…
Cancel
Save