Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Admin unique email constraint is not present in admin form #2633

Open
techi602 opened this issue Jun 21, 2023 · 0 comments
Open

Admin unique email constraint is not present in admin form #2633

techi602 opened this issue Jun 21, 2023 · 0 comments

Comments

@techi602
Copy link
Contributor

Administration form for editing administrator does not have constraint for unique email. Therefore submitting existing administrator e-mail into the form results in error 500 (SQL unique index error).

To Reproduce

In administration go to Administrators --> Administrators overview. Create two administrators with unique e-mail. Then update 2nd administrator and set email from 1st administrator and submit the form.

Additional context

The validation for unique email is already implemented in customer update form.

https://github.com/shopsys/shopsys/blob/master/packages/framework/src/Form/Admin/Administrator/AdministratorFormType.php#L66-L72
vs.
https://github.com/shopsys/shopsys/blob/master/packages/framework/src/Form/Admin/Customer/User/CustomerUserFormType.php#L151

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants