Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

An equal (=) sign in the hash of the product page breaks the tabs functionality. #648

Closed
tzyganu opened this issue Sep 1, 2014 · 2 comments
Assignees

Comments

@tzyganu
Copy link
Contributor

tzyganu commented Sep 1, 2014

If I call a frontend product page and there is an equal (=) sign in the hash the tabs stop working.
I see the reviews tab content even is it's not the first tab and I cannot switch tabs.

How I got here:
I was trying to call a configurable product page and have a configuration already selected.
I used the way I was doing it for 1.x: http://mysite.com/category/product.phtml#120=22 where 120 is the configurable attribute id and 22 is the id of the option I want selected.
The options are selected as expected but I cannot navigate through the tabs (Description, Additional Information, Reviews).

After playing around with the url hash I found that any '=' in it breaks the tabs. Even if I call it simply as product.html#=.

The issue reproduces on FF and Chrome. (didn't test on other browsers but I think this is enough).

@verklov verklov self-assigned this Sep 4, 2014
@verklov
Copy link
Contributor

verklov commented Sep 4, 2014

@tzyganu, thank you for reporting this issue to us! We created the ticket in the internal bug tracker and will get back to you once the team resolves it.

magento-team added a commit that referenced this issue Nov 28, 2014
* Various improvements:
    * Refactored Service Layer of the Magento_Tax Module
    * Stabilized functional tests for the Backend module
    * Stabilized functional tests for the CatalogRule module
    * Stabilized functional tests for the Checkout module
    * Stabilized functional tests for the CurrencySymbol module
    * Stabilized functional tests for the Shipping module
    * Stabilized functional tests for the Tax module
    * Stabilized functional tests for the User module
* Added Readme.md files to the following modules:
    * Magento\RequireJs
    * Magento\Ui
* Fixed bugs:
    * Fixed an issue where product image assignment to a store view was not considered when displaying a product
    * Fixed shipping address area blinking when billing address is filled during checkout with a virtual product
    * Fixed an issue where filter_store.html was not found
    * Fixed an issue where the customer account access menu did not expand on the storefront
    * Fixed an issue where CMS blocks did not open when clicking from a grid
    * Fixed an issue where the Create Product page was completely blocked after closing the New Attribute pop-up
    * Fixed an issue where Stock Status was disabled for Bundle and Grouped products
    * Fixed an issue where a product could not be saved without filling a not required bundle option
    * Fixed broken "per page" selectors on the Customer's account pages
    * Fixed the wrong behavior of JS loaders on the storefront pages
    * Fixed Shopping cart price rule form validation
    * Fixed an issue where the 'Please wait' spinner persisted when creating a customer custom attribute with existing code
    * Fixed a Google Chrome specific issue where subcategories were not displayed correctly on the first hover for category item
    * Fixed an issue where the 'Please wait' spinner did not disappear when creating customer with invalid email
    * Fixed an issue where the Username field auto-focus on admin login page revealed password in case of fast typing
    * Fixed an issue where Bundle Product original Price was not displayed in case of discount
    * Fixed wrong discount calculation for bundle options
    * Fixed an issue where wrong discount and total amounts were displayed on the order creation page when reordering an order with a bundle product in the Admin panel
    * Fixed an issue where admin tax notifications did not appear/disappear unless cache was flushed or disabled
    * Fixed an issue where catalog price and shopping cart price did not match when display currency was different from the base currency
    * Fixed an issue where Tax classes did not allow 'None' as a valid 'product tax class'
    * Fixed an issue where token-based authentication did not work if compilation was enabled
    * Fixed the sample code in index.php illustrating multi websites set up
    * Fixed commands in Setup CLI to match the ones displayed in help
    * Fixed an issue where searching by a part of a product name in Advanced Search did not give correct results
    * Fixed an issue where 404 page is displayed after Search Term mass deletion
    * Fixed an issue where Popular Search Terms were not displayed on the storefront
    * Fixed an issue where it was impossible to add Gift Message during one page checkout
    * Fixed an issue where the optional Postal code setting did not work correctly
    * Fixed an issue where product price details were missing in summary block in the shopping cart when the Back to shopping cart link was clicked on multishipping page
    * Fixed an issue where the 404 error page was displayed instead of the Index Management page after saving mass update
    * Fixed an issue where the "Out of Stock" message was not displayed for a bundle product when there was not enough of one of the associated products in stock
    * Fixed an issue with the Newsletters Report page in the Admin panel
    * Fixed an issue where Catalog price rule was not applying correct rates on specific products
    * Fixed an issue where a fatal error was thrown after clicking a link to a downloadable product
    * Fixed an issue a warning page for Grouped product with enabled MAP
    * Fixed an issue where a configurable product was not displayed in catalog product grid after updating with "Add configurable attributes to the new set based on current"
    * Fixed the inconsistent behavior in the integration tests for the Indexer functionality
    * Fixed an issue where the What's this? information tip link was not presented on product page with configured Minimum Advertised Price (MAP)
* Processed GitHub requests:
    * [#742](#742) -- Admin notifications count overflow
    * [#720](#720) -- https filedriver is not working
    * [#686](#686) -- Product save validation errors in the admin don't hide the overlay
    * [#702](#702) -- Base table or view not found
    * [#652](#652) -- Multishipping checkout not to change the Billing address js issue
    * [#648](#648) -- An equal (=) sign in the hash of the product page to to break the tabs functionality
* Service Contracts:
    * Refactored usage of new API of the Customer module
    * Implemented Service Contracts for the Sales module
    * Refactored Service Contracts for the Catalog module
    * Refactored Service Contracts for the Grouped module
* UI Improvements:
    * Implemented the Form component in Magento UI Library
    * Removed extra JS loaders for category saving
    * Improved the behavior of Categories management in the Admin panel
    * Implemented the keyboard navigation through HTML elements
    * Improved the HTML structure and UI of the Catalog Category Link, Catalog Product Link and CMS Static Block widgets
    * Added UI Library documentation
    * Fixed Blank & Luma themes UI bugs
    * Fixed footer alignment
    * Published the Luma theme and removed the Plushe theme
* Framework Improvements:
    * Added the ability to configure the list of loaded modules before installation
    * Merged SQL and Data Upgrades
    * Moved \Magento\TestFramework\Utility\Files to Magento Framework
* Setup tool improvements:
    * Removed duplication with Framework
    * Deployment configuration is refactored from XML format in local.xml to associated array in config.php
    * Improved performance
* Search improvements:
    * Integrated the Full Text Search library into the Layered Navigation functionality
@verklov
Copy link
Contributor

verklov commented Dec 5, 2014

@tzyganu, we deployed the fix for this issue with version 0.1.0-alpha106. Thanks!

@verklov verklov closed this as completed Dec 5, 2014
okorshenko pushed a commit that referenced this issue Oct 21, 2015
SUPEE-7016: Newsletter subscribers filter for firstname/lastname dosn't work.
magento-engcom-team added a commit that referenced this issue May 2, 2019
…hodAdditional… #648

 - Merge Pull Request magento/graphql-ce#648 from magento/graphql-ce:graphQl-640-remove-braces-from-empty-types
 - Merged commits:
   1. 8a65e48
   2. 0959569
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants