Bug 703313 - Translation Memory searches don't trigger when they should
Summary: Translation Memory searches don't trigger when they should
Alias: None
Product: Zanata
Classification: Retired
Component: Component-UI
Version: unspecified
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
: ---
Assignee: David Mason
QA Contact: Ding-Yi Chen
Depends On:
Blocks: Zanata-1.4
TreeView+ depends on / blocked
Reported: 2011-05-09 23:40 UTC by croe@redhat.com
Modified: 2011-10-28 07:05 UTC (History)
3 users (show)

Fixed In Version: 1.4
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2011-10-28 07:05:41 UTC

Attachments (Terms of Use)

Description croe@redhat.com 2011-05-09 23:40:03 UTC
Description of problem: Down Under Translation Memory
After I translated the Insider Newsletter doc on Zanata/Engineering (internal), I noticed that some strings I just translated on the previous doc (TAM Newsetter), and that were according to'MY (human) memory' 100% matching, did not come up as suggestions in the TM (Translation Memory).
So I intuitively clicked on strings below and came back on my current upper string I was working on, then I noticed 'the suggestion' coming up in the TM. So I clicked on copy, and it worked!
This was quite 'boring' process after a while, so I tried to translate starting from the END of the document = starting with last string to be translated, and then there was no problems at all, all suggestions came up as they should, except it was DOWN UNDER (the other way round).
Could we please fix this bug so that we can have suggestions in TM coming from the top from now on, because so far, translators have missed out on the benefits of the TM capabilities.

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
Actual results:

Expected results: Fonctional TM

Additional info:

Comment 1 Sean Flanigan 2011-05-26 04:01:29 UTC
Sorry, I don't understand the problem description.  I will have to see the problem in action.  Please let me know when you see it again, so I can take a look.

Comment 2 Sean Flanigan 2011-06-07 06:03:04 UTC
Corina showed me two different but related problems.

1. When using the Save button to advance (from message A) to the next message B, Zanata does not perform a TM search for the new message B.

2. If user attempts to force a TM search by clicking on the previous message A, then on B, the search is not triggered.

Only by clicking a lower message, say C, or a message *above A*, can the TM search be triggered.  Clicking on A or B does not trigger a TM search.

Comment 3 Sean Flanigan 2011-08-08 07:24:19 UTC
Note: this bug isn't really about TM, it's about not updating the selected row when we use the Save button.  Thus we don't trigger the TM search.  

Now that we put a border around the selected row, it's easier to see what's going on.

Comment 4 Sean Flanigan 2011-08-12 00:33:31 UTC
This is more serious than I thought, because if the translator doesn't realise what's happening, he or she might click Copy on a "100% match" which isn't really a match at all, because it's a match for a different string.

Comment 5 Sean Flanigan 2011-08-12 01:07:09 UTC
Please ensure that:
(a) The selected row updates whenever navigation occurs, whether using Next/Previous, Next/Previous Fuzzy or Untranslated, Save as Approved buttons, or the keyboard equivalents.
(b) Whenever the selected row changes by any means, the translation memory updates to show matches for that row.

Comment 6 James Ni 2011-08-12 02:13:49 UTC
In commit c52e14, i have added a method of selectRow in gotoRow function of TableEditorView.java. I test with Save as approved button and alt+J/K keyboard short-cuts, both working fine and highlight is moved to next row. And I think Translation Memory is refreshed since row selection event will trigger the Translation memory search, but i only test it in WebTrans-Dev mode and test data is limited.

Please help to make sure that my implementation is correct, thanks

Comment 8 Ding-Yi Chen 2011-08-25 16:46:24 UTC
Will test with 
Keyboard shortcut: 
1. Move up/down
2. Move previous fuzzy/ next fuzzy
3. Save as Approved

Mouse click.
1. Click 1 then 2
2. Click 4 then 3
3. Use previous/next buttons
4. Use previous fuzzy/ next fuzzy
5. Save as Approved

Comment 9 Ding-Yi Chen 2011-08-26 04:28:13 UTC
VERIFIED with 1.4-SNAPSHOT (20110824-1328).

Comment 10 croe@redhat.com 2011-09-16 00:36:18 UTC
I tested it again today on two very similar docs: Newsletters TAM & INSIDER translations, where a lot of text is 100% in these two newsletters.

The same problem occurs: the Translation Memory does not trigger when it should, exactely with the same problems described earlier. You need to go down in the document and make your way upwards for the TM to trigger which is counter productif to the way a translator operates. 
I do not believe this bug has been fixed yet.

Comment 11 David Mason 2011-09-16 00:51:13 UTC
Corina, which zanata version is shown at the bottom of the zanata page?

It will be something like "Zanata version 1.X"

Comment 12 Sean Flanigan 2011-09-16 00:55:47 UTC
VERIFIED means that Dean has validated the fix in our test environment.  Zanata 1.4 hasn't been released or put into production yet.

Note You need to log in before you can comment on or make changes to this bug.