View Issue Details

IDProjectCategoryView StatusLast Update
0005839OXID eShop (all versions)1. ----- eShop frontend -----public2015-12-01 11:12
Reporterleofonic 
PrioritynormalSeverityminorReproducibilityalways
Status acknowledgedResolutionreopened 
Product Version4.8.6 / 5.1.6 
Target VersionFixed in Version 
Summary0005839: Category default sorting gets saved as user sorting
DescriptionCategory default sorting gets saved to session as user sorting and overrides other sortings.
Steps To Reproduce1. 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 InformationRelated to 0005241
TagsNo tags attached.
ThemeAzure
BrowserAll
PHP VersionNot defined
MySQL VersionNot defined

Relationships

related to 0005241 assignedflorian.auer Sorting is saved to session—which is not nice. 

Activities

cesnauskast

2014-07-30 10:35

administrator   ~0010048

Closing this one as it's pretty much about the same like reported in 0005241 and needs PO decision

leofonic

2014-07-30 11:07

reporter   ~0010050

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.

leofonic

2015-07-16 00:48

reporter   ~0011102

Last edited: 2015-07-16 00:52

View 2 revisions

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.

QA

2015-07-16 16:24

administrator   ~0011106

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).

leofonic

2015-07-16 16:42

reporter   ~0011107

This commit does not add a new widget, it just adds writing the current list sorting to session in detail view.