This fixes a navigation loop caused by an unusual flow where properties page (configuration properties) is opened from another properties page (folder properties). The Save & Cancel buttons use a backHref property which depends on the HTTP header's referer.
The issue occurs when the user returns to the folder properties page from the configuration properties page because the backHref of the Save & Close buttons is set to the referer (configuration page).
The suggested fix is to overlay the commerce/gui/content/products/folderproperties with backHref set to the product console page (remove dependency on header.Referer) (. Using an overlay instead of directly changing the backHref formula in commerce module preserves original functionality for modules relying directly on /libs/commerce/gui/content/products/folderproperties.
How Has This Been Tested?
manual testing
Types of changes
[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Description
This fixes a navigation loop caused by an unusual flow where properties page (configuration properties) is opened from another properties page (folder properties). The Save & Cancel buttons use a
backHrefproperty which depends on the HTTP header's referer.${empty header.Referer ? granite:concat("/aem/products.html", granite:encodeURIPath(granite:relativeParent(empty param.item ? requestPathInfo.suffix : param.item, 1))) : header.Referer}The issue occurs when the user returns to the folder properties page from the configuration properties page because the
backHrefof the Save & Close buttons is set to the referer (configuration page).The suggested fix is to overlay the
commerce/gui/content/products/folderpropertieswithbackHrefset to the product console page (remove dependency onheader.Referer) (. Using an overlay instead of directly changing the backHref formula in commerce module preserves original functionality for modules relying directly on/libs/commerce/gui/content/products/folderproperties.How Has This Been Tested?
Types of changes
Checklist: