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

IDEMPIERE-4639 Cross tenant issues with context help (#1848)

This commit is contained in:
Carlos Ruiz 2023-05-20 03:53:49 +02:00 committed by GitHub
parent c05d2cefa9
commit 531aa73ea1
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 36 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
org.adempiere.base/src/org/compiere/model/PO.java
View File

@ -113,7 +113,7 @@ public abstract class PO
/** /**
* *
*/ */
private static final long serialVersionUID = 1672197562752270736L; private static final long serialVersionUID = -7758079724744033518L;
/* String key to create a new record based in UUID constructor */ /* String key to create a new record based in UUID constructor */
public static final String UUID_NEW_RECORD = ""; public static final String UUID_NEW_RECORD = "";
@ -2587,6 +2587,21 @@ public abstract class PO
} }
} }
/**
* Update Value or create new record, used when writing a cross tenant record
* @param trxName transaction
* @throws AdempiereException
* @see #saveEx(String)
*/
public void saveCrossTenantSafeEx() {
try {
PO.setCrossTenantSafe();
saveEx();
} finally {
PO.clearCrossTenantSafe();
}
}
/** /**
* Finish Save Process * Finish Save Process
* @param newRecord new * @param newRecord new
@ -2704,6 +2719,21 @@ public abstract class PO
saveEx(); saveEx();
} }
/**
* Update Value or create new record, used when writing a cross tenant record
* @param trxName transaction
* @throws AdempiereException
* @see #saveEx(String)
*/
public void saveCrossTenantSafeEx(String trxName) {
try {
PO.setCrossTenantSafe();
saveEx(trxName);
} finally {
PO.clearCrossTenantSafe();
}
}
/** /**
* Update Value or create new record. * Update Value or create new record.
* @param trxName transaction * @param trxName transaction

15
org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/window/WCtxHelpSuggestion.java
View File

@ -209,8 +209,6 @@ public class WCtxHelpSuggestion extends Window implements EventListener<Event> {
Object[] params = new Object[]{helpTextbox.getValue(), ctxHelpMsg.get_ID(), ctxHelpMsg.getAD_Client_ID(), Env.getAD_Language(Env.getCtx())}; Object[] params = new Object[]{helpTextbox.getValue(), ctxHelpMsg.get_ID(), ctxHelpMsg.getAD_Client_ID(), Env.getAD_Language(Env.getCtx())};
DB.executeUpdateEx(update.toString(), params, trx.getTrxName()); DB.executeUpdateEx(update.toString(), params, trx.getTrxName());
} else { } else {
try {
PO.setCrossTenantSafe();
/* this whole block code is forcefully writing records on System tenant */ /* this whole block code is forcefully writing records on System tenant */
MCtxHelpSuggestion suggestion = new MCtxHelpSuggestion(Env.getCtx(), 0, trx.getTrxName()); MCtxHelpSuggestion suggestion = new MCtxHelpSuggestion(Env.getCtx(), 0, trx.getTrxName());
suggestion.setClientOrg(0, 0); suggestion.setClientOrg(0, 0);
@ -221,7 +219,7 @@ public class WCtxHelpSuggestion extends Window implements EventListener<Event> {
MCtxHelp ctxHelp = new MCtxHelp(Env.getCtx(), 0, trx.getTrxName()); MCtxHelp ctxHelp = new MCtxHelp(Env.getCtx(), 0, trx.getTrxName());
setContextHelpInfo(po, ctxHelp); setContextHelpInfo(po, ctxHelp);
ctxHelp.setClientOrg(0, 0); ctxHelp.setClientOrg(0, 0);
ctxHelp.saveEx(); ctxHelp.saveCrossTenantSafeEx();
if (po != null) { if (po != null) {
if (po.is_Immutable()) { if (po.is_Immutable()) {
@ -229,11 +227,11 @@ public class WCtxHelpSuggestion extends Window implements EventListener<Event> {
MTable table = MTable.get(po.get_Table_ID()); MTable table = MTable.get(po.get_Table_ID());
PO mutablePO = table.getPO(po.get_ID(), trx.getTrxName()); PO mutablePO = table.getPO(po.get_ID(), trx.getTrxName());
mutablePO.set_ValueOfColumn("AD_CtxHelp_ID", ctxHelp.getAD_CtxHelp_ID()); mutablePO.set_ValueOfColumn("AD_CtxHelp_ID", ctxHelp.getAD_CtxHelp_ID());
mutablePO.saveEx(trx.getTrxName()); mutablePO.saveCrossTenantSafeEx(trx.getTrxName());
po.load(trx.getTrxName()); po.load(trx.getTrxName());
} else { } else {
po.set_ValueOfColumn("AD_CtxHelp_ID", ctxHelp.getAD_CtxHelp_ID()); po.set_ValueOfColumn("AD_CtxHelp_ID", ctxHelp.getAD_CtxHelp_ID());
po.saveEx(trx.getTrxName()); po.saveCrossTenantSafeEx(trx.getTrxName());
} }
} }
@ -242,7 +240,7 @@ public class WCtxHelpSuggestion extends Window implements EventListener<Event> {
msg.setAD_CtxHelp_ID(ctxHelp.getAD_CtxHelp_ID()); msg.setAD_CtxHelp_ID(ctxHelp.getAD_CtxHelp_ID());
msg.setClientOrg(0, 0); msg.setClientOrg(0, 0);
msg.setMsgText(baseContent); msg.setMsgText(baseContent);
msg.saveEx(); msg.saveCrossTenantSafeEx();
suggestion.setAD_CtxHelpMsg_ID(msg.getAD_CtxHelpMsg_ID()); suggestion.setAD_CtxHelpMsg_ID(msg.getAD_CtxHelpMsg_ID());
if (!Util.isEmpty(translatedContent) && !Env.isBaseLanguage(Env.getCtx(), I_AD_CtxHelpMsg.Table_Name)) { if (!Util.isEmpty(translatedContent) && !Env.isBaseLanguage(Env.getCtx(), I_AD_CtxHelpMsg.Table_Name)) {
int id = DB.getSQLValueEx(trx.getTrxName(), "SELECT AD_CtxHelpMsg_ID FROM AD_CtxHelpMsg_Trl WHERE AD_CtxHelpMsg_ID=? AND AD_Client_ID=? " + int id = DB.getSQLValueEx(trx.getTrxName(), "SELECT AD_CtxHelpMsg_ID FROM AD_CtxHelpMsg_Trl WHERE AD_CtxHelpMsg_ID=? AND AD_Client_ID=? " +
@ -280,10 +278,7 @@ public class WCtxHelpSuggestion extends Window implements EventListener<Event> {
suggestion.setMsgText(helpTextbox.getValue()); suggestion.setMsgText(helpTextbox.getValue());
suggestion.setIsSaveAsTenantCustomization(false); suggestion.setIsSaveAsTenantCustomization(false);
suggestion.saveEx(); suggestion.saveCrossTenantSafeEx();
} finally {
PO.clearCrossTenantSafe();
}
} }
this.detach(); this.detach();
} }