Skip to content

Conversation

@kdambekalns
Copy link
Member

The FrontendNodeRoutePartHandler lacked quite some changes from the implementation in Neos, and thus

  • did not resolve shortcuts directly to their target
  • did not know about the nodeType options

Also the exception codes have been "uniqueified".

The FrontendNodeRoutePartHandler lacked quite some changes from the
implementation in Neos, and thus

- did not resolve shortcuts directly to their target
- did not know about the `nodeType` options

Also the exception codes have been "uniqueified".
@kdambekalns kdambekalns self-assigned this Jun 30, 2024
Inspired by PR Flowpack#11 opened by @c4ll-m3-j4ck this adds the missing piece
of the puzzle to support the `uriPathSuffix`.
Copy link
Member

@kitsunet kitsunet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a bunch of Exception use statements as Exception\SomethingException wouldn't resolve in here.

@kdambekalns
Copy link
Member Author

Added a bunch of Exception use statements as Exception\SomethingException wouldn't resolve in here.

There is use Neos\Neos\Routing\Exception; at the top… so it worked. With your change, I do get warnings in PhpStorm now about NoSiteNodeException not existing. 😎

@kitsunet
Copy link
Member

kitsunet commented Jul 3, 2024

but Neos\Neos\Routing\Exception is a class itself? And \Neos\Neos\Routing\Exception\NoSiteNodeException exists for me?

@kitsunet kitsunet merged commit e3de5c8 into Flowpack:master Jul 3, 2024
@kdambekalns
Copy link
Member Author

but Neos\Neos\Routing\Exception is a class itself? And \Neos\Neos\Routing\Exception\NoSiteNodeException exists for me?

Yes, with use Neos\Neos\Routing\Exception in your code you can use

  • Exception to refer to Neos\Neos\Routing\Exception
  • Exception\NoSiteNodeException to refer to Neos\Neos\Routing\Exception\NoSiteNodeException

That's why it worked.

But you changed Exception\NoSiteNodeException to NoSiteNodeException, and forgot to add use Neos\Neos\Routing\Exception\NoSiteNodeException that is then needed…

@kdambekalns kdambekalns deleted the task/update-routepart-handler branch July 3, 2024 07:02
@kitsunet
Copy link
Member

kitsunet commented Jul 3, 2024

Ja! I was not exactly "zurechnungsfähig" anymore ^^ Although I always prefer separate use statements.

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