'auto_delegate' doesn't actually delegate the patches - it merely finds a suitable delegate based on the filename. Rename the function accordingly.
'find_delegate' is also renamed to prevent confusion. Signed-off-by: Stephen Finucane <step...@that.guru> Cc: Paul Jakma <p...@jakma.org> Cc: Tom Rini <tr...@konsulko.com> --- patchwork/parser.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/patchwork/parser.py b/patchwork/parser.py index 2df9278..9d1b79e 100644 --- a/patchwork/parser.py +++ b/patchwork/parser.py @@ -688,7 +688,7 @@ def find_state(mail): return get_default_initial_patch_state() -def auto_delegate(project, filenames): +def find_delegate_by_filename(project, filenames): if not filenames: return None @@ -714,7 +714,7 @@ def auto_delegate(project, filenames): return patch_delegate -def find_delegate(mail): +def find_delegate_by_header(mail): """Return the delegate with the given email or None.""" delegate_email = mail.get('X-Patchwork-Delegate', '').strip() if delegate_email: @@ -784,10 +784,10 @@ def parse_mail(mail, list_id=None): # we delay the saving until we know we have a patch. author.save() - delegate = find_delegate(mail) + delegate = find_delegate_by_header(mail) if not delegate and diff: filenames = find_filenames(diff) - delegate = auto_delegate(project, filenames) + delegate = find_delegate_by_filename(project, filenames) series = find_series(mail) # We will create a new series if: -- 2.9.3 _______________________________________________ Patchwork mailing list Patchwork@lists.ozlabs.org https://lists.ozlabs.org/listinfo/patchwork