Public

Project privacy set to public. By default, its content is available to everyone (authenticated or not). Please note that more restrictive permissions might exist on some items.

stable

Clone or download

Read-only

request #10650: Pullrequest list must not load all the pullrequest details

Download Browse

When we list the pullrequests for a repository in the pullrequest list page, the full pullrequest representation is loaded. If you have a lot of pullrequests and if the git repository is using fine-grained permissions, the requests to load the pullrequests takes too much time. Now we return only a minimal representation in git/:id/pull_requests so that required information is still displayed on the pullrequest list page (labels, author, title, …) Change-Id: I2cbaf6217307bda6c1a787467c45962f63c3efc1

+145 −160
Nicolas Terray (nterray) 2017-10-02 16:50
Nicolas Terray (nterray) 2017-10-03 13:50
1ef69d371011a33edb84a22d7a1250948bd312f0
99bf8f9b81c76bfa2c85d23a4881e3d9b4f821e7
git #tuleap/stable/1ef69d371011a33edb84a22d7a1250948bd312f0

Modified Files

Side by side diff
Inline diff
Name
A plugins/pullrequest/include/PullRequest/REST/v1/PullRequestMinimalRepresentation.php +101 −0 Go to diff View file
M plugins/pullrequest/include/PullRequest/REST/v1/PullRequestRepresentation.php +31 −70 Go to diff View file
M plugins/pullrequest/include/PullRequest/REST/v1/RepositoryResource.php +9 −21 Go to diff View file
M plugins/pullrequest/include/autoload.php +3 −2 Go to diff View file
M plugins/pullrequest/www/js/angular/src/app/overview/_overview.scss +0 −1 Go to diff View file
M plugins/pullrequest/www/js/angular/src/app/pull-request/pull-request-controller.js +1 −26 Go to diff View file
M plugins/pullrequest/www/js/angular/src/app/pull-request/pull-request-controller.spec.js +0 −40 Go to diff View file