Skip to content

Commit

Permalink
TASK: Remove noHomepageException rendering group
Browse files Browse the repository at this point in the history
After the previous adjustments the `SiteDetectionFailedException` will only be thrown in fewer cases, the only real reproducible the `/neos/content` route.
The rendering group is a bit overkill only for this thing and also might swallow 500 errors that should be presented as is and not wrapped by a vibrant welcome page.

The ui should be adjusted instead. neos/neos-ui#3721
  • Loading branch information
mhsdesign committed Feb 16, 2024
1 parent ad29e57 commit cf4190e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 16 deletions.
7 changes: 1 addition & 6 deletions Configuration/Settings.yaml
Expand Up @@ -471,18 +471,13 @@ Neos:
- Neos\Flow\Persistence\Doctrine\Exception\DatabaseException
- Neos\Flow\Persistence\Doctrine\Exception\DatabaseConnectionException
- Neos\Flow\Persistence\Doctrine\Exception\DatabaseStructureException
options: &welcomeTemplate
options:
viewClassName: Neos\Fusion\View\FusionView
viewOptions:
fusionPath: 'Neos/Fusion/DatabaseConnectionExceptions'
fusionPathPatterns: ['resource://Neos.Neos/Private/Fusion/Error/Root.fusion']
enableContentCache: false
templatePathAndFilename: ~

noHomepageException:
matchingExceptionClassNames:
- Neos\Neos\FrontendRouting\SiteDetection\SiteDetectionFailedException
options: *welcomeTemplate
debugger:
ignoredClasses:
Neos\\Neos\\Domain\\Service\\ContentContextFactory: true
Expand Down
10 changes: 0 additions & 10 deletions Resources/Private/Translations/en/Main.xlf
Expand Up @@ -669,16 +669,6 @@
<source>Technical Information</source>
</trans-unit>

<trans-unit id="error.exception.noHomepageException.title" xml:space="preserve">
<source>Missing Homepage</source>
</trans-unit>
<trans-unit id="error.exception.noHomepageException.description" xml:space="preserve">
<source>Either no site has been defined, the site does not contain a homepage or the active site couldn't be determined.</source>
</trans-unit>
<trans-unit id="error.exception.noHomepageException.setupMessage" xml:space="preserve">
<source>You might want to set the site's domain or import a new site in the setup.</source>
</trans-unit>

<trans-unit id="error.exception.databaseConnectionExceptions.title" xml:space="preserve">
<source>Database Error</source>
</trans-unit>
Expand Down

0 comments on commit cf4190e

Please sign in to comment.