Skip to content

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

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

Fix: Fixed issue when category suffix is configured as slash#136
jansentjeu merged 1 commit intobetafrom
fix/TW-107-slash-category-suffix

Conversation

@jansentjeu
Copy link
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
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

Successfully merging this pull request may close these issues.

2 participants