« Previous | Next » 

Revision bbf90a4a

IDbbf90a4adefdad23228ac4da7a1da3113a989723
Parent a30578bd
Child 036921d3

Added by Christian Müller about 7 years ago

[BUGFIX] Merge request arguments instead of overwrite

A sub-request losing its arguments if a route is defined f.e. like this:
-
name: 'Basket Steps'
uriPattern: '{node}/{--pipeu_factura_nodetypes-basket.@action}.{@format}'
defaults:
'@package': 'TYPO3.Neos'
'@controller': 'Frontend\Node'
'@format': 'html'
'@action': 'show'
'--pipeu_factura_nodetypes-basket':
'@package': 'PIPEU.Factura.NodeTypes'
'@controller': 'Basket'
routeParts:
node:
handler: TYPO3\Neos\Routing\FrontendNodeRoutePartHandler
appendExceedingArguments: true

In this case '--pipeu_factura_nodetypes-basket' will override
any pluginArgument because of a missing check for existing
index in pluginArguments. Its getting replaced instead of merged.

This wrong behavior is fixed with this patch by merging instead of
overwriting.

Same for internal arguments.

Was fixed in master by the related feature.

Related: I1d192231810757f38aecfc3f7dc520b118ee3feb
Change-Id: Ib067912327cb07cc6fff91d1fe5b3e5c6cd9ccdb
Resolves: #59404
Releases: master, 2.2, 2.1

  • added
  • modified
  • copied
  • renamed
  • deleted