Thanks to visit codestin.com
Credit goes to github.com

Skip to content

doctrine:mapping:import bugfix #41

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

Closed
wants to merge 1 commit into from

Conversation

chesteroni
Copy link
Contributor

I tried to create mapping from existing MySQL database. I got Fatal error:
PHP Fatal error: Call to a member function getConfiguration() on a non-object in /home/chester/workspace/MyProject/src/vendor/doctrine/lib/Doctrine/ORM/Mapping/ClassMetadataFactory.php on line 135
with clue in stack trace:
(...)
PHP 7. Doctrine\ORM\Mapping\ClassMetadataFactory->getAllMetadata() /home/chester/workspace/lingus/src/vendor/symfony/src/Symfony/Bundle/DoctrineBundle/Command/ImportMappingDoctrineCommand.php:97
(...)
After googling I found this bug report at Doctrine:
http://www.doctrine-project.org/jira/browse/DDC-939?page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel
So I fixed the code in symfony.
It worked.

However - somebody should see, if this bugfix possibly should be applied in some other places in framework - my knowledge about symfony/doctrine is rather small.

@chesteroni
Copy link
Contributor Author

BTW: Doctrine documentation has an error, my fix is against the documentation, but in mentioned doctrine-jira bug they say that it is a documentation bug.

@jwage
Copy link
Contributor

jwage commented Jan 4, 2011

Fabien, this can be merged. It was another location where we did not setEntityManager after the change in Doctrine!

fabpot added a commit that referenced this pull request Jan 10, 2012
Commits
-------

63e7f95 updated hungarian translations

Discussion
----------

Hungarian translations for 2.0

Bug fix: no
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
Fixes the following tickets: -

I have translated the trans-unit #41.
SofHad pushed a commit to SofHad/symfony that referenced this pull request Oct 12, 2015
This PR was squashed before being merged into the master branch (closes symfony#41).

Discussion
----------

Translate to Russian and Ukrainian

Commits
-------

2e7a2b7 Translate to Russian and Ukrainian
jderusse pushed a commit to jderusse/symfony that referenced this pull request Mar 30, 2020
This PR was merged into the master branch.

Discussion
----------

Update Contributing section

Add Validate step to the contributing section.

Commits
-------

0432dfc Update README.md
This pull request was closed.
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.

2 participants