QA: Also check for number of people in CC

1 files changed, 8 insertions, 6 deletions

diff --git a/qa/bugzillaChecker.py b/qa/bugzillaChecker.py
index 0c7edfff..39162274 100755
--- a/qa/bugzillaChecker.py
+++ b/qa/bugzillaChecker.py
@@ -49,6 +49,7 @@ versionsToCheck = ('5', '6')
 crashReportDomain = "https://crashreport.libreoffice.org/stats/signature/"
 
 minNumOfDupes = 5
+minNumOfCC = 20
 
 def util_create_statList_checkers():
     return {
@@ -89,14 +90,15 @@ def analyze_bugzilla_checkers(statList, bugzillaData, cfg):
             common.util_check_bugzilla_mail(statList, creatorMail, row['creator_detail']['real_name'], creationDate, rowId)
 
             if common.isOpen(rowStatus) and rowId not in dupesBugs:
-                dupesBugs[rowId] = { 'total': 0, 'priority': row['priority'], 'severity': row['severity'] }
+                dupesBugs[rowId] = { 'totalDupes': 0, 'totalCC': len(row['cc']), 'priority': row['priority'], 'severity': row['severity'] }
 
             rowDupeOf = common.util_check_duplicated(bugzillaData, rowId)
             if rowDupeOf and common.isOpen(bugzillaData['bugs'][str(rowDupeOf)]['status']):
                 if rowDupeOf not in dupesBugs:
-                    dupesBugs[rowDupeOf] = { 'total': 0, 'priority': bugzillaData['bugs'][str(rowDupeOf)]['priority'],
+                    dupesBugs[rowDupeOf] = { 'totalDupes': 0, 'totalCC': len(bugzillaData['bugs'][str(rowDupeOf)]['cc']),
+                            'priority': bugzillaData['bugs'][str(rowDupeOf)]['priority'],
                             'severity': bugzillaData['bugs'][str(rowDupeOf)]['severity'] }
-                dupesBugs[rowDupeOf]['total'] += 1
+                dupesBugs[rowDupeOf]['totalDupes'] += 1
 
             crashSignature = row['cf_crashreport']
             if crashSignature:
@@ -401,14 +403,14 @@ def analyze_bugzilla_checkers(statList, bugzillaData, cfg):
 
     for k, v in dupesBugs.items():
         if v['severity'] == 'enhancement':
-            if v['total'] < minNumOfDupes and (v['priority'] == 'high' or v['priority'] == 'highest'):
+            if v['totalDupes'] < minNumOfDupes and v['totalCC'] < minNumOfCC and (v['priority'] == 'high' or v['priority'] == 'highest'):
                 value = [k, '', '']
                 util_add_to_result(lResults, 'change_enhancement_priority_to_medium', value)
-            elif v['total'] >= minNumOfDupes and (v['priority'] != 'high' and v['priority'] != 'highest'):
+            elif (v['totalDupes'] >= minNumOfDupes or v['totalCC'] >= minNumOfCC) and (v['priority'] != 'high' and v['priority'] != 'highest'):
                 value = [k, '', '']
                 util_add_to_result(lResults, 'change_enhancement_priority_to_high', value)
         else:
-            if v['total'] >= minNumOfDupes and (v['priority'] != 'high' and v['priority'] != 'highest'):
+            if (v['totalDupes'] >= minNumOfDupes or v['totalCC'] >= minNumOfCC) and (v['priority'] != 'high' and v['priority'] != 'highest'):
                 value = [k, '', '']
                 util_add_to_result(lResults, 'change_bug_priority_to_high', value)