code cleanup
Bug #494540 reported by
Edwin Grubbs
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
High
|
Edwin Grubbs |
Bug Description
In c/l/fields/
* Collapse if/else statements that return True or False.
* Rename is_valid_
Add test for double-click handling in renew_action() in lp/registry/
Split up tests in lp/registry/
Related branches
lp://qastaging/~edwin-grubbs/launchpad/bug-494540-cleanup
- Björn Tillenius (community): Approve
-
Diff: 198 lines (+53/-33)4 files modifiedlib/canonical/launchpad/fields/__init__.py (+10/-19)
lib/lp/registry/interfaces/person.py (+6/-5)
lib/lp/registry/stories/teammembership/xx-member-renewed-membership.txt (+18/-0)
lib/lp/registry/tests/test_productseries_vocabularies.py (+19/-9)
Changed in launchpad-registry: | |
status: | In Progress → Fix Committed |
Changed in launchpad-registry: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.