Skip to content

Conversation

@dbu
Copy link
Member

@dbu dbu commented Jul 25, 2014

Q A
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Tests pass? travis
Fixed tickets -
License MIT
Doc PR TODO

This is a backwards compatible change for adding support for namespace alias as with auto-mapped documents in the default Document folder.

The alias only work with symfony/symfony#10853 , are ignored otherwise. Lets still wait for the symfony core PR to be merged before merging this.

Copy link
Contributor

Choose a reason for hiding this comment

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

dri.ver => driver

@dbu
Copy link
Member Author

dbu commented Jul 25, 2014

cleaned up naming

fabpot added a commit to symfony/symfony that referenced this pull request Aug 8, 2014
…pace aliases (dbu) This PR was merged into the 2.6-dev branch. Discussion ---------- provide a compiler pass for doctrine to register namespace aliases | Q | A | ------------- | --- | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | yes | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | symfony/symfony-docs#4069 I realized that the compiler pass only allows bundles to register folders to load entities / documents but not namespace alias to make things like `CmfRoutingBundle:Route` work. This is an attempt to provide a separate compiler pass for this. Maybe we should instead provide a new combined compiler pass and have users switch to the new pass if available. * [x] [Doctrine ORM](doctrine/DoctrineBundle#316) * [x] [DoctrinePHPCRBundle](doctrine/DoctrinePHPCRBundle#158) * [ ] Doctrine CouchDB * [ ] Doctrine MongoDB Commits ------- 9a62ab3 provide a compiler pass for doctrine to register namespace aliases and not only mapping drivers
@dbu dbu removed the in progress label Aug 8, 2014
dbu added a commit that referenced this pull request Aug 8, 2014
add support for bundle namespace alias
@dbu dbu merged commit ea1db1b into master Aug 8, 2014
@dbu dbu deleted the support-namespace-alias branch August 8, 2014 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants