Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKevin Barbe2016-10-14 06:29:56 -0400
committerMarvin Mueller2016-10-31 08:54:53 -0400
commite40eee661a99f974938e8be01d6027bcf2bfcdb9 (patch)
tree021ab445c122367846195d6f036430b6a58d8840
parent2c69d155f0064e461eaa9a1c0735415cb654d085 (diff)
downloadorg.eclipse.jubula.core-e40eee661a99f974938e8be01d6027bcf2bfcdb9.tar.gz
org.eclipse.jubula.core-e40eee661a99f974938e8be01d6027bcf2bfcdb9.tar.xz
org.eclipse.jubula.core-e40eee661a99f974938e8be01d6027bcf2bfcdb9.zip
JUB-1775: Fix for Bug 500914 - Value is stored in the wrong cell when editing a cell in the Data Sets View and clicking on another cell after editing
Change-Id: Iee418c6daba97f99d029c1a7bf3190db318c5ba2 Signed-off-by: Kevin Barbe <kevin.barbe@bredex.de>
-rw-r--r--org.eclipse.jubula.client.ui.rcp/src/org/eclipse/jubula/client/ui/rcp/views/dataset/AbstractDataSetPage.java11
1 files changed, 6 insertions, 5 deletions
diff --git a/org.eclipse.jubula.client.ui.rcp/src/org/eclipse/jubula/client/ui/rcp/views/dataset/AbstractDataSetPage.java b/org.eclipse.jubula.client.ui.rcp/src/org/eclipse/jubula/client/ui/rcp/views/dataset/AbstractDataSetPage.java
index 9b21d7e64..303c3ec88 100644
--- a/org.eclipse.jubula.client.ui.rcp/src/org/eclipse/jubula/client/ui/rcp/views/dataset/AbstractDataSetPage.java
+++ b/org.eclipse.jubula.client.ui.rcp/src/org/eclipse/jubula/client/ui/rcp/views/dataset/AbstractDataSetPage.java
@@ -1082,6 +1082,8 @@ public abstract class AbstractDataSetPage extends Page
}
}
};
+ /** The index of the cell the editor was last activated at */
+ private int m_currentEditorIndex;
/**
* @param parent parent
@@ -1149,11 +1151,9 @@ public abstract class AbstractDataSetPage extends Page
private void writeDataSetData(String property, Object value,
AbstractJBEditor edit) {
final int langIndex = getColumnIndexOfProperty(property);
- final int dsNumber = getSelectedDataSet();
- final int paramIndex = getTable()
- .getSelectionIndex();
- setValueToModel(value, edit, paramIndex, dsNumber);
- getTable().getItem(paramIndex).setText(langIndex,
+ setValueToModel(value, edit, m_currentEditorIndex,
+ m_currentEditorIndex);
+ getTable().getItem(m_currentEditorIndex).setText(langIndex,
value == null ? StringConstants.EMPTY : (String) value);
}
@@ -1279,6 +1279,7 @@ public abstract class AbstractDataSetPage extends Page
}
// end http://eclip.se/390800
TextControlBP.selectAll(m_editor.getEditor());
+ m_currentEditorIndex = getSelectedDataSet();
}
}

Back to the top