Bug #92517
closedNo alternative namespace supported by ExtbasePluginEnhancer
100%
Description
It is possible to configure an alternative pluginNamespace for an Extbase Plugin.
This can be done via TypoScript:
plugin { tx_extname_pluginname { view { pluginNamespace = search } } }
As this is not supported by the ExtbasePluginEnhancer, those can not be combined. It is not possible to provide routing configuration when using a custom configured pluginNamespace.
I would suggest to respect the "namespace" option, which is respected by PluginEnhancer, which is extended by ExtbasePluginEnhancer. This way integrator are able to configure an alternative namespace in both places.
Our use case: Map only some parts of routes, but keep others, e.g. search parameters. Those should not add unnecessary garbage to url like "tx_extname_pluginname" but "search" instead.
The option was initially introduced to access the arguments from another plugin (as far as I know).
Steps to reproduce:
Use any existing Extbase Plugin (which is using arguments) and insert it into a page. Configure an alternative pluginNamespace as shown above. The arguments should no longer be mapped. Add "namespace" with same value to the configuration of the Enhaner, it should work with the patch.
In case one follows this approach, one can omit the extension and plugin option of the Enhancer. Those are only used to generate the namespace.
Updated by Gerrit Code Review almost 4 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66083
Updated by Daniel Siepmann almost 4 years ago
Actually it is possible by omitting extension and or plugin, just like documented here: https://docs.typo3.org/m/typo3/reference-coreapi/master/en-us/ApiOverview/Routing/AdvancedRoutingConfiguration.html?highlight=extbase%20enhancer#extbase-plugin-enhancer
Updated by Daniel Siepmann almost 4 years ago
- Status changed from Under Review to Rejected
Updated by Gerrit Code Review almost 4 years ago
- Status changed from Rejected to Under Review
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66083
Updated by Gerrit Code Review almost 4 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66083
Updated by Gerrit Code Review over 3 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66083
Updated by Gerrit Code Review over 2 years ago
Patch set 5 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66083
Updated by Gerrit Code Review over 1 year ago
Patch set 6 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66083
Updated by Gerrit Code Review over 1 year ago
Patch set 7 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66083
Updated by Gerrit Code Review over 1 year ago
Patch set 8 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66083
Updated by Gerrit Code Review over 1 year ago
Patch set 9 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/66083
Updated by Daniel Siepmann over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 64752c95af4819996c52d1e3e34b7c10b60c02da.