From 3d4c5fb1235b19e05e32f1a1d9b35833c82aeb54 Mon Sep 17 00:00:00 2001 From: Carlos Ruiz Date: Sat, 26 Feb 2011 20:46:37 -0500 Subject: [PATCH] Revert rev 12643 - Invalid "Record on parent tab was changed" 6334 aa2da671ac3a http://sourceforge.net/support/tracker.php?aid=3017560 According to the description of the tracker the test scenario is not valid because it could be possible that process completed the document and in this case the warning is valid. Checking with the flag manualcmd is not good as this is used to indicate a navigation button was pushed. I tested again the issue described in tracker 1985481 and it needs further work, the validation is raised when the user push the save button, but not when the user pushes next or previous record buttons. http://sourceforge.net/support/tracker.php?aid=1985481 --- org.adempiere.base/src/org/compiere/model/GridTab.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/org.adempiere.base/src/org/compiere/model/GridTab.java b/org.adempiere.base/src/org/compiere/model/GridTab.java index d5a6d7f3ef..45c492f08c 100644 --- a/org.adempiere.base/src/org/compiere/model/GridTab.java +++ b/org.adempiere.base/src/org/compiere/model/GridTab.java @@ -974,11 +974,7 @@ public class GridTab implements DataStatusListener, Evaluatee, Serializable try { if (hasChangedCurrentTabAndParents()) - { - // Fail only if it's a true change - teo_sarca [ 3017560 ] - if (manualCmd || m_mTable.hasChanged(m_currentRow)) - return false; - } + return false; boolean retValue = (m_mTable.dataSave(manualCmd) == GridTable.SAVE_OK); if (manualCmd)