From c9d56568b3a8d0ec594ff955302bbe4c3c2f204c Mon Sep 17 00:00:00 2001 From: Heng Sin Low Date: Mon, 30 Sep 2013 18:34:30 +0800 Subject: [PATCH] 1002807 - Fixed ReadonlyLogic+IsProcessed() validation create confusing UI state. IDEMPIERE-54 --- org.adempiere.base/src/org/compiere/model/GridTab.java | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/org.adempiere.base/src/org/compiere/model/GridTab.java b/org.adempiere.base/src/org/compiere/model/GridTab.java index 3328330fef..523668a748 100644 --- a/org.adempiere.base/src/org/compiere/model/GridTab.java +++ b/org.adempiere.base/src/org/compiere/model/GridTab.java @@ -1160,6 +1160,10 @@ public class GridTab implements DataStatusListener, Evaluatee, Serializable } // Prevent New Where Main Record is processed // but not apply for TabLevel=0 - teo_sarca [ 1673902 ] + // hengsin: together with readonly logic, the following validation create confusing situation for user. + // i.e, if readonly logic enable the new button on toolbar, it will just does nothing due to the validation below. + // better let everything decide using just the tab's readonly logic instead. + /* if (m_vo.TabLevel > 0 && m_vo.TabNo > 0) { boolean processed = isProcessed(); @@ -1170,7 +1174,7 @@ public class GridTab implements DataStatusListener, Evaluatee, Serializable return false; } if (log.isLoggable(Level.FINEST)) log.finest("Processed=" + processed); - } + }*/ //hengsin, don't create new when parent is empty if (isDetail() && m_parentNeedSave)