Use Contributor class instead of hash

1 files changed, 10 insertions, 15 deletions

diff --git a/app/models/repository.rb b/app/models/repository.rb
index a2285bb5dab..89bf12ce9ef 100644
--- a/app/models/repository.rb
+++ b/app/models/repository.rb
@@ -244,28 +244,23 @@ class Repository
   end
 
   def contributors
-    contributors = {}
     log = graph_log.group_by { |i| i[:author_email] }
 
-    log.each do |email, contributions|
-      contributors[email] = {
-        email: email,
-        commits: 0,
-        additions: 0,
-        deletions: 0,
-      }
+    log.map do |email, contributions|
+      contributor = Gitlab::Contributor.new
+      contributor.email = email
 
       contributions.each do |contribution|
-        if contributors[email][:name].blank?
-          contributors[email][:name] = contribution[:author_name]
+        if contributor.name.blank?
+          contributor.name = contribution[:author_name]
         end
 
-        contributors[email][:commits] += 1
-        contributors[email][:additions] += contribution[:additions] || 0
-        contributors[email][:deletions] += contribution[:deletions] || 0
+        contributor.commits += 1
+        contributor.additions += contribution[:additions] || 0
+        contributor.deletions += contribution[:deletions] || 0
       end
-    end
 
-    contributors.values
+      contributor
+    end
   end
 end