Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0036160 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Discounts and Promotions | major | always | 2017-06-01 11:59 | 2017-06-26 10:52 | |||
Reporter | malsasua | View Status | public | |||||
Assigned To | ranjith_qualiantech_com | |||||||
Priority | immediate | Resolution | fixed | Fixed in Version | RR17Q3 | |||
Status | closed | Fix in branch | Fixed in SCM revision | a99eaea6ebd2 | ||||
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 | Production - Confirmed Stable | |||||||
Regression date | 2017-02-17 | |||||||
Regression introduced in release | RR17Q2 | |||||||
Regression introduced by commit | https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/eb6b4b84a3416e104ee4aa2f2704ecaa732cc084 [^] | |||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0036160: Discounts are not applied when a product is not included in the store assortment | |||||||
Description | when a discount is configured with "included product: All excluding defined", and there is some product in product tab, then the discount is not applied for any product | |||||||
Steps To Reproduce | in backend: . discount and promotion window: . discount: GPS_10_per . change "included product" field, from "Only those defined" to "All excluding defined" in product tab there are two products: gps handled and gps touch screen in POS: . add new product: inspect repelent expected behavior: discount GPS_10_per should be applied, but it is not applied. It is not applied for any discount | |||||||
Proposed Solution | in discount master query, this part should be removed: hql += "and ((p.includedProducts = 'Y' and not exists (select 1 "; hql += " from PricingAdjustmentProduct pap, OBRETCO_Prol_Product ppl "; hql += " where pap.active = true and pap.priceAdjustment = p "; hql += " and pap.product.id = ppl.product.id "; hql += " and ppl.obretcoProductlist.id ='" + productList.getId() + "')) "; | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||||||||||||||||
|
Notes | |
(0097320) hgbot (developer) 2017-06-12 16:13 |
Repository: erp/pmods/org.openbravo.retail.posterminal Changeset: a99eaea6ebd2ae64996cbbabdddf5e2d76131638 Author: Ranjith S R <ranjith <at> qualiantech.com> Date: Mon Jun 12 19:42:45 2017 +0530 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a99eaea6ebd2ae64996cbbabdddf5e2d76131638 [^] Fixed issue 36160 : Validate Discount Product if includeProduct is set as "all excluding defined" * For Discount Product, if includeProduct is set as "all excluding defined" then it should not be checked with assortment --- M src/org/openbravo/retail/posterminal/master/Discount.java --- |
(0097321) hgbot (developer) 2017-06-12 16:14 |
Repository: tools/automation/pi-mobile Changeset: f86a6cbf1ea6a207f4ee8016779f350aae159485 Author: Ranjith S R <ranjith <at> qualiantech.com> Date: Mon Jun 12 19:42:01 2017 +0530 URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/f86a6cbf1ea6a207f4ee8016779f350aae159485 [^] Verifies issue 36160 : Added automated test 'I36160_VerifyDiscountWithIncludeExcludeProducts' --- A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/discountsandpromotions/I36160_VerifyDiscountWithIncludeExcludeProducts.java --- |
(0097322) ranjith_qualiantech_com (developer) 2017-06-12 16:14 |
Reopened to create backports |
Issue History | |||
Date Modified | Username | Field | Change |
2017-06-01 11:59 | malsasua | New Issue | |
2017-06-01 11:59 | malsasua | Assigned To | => Retail |
2017-06-01 11:59 | malsasua | Resolution time | => 1498773600 |
2017-06-01 11:59 | malsasua | Regression level | => Production - Confirmed Stable |
2017-06-01 11:59 | malsasua | Triggers an Emergency Pack | => No |
2017-06-05 10:04 | malsasua | Relationship added | related to 0035254 |
2017-06-08 09:22 | ranjith_qualiantech_com | Assigned To | Retail => ranjith_qualiantech_com |
2017-06-08 09:22 | ranjith_qualiantech_com | Status | new => scheduled |
2017-06-12 08:04 | ranjith_qualiantech_com | Regression date | => 2017-02-17 |
2017-06-12 08:04 | ranjith_qualiantech_com | Regression introduced in release | => RR17Q2 |
2017-06-12 08:04 | ranjith_qualiantech_com | Regression introduced by commit | => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/eb6b4b84a3416e104ee4aa2f2704ecaa732cc084 [^] |
2017-06-12 16:13 | hgbot | Checkin | |
2017-06-12 16:13 | hgbot | Note Added: 0097320 | |
2017-06-12 16:13 | hgbot | Status | scheduled => resolved |
2017-06-12 16:13 | hgbot | Resolution | open => fixed |
2017-06-12 16:13 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a99eaea6ebd2ae64996cbbabdddf5e2d76131638 [^] |
2017-06-12 16:14 | hgbot | Checkin | |
2017-06-12 16:14 | hgbot | Note Added: 0097321 | |
2017-06-12 16:14 | ranjith_qualiantech_com | Note Added: 0097322 | |
2017-06-12 16:14 | ranjith_qualiantech_com | Status | resolved => new |
2017-06-12 16:14 | ranjith_qualiantech_com | Resolution | fixed => open |
2017-06-12 16:15 | ranjith_qualiantech_com | Status | new => scheduled |
2017-06-12 16:15 | ranjith_qualiantech_com | Status | scheduled => resolved |
2017-06-12 16:15 | ranjith_qualiantech_com | Resolution | open => fixed |
2017-06-19 18:29 | marvintm | Review Assigned To | => marvintm |
2017-06-19 18:29 | marvintm | Status | resolved => closed |
2017-06-19 18:29 | marvintm | Fixed in Version | => RR17Q3 |
2017-06-26 10:52 | dmitry_mezentsev | Relationship replaced | caused by 0035254 |
Copyright © 2000 - 2009 MantisBT Group |