Remove PersonLocation-related code
Bug #933911 reported by
Guilherme Salgado
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Guilherme Salgado | ||
Bug Description
We no longer have any views to edit a person's location and most of the code that deals with that is currently unused.
Related branches
lp:~salgado/launchpad/remove-map-views
- Ian Booth (community): Approve
-
Diff: 1945 lines (+95/-1416)26 files modifiedlib/lp/app/widgets/doc/location-widget.txt (+0/-204)
lib/lp/app/widgets/location.py (+0/-192)
lib/lp/app/widgets/templates/location.pt (+0/-2)
lib/lp/blueprints/browser/tests/sprintattendance-views.txt (+0/-20)
lib/lp/registry/browser/configure.zcml (+1/-28)
lib/lp/registry/browser/person.py (+21/-71)
lib/lp/registry/browser/team.py (+0/-116)
lib/lp/registry/browser/tests/team-views.txt (+0/-129)
lib/lp/registry/doc/personlocation.txt (+17/-195)
lib/lp/registry/interfaces/location.py (+12/-6)
lib/lp/registry/interfaces/person.py (+1/-35)
lib/lp/registry/model/person.py (+18/-113)
lib/lp/registry/stories/location/personlocation-edit.txt (+7/-28)
lib/lp/registry/stories/location/personlocation.txt (+0/-18)
lib/lp/registry/stories/location/team-map.txt (+0/-86)
lib/lp/registry/stories/person/xx-person-home.txt (+5/-20)
lib/lp/registry/stories/webservice/xx-personlocation.txt (+9/-25)
lib/lp/registry/templates/person-index.pt (+0/-2)
lib/lp/registry/templates/person-portlet-contact-details.pt (+1/-1)
lib/lp/registry/templates/person-portlet-map.pt (+0/-21)
lib/lp/registry/templates/team-map-data.pt (+0/-17)
lib/lp/registry/templates/team-map.pt (+0/-45)
lib/lp/registry/templates/team-portlet-map.pt (+0/-37)
lib/lp/registry/tests/test_person.py (+1/-2)
lib/lp/registry/tests/test_personset.py (+1/-2)
lib/lp/registry/xmlrpc/canonicalsso.py (+1/-1)
| Changed in launchpad: | |
| assignee: | nobody → Guilherme Salgado (salgado) |
| status: | New → In Progress |
| tags: | added: tech-debt users |
| Changed in launchpad: | |
| importance: | Undecided → Low |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
Revision history for this message
| William Grant (wgrant) wrote | #2 |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r14834 <http://