Openbravo Issue Tracking System - Openbravo ERP | |||||||||||||||||||||||||||||||||||||
| View Issue Details | |||||||||||||||||||||||||||||||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||||||||||||||||||||||||||
| 0036183 | Openbravo ERP | A. Platform | public | 2017-06-06 12:18 | 2017-07-24 12:12 | ||||||||||||||||||||||||||||||||
| Reporter | adrianromero | ||||||||||||||||||||||||||||||||||||
| Assigned To | platform | ||||||||||||||||||||||||||||||||||||
| Priority | low | Severity | minor | Reproducibility | have not tried | ||||||||||||||||||||||||||||||||
| Status | closed | Resolution | no change required | ||||||||||||||||||||||||||||||||||
| Platform | OS | 5 | OS Version | ||||||||||||||||||||||||||||||||||
| Product Version | |||||||||||||||||||||||||||||||||||||
| Target Version | Fixed in Version | ||||||||||||||||||||||||||||||||||||
| Merge Request Status | |||||||||||||||||||||||||||||||||||||
| Review Assigned To | alostale | ||||||||||||||||||||||||||||||||||||
| OBNetwork customer | |||||||||||||||||||||||||||||||||||||
| Web browser | |||||||||||||||||||||||||||||||||||||
| Modules | Core | ||||||||||||||||||||||||||||||||||||
| Support ticket | |||||||||||||||||||||||||||||||||||||
| Regression level | |||||||||||||||||||||||||||||||||||||
| Regression date | |||||||||||||||||||||||||||||||||||||
| Regression introduced in release | |||||||||||||||||||||||||||||||||||||
| Regression introduced by commit | |||||||||||||||||||||||||||||||||||||
| Triggers an Emergency Pack | No | ||||||||||||||||||||||||||||||||||||
| Summary | 0036183: Platform: EntityPersistenceEventObserver validations has a high log level | ||||||||||||||||||||||||||||||||||||
| Description | Entity validations created extending EntityPersistenceEventObserver, when the validation fails an exception is thrown, and it is registered in the openbravo log file. These kind of exceptions are functional validations and should not be registered in the openbravo log with a high level | ||||||||||||||||||||||||||||||||||||
| Steps To Reproduce | For example to make fail the validation implemented in org.openbravo.retail.posterminal.event.ProductEventHandler, try to create a product with type "Service" and checked "Show stock". The validation appears and the record cannot be saved. Observe the openbravo log a exception is registered. | ||||||||||||||||||||||||||||||||||||
| Proposed Solution | The infrastructure does not log anything by default. The problem is some of these observers are throwing OBException, which by default log stack traces. Candidates to check: $ find -name *.java | xargs grep "extends.*EntityPersistenceEventObserver" -l | xargs grep OBException -l | sort == Functional == ./modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/event/FIN_FinaccTransactionEventListener.java ./modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/event/FIN_PaymentEventListener.java ./modules/org.openbravo.client.application/src/org/openbravo/client/application/event/AcctSchemaEventHandler.java ./modules/org.openbravo.client.application/src/org/openbravo/client/application/event/LocationSettingsHandler.java ./src/org/openbravo/event/ConversionRateEventHandler.java ./src/org/openbravo/event/DocTypeEventHandler.java ./src/org/openbravo/event/GLJournalLineEventHandler.java ./src/org/openbravo/event/InvoiceTaxEventHandler.java ./src/org/openbravo/event/MInOutLineEventHandler.java ./src/org/openbravo/event/OrderDiscountEventHandler.java ./src/org/openbravo/event/PeriodEventHandler.java ./src/org/openbravo/event/ProductAumEventHandler.java ./src/org/openbravo/event/ProductCharacteristicEventHandler.java ./src/org/openbravo/event/ProductEventHandler.java ./src/org/openbravo/event/ProductionLineEventHandler.java ./src/org/openbravo/event/ServiceOrderLineEventHandler.java ./src/org/openbravo/event/ServicePriceRuleRangeEventHandler.java ./src/org/openbravo/event/SimpleGLJournalEventHandler.java ./src/org/openbravo/event/UOMConversionEventHandler.java == Platform == ./modules/org.openbravo.client.application/src/org/openbravo/client/application/attachment/AttachmentConfigEventHandler.java ./modules/org.openbravo.client.application/src/org/openbravo/client/application/attachment/AttachmentMetadataEventHandler.java ./modules/org.openbravo.client.application/src/org/openbravo/client/application/event/FieldHandler.java ./modules/org.openbravo.client.application/src/org/openbravo/client/application/event/NoteEventHandler.java ./modules/org.openbravo.client.application/src/org/openbravo/client/application/event/RoleAccessUniqueHandler.java ./modules/org.openbravo.client.application/src/org/openbravo/client/application/event/TableEventHandler.java ./modules/org.openbravo.client.application/src/org/openbravo/client/application/event/TableTreeEventHandler.java ./modules/org.openbravo.client.application/src/org/openbravo/client/application/event/WindowPersonalizationEventHandler.java ./src/org/openbravo/event/ADTableEventHandler.java ./src/org/openbravo/event/ProcessRequestEventHandler.java == Retail == ./modules/org.openbravo.mobile.core/src/org/openbravo/mobile/core/servercontroller/MultiServerPreferenceEventHandler.java ./modules/org.openbravo.mobile.core/src/org/openbravo/mobile/core/servercontroller/ServerStateBackgroundEventHandler.java ./modules/org.openbravo.retail.discounts/src/org/openbravo/retail/discounts/event/MOfferEventHandler.java ./modules/org.openbravo.retail.discounts/src/org/openbravo/retail/discounts/event/MOfferProductEventHandler.java ./modules/org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/event/ADOrgWarehouseEventHandler.java ./modules/org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/event/AppPaymentEventHandler.java ./modules/org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/event/PaymentMethodEventHandler.java ./modules/org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/event/PaymentMethodTypeEventHandler.java ./modules/org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/event/ProductEventHandler.java ./modules/org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/event/TerminalTypePaymentMethodEventHandler.java ./modules/org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/importprocess/POSImportEntryProcessor.java ./modules/org.openbravo.retail.posterminal/src/org/openbravo/retail/posterminal/PaymentMethodHandler.java | ||||||||||||||||||||||||||||||||||||
| Additional Information | |||||||||||||||||||||||||||||||||||||
| Tags | SER-QA | ||||||||||||||||||||||||||||||||||||
| Relationships |
| ||||||||||||||||||||||||||||||||||||
| Attached Files | |||||||||||||||||||||||||||||||||||||
| Issue History | |||||||||||||||||||||||||||||||||||||
| Date Modified | Username | Field | Change | ||||||||||||||||||||||||||||||||||
| 2017-06-06 12:18 | adrianromero | New Issue | |||||||||||||||||||||||||||||||||||
| 2017-06-06 12:18 | adrianromero | Assigned To | => platform | ||||||||||||||||||||||||||||||||||
| 2017-06-06 12:18 | adrianromero | Modules | => Core | ||||||||||||||||||||||||||||||||||
| 2017-06-06 12:18 | adrianromero | Resolution time | => 1497909600 | ||||||||||||||||||||||||||||||||||
| 2017-06-06 12:18 | adrianromero | Triggers an Emergency Pack | => No | ||||||||||||||||||||||||||||||||||
| 2017-06-06 12:19 | adrianromero | Tag Attached: SER-QA | |||||||||||||||||||||||||||||||||||
| 2017-06-08 11:03 | alostale | Relationship added | blocks 0036162 | ||||||||||||||||||||||||||||||||||
| 2017-06-08 11:03 | alostale | Note Added: 0097204 | |||||||||||||||||||||||||||||||||||
| 2017-06-08 11:03 | alostale | Priority | normal => low | ||||||||||||||||||||||||||||||||||
| 2017-06-08 11:03 | alostale | Severity | major => minor | ||||||||||||||||||||||||||||||||||
| 2017-06-08 11:03 | alostale | Status | new => acknowledged | ||||||||||||||||||||||||||||||||||
| 2017-06-08 11:03 | alostale | Resolution time | 1497909600 => | ||||||||||||||||||||||||||||||||||
| 2017-06-08 13:35 | alostale | Note Added: 0097216 | |||||||||||||||||||||||||||||||||||
| 2017-06-08 14:27 | alostale | Proposed Solution updated | |||||||||||||||||||||||||||||||||||
| 2017-06-08 14:27 | alostale | Note Deleted: 0097216 | |||||||||||||||||||||||||||||||||||
| 2017-06-08 14:28 | alostale | Issue cloned | 0036207 | ||||||||||||||||||||||||||||||||||
| 2017-06-08 14:28 | alostale | Relationship added | related to 0036207 | ||||||||||||||||||||||||||||||||||
| 2017-06-08 14:30 | alostale | Issue cloned | 0036208 | ||||||||||||||||||||||||||||||||||
| 2017-06-08 14:30 | alostale | Relationship added | related to 0036208 | ||||||||||||||||||||||||||||||||||
| 2017-06-08 14:34 | alostale | Summary | EntityPersistenceEventObserver validations has a high log level => Platform: EntityPersistenceEventObserver validations has a high log level | ||||||||||||||||||||||||||||||||||
| 2017-06-08 14:34 | alostale | Proposed Solution updated | |||||||||||||||||||||||||||||||||||
| 2017-07-24 12:06 | alostale | Relationship added | related to 0036527 | ||||||||||||||||||||||||||||||||||
| 2017-07-24 12:11 | alostale | Status | acknowledged => scheduled | ||||||||||||||||||||||||||||||||||
| 2017-07-24 12:12 | alostale | Status | scheduled => feedback | ||||||||||||||||||||||||||||||||||
| 2017-07-24 12:12 | alostale | Review Assigned To | => alostale | ||||||||||||||||||||||||||||||||||
| 2017-07-24 12:12 | alostale | Note Added: 0098198 | |||||||||||||||||||||||||||||||||||
| 2017-07-24 12:12 | alostale | Status | feedback => closed | ||||||||||||||||||||||||||||||||||
| 2017-07-24 12:12 | alostale | Resolution | open => no change required | ||||||||||||||||||||||||||||||||||
| Notes | |||||
|
|
|||||
|
|
||||
|
|
|||||
|
|
||||