Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0032936Retail ModulesReturn Receiptpublic2016-05-06 23:022016-06-07 14:08
ReporterOsmany 
Assigned Tomario_castello 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q2Fixed in VersionRR16Q2 
Merge Request Status
Review Assigned Toguilleaer
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/d7e8be70a587 [^]
Triggers an Emergency PackNo
Summary0032936: [SERQA 1045] Verified Return not using original customer
DescriptionWhen you select a Ticket of a client to do a verified return the new ticket is created with anonymous customer instead of original's one.

This issue happens only when is activated 'Enable Remote for Customer' Preference
Steps To ReproduceIn ERP Backend:
- Preference: Enable Remote for Customer - value Y

Log into WebPOS
- Click on verified returns
- Select a ticket with a client different of anonymus client
- Realize that the new ticket is created with anonymus ticket
Proposed SolutionAttached two different proposals, one in order.js and the other in modalReturnLines.js. Need to decide what is the better approach.

It seems the proposals attached does not solve the root of the problem and just the consequences. So it is needed to research what is the best solution.
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0032898 closed mario_castello [SERQA 1045] Verified Return not using original customer 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-05-11 18:06OrekariaTypedefect => backport
2016-05-11 18:06OrekariaTarget Version => RR16Q2
2016-06-02 23:35hgbotCheckin
2016-06-02 23:35hgbotNote Added: 0086966
2016-06-02 23:35hgbotStatusscheduled => resolved
2016-06-02 23:35hgbotResolutionopen => fixed
2016-06-02 23:35hgbotFixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR16Q2/org.openbravo.retail.returns/rev/eb869c3f509d191c702861c21c1b4d81c6621bb3 [^]
2016-06-02 23:39mario_castelloNote Edited: 0086966bug_revision_view_page.php?bugnote_id=0086966#r12309
2016-06-07 14:08guilleaerReview Assigned To => guilleaer
2016-06-07 14:08guilleaerStatusresolved => closed
2016-06-07 14:08guilleaerFixed in Version => RR16Q2

Notes
(0086966)
hgbot   
2016-06-02 23:35   
(edited on: 2016-06-02 23:39)
Repository: retail/backports/3.0RR16Q2/org.openbravo.retail.returns
Changeset: eb869c3f509d191c702861c21c1b4d81c6621bb3
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Thu Jun 02 15:32:52 2016 -0600
URL: http://code.openbravo.com/retail/backports/3.0RR16Q2/org.openbravo.retail.returns/rev/eb869c3f509d191c702861c21c1b4d81c6621bb3 [^]

Fixed issue 32936: Deleted unnecessary code in verified returns lines.

It is not necessary to execute removeTemporaly and saveTemporaly for customer in the verified returns process.
This funcionality has been implemented on setBPandBPLoc function in order.js.

---
M web/org.openbravo.retail.returns/js/modalReturnLines.js
---