Openbravo Issue Tracking System - Retail Modules
View Issue Details
0047968Retail ModulesWeb POSpublic2021-11-03 14:552021-12-10 07:51
shuehner 
rqueralta 
normalminorhave not tried
closedfixed 
5
 
 
No
0047968: org.openbravo.retail.avalaraconnector: uses bad logging (System.out.println and printStrace
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
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();
No tags attached.
blocks design defect 0036162 acknowledged Triage Platform Base Openbravo ERP clean up openbravo.log 
Issue History
2021-11-03 14:55shuehnerNew Issue
2021-11-03 14:55shuehnerAssigned To => Retail
2021-11-03 14:55shuehnerTriggers an Emergency Pack => No
2021-11-03 14:56shuehnerRelationship addedblocks 0036162
2021-11-03 15:01shuehnerSummaryorg.openbravo.retail.avalaraconnector: uses bad logging (System.out.println in commented out code) => org.openbravo.retail.avalaraconnector: uses bad logging (System.out.println and printStrace
2021-11-03 15:01shuehnerDescription Updatedbug_revision_view_page.php?rev_id=23266#r23266
2021-11-03 15:01shuehnerSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=23268#r23268
2021-11-04 08:17marvintmStatusnew => scheduled
2021-11-04 08:17marvintmAssigned ToRetail => justin_patterson
2021-11-10 15:54marvintmAssigned Tojustin_patterson => Retail
2021-11-16 16:42rqueraltaAssigned ToRetail => rqueralta
2021-11-26 07:57marvintmStatusscheduled => acknowledged
2021-11-26 07:57marvintmStatusacknowledged => scheduled
2021-11-26 07:57marvintmStatusscheduled => feedback
2021-11-26 07:58marvintmAssigned Torqueralta => Retail
2021-11-26 07:58marvintmStatusfeedback => new
2021-12-01 15:58rqueraltaAssigned ToRetail => rqueralta
2021-12-10 04:18hgbotNote Added: 0133593
2021-12-10 07:51hgbotResolutionopen => fixed
2021-12-10 07:51hgbotStatusnew => closed
2021-12-10 07:51hgbotNote Added: 0133594
2021-12-10 07:51hgbotNote Added: 0133595

Notes
(0133593)
hgbot   
2021-12-10 04:18   
Merge Request created: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.avalaraconnector/-/merge_requests/1 [^]
(0133594)
hgbot   
2021-12-10 07:51   
Directly closing issue as related merge request is already approved.

Repository: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.avalaraconnector [^]
Changeset: e7bcd02fc193956323c14a1793a4ed675cebd570
Author: Rafael Queralta <rafaelcuba81@gmail.com>
Date: 2021-12-09T14:53:26-05:00
URL: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.avalaraconnector/-/commit/e7bcd02fc193956323c14a1793a4ed675cebd570 [^]

Fixed BUG-47968: Removed bad logging and replace printStrace by log4j.error

---
M src/org/openbravo/retail/avalaraconnector/edl/OBAVCODataProcessor.java
M src/org/openbravo/retail/avalaraconnector/hooks/PreOrderLoader.java
---
(0133595)
hgbot   
2021-12-10 07:51   
Merge request merged: https://gitlab.com/openbravo/product/pmods/org.openbravo.retail.avalaraconnector/-/merge_requests/1 [^]