StofDoctrineExtensionsBundle icon indicating copy to clipboard operation
StofDoctrineExtensionsBundle copied to clipboard

Add cross OM references support

Open alemorozov opened this issue 12 years ago • 16 comments
trafficstars

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

alemorozov avatar Apr 27 '13 11:04 alemorozov

When the extension was contributed, I commented saying it should receive ManagerRegistry instances instead of managers. @jwage said he would work on it: https://github.com/Atlantic18/DoctrineExtensions/pull/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

stof avatar Apr 27 '13 15:04 stof

+1 for that

soerenmartius avatar Jun 25 '13 14:06 soerenmartius

+1 bump !

azr avatar Jul 16 '13 18:07 azr

+1 :)

andreasschmidt avatar Nov 26 '13 07:11 andreasschmidt

+1

solocommand avatar Dec 03 '13 16:12 solocommand

+1

dovys avatar Dec 17 '13 08:12 dovys

:+1:

bmeynell avatar Aug 21 '14 16:08 bmeynell

:+1: Came here from: http://stackoverflow.com/questions/17296659/sharing-objects-between-doctrine2-odm-and-orm-in-symfony2-with-doctrine-extensio#17304090

webdevilopers avatar Sep 10 '14 11:09 webdevilopers

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.

dbojdo avatar May 18 '15 12:05 dbojdo

Good luck @dbojdo You might already saw this relevant thread https://github.com/Atlantic18/DoctrineExtensions/issues/821

It was already implemented in https://github.com/Atlantic18/DoctrineExtensions/commit/668ea01fa3ced27667fdf342d3ee76306d95d64d and https://github.com/Atlantic18/DoctrineExtensions/commit/0d6a4c8d0c9a64c8b6e735ed4c0dae03e1e31d59 but this is from WIP doctrine extensions 3.0 mega branch, which wasn't merged for several years already.

diimpp avatar May 18 '15 12:05 diimpp

@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: https://github.com/dbojdo/DoctrineExtensions/commit/54b19d5410063f6a62cce8f3b98e49cdfb015579

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

dbojdo avatar May 18 '15 13:05 dbojdo

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

diimpp avatar May 18 '15 13:05 diimpp

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

dbojdo avatar May 18 '15 13:05 dbojdo

@dbojdo did u re-factor it ?

chandon avatar Jun 08 '15 16:06 chandon

@chandon Hi! Yes, it's working fine see: https://github.com/Atlantic18/DoctrineExtensions/pull/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)

dbojdo avatar Jun 08 '15 17:06 dbojdo

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

chandon avatar Jun 09 '15 05:06 chandon