midsuit
/
core-jgi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

IDEMPIERE-2253 a field invisible at gridview by display logic don't real invisible (user can see data behind) / IDEMPIERE-2148 / IDEMPIERE-2935

This commit is contained in:
Carlos Ruiz 2015-12-16 08:20:43 -05:00
parent 93c31a73a4
commit f7596139e2
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/adwindow/GridTabRowRenderer.java
View File

@ -17,6 +17,7 @@ import java.util.LinkedHashMap;
import java.util.List;
import java.util.Map;
import java.util.Map.Entry;
import java.util.Properties;
import org.adempiere.util.GridRowCtx;
import org.adempiere.webui.LayoutUtils;
@ -504,10 +505,11 @@ public class GridTabRowRenderer implements RowRenderer<Object[]>, RowRendererExt
else if (DisplayType.isNumeric(gridPanelFields[i].getDisplayType())) {
divStyle = CELL_DIV_STYLE_ALIGN_RIGHT;
}
if (!gridPanelFields[i].isDisplayedGrid()){
GridRowCtx ctx = new GridRowCtx(Env.getCtx(), gridTab, rowIndex);
if (!gridPanelFields[i].isDisplayed(ctx, true)){
// IDEMPIERE-2253
div.removeChild(component);
component.setVisible(false);
}
}
div.setStyle(divStyle);
@ -642,11 +644,14 @@ public class GridTabRowRenderer implements RowRenderer<Object[]>, RowRendererExt
div.appendChild(popupMenu);
popupMenu.addContextElement((XulElement) editor.getComponent());
}
Properties ctx = isDetailPane() ? new GridRowCtx(Env.getCtx(), gridTab, gridTab.getCurrentRow())
: gridPanelFields[i].getVO().ctx;
//check context
if (!gridPanelFields[i].isDisplayedGrid()){
if (!gridPanelFields[i].isDisplayed(ctx, true)){
// IDEMPIERE-2253
div.removeChild(editor.getComponent());
editor.getComponent().setVisible(false);
}
editor.setReadWrite(gridPanelFields[i].isEditableGrid(true));