Skip to content

Commit d8fa5e7

Browse files
authored
Merge pull request #1998 from CarlSargunar/bug/16601-Login-error
Bugfix: Installer does not take the minimum password length into account
2 parents 91db741 + f26a5c9 commit d8fa5e7

File tree

1 file changed

+14
-6
lines changed

1 file changed

+14
-6
lines changed

src/apps/installer/user/installer-user.element.ts

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,28 +9,35 @@ export class UmbInstallerUserElement extends UmbLitElement {
99
@state()
1010
private _userFormData?: { name: string; password: string; email: string; subscribeToNewsletter: boolean };
1111

12-
private _installerContext?: UmbInstallerContext;
12+
@state()
13+
private _minimumPasswordLength = 10;
14+
15+
#installerContext?: UmbInstallerContext;
1316

1417
constructor() {
1518
super();
1619

1720
this.consumeContext(UMB_INSTALLER_CONTEXT, (installerContext) => {
18-
this._installerContext = installerContext;
21+
this.#installerContext = installerContext;
1922
this._observeInstallerData();
2023
});
2124
}
2225

2326
private _observeInstallerData() {
24-
if (!this._installerContext) return;
27+
if (!this.#installerContext) return;
2528

26-
this.observe(this._installerContext.data, ({ user }) => {
29+
this.observe(this.#installerContext.data, ({ user }) => {
2730
this._userFormData = {
2831
name: user.name,
2932
password: user.password,
3033
email: user.email,
3134
subscribeToNewsletter: user.subscribeToNewsletter ?? false,
3235
};
3336
});
37+
38+
this.observe(this.#installerContext.settings, (settings) => {
39+
this._minimumPasswordLength = settings?.user.minCharLength ?? this._minimumPasswordLength;
40+
});
3441
}
3542

3643
private _handleSubmit = (e: SubmitEvent) => {
@@ -48,8 +55,8 @@ export class UmbInstallerUserElement extends UmbLitElement {
4855
const email = formData.get('email') as string;
4956
const subscribeToNewsletter = formData.has('subscribeToNewsletter');
5057

51-
this._installerContext?.appendData({ user: { name, password, email, subscribeToNewsletter } });
52-
this._installerContext?.nextStep();
58+
this.#installerContext?.appendData({ user: { name, password, email, subscribeToNewsletter } });
59+
this.#installerContext?.nextStep();
5360
};
5461

5562
override render() {
@@ -87,6 +94,7 @@ export class UmbInstallerUserElement extends UmbLitElement {
8794
id="password"
8895
name="password"
8996
label="password"
97+
minlength=${this._minimumPasswordLength}
9098
.value=${this._userFormData?.password}
9199
required
92100
required-message="Password is required"></uui-input-password>

0 commit comments

Comments
 (0)