Project:
View Revisions: Issue #47968 | [ Back to Issue ] | ||
Summary | 0047968: org.openbravo.retail.avalaraconnector: uses bad logging (System.out.println and printStrace | ||
Revision | 2021-11-03 15:01 by shuehner | ||
Steps To Reproduce | https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.avalaraconnector/-/blob/master/src/org/openbravo/retail/avalaraconnector/hooks/PreOrderLoader.java#L40 [^] pmods/org.openbravo.retail.avalaraconnector/src/org/openbravo/retail/avalaraconnector/edl/OBAVCODataProcessor.java: e.printStackTrace(); pmods/org.openbravo.retail.avalaraconnector/src/org/openbravo/retail/avalaraconnector/edl/OBAVCODataProcessor.java: e.printStackTrace(); |
||
Revision | 2021-11-03 15:01 by shuehner | ||
Description | The PreOrderLoader has apparently some leftover developer debugging code https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.avalaraconnector/-/blob/master/src/org/openbravo/retail/avalaraconnector/hooks/PreOrderLoader.java#L40 [^] That should either be deleted to not have bad code example or alternatively moved to be log4j.debug also is uses printStrace in 2 catch blocks to be moved to log4j.error |
||
Revision | 2021-11-03 14:56 by shuehner | ||
Steps To Reproduce | https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.avalaraconnector/-/blob/master/src/org/openbravo/retail/avalaraconnector/hooks/PreOrderLoader.java#L40 [^] | ||
Revision | 2021-11-03 14:56 by shuehner | ||
Description | The PreOrderLoader has apparently some leftover developer debugging code https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.avalaraconnector/-/blob/master/src/org/openbravo/retail/avalaraconnector/hooks/PreOrderLoader.java#L40 [^] That should either be deleted to not have bad code example or alternatively moved to be log4j.debug |
Copyright © 2000 - 2009 MantisBT Group |