Use Contributor class instead of hash

3 files changed, 6 insertions, 2 deletions

diff --git a/lib/api/entities.rb b/lib/api/entities.rb
index b190646a1e3..09fb97abf29 100644
--- a/lib/api/entities.rb
+++ b/lib/api/entities.rb
@@ -218,5 +218,9 @@ module API
 
       expose :same, as: :compare_same_ref
     end
+
+    class Contributor < Grape::Entity
+      expose :name, :email, :commits, :additions, :deletions
+    end
   end
 end
diff --git a/lib/api/repositories.rb b/lib/api/repositories.rb
index 241036729b5..d091fa4f035 100644
--- a/lib/api/repositories.rb
+++ b/lib/api/repositories.rb
@@ -160,7 +160,7 @@ module API
       get ':id/repository/contributors' do
         authorize! :download_code, user_project
 
-        user_project.repository.contributors
+        present user_project.repository.contributors, with: Entities::Contributor
       end
     end
   end
diff --git a/lib/gitlab/contributors.rb b/lib/gitlab/contributors.rb
index 834532470aa..c41e92b620f 100644
--- a/lib/gitlab/contributors.rb
+++ b/lib/gitlab/contributors.rb
@@ -3,7 +3,7 @@ module Gitlab
     attr_accessor :email, :name, :commits, :additions, :deletions
 
     def initialize
-
+      @commits, @additions, @deletions = 0, 0, 0
     end
   end
 end