DistroReleaseQueue.status could use the immutable flag
Bug #29663 reported by
Christian Reis
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Triaged
|
Low
|
Unassigned |
Bug Description
As per Salgado/Spiv's review on 2006-01-20 we should use the immutable=True flag on the status attribute, instead of the rather awkward _set_status() method we're using.
Changed in launchpad: | |
status: | Unconfirmed → Confirmed |
Changed in soyuz: | |
assignee: | Celso Providelo (cprov) → nobody |
Changed in launchpad: | |
importance: | Medium → Low |
To post a comment you must log in.
Salgado also suggested:
> This seems a bit evil to me. Have you checked with Andrew if there isn't a
> better way of doing this? Maybe having the DB column as a private attribute
> (not declared in the interface) and then providing a read-only property that
> only reads this private attribute. This way you won't need this _set_status()
> method and all the set_foo() methods could simply do "self._status = foo".