Refactor the LaunchpadDatabasePolicy implementation
Bug #274132 reported by
Francis J. Lacoste
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Medium
|
Francis J. Lacoste | ||
Bug Description
We have an IDatabasePolicy interface (that doesn't match the implementation).
The current implementation contains a bunch of if/else statement looking at the request type, since the policy is an adapter look-up, we should extract those into different adapters for each specific request type.
| Changed in launchpad-foundations: | |
| assignee: | nobody → flacoste |
| importance: | Undecided → Medium |
| milestone: | none → 2.1.10 |
| status: | New → Triaged |
| Changed in launchpad-foundations: | |
| milestone: | 2.1.10 → 2.1.11 |
Revision history for this message
| Francis J. Lacoste (flacoste) wrote | #1 |
| Changed in launchpad-foundations: | |
| status: | Triaged → In Progress |
Revision history for this message
| Francis J. Lacoste (flacoste) wrote | #2 |
| Changed in launchpad-foundations: | |
| status: | In Progress → Fix Committed |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed at the same time than bug 294842.