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

Redirection error if the url contains parameters #811

Closed
ingoKrOffice opened this issue May 8, 2024 · 1 comment · Fixed by #812
Closed

Redirection error if the url contains parameters #811

ingoKrOffice opened this issue May 8, 2024 · 1 comment · Fixed by #812
Labels
bug Indicates an unexpected problem or unintended behaviour
Milestone

Comments

@ingoKrOffice
Copy link
Contributor

ingoKrOffice commented May 8, 2024

In v4.16 Detail Page > Needed Filter Hook or Option to stopping redirection we implemented a feature that is causing a bug for our customers.

Currently the "oo_is_detailpage_redirection" filter is being overridden in the plugin, rendering it useless. Additionally, the plugin performs a redirect that removes parameters from URL, even when redirecting from the regular ID-URL to the "speaking" Title-URL (www.website.de/detail/123/ --> www.website.de/detail/123-house-in-aachen/)

The expected behavior is that the filter can be properly set to true or false and that the parameters in the URL remain intact even when the plugin performs a redirect. These changes are intended to restore the desired behavior of the plugin and improve the user experience.

Please revise the feature for any other bugs and inconsistencies.

@ingoKrOffice ingoKrOffice linked a pull request May 8, 2024 that will close this issue
@ingoKrOffice ingoKrOffice added the bug Indicates an unexpected problem or unintended behaviour label May 8, 2024
@fredericalpers fredericalpers added this to the v4.24 milestone May 8, 2024
@fredericalpers
Copy link
Contributor

also see the linked Pull Request

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behaviour
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants