View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006083 | OXID eShop (all versions) | 1.01. Products (product, categories, manufacturer, promotions etc.) | public | 2015-03-11 17:52 | 2017-09-04 11:55 |
| Reporter | l.niewalda | Assigned To | |||
| Priority | high | Severity | major | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Product Version | 4.9.3 / 5.2.3 | ||||
| Fixed in Version | 4.10.0 / 5.3.0 | ||||
| Summary | 0006083: article sorting fields not verified against valid field from admin settings | ||||
| Description | using a listorderby field in alist which is not a aviable in database schema results in a mysql query error and redirects to a "Shop offline!". the invalid sorting row parameter is also saved in session which results in a "Shop offline!" message after trying to open any category page after that. no check against possible/valid sortfields entered in the "Database Fields Product Lists can be sorted after" in admin is performed. | ||||
| Steps To Reproduce | insert custom not aviable row in listorderby get parameter (here foobar): http://demoshop.oxid-esales.com/professional-edition/Kiteboarding/Kites/?ldtype=infogrid&_artperpage=10&listorderby=foobar&listorder=asc&pgNr=0&cl=alist&searchparam=&cnid=0f4fb00809cec9aa0910aa9c8fe36751 try to open any category after that: http://demoshop.oxid-esales.com/professional-edition/Kiteboarding/Kites/ | ||||
| Tags | No tags attached. | ||||
| Theme | Azure | ||||
| Browser | All | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
| parent of | 0006579 | resolved | saulius.stasiukaitis | OXID eShop (all versions) | Sorting by self-created fields in the oxarticles does not work |
| parent of | 0006445 | resolved | saulius.stasiukaitis | Flow theme | sorting by price doesn't work |
| related to | 0005241 | confirmed | SvenBrunk | OXID eShop (all versions) | Sorting is saved to session—which is not nice. |
|
|
https://github.com/OXID-eSales/oxideshop_ce/pull/394 |
|
|
Thank you, the pull request was just merged to master and ported to 5.3 branch. |
