Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0030176 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Discounts and Promotions | trivial | always | 2015-06-15 10:28 | 2015-09-04 15:03 | |||
Reporter | asiermartirena | View Status | public | |||||
Assigned To | jorge-garcia | |||||||
Priority | immediate | Resolution | fixed | Fixed in Version | RR15Q4 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 3037b3f02cf7 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Review Assigned To | marvintm | |||||||
Regression level | Code review | |||||||
Regression date | 2015-05-05 | |||||||
Regression introduced in release | main | |||||||
Regression introduced by commit | https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a1c1c77a3384b2225f92d600ae03073ada0c5c0d [^] | |||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0030176: Problem with identifier attribute created to show discounts | |||||||
Description | When adding a promotion to a product list, an identifier attribute is generated to show a customized name for that promotion. Then, when rendering the line that identifier value is taken and assigned to show in the discount line. This identifier is not stored in any column in database, so when someone tries to generate again the ticket (for example when creating a return ticket), the line renderization tries to access to the not stored identifier value. This casuses that 'undefined' value is shown for the recovered discount line. | |||||||
Steps To Reproduce | Create a ticket and add a discount to a product. Pay the ticket. Return the ticket and select the product with the discount. In the order created the discount will be shown with the name 'undefined'. | |||||||
Proposed Solution | Create a new column to store the identifier value, and only fill it if the identifier is needed. Then, when creating the discount component don't assign the identifier directly. Check if the identifier is filled, and else assign the name value. | |||||||
Tags | No tags attached. | |||||||
Attached Files | 30176_posterminal.diff [^] (2,795 bytes) 2015-07-01 13:10 [Show Content]
30176_discounts.diff [^] (5,879 bytes) 2015-07-01 13:10 [Show Content] | |||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | |||||||||||||||
|
Notes | |
(0079358) hgbot (developer) 2015-08-14 13:26 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: aa0b535fc3e586c30aee23dbe8665b97736c6c43 Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Tue Aug 11 09:59:45 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/aa0b535fc3e586c30aee23dbe8665b97736c6c43 [^] Fixes issue 30176, port of issue 30298: Changed needed in posterminal module order.js file must be changed too. --- M web/org.openbravo.retail.posterminal/js/model/order.js --- |
(0079360) hgbot (developer) 2015-08-14 13:26 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: acfc788c2c5ce9401c9fe9a1d1a4e5a30d9082aa Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Fri Aug 07 14:10:39 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/acfc788c2c5ce9401c9fe9a1d1a4e5a30d9082aa [^] Fixes issue 30176, port of issue 30298: Changed needed in posterminal module Some changes are required for the implementation of the discounts module changes. --- M src/org/openbravo/retail/posterminal/OrderLoader.java M src/org/openbravo/retail/posterminal/PaidReceipts.java M web/org.openbravo.retail.posterminal/js/components/renderorderline.js --- |
(0079363) hgbot (developer) 2015-08-14 14:26 |
Repository: erp/pmods/org.openbravo.retail.discounts Changeset: 3037b3f02cf7bca0c99b8a77974782465772108f Author: Jorge Garcia <jorge.garcia <at> openbravo.com> Date: Mon Aug 03 09:46:20 2015 +0200 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/3037b3f02cf7bca0c99b8a77974782465772108f [^] Fixes issue 30176, port of issue 30298: Problem with identifier attribute created to show discounts The problem was that the identifier of the issue is not saved in the database. The solution is to create a new column that stores the identifier, and retrieve the discount identifier from the database. --- M src-db/database/model/modifiedTables/C_ORDERLINE_OFFER.xml M src-db/database/sourcedata/AD_COLUMN.xml M src-db/database/sourcedata/AD_ELEMENT.xml --- |
(0080126) hgbot (developer) 2015-09-04 15:03 |
Repository: tools/automation/pi-mobile Changeset: f265fb38240e589f36858e0ea22a5ed334b7bef4 Author: Rafa Alonso <ral <at> openbravo.com> Date: Fri Sep 04 15:02:58 2015 +0200 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/f265fb38240e589f36858e0ea22a5ed334b7bef4 [^] Related to issue 30176: Added the 'I30176_UndefinedDiscountWhenReturning' test --- M src-test/org/openbravo/test/mobile/common/selenium/javascript/TestId.java A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/discountsandpromotions/I30176_UndefinedDiscountWhenReturning.java --- |
Issue History | |||
Date Modified | Username | Field | Change |
2015-06-15 10:28 | asiermartirena | New Issue | |
2015-06-15 10:28 | asiermartirena | Assigned To | => Retail |
2015-06-15 10:28 | asiermartirena | Regression level | => Code review |
2015-06-15 10:28 | asiermartirena | Regression date | => 2015-05-05 |
2015-06-15 10:28 | asiermartirena | Regression introduced in release | => main |
2015-06-15 10:28 | asiermartirena | Regression introduced by commit | => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a1c1c77a3384b2225f92d600ae03073ada0c5c0d [^] |
2015-06-15 10:28 | asiermartirena | Triggers an Emergency Pack | => No |
2015-06-29 16:21 | asiermartirena | File Added: 30176_discounts.diff | |
2015-06-29 16:21 | asiermartirena | File Added: 30176_posterminal.diff | |
2015-07-01 13:09 | asiermartirena | File Deleted: 30176_discounts.diff | |
2015-07-01 13:10 | asiermartirena | File Deleted: 30176_posterminal.diff | |
2015-07-01 13:10 | asiermartirena | File Added: 30176_posterminal.diff | |
2015-07-01 13:10 | asiermartirena | File Added: 30176_discounts.diff | |
2015-07-02 10:03 | marvintm | Status | new => scheduled |
2015-07-17 09:08 | lorenzofidalgo | Relationship added | related to 0030385 |
2015-08-14 13:26 | hgbot | Checkin | |
2015-08-14 13:26 | hgbot | Note Added: 0079358 | |
2015-08-14 13:26 | hgbot | Status | scheduled => resolved |
2015-08-14 13:26 | hgbot | Resolution | open => fixed |
2015-08-14 13:26 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/aa0b535fc3e586c30aee23dbe8665b97736c6c43 [^] |
2015-08-14 13:26 | hgbot | Checkin | |
2015-08-14 13:26 | hgbot | Note Added: 0079360 | |
2015-08-14 13:26 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/aa0b535fc3e586c30aee23dbe8665b97736c6c43 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/acfc788c2c5ce9401c9fe9a1d1a4e5a30d9082aa [^] |
2015-08-14 14:26 | hgbot | Checkin | |
2015-08-14 14:26 | hgbot | Note Added: 0079363 | |
2015-08-14 14:26 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/acfc788c2c5ce9401c9fe9a1d1a4e5a30d9082aa [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.discounts/rev/3037b3f02cf7bca0c99b8a77974782465772108f [^] |
2015-08-31 09:19 | marvintm | Assigned To | Retail => jorge-garcia |
2015-08-31 09:19 | marvintm | Review Assigned To | => marvintm |
2015-08-31 09:19 | marvintm | Status | resolved => closed |
2015-08-31 09:19 | marvintm | Fixed in Version | => RR15Q4 |
2015-09-04 15:03 | hgbot | Checkin | |
2015-09-04 15:03 | hgbot | Note Added: 0080126 |
Copyright © 2000 - 2009 MantisBT Group |