Skip to content

Fix: Fixed issue when category suffix is configured as slash - #136

Merged
jansentjeu merged 1 commit into
betafrom
fix/TW-107-slash-category-suffix
Jan 23, 2026
Merged

jansentjeu merged 1 commit into
betafrom
fix/TW-107-slash-category-suffix

Conversation

@jansentjeu

Copy link
Copy Markdown
Collaborator

The trailing slash was always trimmed from the URL rewrite, even when a slash was set as the category suffix. This is now checked for proper slash trimming. If the suffix is ​​set to a slash, it will not be trimmed from the URL rewrite.

Steps to reproduce the bug:

  1. Set the Stores -> Configuration -> Catalog -> Catalog -> Search Engine Optimization -> Category URL Suffix configuration to /
  2. Set the Stores -> Configuration -> Emico Extensions -> Attribute landing pages -> General -> Append Category url suffix to landingpage Urls configuration to Yes
  3. Create an attribute landing page
  4. Go in the admin to Marketing -> URL Rewrites and search for your attribute landing page URL rewrites.
  5. Check if the request path of the URL rewrites does NOT contain a trailing slash, even though you have set it to do so

Test scenario after bugfix:

  1. Make sure configurations are set correctly as described above
  2. Create an attribute landing page
  3. Go in the admin to Marketing -> URL Rewrites and search for your attribute landing page URL rewrites.
  4. Check if the request path of the URL rewrites does contain a trailing slash
  5. Change the Category URL Suffix to .html
  6. Check if trailing slash is changed to .html in the URL rewrites
  7. Change the Category URL Suffix back to /
  8. Check if .html is changed to / in the URL rewrites

@jansentjeu
jansentjeu merged commit d58436e into beta Jan 23, 2026
1 check passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 6.0.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants