Skip to content

Commit 9bbd9e7

Browse files
committed
Fix indents of multiline if that are confusable with next block
This resolves pycodestyle warnings of the form: "E129 visually indented line with same indent as next logical line"
1 parent 96318bc commit 9bbd9e7

4 files changed

Lines changed: 10 additions & 10 deletions

File tree

courier/control.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -280,7 +280,7 @@ def del_recipient(control_paths, recipient):
280280
rcpts = _get_recipients_from_file(control_path)
281281
for rcpt in rcpts:
282282
if (rcpt[1] is False # Delivery is not complete for this recipient
283-
and rcpt[2][0] == recipient):
283+
and rcpt[2][0] == recipient):
284284
_mark_complete(control_path, rcpt[0])
285285
return
286286

@@ -306,7 +306,7 @@ def del_recipient_data(control_paths, recipient_data):
306306
rcpts = _get_recipients_from_file(cf)
307307
for x in rcpts:
308308
if (x[1] is False # Delivery is not complete for this recipient
309-
and x[2] == recipient_data):
309+
and x[2] == recipient_data):
310310
_mark_complete(cf, x[0])
311311
return
312312

courier/quarantine.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ def send_notice(message, address, sender=None):
7373
# available, or if it is present and a true value.
7474
rcpts = []
7575
if ('notify_recipient' not in config
76-
or config['notify_recipient']):
76+
or config['notify_recipient']):
7777
rcpts.append(address)
7878
if 'also_notify' in config and config['also_notify']:
7979
rcpts.append(config['also_notify'])
@@ -123,9 +123,9 @@ def quarantine(body_path, control_paths, explanation):
123123
# Some sites would prefer that only admins release messages from the
124124
# quarantine.
125125
if ('user_release' in config
126-
and config['user_release'] == 0
127-
and 'also_notify' in config
128-
and config['also_notify']):
126+
and config['user_release'] == 0
127+
and 'also_notify' in config
128+
and config['also_notify']):
129129
release_addr = config['also_notify']
130130
else:
131131
release_addr = 'quarantine-%s-%s@%s' % (config['siteid'],
@@ -189,8 +189,8 @@ def release(requested_id, address):
189189
return
190190
for x in courier.control.get_control_data(quarantine_paths[1])['r']:
191191
if (x[0] == address or
192-
x[1] == address or
193-
x[1] == '%s%s' % ('rfc822;', address)):
192+
x[1] == address or
193+
x[1] == '%s%s' % ('rfc822;', address)):
194194
courier.sendmail.sendmail('', address, open(quarantine_paths[0], 'rb').readlines())
195195
return
196196
# If no address matched, alert the user that the request was invalid.

filters/pythonfilter/deliveredto.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ def do_filter(body_path, control_paths):
4545
if len(dparts) != 2:
4646
return ''
4747
if (courier.config.is_local(dparts[1]) or
48-
courier.config.is_hosteddomain(dparts[1])):
48+
courier.config.is_hosteddomain(dparts[1])):
4949
return '501 Mail loop - already have my Delivered-To: header.'
5050
return ''
5151

filters/pythonfilter/localsenders.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ def do_filter(body_path, control_paths):
5353
if sender_info is None:
5454
return '517 Sender does not exist: %s' % sender
5555
if (require_auth and
56-
courier.control.get_auth_user(control_paths, body_path) is None):
56+
courier.control.get_auth_user(control_paths, body_path) is None):
5757
return '517 Policy requires local senders to authenticate.'
5858
return ''
5959

0 commit comments

Comments
 (0)