Openbravo Issue Tracking System - Java Client POS
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0004603Java Client POS(No Category)public2008-08-11 08:342014-04-02 09:02
Reporterthelurker 
Assigned To 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version2.10 MP1 
Target Version2.30Fixed in Version2.30 beta 
Regression date
Regression introduced by commit
Regression level
Review Assigned To
OBNetwork customer
Support ticket
Regression introduced in release
Summary0004603: PaymentInfoRefundMagCard does not have enough context information
DescriptionSome payment gateways require additional context information to apply a credit (refund). This information includes original total, reference number or order number from the gateway.

Currently the system does not even supply the original transaction id to enable a lookup of this information.

So minimally to implement refunds for certain gateways the original openbravopos transaction id needs to be provided to enable the gateway implementation to lookup persistent data (in an additional table etc.). The edit Sales interface looks up the receipt but it appears that the authorization code from the PaymentGateway PaymentInfo class is not persisted.


Steps To ReproduceFunctionality is not present and the current API prevents an implementation.
Proposed SolutionI propose that the original transaction id (or better yet the original PaymentInfo instance) is passed through to the PaymentInfoRefundMagCard class so that at least some sort of persistent lookup can be implemented. If the original transaction's PaymentInfo is persisted as a serialized object and restored upon lookup then the UI can show exactly what the Printer (Screen) shows and also would be able to pass along the exact information that the PaymentGateway implemention would need to implement refund.
Additional Information
TagsNo tags attached.
Relationships
depends on feature request 00053582.30 closed adrianromero Payments gateways 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2008-08-11 08:34thelurkerNew Issue
2008-08-11 08:34thelurkersf_bug_id0 => 2046006
2008-08-11 18:05adrianromeroStatusnew => acknowledged
2008-10-07 10:42adrianromeroRelationship addeddepends on 0005358
2008-10-07 10:42adrianromeroStatusacknowledged => scheduled
2008-10-07 10:42adrianromeroTarget Version => 2.30
2008-12-10 11:07svnbotCheckin
2008-12-10 11:07svnbotNote Added: 0011130
2008-12-10 11:07svnbotStatusscheduled => resolved
2008-12-10 11:07svnbotResolutionopen => fixed
2008-12-10 11:07svnbotsvn_revision => 385
2009-03-02 15:42adrianromeroTarget Version2.30 => 2.30 beta
2009-03-02 15:45adrianromeroTarget Version2.30 beta => 2.30
2009-03-02 15:46adrianromeroFixed in Version => 2.30 beta
2012-11-07 09:40priyamCategory01 - General => (No Category)
2014-04-02 09:02plujanNote Added: 0065891
2014-04-02 09:02plujanStatusresolved => closed

Notes
(0011130)
svnbot   
2008-12-10 11:07   
Repository: openbravopos
Revision: 385
Author: mirurita
Date: 2008-12-10 11:07:48 +0100 (Wed, 10 Dec 2008)

Fixes issue 0005358: Payments gateways
Fixes issue 0005905: CYBERAUTHORIZE Gateway Integration
Fixes issue 0004603: PaymentInfoRefundMagCard does not have enough context information

---
U trunk/locales/pos_messages.properties
U trunk/src-pos/com/openbravo/pos/config/JPanelConfigPayment.form
U trunk/src-pos/com/openbravo/pos/config/JPanelConfigPayment.java
U trunk/src-pos/com/openbravo/pos/forms/AppConfig.java
U trunk/src-pos/com/openbravo/pos/forms/DataLogicSales.java
A trunk/src-pos/com/openbravo/pos/payment/ConfigPaymentPanelGeneric.form
A trunk/src-pos/com/openbravo/pos/payment/ConfigPaymentPanelGeneric.java
U trunk/src-pos/com/openbravo/pos/payment/JPaymentCashPos.java
U trunk/src-pos/com/openbravo/pos/payment/JPaymentCheque.java
U trunk/src-pos/com/openbravo/pos/payment/JPaymentDebt.java
U trunk/src-pos/com/openbravo/pos/payment/JPaymentFree.java
U trunk/src-pos/com/openbravo/pos/payment/JPaymentInterface.java
U trunk/src-pos/com/openbravo/pos/payment/JPaymentMagcard.java
U trunk/src-pos/com/openbravo/pos/payment/JPaymentPaper.java
U trunk/src-pos/com/openbravo/pos/payment/JPaymentRefund.java
U trunk/src-pos/com/openbravo/pos/payment/JPaymentSelect.java
U trunk/src-pos/com/openbravo/pos/payment/JPaymentSelectRefund.java
A trunk/src-pos/com/openbravo/pos/payment/PaymentConfiguration.java
U trunk/src-pos/com/openbravo/pos/payment/PaymentGatewayAuthorizeNet.java
A trunk/src-pos/com/openbravo/pos/payment/PaymentGatewayCyberauthorize.java
U trunk/src-pos/com/openbravo/pos/payment/PaymentGatewayExt.java
U trunk/src-pos/com/openbravo/pos/payment/PaymentGatewayFac.java
A trunk/src-pos/com/openbravo/pos/payment/PaymentGatewayPGNET.java
A trunk/src-pos/com/openbravo/pos/payment/PaymentGatewayPayPoint.java
U trunk/src-pos/com/openbravo/pos/payment/PaymentInfo.java
U trunk/src-pos/com/openbravo/pos/payment/PaymentInfoCash.java
U trunk/src-pos/com/openbravo/pos/payment/PaymentInfoFree.java
U trunk/src-pos/com/openbravo/pos/payment/PaymentInfoMagcard.java
U trunk/src-pos/com/openbravo/pos/payment/PaymentInfoMagcardRefund.java
U trunk/src-pos/com/openbravo/pos/payment/PaymentInfoTicket.java
U trunk/src-pos/com/openbravo/pos/sales/JPanelTicket.java
U trunk/src-pos/com/openbravo/pos/sales/JTicketsBagTicket.java
U trunk/src-pos/com/openbravo/pos/scripts/hsqldb-create.sql
U trunk/src-pos/com/openbravo/pos/scripts/hsqldb-upgrade-2.00.sql
U trunk/src-pos/com/openbravo/pos/scripts/hsqldb-upgrade-2.10.sql
U trunk/src-pos/com/openbravo/pos/scripts/hsqldb-upgrade-2.20.sql
U trunk/src-pos/com/openbravo/pos/scripts/mysql-create.sql
U trunk/src-pos/com/openbravo/pos/scripts/mysql-upgrade-2.00.sql
U trunk/src-pos/com/openbravo/pos/scripts/mysql-upgrade-2.10.sql
U trunk/src-pos/com/openbravo/pos/scripts/mysql-upgrade-2.20.sql
U trunk/src-pos/com/openbravo/pos/scripts/oracle-create.sql
U trunk/src-pos/com/openbravo/pos/scripts/oracle-upgrade-2.00.sql
U trunk/src-pos/com/openbravo/pos/scripts/oracle-upgrade-2.10.sql
U trunk/src-pos/com/openbravo/pos/scripts/oracle-upgrade-2.20.sql
U trunk/src-pos/com/openbravo/pos/scripts/postgresql-create.sql
U trunk/src-pos/com/openbravo/pos/scripts/postgresql-upgrade-2.00.sql
U trunk/src-pos/com/openbravo/pos/scripts/postgresql-upgrade-2.10.sql
U trunk/src-pos/com/openbravo/pos/scripts/postgresql-upgrade-2.20.sql
U trunk/src-pos/com/openbravo/pos/ticket/TicketInfo.java
---

https://dev.openbravo.com/websvn/openbravopos/?rev=385&sc=1 [^]
(0065891)
plujan   
2014-04-02 09:02   
Closed as it was in Resolved for too long