-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Closed
Labels
Area: FrontendComponent: CatalogFixed in 2.4.xThe issue has been fixed in 2.4-develop branchThe issue has been fixed in 2.4-develop branchIssue: Clear DescriptionGate 2 Passed. Manual verification of the issue description passedGate 2 Passed. Manual verification of the issue description passedIssue: ConfirmedGate 3 Passed. Manual verification of the issue completed. Issue is confirmedGate 3 Passed. Manual verification of the issue completed. Issue is confirmedIssue: Format is validGate 1 Passed. Automatic verification of issue format passedGate 1 Passed. Automatic verification of issue format passedIssue: Ready for WorkGate 4. Acknowledged. Issue is added to backlog and ready for developmentGate 4. Acknowledged. Issue is added to backlog and ready for developmentProgress: PR CreatedIndicates that Pull Request has been created to fix issueIndicates that Pull Request has been created to fix issueReproduced on 2.4.xThe issue has been reproduced on latest 2.4-develop branchThe issue has been reproduced on latest 2.4-develop branch
Description
Preconditions (*)
- Magento 2.3.2-p2
- Single category with few products
Steps to reproduce (*)
- Go to Category page that has filters by price
- To the end of URL add
?price=-100%27[0]
, for instance https://example.com/my-category/?price=-100%27[0] - See logs
Expected result (*)
- We should not have error logs
- Layered navigation block should be shown
Actual result (*)
- We have following logs in system.log:
[2019-12-05 11:25:41] main.CRITICAL: Notice: A non well formed numeric value encountered in /var/www/magento2/vendor/magento/module-catalog-search/Model/Layer/Filter/Price.php on line 180 [] []
[2019-12-05 11:25:43] main.CRITICAL: The "catalog.leftnav" block threw an exception, and it can't be rendered. [] []
- Layered navigation block isn't shown
Note: this value isn't passed to DB, so no security issue there
Metadata
Metadata
Assignees
Labels
Area: FrontendComponent: CatalogFixed in 2.4.xThe issue has been fixed in 2.4-develop branchThe issue has been fixed in 2.4-develop branchIssue: Clear DescriptionGate 2 Passed. Manual verification of the issue description passedGate 2 Passed. Manual verification of the issue description passedIssue: ConfirmedGate 3 Passed. Manual verification of the issue completed. Issue is confirmedGate 3 Passed. Manual verification of the issue completed. Issue is confirmedIssue: Format is validGate 1 Passed. Automatic verification of issue format passedGate 1 Passed. Automatic verification of issue format passedIssue: Ready for WorkGate 4. Acknowledged. Issue is added to backlog and ready for developmentGate 4. Acknowledged. Issue is added to backlog and ready for developmentProgress: PR CreatedIndicates that Pull Request has been created to fix issueIndicates that Pull Request has been created to fix issueReproduced on 2.4.xThe issue has been reproduced on latest 2.4-develop branchThe issue has been reproduced on latest 2.4-develop branch