Skip to content
Snippets Groups Projects
Commit e238773a authored by Jiří Kalvoda's avatar Jiří Kalvoda
Browse files

Přidáno upozornění v případě vytváření orga s duplicitním jménem.

Closes #118

Pro potvrzení vytvoření duplicity jsem přidal do formuláře UserEditForm
zaškrtávací položku is_duplicit_name, kterou vždy, když není potřeba, mažu.
parent d1b513a4
No related branches found
No related tags found
No related merge requests found
This commit is part of merge request !87. Comments created here will be created in the context of that merge request.
...@@ -356,6 +356,7 @@ class UserEditForm(FlaskForm): ...@@ -356,6 +356,7 @@ class UserEditForm(FlaskForm):
email = wtforms.StringField("E-mail", validators=[Required()]) email = wtforms.StringField("E-mail", validators=[Required()])
note = wtforms.TextAreaField("Poznámka") note = wtforms.TextAreaField("Poznámka")
is_test = wtforms.BooleanField("Testovací účet") is_test = wtforms.BooleanField("Testovací účet")
allow_duplicity_name = wtforms.BooleanField("Přidat účet s duplicitním jménem")
submit = wtforms.SubmitField("Uložit") submit = wtforms.SubmitField("Uložit")
def validate_email(form, field): def validate_email(form, field):
...@@ -385,6 +386,7 @@ def org_user_edit(id: int): ...@@ -385,6 +386,7 @@ def org_user_edit(id: int):
raise werkzeug.exceptions.Forbidden() raise werkzeug.exceptions.Forbidden()
form = UserEditForm(obj=user) form = UserEditForm(obj=user)
del form.allow_duplicity_name
if (user.is_org or user.is_admin) and not g.user.is_admin: if (user.is_org or user.is_admin) and not g.user.is_admin:
# emaily u organizátorů může editovat jen správce # emaily u organizátorů může editovat jen správce
del form.email del form.email
...@@ -433,6 +435,7 @@ def org_user_new(): ...@@ -433,6 +435,7 @@ def org_user_new():
form = UserEditForm() form = UserEditForm()
form.submit.label.text = 'Vytvořit' form.submit.label.text = 'Vytvořit'
is_duplicit_name = False
if form.validate_on_submit(): if form.validate_on_submit():
check = True check = True
...@@ -440,6 +443,15 @@ def org_user_new(): ...@@ -440,6 +443,15 @@ def org_user_new():
flash('Účet s daným e-mailem již existuje', 'danger') flash('Účet s daným e-mailem již existuje', 'danger')
check = False check = False
if is_org:
if (mo.db.get_session().query(db.User)
.filter_by(first_name=form.first_name.data, last_name=form.last_name.data, is_org=True)
.first() is not None):
is_duplicit_name = True
if not form.allow_duplicity_name.data:
flash('Organizátor s daným jménem již existuje. V případě, že se nejedná o chybu, zaškrtněte políčko ve formuláři.', 'danger')
check = False
if check: if check:
new_user = db.User() new_user = db.User()
form.populate_obj(new_user) form.populate_obj(new_user)
...@@ -470,4 +482,6 @@ def org_user_new(): ...@@ -470,4 +482,6 @@ def org_user_new():
return redirect(url_for('org_org', id=new_user.user_id)) return redirect(url_for('org_org', id=new_user.user_id))
return redirect(url_for('org_user', id=new_user.user_id)) return redirect(url_for('org_user', id=new_user.user_id))
if not is_duplicit_name:
del form.allow_duplicity_name
return render_template('org_user_new.html', form=form, is_org=is_org) return render_template('org_user_new.html', form=form, is_org=is_org)
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment