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

added documentation for the new Debug component#2479

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
weaverryan merged 1 commit intosymfony:masterfromfabpot:debug-component
Apr 11, 2013

Conversation

fabpot
Copy link
Member

QA
Doc fix?no
New docs?yes (symfony/symfony#7441)
Applies to2.3+
Fixed ticketsn/a

fabpot added a commit to symfony/symfony that referenced this pull requestApr 7, 2013
This PR was merged into the master branch.Discussion----------[Debug] added the component (closes#6828,closes#6834,closes#7330)| Q             | A| ------------- | ---| Bug fix?      | no| New feature?  | yes| BC breaks?    | no| Deprecations? | yes| Tests pass?   | yes| Fixed tickets |#6828,#6834,#7330| License       | MIT| Doc PR        |symfony/symfony-docs#2479You can use the individual tools, or register them all:```phpuse Symfony\Component\Debug\Debug;Debug::enable();```Changes in Symfony SE:symfony/symfony-standard#523Commits-------f693128 fixed typos1ab1146 [Debug] fixed minor bugsdaa3a3c [Debug] changed composer to accept more versionse455269 [Debug] ensured that the Debug tools can only be registered once946bfb2 [Debug] made the exception handler independant of HttpFoundation2b305c2 added a main Debug class to ease integration2ff0927 [Debug] added the component (closes#6828,closes#6834,closes#7330)
Debug::enable();

The :method:`Symfony\\Component\\Debug\\Debug::enable` method registers an
error handler and an exception handler. If the Symfony ClassLoader component
Copy link
Member

Choose a reason for hiding this comment

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

add some interlinking to the ClassLoader component docs here.

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

done

@wouterj
Copy link
Member

you forgot to add this article in themap.rst.inc file.

The Debug Component
===================

The Debug Component provides tools to ease debugging PHP code.
Copy link
Member

Choose a reason for hiding this comment

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

You should add the note saying it is new in 2.3, extracting code from HttpKernel (see how it is done for PropertyAccess)

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

fixed

@fabpot
Copy link
MemberAuthor

everything should be ok now.


The :method:`Symfony\\Component\\Debug\\Debug::enable` method registers an
error handler and an exception handler. If the :doc:`ClassLoader component
</components/class_loader>` is available, a special class loader is also
Copy link
Member

Choose a reason for hiding this comment

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

I prefer to have rules on 1 line:

[...] If the:doc:`ClassLoader component</components/class_loader>` is available,a special [...]

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

That's not what we are doing elsewhere. I don't format things by hand but via my text editor.

Copy link
Member

Choose a reason for hiding this comment

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

It's something I always say on PRs and it's something that's done in all documents as far as I can see.

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

ok, I will let you reformat what you think is best.

weaverryan added a commit that referenced this pull requestApr 11, 2013
added documentation for the new Debug component
@weaverryanweaverryan merged commit8c270f7 intosymfony:masterApr 11, 2013
weaverryan added a commit that referenced this pull requestApr 11, 2013
@fabpotfabpot deleted the debug-component branchMarch 3, 2014 12:23
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@fabpot@wouterj@stof@weaverryan

[8]ページ先頭

©2009-2025 Movatter.jp