Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include default typoscript installation-wide #55

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

dimitri-koenig
Copy link
Contributor

@dimitri-koenig dimitri-koenig commented Apr 11, 2024

Currently the dashboard template cannot be loaded, see #35. This fixes it.

Temporary workaround: patch via composer-patches:

        "pluswerk/mail-logger": {
            "Fix backend view path": "https://github.com/dimitri-koenig/mail_logger/pull/1.patch"
        }

Fixes #35

@Kanti
Copy link
Member

Kanti commented Apr 12, 2024

Instead of this workaround, you should probably include the TypoScript of this extension in your Sites TypoScript

Add @import 'EXT:mail_logger/Configuration/TypoScript/setup.typoscript' in your setup
And @import 'EXT:mail_logger/Configuration/TypoScript/constant.typoscript' in your constants.

@Kanti Kanti closed this Apr 12, 2024
@dimitri-koenig
Copy link
Contributor Author

dimitri-koenig commented Apr 12, 2024

@Kanti, that would work for Frontend, but not for Backend, wouldn't it?

@dimitri-koenig dimitri-koenig changed the title Set custom path for backend views Include default typoscript installation-wide Apr 20, 2024
@dimitri-koenig
Copy link
Contributor Author

I've changed the code, removed the initializeView part, and added an ext_localconf.php file which includes the typoscript installation-wide. I had some warnings when opening up the backend module. This fixes it.

@josefglatz
Copy link

josefglatz commented Apr 25, 2024

My workaround was also to add the proper TypoScript in our SitePackage (TYPO3 11.5)

Once this get's merged I can remove my workaround in the project.

Just for the reference, I had to provide key, type and content:

\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addTypoScript(
    'mail_logger',
    'setup',
    '@import \'EXT:mail_logger/Configuration/TypoScript/setup.typoscript\''
);
\TYPO3\CMS\Core\Utility\ExtensionManagementUtility::addTypoScript(
    'mail_logger',
    'constants',
    '@import \'EXT:mail_logger/Configuration/TypoScript/constants.typoscript\''
);

@Kanti Kanti merged commit 400338f into pluswerk:master Apr 25, 2024
7 checks passed
@Kanti
Copy link
Member

Kanti commented Apr 25, 2024

I had time to test it with our Projects, did does not break anything, so I will release this.
If the first site root has the typoscript incldued indriectly as in our Projects, the change is not neccessery as the first SiteRoots Typoscript will always be loaded in the Backend Module:
It is like this since TYPO3 7 LTS
https://github.com/TYPO3/typo3/blob/main/typo3/sysext/extbase/Classes/Configuration/BackendConfigurationManager.php#L303
grafik

@dimitri-koenig
Copy link
Contributor Author

@Kanti, thx

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.

Backend module doesn't work in 10.4 LTS
3 participants