Add "max" query parameter for get_recently_added_contributors

tests/test_contributors_views.py

@@ -104,3 +104,14 @@ def test_get_recently_added_contributors(self):
         project.reload()
         recent = [c for c in self.user.recently_added if c.is_active()]
         assert_equal(len(res.json['contributors']), len(recent))
+
+
+    def test_get_recently_added_contributors_with_limit(self):
+        project = ProjectFactory(creator=self.user)
+        for _ in range(5):
+            project.add_contributor(AuthUserFactory(), auth=self.auth)
+        project.save()
+        url = self.project.api_url_for('get_recently_added_contributors', max=4)
+        res = self.app.get(url, auth=self.user.auth)
+        project.reload()
+        assert_equal(len(res.json['contributors']), 4)

website/project/views/contributor.py

@@ -148,9 +148,18 @@ def get_most_in_common_contributors(auth, **kwargs):
 def get_recently_added_contributors(auth, **kwargs):
     node = kwargs['node'] or kwargs['project']
 
+    try:
+        max_results = int(request.args.get('max'))
+    except (TypeError, ValueError):
+        raise HTTPError(http.BAD_REQUEST)
+    if max_results:
+        recently_added = auth.user.recently_added[:max_results]
+    else:
+        recently_added = auth.user.recently_added
+
     contribs = [
         utils.add_contributor_json(contrib, get_current_user())
-        for contrib in auth.user.recently_added
+        for contrib in recently_added
         if contrib.is_active()
         if contrib._id not in node.contributors
     ]