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

[cookbook][doctrine] fixed reverse_engineering doc#2578

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 4 commits intosymfony:2.1from77web:fix-reverse_engineering-doc
Jun 4, 2013

Conversation

77web
Copy link
Contributor

QA
Doc fix?yes
New docs?no
Applies to2.1+
Fixed ticketsN/A

Sorry but I could find no evidence of update from the code.
I wrote the way that actually works for me (in Symfony SE 2.1/2.2).
If the evidence for this fix is required, I will dig into code...
Maybe they are in doctrine/DoctrineBundle or doctrine/orm?

@wouterj
Copy link
Member

I did no further research, but the code for the command can be found in the doctrinebindle:https://github.com/doctrine/DoctrineBundle/blob/master/Command/Proxy/ConvertMappingDoctrineCommand.php

@77web
Copy link
ContributorAuthor

@wouterj Thanks!
The method "getExporter()" in that class describes9f2fb49.

weaverryan added a commit that referenced this pull requestJun 4, 2013
[cookbook][doctrine] fixed reverse_engineering doc
@weaverryanweaverryan merged commit02c73c2 intosymfony:2.1Jun 4, 2013
@77web77web deleted the fix-reverse_engineering-doc branchJune 4, 2013 03:11
weaverryan added a commit that referenced this pull requestJun 4, 2013
@weaverryan
Copy link
Member

Hi Hiromi!

These are all excellent changes - I just worked through the exercises to verify :).

Thanks very much for your contribution!

@77web
Copy link
ContributorAuthor

Thank you for your kind help, too!

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.

3 participants
@77web@wouterj@weaverryan

[8]ページ先頭

©2009-2025 Movatter.jp