Bug 844239 - [WebTrans] WebUI shows Save FAILED on page 2 or later

zanata · Aug 2, 2012 · 3192a92 · 3192a92
1 parent ae842b4
commit 3192a92
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/zanata-war/src/main/java/org/zanata/webtrans/client/editor/table/TableEditorPresenter.java b/zanata-war/src/main/java/org/zanata/webtrans/client/editor/table/TableEditorPresenter.java
@@ -759,7 +759,8 @@ public void onSuccess(UpdateTransUnitResult result)
                {
                   UndoLink undoLink = undoLinkProvider.get();
                   undoLink.prepareUndoFor(result);
-                  targetContentsPresenter.addUndoLink(row, undoLink);
+                  int rowIndex = row % display.getPageSize();
+                  targetContentsPresenter.addUndoLink(rowIndex, undoLink);
                   eventBus.fireEvent(new NotificationEvent(Severity.Info, messages.notifyUpdateSaved()));
                }
                else

diff --git a/...ta-war/src/main/java/org/zanata/webtrans/client/editor/table/TargetContentsPresenter.java b/...ta-war/src/main/java/org/zanata/webtrans/client/editor/table/TargetContentsPresenter.java
@@ -812,7 +812,10 @@ public void concealDisplay()
 
    public void addUndoLink(int row, UndoLink undoLink)
    {
-      TargetContentsDisplay targetContentsDisplay = displayList.get(row);
-      targetContentsDisplay.addUndo(undoLink);
+      if (row >=0 && row < displayList.size())
+      {
+         TargetContentsDisplay targetContentsDisplay = displayList.get(row);
+         targetContentsDisplay.addUndo(undoLink);
+      }
    }
 }