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

IDEMPIERE 5929 - Looping on ZK happening on Exceptions caused by IServerPushCallback.updateUI implementations (#2106) 

* IDEMPIERE-5929 - Fix replace tab method to add a window if there is no refTab reference

* IDEMPIERE-5929 - Removes unnecessary NPE handling when replacing tabs

* IDEMPIERE-5929 - Adjust code indentation/style

* IDEMPIERE-5929 - Restores NPE handling for replacing tabs

* IDEMPIERE-5929 - Reverts changes on WindowContainer

This commit reverts three other commits:
4433f834f5a3892aec2c1486e30b846fbf957f8f
c6c6d2abf9a44d51f41d68a9e3e18ae04863435c
64916156eb43da481601889837df8571a5e1d596
This commit is contained in:
Marcos Gabriel de Oliveira Favaretto 2023-11-16 09:51:37 -03:00 committed by GitHub
parent af67186e51
commit bcbfcbfeb9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/desktop/TabbedDesktop.java
View File

@ -287,7 +287,12 @@ public abstract class TabbedDesktop extends AbstractDesktop {
int windowNo = (Integer)windowNoAttribute; int windowNo = (Integer)windowNoAttribute;
refTab = windowContainer.getTab(windowNo); refTab = windowContainer.getTab(windowNo);
} }
if (refTab == null)
windowContainer.addWindow(tabPanel, title, true, null);
else
windowContainer.replace(refTab, window, title); windowContainer.replace(refTab, window, title);
} }
else { else {
windowContainer.addWindow(tabPanel, title, true, null); windowContainer.addWindow(tabPanel, title, true, null);