Merge lp://qastaging/~jtv/launchpad/bug-682933 into lp://qastaging/launchpad/db-devel
Proposed by
Jeroen T. Vermeulen
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Jeroen T. Vermeulen | ||||
| Approved revision: | no longer in the source branch. | ||||
| Merged at revision: | 10023 | ||||
| Proposed branch: | lp://qastaging/~jtv/launchpad/bug-682933 | ||||
| Merge into: | lp://qastaging/launchpad/db-devel | ||||
| Diff against target: |
14 lines (+2/-2) 1 file modified
lib/lp/translations/scripts/migrate_current_flag.py (+2/-2) |
||||
| To merge this branch: | bzr merge lp://qastaging/~jtv/launchpad/bug-682933 | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Henning Eggers (community) | code | Approve | |
|
Review via email:
|
|||
Commit message
[r=henninge]
Description of the change
= Bug 682933 =
This fixes a problem with the migration to the "Recife" data model. It works around Storm bug 682989, where the Reference class gets compiled to SQL that does not take ClassAliases into account. This resulted in a seemingly correct query that nevertheless didn't do quite what we meant it to.
Kudos to Thomas Hervé, who came up with the simple workaround on very short notice.
{{{
./bin/test -vvc lp.translations -t migrate.
}}}
Jeroen
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
If you say so ... ;-) Thanks for figuring this out!