#173 closed defect (fixed)

When re importing originals unchanged a +active translation becomes obsolete

Reported by: petervanderdoes Owned by: somebody
Priority: critical Milestone:
Component: general Version: 0.1
Keywords: has-patch Cc: nbachiyski

Description

When you re import a original translation file, all translation marked as +active that were unchanged in the original translation file are set to -obsolete,

Attachments (2)

173.0.patch (3.2 KB) - added by petervanderdoes 16 months ago.
Fixes the reimporting problem
gp.173.diff (2.1 KB) - added by nacin 16 months ago.

Download all attachments as: .zip

Change History (7)

Fixes the reimporting problem

  • Keywords has-patch added
  • Cc nbachiyski added

Works for me, needs blessing from nbachiyski

  • Resolution set to fixed
  • Status changed from new to closed

(In [657]) Do not leave unchanged active strings with their tempopary obsolete state

First, we used to mark all active strings as +obsolete, then mark all
current ones as +active.

Then, updating all strings on every POT import started causing performance
problems, so in [643] we stopped updating unchanged strings. This caused
unchanged active strings to be left in obsolete state.

The current approach is to leave all originals active, mark the new ones
as active, too and then, go through all removed and mark only them
as oboslete.

Historically, removed strings are fewer than added, that's why we chose to
remove them one by one, instead of marking only news ones.

Fixes #173, props petervanderdoes

  • Resolution fixed deleted
  • Status changed from closed to reopened

[657] uses the wrong variable name in one spot, meaning that old strings won't get obsoleted. Patch attached.

nacin16 months ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [663]) Obsolete the right string when importing originals

We were using the variable $original, which isn't
the one from the loop, but has been used in previous
foreachs and thus have a value and we didn't get
errors.

The test needed to obsolete at least two originals, because
otherwise by chance we were obsoleting the right original.

Fixes #173, props nacin

Note: See TracTickets for help on using tickets.