launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #27271
Re: [Merge] ~twom/launchpad:gdbr-add-bzr-branches into launchpad:master
Little refactor suggested.
Diff comments:
> diff --git a/lib/lp/registry/model/person.py b/lib/lp/registry/model/person.py
> index edac8e9..dbadb17 100644
> --- a/lib/lp/registry/model/person.py
> +++ b/lib/lp/registry/model/person.py
> @@ -4084,13 +4085,33 @@ class PersonSet:
> return {"status": "no data held"}
>
> account = email_results.one()[1]
> - # This is only an 'account' in terms of the end user view,
> - # it does not refer to an `IAccount`.
> - return_data = {"status": "account only; no other data"}
> + return_data = {}
> return_data["person"] = canonical_url(account)
> # Get the data behind the overview screen
> overview = self.getUserOverview(account)
> return_data.update(overview)
> +
> + # bzr branches
Maybe this whole block can be split into a specific method as we did with the `getUserOverview`? Mainly envisioning this to grow quickly.
> + branches = account.getBranches()
> + if not branches.is_empty():
> + search_url = canonical_url(
> + account, rootsite='code', view_name='+branches')
> + req = PreparedRequest()
> + req.prepare_url(search_url, {
> + "field.category": "OWNED",
> + "field.category-empty-marker": "1",
> + "field.lifecycle": "ALL",
> + "field.lifecycle-empty-marker": "1",
> + "field.sort_by": "most recently changed first",
> + "field.sort_by-empty-marker": "1"})
> + return_data['branches'] = req.url
> +
> + # This is only an 'account' in terms of the end user view,
> + # it does not refer to an `IAccount`.
> + if len(return_data.keys()) > 1:
> + return_data["status"] = "account with data"
> + else:
> + return_data["status"] = "account only; no other data"
> return return_data
>
> def getUserOverview(self, person):
--
https://code.launchpad.net/~twom/launchpad/+git/launchpad/+merge/405703
Your team Launchpad code reviewers is requested to review the proposed merge of ~twom/launchpad:gdbr-add-bzr-branches into launchpad:master.
References