darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-23 15:57:23 -04:00
Code Issues Packages Projects Releases Wiki Activity

Prevent duplicate domains

Browse Source
This commit is contained in:
Chris Hill-Scott
2019-03-22 14:46:03 +00:00
parent de68d720c7
commit 8b81d3594a
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
app/main/views/organisations.py
View File

@@ -1,3 +1,5 @@
from collections import OrderedDict
from flask import flash, redirect, render_template, request, session, url_for from flask import flash, redirect, render_template, request, session, url_for
from flask_login import current_user, login_required from flask_login import current_user, login_required
from notifications_python_client.errors import HTTPError from notifications_python_client.errors import HTTPError
@@ -416,7 +418,10 @@ def edit_organisation_domains(org_id):
if form.validate_on_submit(): if form.validate_on_submit():
organisations_client.update_organisation( organisations_client.update_organisation(
org_id, org_id,
domains=list(filter(None, form.domains.data)), domains=list(OrderedDict.fromkeys(
domain.lower()
for domain in filter(None, form.domains.data)
)),
) )
return redirect(url_for('.organisation_settings', org_id=org_id)) return redirect(url_for('.organisation_settings', org_id=org_id))

2
tests/app/main/views/organisations/test_organisation_invites.py
View File

@@ -736,6 +736,8 @@ def test_view_organisation_domains(
( (
{ {
'domains-0': 'example.gov.uk', 'domains-0': 'example.gov.uk',
'domains-2': 'example.gov.uk',
'domains-3': 'EXAMPLE.GOV.UK',
'domains-5': 'test.gov.uk', 'domains-5': 'test.gov.uk',
}, },
{ {