# HG changeset patch
# User Mads Kiilerich <mads@kiilerich.com>
# Date 1588454119 -7200
# Node ID 4c7eca54cd55ebb5ea9b5ed4bc1f11ef5e275f6a
# Parent  b77c22e8a39cf26691c97714ff0df92022f24e50
issue: log more helpful errors for "skipping incomplete issue pattern"

diff -r b77c22e8a39c -r 4c7eca54cd55 kallithea/lib/helpers.py
--- a/kallithea/lib/helpers.py	Sat May 02 23:02:02 2020 +0200
+++ b/kallithea/lib/helpers.py	Sat May 02 23:15:19 2020 +0200
@@ -1194,8 +1194,14 @@
             issue_prefix = CONFIG.get('issue_prefix%s' % suffix)
             if issue_prefix:
                 log.error('found unsupported issue_prefix%s = %r - use issue_sub%s instead', suffix, issue_prefix, suffix)
-            if not issue_pat or not issue_server_link or issue_sub is None: # issue_sub can be empty but should be present
-                log.error('skipping incomplete issue pattern %r: %r -> %r %r', k, issue_pat, issue_server_link, issue_sub)
+            if not issue_pat:
+                log.error('skipping incomplete issue pattern %r: it needs a regexp', k)
+                continue
+            if not issue_server_link:
+                log.error('skipping incomplete issue pattern %r: it needs issue_server_link%s', k, suffix)
+                continue
+            if issue_sub is None: # issue_sub can be empty but should be present
+                log.error('skipping incomplete issue pattern %r: it needs (a potentially empty) issue_sub%s', k, suffix)
                 continue
 
             # Wrap tmp_urlify_issues_f with substitution of this pattern, while making sure all loop variables (and compiled regexpes) are bound