-
-
Notifications
You must be signed in to change notification settings - Fork 351
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
routeEnhancers with overwriteDemand/* not working in 12 #2146
Comments
Hi Georg, I can confirm that bug too in TYPO3v11 / news v 11.1.2 when overwriting stickylist via DateMenu. Kind regards Andrea |
tested this with simple routeEnhancers:
PageTypeSuffix:
type: PageType
default: ''
map:
feed.xml: 9818
calendar.ical: 9819
News:
type: Extbase
extension: News
plugin: Pi1
routes:
- routePath: '/'
_controller: 'News::list'
- routePath: '/category/{category-name}'
_controller: 'News::list'
_arguments:
category-name: overwriteDemand/categories
- routePath: '/tag/{tag-name}'
_controller: 'News::list'
_arguments:
tag-name: overwriteDemand/tags
defaultController: 'News::list'
defaults:
page: '0'
aspects:
category-name:
type: PersistedAliasMapper
tableName: sys_category
routeFieldName: slug and looks good - imho the issue must be in core or ordering of routing :/ |
please try
istead of
|
this works now but I could swear to have tried that too ... probably with an older version ... |
i feel the same, maybe test with previous core version - fixed there by accident? ^^ |
Bug Report
I am not sure if it is a news or Typo3 bug. But since Typo 12 the mapping of the overwriteDemand parameters in combination with the NewsCategoryMapper or PersistedAliasMapper doesn't seem to work anymore.
Current Behavior
A call from
news/testcat
returns no results but a call with:
news/?tx_news_pi1[overwriteDemand][categories]=1&cHash=0e70fa1f6a2ce75e2db9713a4ddb5aeb
returns results.
The array overwriteDemand always arrives in the controller with null in the first case.
Environment
The text was updated successfully, but these errors were encountered: