Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0032898 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Retail Modules] Return Receipt | major | always | 2016-05-06 23:02 | 2016-05-17 17:17 | |||
| Reporter | Osmany | View Status | public | |||||
| Assigned To | mario_castello | |||||||
| Priority | high | Resolution | fixed | Fixed in Version | RR16Q3 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | e5eebca33a4b | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | SCM revision | |||||||
| Merge Request Status | ||||||||
| Review Assigned To | guilleaer | |||||||
| OBNetwork customer | OBPS | |||||||
| Support ticket | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | https://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/d7e8be70a587 [^] | |||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0032898: [SERQA 1045] Verified Return not using original customer | |||||||
| Description | When 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 Reproduce | In 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 Solution | Attached 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. | |||||||
| Tags | SER-QA | |||||||
| Attached Files |  proposalsolution.diff [^] (816 bytes) 2016-05-06 23:02 [Show Content]
32898ReturnLines.diff [^] (1,129 bytes) 2016-05-09 10:23 [Show Content]
| |||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0086421) hgbot (developer) 2016-05-12 16:38 |
Repository: tools/automation/pi-mobile Changeset: f7c88b365d47527098534b5b6a96037d2133f707 Author: Mario Castello <mario.castello <at> peoplewalking.com> Date: Thu May 12 08:38:35 2016 -0600 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/f7c88b365d47527098534b5b6a96037d2133f707 [^] Related to issue 32898: Added automated test --- A src-test/org/openbravo/test/mobile/quarantine/pack/MCA/returns/I32898_VerifiedReturnsAndRemoteCustomer.java --- |
|
(0086430) hgbot (developer) 2016-05-12 18:55 edited on: 2016-05-12 19:05 |
Repository: erp/pmods/org.openbravo.retail.returns Changeset: e5eebca33a4b939ad3168a1c80a20ad203146c8a Author: Mario Castello <mario.castello <at> peoplewalking.com> Date: Tue May 10 11:53:36 2016 -0600 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/e5eebca33a4b939ad3168a1c80a20ad203146c8a [^] Fixed issue 32898: 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 --- |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2016-05-06 23:02 | Osmany | New Issue | |
| 2016-05-06 23:02 | Osmany | Assigned To | => Retail |
| 2016-05-06 23:03 | Osmany | File Added: proposalsolution.diff | |
| 2016-05-06 23:03 | Osmany | OBNetwork customer | => No |
| 2016-05-06 23:03 | Osmany | Triggers an Emergency Pack | => No |
| 2016-05-09 10:23 | adrianromero | File Added: 32898ReturnLines.diff | |
| 2016-05-09 10:24 | adrianromero | Proposed Solution updated | |
| 2016-05-09 10:25 | adrianromero | OBNetwork customer | No => Yes |
| 2016-05-09 10:25 | adrianromero | Resolution time | => 1463349600 |
| 2016-05-09 10:25 | adrianromero | Tag Attached: SER-QA | |
| 2016-05-09 17:06 | Osmany | Regression introduced by commit | => https://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/d7e8be70a587 [^] |
| 2016-05-10 10:30 | Orekaria | Resolution time | 1463349600 => 1463695200 |
| 2016-05-10 10:30 | Orekaria | Status | new => acknowledged |
| 2016-05-10 10:31 | Orekaria | Summary | Verified Return not changes the customer => Verified Return not using original customer |
| 2016-05-10 15:38 | mario_castello | Assigned To | Retail => mario_castello |
| 2016-05-10 15:38 | mario_castello | Status | acknowledged => scheduled |
| 2016-05-10 18:10 | adrianromero | Proposed Solution updated | |
| 2016-05-11 18:04 | amayavicente | Summary | Verified Return not using original customer => [SERQA 1045] Verified Return not using original customer |
| 2016-05-11 18:06 | Orekaria | Status | scheduled => feedback |
| 2016-05-11 18:06 | Orekaria | Status | feedback => scheduled |
| 2016-05-12 16:38 | hgbot | Checkin | |
| 2016-05-12 16:38 | hgbot | Note Added: 0086421 | |
| 2016-05-12 18:55 | hgbot | Checkin | |
| 2016-05-12 18:55 | hgbot | Note Added: 0086430 | |
| 2016-05-12 18:55 | hgbot | Status | scheduled => resolved |
| 2016-05-12 18:55 | hgbot | Resolution | open => fixed |
| 2016-05-12 18:55 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.returns/rev/e5eebca33a4b939ad3168a1c80a20ad203146c8a [^] |
| 2016-05-12 19:05 | mario_castello | Note Edited: 0086430 | View Revisions |
| 2016-05-17 17:17 | guilleaer | Review Assigned To | => guilleaer |
| 2016-05-17 17:17 | guilleaer | Status | resolved => closed |
| 2016-05-17 17:17 | guilleaer | Fixed in Version | => RR16Q3 |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |