Person.account_status and friends must go

Bug #237280 reported by Stuart Bishop
4
Affects Status Importance Assigned to Milestone
Launchpad itself
Won't Fix
Low
Unassigned

Bug Description

After the auth-person-split branches, Person.account_status, account_status_comment, openid_identifier and password attributes become properties, proxying for Account attributes. They just exist to reduce the impact of the auth-person-split branch on existing code.

These properties should be removed, and call sites changed to use the Account attributes directly. This may involve changing the security declarations on Account as it is currently very tight (and Person too lose).

Changed in launchpad:
status: New → Confirmed
Curtis Hovey (sinzui)
Changed in launchpad-registry:
importance: Undecided → Low
status: Confirmed → Triaged
Curtis Hovey (sinzui)
tags: added: tech-debt
removed: cleanup
Revision history for this message
Guilherme Salgado (salgado) wrote :

In fact, they need to stay now that Account will be gone.

Changed in launchpad-registry:
status: Triaged → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers