Skip to content
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

Add cross OM references support #192

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alexeymorozov
Copy link

Support for this was added to DoctrineExtensions in https://github.com/l3pp4rd/DoctrineExtensions/pull/625

Conflicts:
	DependencyInjection/Configuration.php
	DependencyInjection/StofDoctrineExtensionsExtension.php
	Resources/config/listeners.xml
@stof
Copy link
Owner

stof commented Apr 27, 2013

When the extension was contributed, I commented saying it should receive ManagerRegistry instances instead of managers. @jwage said he would work on it: doctrine-extensions/DoctrineExtensions#625
I think it would be better to refactor it this way first as it would not require changing the listener anymore in the bundle

@soerenmartius
Copy link

+1 for that

@azr
Copy link

azr commented Jul 16, 2013

+1 bump !

@andreasschmidt
Copy link

+1 :)

@solocommand
Copy link

+1

2 similar comments
@dovys
Copy link

dovys commented Dec 17, 2013

+1

@bmeynell
Copy link

👍

@dbojdo
Copy link

dbojdo commented May 18, 2015

Hi! I see, this feature got stuck. I need it very much (different managers), so I'm going to re-factor it. I'll also provide the XML and YAML mapping drivers.

@diimpp
Copy link

diimpp commented May 18, 2015

Good luck @dbojdo
You might already saw this relevant thread doctrine-extensions/DoctrineExtensions#821

It was already implemented in
doctrine-extensions/DoctrineExtensions@668ea01
and
doctrine-extensions/DoctrineExtensions@0d6a4c8
but this is from WIP doctrine extensions 3.0 mega branch, which wasn't merged for several years already.

@dbojdo
Copy link

dbojdo commented May 18, 2015

@diimpp thanks :/ Yeah. I've seen that, we can wait for the merging forever. Btw, why did you inject only one ManagerRegistry? As I know, DoctrineBundle and DoctrineMongodbBundle provide a independent ManagerRegistry. So you can't find the Manager for MongoDb Document in the Entity one.

See: dbojdo/DoctrineExtensions@54b19d5

Anyway, I need this feature very much. If nothing change, I'll just create my own Extension based on this one.

@diimpp
Copy link

diimpp commented May 18, 2015

No credits for me, I'm just posting links :) For same reason I don't know reasoning behind implementation.

@dbojdo
Copy link

dbojdo commented May 18, 2015

@diimpp oh, sorry, I haven't noted :) Thanks anyway.

@chandon
Copy link

chandon commented Jun 8, 2015

@dbojdo did u re-factor it ?

@dbojdo
Copy link

dbojdo commented Jun 8, 2015

@chandon Hi! Yes, it's working fine see: doctrine-extensions/DoctrineExtensions#1320
I don't know, where they will merge it (if ever?) but you can use https://github.com/DXI-Ltd/doctrine-extension and https://github.com/DXI-Ltd/doctrine-extension-bundle as long is stuck (there is ported Reference extension)

@chandon
Copy link

chandon commented Jun 9, 2015

@dbojdo thanks, i installed it from your DXI bundle, it's working well !

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.