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

Note and link to Logging API #5214

Merged
merged 2 commits into from
Jan 16, 2025
Merged

Conversation

julianhofmann
Copy link
Contributor

In the Performance section, not only the direct configuration of the core should be taken into account, but at least measures for optimizing logging via API should be pointed out.

In the Performance section, not only the direct configuration of the core should be taken into account, but at least measures for optimizing logging via API should be pointed out.
@linawolf linawolf self-assigned this Jan 16, 2025
@linawolf linawolf added backport 12.4 Backport to TYPO3v12 backport 13.4 Backport to TYPO3v13 labels Jan 16, 2025
@linawolf linawolf merged commit 894d168 into TYPO3-Documentation:main Jan 16, 2025
7 checks passed
@julianhofmann
Copy link
Contributor Author

Oh, :PHP:\Psr\Log\LogLevel::ERROR behaves differently than :php:\Psr\Log\LogLevel::ERROR and swallows the backslashes.

@linawolf
Copy link
Member

@jaapio does it make sense that textroles are case-sensitiv?

@linawolf
Copy link
Member

@julianhofmann could you make a follow up please?

julianhofmann added a commit to julianhofmann/TYPO3CMS-Reference-CoreApi that referenced this pull request Jan 16, 2025
Uppercase-written TextRoles are not (or in another way) interpreted. Concretely, for this PR, backslashes have been swallowed.

Releated: TYPO3-Documentation#5214
@julianhofmann julianhofmann mentioned this pull request Jan 16, 2025
froemken pushed a commit that referenced this pull request Jan 16, 2025
Uppercase-written TextRoles are not (or in another way) interpreted. Concretely, for this PR, backslashes have been swallowed.

Releated: #5214
github-actions bot pushed a commit that referenced this pull request Jan 16, 2025
Uppercase-written TextRoles are not (or in another way) interpreted. Concretely, for this PR, backslashes have been swallowed.

Releated: #5214
github-actions bot pushed a commit that referenced this pull request Jan 16, 2025
Uppercase-written TextRoles are not (or in another way) interpreted. Concretely, for this PR, backslashes have been swallowed.

Releated: #5214
froemken pushed a commit that referenced this pull request Jan 16, 2025
Uppercase-written TextRoles are not (or in another way) interpreted. Concretely, for this PR, backslashes have been swallowed.

Releated: #5214

Co-authored-by: Julian Hofmann <[email protected]>
froemken pushed a commit that referenced this pull request Jan 16, 2025
Uppercase-written TextRoles are not (or in another way) interpreted. Concretely, for this PR, backslashes have been swallowed.

Releated: #5214

Co-authored-by: Julian Hofmann <[email protected]>
@jaapio
Copy link
Contributor

jaapio commented Jan 17, 2025

does it make sense that textroles are case-sensitiv?

We would need to check how tools work. But I think it doesn't make sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 12.4 Backport to TYPO3v12 backport 13.4 Backport to TYPO3v13
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants