#3198 Fix stats for commits without author
Merged 6 years ago by pingou. Opened 6 years ago by lsedlar.
lsedlar/pagure empty-email  into  master

file modified
+3
@@ -853,6 +853,9 @@ 

          stats[(author, email)] += 1

  

      for (name, email), val in stats.items():

+         if not email:

+             # Author email is missing in the git commit.

+             continue

          # For each recorded user info, check if we know the e-mail address of

          # the user.

          user = pagure.lib.search_user(session, email=email)

Fixes: #3196

I can't reproduce it, so I'm not completely sure this works.

rebased onto 83bfcffee26737ad84df9afa40b33ed89fe5f04e

6 years ago

rebased onto 4c386fb

6 years ago

I see the traceback and I think the fix is right so I'm going to merge this but grr it's annoying not to be able to replicate this :(

Pull-Request has been merged by pingou

6 years ago
Metadata