diff --git a/org.adempiere.base/src/org/compiere/model/MPackage.java b/org.adempiere.base/src/org/compiere/model/MPackage.java index fb50e5a669..18076db9f7 100644 --- a/org.adempiere.base/src/org/compiere/model/MPackage.java +++ b/org.adempiere.base/src/org/compiere/model/MPackage.java @@ -319,7 +319,7 @@ public class MPackage extends X_M_Package if (line.getM_PackageMPS_ID() == 0) continue; - MPackageMPS packageMPS = new MPackageMPS(getCtx(), line.getM_PackageMPS_ID(), get_TrxName()); + MPackageMPS packageMPS = new MPackageMPS(getCtx(), line.getM_PackageMPS_ID(), null); packageMPS.setMasterTrackingNo(line.getMasterTrackingNo()); packageMPS.setTrackingNo(line.getTrackingNo()); packageMPS.setPrice(line.getPrice()); @@ -369,11 +369,13 @@ public class MPackage extends X_M_Package log.log(Level.SEVERE, "processOnline", e); setErrorMessage(Msg.getMsg(Env.getCtx(), "ShippingNotProcessed") + ": " + e.getMessage()); } - - if (trx != null) - { - trx.commit(); - trx.close(); + finally + { + if (trx != null) + { + trx.commit(); + trx.close(); + } } return ok; diff --git a/org.idempiere.webservices/WEB-INF/src/org/idempiere/adinterface/Process.java b/org.idempiere.webservices/WEB-INF/src/org/idempiere/adinterface/Process.java index edb8d133e5..e90ff8659d 100644 --- a/org.idempiere.webservices/WEB-INF/src/org/idempiere/adinterface/Process.java +++ b/org.idempiere.webservices/WEB-INF/src/org/idempiere/adinterface/Process.java @@ -342,6 +342,7 @@ public class Process { if (!processOK || pi.isError()) { r.setSummary(pi.getSummary()); + r.setError(pi.getSummary()); r.setLogInfo(pi.getLogInfo(true)); r.setIsError( true ); processOK = false; @@ -349,6 +350,7 @@ public class Process { else { r.setSummary(pi.getSummary()); + r.setError(pi.getSummary()); r.setLogInfo(pi.getLogInfo(true)); r.setIsError( false ); }