[zanata-bugs] [Bug 867196] New: [WebTrans] click source on different row will trigger validation on mismatch source and target

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=867196

            Bug ID: 867196
        QA Contact: dchen at redhat.com
          Severity: unspecified
           Version: development
          Priority: unspecified
                CC: zanata-bugs at redhat.com
          Assignee: runab at redhat.com
           Summary: [WebTrans] click source on different row will trigger
                    validation on mismatch source and target
        Regression: ---
      Story Points: ---
    Classification: Community
                OS: Unspecified
          Reporter: pahuang at redhat.com
              Type: Bug
     Documentation: ---
          Hardware: Unspecified
        Mount Type: ---
            Status: NEW
         Component: Component-Logic
           Product: Zanata

Description of problem:
In editor, click on source will trigger validation. But if click on source on
different row it will try to validate current source against the clicked row's
target and get wrong validation warning.

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

How reproducible:
always

Steps to Reproduce:
1. Open editor and select a row without any variables(i.e.%s) in source
2. Enter some text in editor without any variables
3. Click on a different row with variable(i.e. %s) in source

Actual results:
Row selection changes but previous selected row will be marked as error.

Expected results:
Row selection changes and previous selected row should not have any error.

Additional info:
This is due to a RequestValidationEvent is triggered on source click. When
clicking on different row, it will fire this and then compare this source
against the old selection's target and get wrong result. We need to ensure row
selection change happen before this event so that validation only occur between
source and target on the same row.

-- 
You are receiving this mail because:
You are on the CC list for the bug.