View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006284 | OXID eShop (all versions) | 1.02. Price calculations (discounts, coupons, additional costs etc.) | public | 2015-12-10 15:17 | 2015-12-28 12:57 |
| Reporter | michael_keiluweit | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 4.9.6 / 5.2.6 | ||||
| Fixed in Version | 6.0.0-beta.1 | ||||
| Summary | 0006284: It's possible to generate coupons without a coupon number | ||||
| Description | During the generation of coupons it isn't checked if the field for the coupon number was filled. So it's possible to generate coupons without a coupon number. The basket prevents you from activating a coupon without a number. But if you disable javascript in your browser, then you can just click the button "submit coupon" and you will get the coupon which hasn't a number. So we have two situations here: Either the backend has to be improved that coupons must have at least one char or the basket must accept empty codes even if javascript is activated. | ||||
| Steps To Reproduce | backend 1. Shop settings -> Coupon Series -> Create new Coupon Serie 2. add a discount, give it a name, save it 3. now, activate the radio box for the field "Coupon Number" but let the input field blank 4. type a number for the field "Quantity" 5. save it frontend 1. put an article in the basket 2. goto basket 3. disable javascript 4. click the button "submit button" | ||||
| Additional Information | Please note: This will only work if there is at least one coupon serie which was generated without a coupon number. If the solution will be that you can't generate a coupon serie without a coupon number, then the basket must give the users with disabled javascript an error message that the field for the coupon code musst be filled. Currently only users with activated javascript will get an error message - users without javascript not. | ||||
| Tags | No tags attached. | ||||
| Theme | Azure | ||||
| Browser | Not defined | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
|
|
There is a pull-request related to this issue: https://github.com/OXID-eSales/oxideshop_ce/pull/301 |
|
|
https://github.com/OXID-eSales/oxideshop_ce/pull/301 |
