Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Subtract "missing key" entries from total translated keys, so that percentage is more accurate #106

Open
svogt opened this issue Jun 18, 2013 · 2 comments
Labels

Comments

@svogt
Copy link
Collaborator

svogt commented Jun 18, 2013

It would be good if we subtracted entries in "missing key" state from the total of translated keys. That way, the total count should better match what is actually translated and the overall percentage should be correct again.

@jutzig
Copy link
Owner

jutzig commented Jun 18, 2013

This is actually more tricky than one would think.
Reviews cannot be used for this, because they are computed asynchronously after a property file has been persisted.
The completion state however is computed directly as part of the serialization process and for performance reasons it simply compares the amount of serialized Key/Value pairs with the amount of keys in the template. The Template is not loaded or evaluated for this, the value comes directly from the Database.
Currently I only see two possibilites:

  1. The Missing Key review actively modifies the completion state of resource
  2. During serialization the template must be loaded as well and each translation key must be compared against the template. That comes with a performance penalty.

None of these sound like a perfect solution. Do you have an idea for an alterantive?

@svogt
Copy link
Collaborator Author

svogt commented Jun 19, 2013

Not yet, though I would not object going for no. 1 first and maybe find a better solution later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants