-
-
Notifications
You must be signed in to change notification settings - Fork 241
Closed
Description
Routing in plugins like described here neos/Neos.Demo#190 (comment) is currently broken.
The EventSourcedFrontendNodeRoutePartHandler expects a NodeAdress instance and not a serialized string:
Line 265 in a2dd063
| if (!$nodeAddress instanceof NodeAddress) { |
Fix:
if (!$nodeAddress instanceof NodeAddress) {
$nodeAddress = NodeAddressFactory::create($this->contentRepositoryRegistry->get($currentRequestSiteDetectionResult->contentRepositoryId))
->createFromUriString($nodeAddress);
}But the real question is why was it a string in the first place? I fear because of this weird thing?
neos-development-collection/Neos.Neos/Classes/Routing/NodeIdentityConverterAspect.php
Line 34 in 5f80a56
| class NodeIdentityConverterAspect |
With this fix the Neos.Neos:Plugin works as described here neos/Neos.Demo#190
Things i tested in the controller (we should ad an behat test for each):
- forward (content of other action)
- throwStatus (500 will be upmerged and plugin will show text inside:
500 Internal Server Error - redirect (uri will change)
- set arbitrary header
x-mhs: foobar - build uri to another action (navigation)
- render fluid view
- render string
Reactions are currently unavailable
Metadata
Metadata
Assignees
Type
Projects
Status
Done ✅