Do not log unchanged email addresses (#646)

This commit is contained in:
Tobias Kunze
2017-10-19 22:03:12 +02:00
committed by Raphael Michel
parent 1b8b12cbc3
commit 60212dcbcc

View File

@@ -645,7 +645,11 @@ class OrderContactChange(OrderView):
def post(self, *args, **kwargs): def post(self, *args, **kwargs):
old_email = self.order.email old_email = self.order.email
changed = False
if self.form.is_valid(): if self.form.is_valid():
new_email = self.form.cleaned_data['email']
if new_email != old_email:
changed = True
self.order.log_action( self.order.log_action(
'pretix.event.order.contact.changed', 'pretix.event.order.contact.changed',
data={ data={
@@ -655,6 +659,7 @@ class OrderContactChange(OrderView):
user=self.request.user, user=self.request.user,
) )
if self.form.cleaned_data['regenerate_secrets']: if self.form.cleaned_data['regenerate_secrets']:
changed = True
self.order.secret = generate_secret() self.order.secret = generate_secret()
for op in self.order.positions.all(): for op in self.order.positions.all():
op.secret = generate_position_secret() op.secret = generate_position_secret()
@@ -664,7 +669,10 @@ class OrderContactChange(OrderView):
self.order.log_action('pretix.event.order.secret.changed', user=self.request.user) self.order.log_action('pretix.event.order.secret.changed', user=self.request.user)
self.form.save() self.form.save()
if changed:
messages.success(self.request, _('The order has been changed.')) messages.success(self.request, _('The order has been changed.'))
else:
messages.success(self.request, _('Nothing about the order had to be changed.'))
return redirect(self.get_order_url()) return redirect(self.get_order_url())
return self.get(*args, **kwargs) return self.get(*args, **kwargs)