Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0031650
TypeCategorySeverityReproducibilityDate SubmittedLast Update
feature request[Retail Modules] Web POSminorhave not tried2015-12-07 09:212015-12-29 14:09
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in VersionRR16Q1
StatusclosedFix in branchFixed in SCM revision0bea678f20cf
ProjectionnoneETAnoneTarget VersionRR16Q1
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Merge Request Status
Review Assigned ToOrekaria
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0031650: Change POSImportEntryProcessor to use Business Event Handler concept

DescriptionThe POSImportEntryProcessor class [1] uses the concept of a ImportEntryPreprocessor to set the pos terminal. But it is better to use an entity event listener as this covers all the cases when a new import entry is created through code.

[1]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/file/aadb5f0c8865/src/org/openbravo/retail/posterminal/importprocess/POSImportEntryProcessor.java [^]

[2]
http://wiki.openbravo.com/wiki/How_to_implement_a_business_event_handler [^]
Steps To ReproduceSee the code, change to use an entity event listener
Proposed SolutionNote, the code is touched/used by all the retail test cases. Any change here will be thoroughly tested through automated tests.
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]

-  Notes
(0082586)
hgbot (developer)
2015-12-07 11:22

 Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 0bea678f20cf910f8895ff807083f070751fdad7
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Dec 07 09:41:12 2015 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/0bea678f20cf910f8895ff807083f070751fdad7 [^]

Fixes issue 31650: Change POSImportEntryProcessor to use Business Event Handler concept
use business event handler instead of pre-processor as it covers more cases
when persisting new import entries

---
M src/org/openbravo/retail/posterminal/importprocess/POSImportEntryProcessor.java
---
(0082994)
mtaal (viewer)
2015-12-24 14:22

 Api change noted, approved and recorded here:
http://wiki.openbravo.com/wiki/Retail:API_changes#PI [^]

- Issue History
Date Modified Username Field Change
2015-12-07 09:21 mtaal New Issue
2015-12-07 09:21 mtaal Assigned To => mtaal
2015-12-07 09:21 mtaal OBNetwork customer => No
2015-12-07 09:21 mtaal Triggers an Emergency Pack => No
2015-12-07 11:22 mtaal Review Assigned To => marvintm
2015-12-07 11:22 hgbot Checkin
2015-12-07 11:22 hgbot Note Added: 0082586
2015-12-07 11:22 hgbot Status new => resolved
2015-12-07 11:22 hgbot Resolution open => fixed
2015-12-07 11:22 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/0bea678f20cf910f8895ff807083f070751fdad7 [^]
2015-12-24 14:22 mtaal Note Added: 0082994
2015-12-29 12:27 Orekaria Review Assigned To marvintm => Orekaria
2015-12-29 12:32 Orekaria Status resolved => new
2015-12-29 12:32 Orekaria Resolution fixed => open
2015-12-29 12:32 Orekaria Status new => scheduled
2015-12-29 14:09 Orekaria Status scheduled => resolved
2015-12-29 14:09 Orekaria Fixed in Version => RR16Q1
2015-12-29 14:09 Orekaria Resolution open => fixed
2015-12-29 14:09 Orekaria Status resolved => closed

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker