diff --git a/gracedb/alerts/views.py b/gracedb/alerts/views.py
index 0ea11cce0e7225e540eb5543c8106af8d5609a5d..df94588a203516dcf52b331748f96cd34d3cd353 100644
--- a/gracedb/alerts/views.py
+++ b/gracedb/alerts/views.py
@@ -65,8 +65,7 @@ class CreateNotificationView(MultipleFormView):
return kw
def form_valid(self, form):
- if form.cleaned_data.has_key('key_field'):
- form.cleaned_data.pop('key_field')
+ form.cleaned_data.pop('key_field', None)
# Add user (from request) and category (stored on form class) to
# the form instance, then save
@@ -169,8 +168,7 @@ class CreateContactView(MultipleFormView):
def form_valid(self, form):
# Remove key_field, add user, and save form
- if form.cleaned_data.has_key('key_field'):
- form.cleaned_data.pop('key_field')
+ form.cleaned_data.pop('key_field', None)
form.instance.user = self.request.user
form.save()
diff --git a/gracedb/api/v1/events/views.py b/gracedb/api/v1/events/views.py
index bfbd37e8c573244ab4da7e3e560759f829a0faa8..c3a4641c3f267b5d3ef1f1e5f47b5ed0214c9909 100644
--- a/gracedb/api/v1/events/views.py
+++ b/gracedb/api/v1/events/views.py
@@ -489,7 +489,7 @@ class EventList(InheritPermissionsAPIView):
# TP (21 Nov 2017): Hack to allow basic event submission without
# labels to function with versions of gracedb-client before 1.26.
# Should be removed eventually.
- if (request.data.has_key('labels') and request.data['labels'] == ''):
+ if (request.data.get('labels') == ''):
request.data.pop('labels', None)
form = CreateEventForm(request.data, request.data)
@@ -746,7 +746,7 @@ class EventNeighbors(InheritPermissionsAPIView):
# and TSV renderers.
@event_and_auth_required
def get(self, request, event):
- if request.query_params.has_key('neighborhood'):
+ if 'neighborhood' in request.query_params:
delta = request.query_params['neighborhood']
try:
if delta.find(',') < 0:
diff --git a/gracedb/api/v1/superevents/url_templates.py b/gracedb/api/v1/superevents/url_templates.py
index 49433e0bb5fe910f418ee4a3b78b96a1eacbaa6b..9085b757a19e222e54867f7d08ac845a5630f858 100644
--- a/gracedb/api/v1/superevents/url_templates.py
+++ b/gracedb/api/v1/superevents/url_templates.py
@@ -60,12 +60,12 @@ def construct_url_templates(request=None):
# keys are like '{view_name}-template'
# values are URLs with placeholder parameters
templates = {view_name + '-template': sr(view_name, args=args)
- for view_name, args in views.iteritems()}
+ for view_name, args in views.items()}
# Replace URL placeholder parameters with string formatting placeholders
# Ex: replace 'G1234' with '{graceid}'
- for k,v in templates.iteritems():
- for pattern,placeholder in PH.iteritems():
+ for k,v in templates.items():
+ for pattern,placeholder in PH.items():
if placeholder in v:
v = v.replace(placeholder, "{{{0}}}".format(pattern))
templates[k] = v
diff --git a/gracedb/events/tests/test_label_search.py b/gracedb/events/tests/test_label_search.py
index 3219c5653c6f6540bc3a83f0098ff643a10c7fae..14063ae50c97ffc4349cc3d976d06d34e4f3d131 100644
--- a/gracedb/events/tests/test_label_search.py
+++ b/gracedb/events/tests/test_label_search.py
@@ -53,7 +53,7 @@ class LabelSearchTestCase(TestCase):
Labelling.objects.create(event=e, label=label, creator=submitter)
def test_all_queries(self):
- for key, d in QUERY_CASES.iteritems():
+ for key, d in QUERY_CASES.items():
print "Checking %s ... " % key
# Explicitly search for test events
query = 'Test ' + d['query']
diff --git a/gracedb/ligoauth/middleware.py b/gracedb/ligoauth/middleware.py
index a4b879fbf9c2997351e189eb720617da80590603..e08bd5f645cb13c1cd6bdbaf86a72e8c78e2a458 100644
--- a/gracedb/ligoauth/middleware.py
+++ b/gracedb/ligoauth/middleware.py
@@ -125,7 +125,7 @@ class ControlRoomMiddleware(object):
user_ip = self.get_client_ip(request)
# Add user to control room group(s)
- for ifo, ip in settings.CONTROL_ROOM_IPS.iteritems():
+ for ifo, ip in settings.CONTROL_ROOM_IPS.items():
if (ip == user_ip):
control_room_group = DjangoGroup.objects.get(name=
(ifo.lower() + self.control_room_group_suffix))
diff --git a/gracedb/search/query/superevents.py b/gracedb/search/query/superevents.py
index b6c34aa11255c97c3fb0a967ac637b1e10d7c664..fae5156e7e8bf13f868fb8f1da47b356c0569963 100644
--- a/gracedb/search/query/superevents.py
+++ b/gracedb/search/query/superevents.py
@@ -227,21 +227,21 @@ parameter_dicts = {
# Compile a list of expressions to try to match
expr_list = []
-for k,p in parameter_dicts.iteritems():
+for k,p in parameter_dicts.items():
# Define val and set name
val = p['value']
val.setName(k)
# Add range with format: parameter .. parameter
- if p.has_key('doRange') and p['doRange']:
+ if p.get('doRange'):
range_val = val + Suppress("..") + val
val ^= range_val
# Add keyword. Format is keyword: value
- if p.has_key('keyword'):
+ if 'keyword' in p:
if isinstance(p['keyword'], list):
- if p.has_key('keywordOptional') and p['keywordOptional']:
+ if p.get('keywordOptional'):
keyword_list = [Optional(Suppress(Keyword(k + ":"))) for k in
p['keyword']]
else:
@@ -249,7 +249,7 @@ for k,p in parameter_dicts.iteritems():
keyword = reduce(lambda x,y: x^y, keyword_list)
else:
keyword = Suppress(Keyword(p['keyword'] + ":"))
- if p.has_key('keywordOptional') and p['keywordOptional']:
+ if p.get('keywordOptional'):
keyword = Optional(keyword)
# Combine keyword and value into a single expression
diff --git a/gracedb/superevents/mixins.py b/gracedb/superevents/mixins.py
index 7d62075b373bfd136824fd9cba2e1a8f13e6bfa8..a7cf26adb4997d18e7b520a8648b42a824e5d26e 100644
--- a/gracedb/superevents/mixins.py
+++ b/gracedb/superevents/mixins.py
@@ -220,7 +220,7 @@ class ConfirmGwFormMixin(ContextMixin):
'is_test': 'superevents.confirm_gw_test_superevent',
'is_mdc': 'superevents.confirm_gw_mdc_superevent',
}
- for method_name, perm_name in method_perm_pairs.iteritems():
+ for method_name, perm_name in method_perm_pairs.items():
is_category = getattr(self.object, method_name)
if (is_category() and self.request.user.has_perm(perm_name)):
context['show_gw_status_form'] = True
diff --git a/gracedb/superevents/utils.py b/gracedb/superevents/utils.py
index cec67f8203e27e6e6614734aaafec7a997faff0e..6422150ce017d0c785f96e3479ecc426ac22c3ad 100644
--- a/gracedb/superevents/utils.py
+++ b/gracedb/superevents/utils.py
@@ -130,13 +130,13 @@ def update_superevent(superevent, updater, add_log_message=True,
# Extract "updatable" superevent params from kwargs
param_names = ['t_start', 't_0', 't_end', 'preferred_event']
- new_params = {k: v for k,v in kwargs.iteritems() if k in param_names}
+ new_params = {k: v for k,v in kwargs.items() if k in param_names}
# Get old parameters
old_params = {k: getattr(superevent, k) for k in new_params.keys()}
# Update superevent object
- for k,v in new_params.iteritems():
+ for k,v in new_params.items():
setattr(superevent, k, v)
superevent.save()
@@ -709,7 +709,7 @@ def update_signoff(signoff, user, status, comment, add_log_message=True,
superevent = signoff.superevent
# Update signoff values
- for k,v in updated_attributes.iteritems():
+ for k,v in updated_attributes.items():
setattr(signoff, k, v)
signoff.save(update_fields=list(updated_attributes))
diff --git a/gracedb/templates/gracedb/performance.html b/gracedb/templates/gracedb/performance.html
index 2cf558bd611caf37579b94db5644795efeb934c8..fdaf479097a331b4ad7c72d0842b0f30307406b8 100644
--- a/gracedb/templates/gracedb/performance.html
+++ b/gracedb/templates/gracedb/performance.html
@@ -25,7 +25,7 @@
<h4>Details</h4>
<table style="border-spacing:0px;">
<tr> <th>Status</th> <th>Number of responses</th> </tr>
- {% for key, value in totals_by_status.create.iteritems %}
+ {% for key, value in totals_by_status.create.items %}
<tr> <td> {{ key }} </td> <td> {{ value }} </td> </tr>
{% endfor %}
</table>
@@ -45,7 +45,7 @@
<h4>Details</h4>
<table style="border-spacing:0px;">
<tr> <th>Status</th> <th>Number of responses</th> </tr>
- {% for key, value in totals_by_status.annotate.iteritems %}
+ {% for key, value in totals_by_status.annotate.items %}
<tr> <td> {{ key }} </td> <td> {{ value }} </td> </tr>
{% endfor %}
</table>