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

IDEMPIERE-2771 Improve 2pack tracing for errors / AP2-694 Packin sometimes notifies Success when is a Failure

This commit is contained in:
Carlos Ruiz 2018-10-13 10:33:53 +02:00
parent f70f76d732
commit 069bc5e5ba
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
org.adempiere.pipo/src/org/adempiere/pipo/srv/PipoDictionaryService.java
View File

@ -111,7 +111,8 @@ public class PipoDictionaryService implements IDictionaryService {
if (logger.isLoggable(Level.INFO)) logger.info("commit " + trxName); if (logger.isLoggable(Level.INFO)) logger.info("commit " + trxName);
} catch (Exception e) { } catch (Exception e) {
adPackageImp.setP_Msg(e.getLocalizedMessage()); adPackageImp.setP_Msg(e.getLocalizedMessage());
packIn.getNotifier().addStatusLine(e.getLocalizedMessage()); packIn.getNotifier().addFailureLine(e.getLocalizedMessage());
packIn.setSuccess(false);
logger.log(Level.SEVERE, "importXML:", e); logger.log(Level.SEVERE, "importXML:", e);
throw e; throw e;
} finally { } finally {

3
org.adempiere.pipo/src/org/adempiere/pipo2/PackInProcess.java
View File

@ -147,7 +147,8 @@ public class PackInProcess extends SvrProcess {
commitEx(); // we need to commit to capture HERE when the deferred validation of foreign keys can fail commitEx(); // we need to commit to capture HERE when the deferred validation of foreign keys can fail
} catch (Exception e) { } catch (Exception e) {
adPackageImp.setP_Msg(e.getLocalizedMessage()); adPackageImp.setP_Msg(e.getLocalizedMessage());
packIn.getNotifier().addStatusLine(e.getLocalizedMessage()); packIn.getNotifier().addFailureLine(e.getLocalizedMessage());
packIn.setSuccess(false);
log.log(Level.SEVERE, "importXML:", e); log.log(Level.SEVERE, "importXML:", e);
throw e; throw e;
} finally { } finally {