View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005839 | OXID eShop (all versions) | 1. ----- eShop frontend ----- | public | 2014-07-29 13:19 | 2023-12-06 11:34 |
| Reporter | leofonic | Assigned To | |||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | confirmed | Resolution | reopened | ||
| Product Version | 4.8.6 / 5.1.6 | ||||
| Summary | 0005839: Category default sorting gets saved as user sorting | ||||
| Description | Category default sorting gets saved to session as user sorting and overrides other sortings. | ||||
| Steps To Reproduce | 1. In Backend, set default sorting (Fast Sorting) of "Kites" to price asc and also set some different default sortings for other categories. 2. Switch between categories to verify all sortings are as set in admin 3. Go to "Kites" and switch listview to "grid" to start session 4. Click an article for detail view and then click "back to overview" 5. All categories, search and manufacturer pages are now sorted price asc | ||||
| Additional Information | Related to 0005241 | ||||
| Tags | No tags attached. | ||||
| Theme | Azure | ||||
| Browser | All | ||||
| PHP Version | Not defined | ||||
| Database Version | Not defined | ||||
|
|
Closing this one as it's pretty much about the same like reported in 0005241 and needs PO decision |
|
|
This is not the same as 0005241. It is one thing to save a setting to the session that the user has made deliberately, and another thing to save this setting without any user intervention. While agree with 0005241 that it is not good to save this setting without the possibility to undo, this bug report is about the sorting being saved to session without selecting any kind of sorting. This renders the option to set category fast sorting for each category quite useless. So 0005241 is more a feature request, while this report is a bug. You can even remove the option to select sorting in templates and sorting will still get saved to session. |
|
|
This bug is caused by this commit: https://github.com/OXID-eSales/oxideshop_ce/commit/024dc71922aa9205d61a424fc93d07cc8298cc0a What was the purpose of this commit, i can't see the benefit of the added code? And please change "has duplicate" as this is not a duplicate. |
|
|
Cause of said commit: https://www.oxid-esales.com/de/support-services/dokumentation-und-hilfe/oxid-eshop/releases/releases-2013/oxid-eshop-480510.html (Abschnitt Neue Widgets) (unfortunately, only german release notes are currently available). |
|
|
This commit does not add a new widget, it just adds writing the current list sorting to session in detail view. |
|
|
I guess we currently cannot tell apart the intentionally and non-intentionally made changes to the view parameters (or if there were any changes at all). We need to think about a better solution. |
