Count of changed in LP messages is too big
Bug #455680 reported by
Kuvaly [LCT]
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Данило Шеган | ||
Bug Description
Good day,
at page https:/
Thank you.
Related branches
lp:~danilo/launchpad/bug-455680
- Jeroen T. Vermeulen (community): Approve
-
Diff: 131 lines2 files modifiedlib/lp/translations/model/pofile.py (+2/-1)
lib/lp/translations/tests/test_pofile.py (+47/-12)
Revision history for this message
| Kuvaly [LCT] (kuvaly) wrote | #1 |
| affects: | launchpad → rosetta |
| Changed in rosetta: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| tags: | added: message-sharing |
| Changed in rosetta: | |
| assignee: | nobody → Данило Шеган (danilo) |
| milestone: | none → 3.1.10 |
Revision history for this message
| Данило Шеган (danilo) wrote | #2 |
| Changed in rosetta: | |
| status: | Triaged → In Progress |
| summary: |
- translation status appears too sometimes over debt and sometimes too - short + Count of changed in LP messages is too big |
Revision history for this message
| Ursula Junque (ursinha) wrote A commit mentioned this bug | #3 |
| Changed in rosetta: | |
| status: | In Progress → Fix Committed |
| tags: | added: qa-needstesting |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| Данило Шеган (danilo) wrote | #4 |
| Changed in rosetta: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
It is a bug in how we figure out if a message is changed in Launchpad, so we let some slip in even if they should not be counted.
Technical details ahead :)
What happens is that with some old data, possibly resulting from message sharing migration, we've got a weird situation:
1. There is a shared translation which is both current and imported
2. There is a diverged translation which is only imported (and not current)
3. There is no diverged current translation
Technically, this one is a changed translation, because the current one (shared, imported one) is different from the most relevant imported translation (diverged one). However, since this is a situation one can't get into with the latest code, and considering how we want to get rid of all the divergence for imported messages (bug #456597), let's just not count them anyway.