esc-reporting/esc-analyze.py | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
New commits:
commit 970abc0ae1ea75400352663fdbc416f38babc405
Author: jan Iversen <j...@libreoffice.org>
Date: Mon May 15 07:42:13 2017 +0200
esc-report: not all committers have a gerrit account
diff --git a/esc-reporting/esc-analyze.py b/esc-reporting/esc-analyze.py
index 0b45f70..7680be1 100755
--- a/esc-reporting/esc-analyze.py
+++ b/esc-reporting/esc-analyze.py
@@ -664,19 +664,20 @@ def analyze_reports():
if cntReview == 0 and not
statList['people'][ownerEmail]['isCommitter']:
tmpListToReview.append(entry['id'])
+ defaultEmail = util_check_mail('',
cfg['automate']['gerritRewiewUserEmail'])
for id in tmpListToReview:
- reviewEmail = util_check_mail('',
cfg['automate']['gerritRewiewUserEmail'])
+ reviewEmail = defaultEmail
txt = gerritData['patch'][id]['subject']
if txt.startswith('tdf#'):
try:
row = bugzillaData['bugs'][re.findall('\d+', txt)[0]]
ownerEmail = util_check_mail(row['creator_detail']['name'],
row['creator_detail']['email'])
- if statList['people'][ownerEmail]['isCommitter']:
+ if 'reviewName' in statList['people'][ownerEmail]['gerrit']:
reviewEmail = ownerEmail
else:
for comment in row['comments']:
email = util_check_mail('', comment['creator'])
- if not email == 'aniste...@gmail.com' and not email ==
'j...@documentfoundation.org' and statList['people'][email]['isCommitter']:
+ if not email == 'aniste...@gmail.com' and not email ==
'j...@documentfoundation.org' and 'reviewName' in
statList['people'][ownerEmail]['gerrit']:
reviewEmail = email
break
except Exception as e:
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
