Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Use callable classes for custom Monolog processors#10125

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@jaikdean
Copy link

The built-in Monolog processors all use callable classes with an__invoke() method, whereas the Symfony documentation uses aprocessRecord() method. Using__invoke() provides consistency with the standard processors and also removes the need to specify the method name in the service configuration.

@javiereguiluz
Copy link
Member

@jaikdean I've just checked Monolog and you are totally right! Let's make this change because it's consistent and simplifies things a lot. Thanks for helping us improve Symfony Docs and congrats on your first contribution to this repository!

@javiereguiluzjaviereguiluz merged commit6f1a0f3 intosymfony:2.8Jul 27, 2018
javiereguiluz added a commit that referenced this pull requestJul 27, 2018
…dean)This PR was merged into the 2.8 branch.Discussion----------Use callable classes for custom Monolog processorsThe built-in Monolog processors all use callable classes with an `__invoke()` method, whereas the Symfony documentation uses a `processRecord()` method. Using `__invoke()` provides consistency with the standard processors and also removes the need to specify the method name in the service configuration.Commits-------6f1a0f3 Use callable classes for custom Monolog processors
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Projects

None yet

Milestone

2.8

Development

Successfully merging this pull request may close these issues.

3 participants

@jaikdean@javiereguiluz@carsonbot

[8]ページ先頭

©2009-2025 Movatter.jp