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

the name doctrine bridge is already taken #1

Open
dbu opened this issue Aug 7, 2013 · 14 comments
Open

the name doctrine bridge is already taken #1

dbu opened this issue Aug 7, 2013 · 14 comments

Comments

@dbu
Copy link

dbu commented Aug 7, 2013

sorry, one more thing i notice is that we have a name clash. there is https://github.com/symfony/DoctrineBridge which has the purpose of bridging doctrine into symfony. its a really different type of bridge. but i think we should find a different name to make this work. what about DoctrineInterlinkBundle or DoctrineConnectorBundle?

/cc @sjopet

btw, as this is supposed to eventually go into the doctrine project, why not use the doctrine namespace even now? would make it easier to migrate.

@sjopet
Copy link

sjopet commented Aug 7, 2013

yes I agree. If this is going to be a doctrine bundle, which would make sense, why not create it in the doctrine namespace. I still think BridgeBundle is a good name though. I hope @markri who wrote our initial version would like to have a look.

@dbu
Copy link
Author

dbu commented Aug 7, 2013

i like the BridgeBundle name, it sounds right. the only issue is that
its confusing with the doctrine bridge of symfony that is something
different.
btw, would it make sense to have a Brigde|Interlink|whatever library
that can be used outside of symfony, and the bundle just leveraging that
library?

@hacfi
Copy link
Owner

hacfi commented Aug 7, 2013

@dbu I’ll change it to doctrine namespace when I rename the bundle later today.

More suggestions:

  • ObjectBridgeBundle
  • MapperBridgeBundle

I also link the Interlink part.

I don’t care much about the name although I think DoctrineConnectorBundle sounds to generic (could be a bundle to connect DBAL or something).

@dbu
Copy link
Author

dbu commented Aug 7, 2013

i like ObjectBridgeBundle

@sjopet
Copy link

sjopet commented Aug 7, 2013

ObjectBridgeBundle or ModelBridgeBundle??

@GromNaN
Copy link

GromNaN commented Aug 7, 2013

After reading the README, the following names come to my mind:

  • DoctrineRelationBundle
  • DoctrineReferenceBundle
  • EntityRelationBundle
  • EntityReferenceBundle
  • ModelRelationBundle
  • ModelReferenceBundle

@stof
Copy link

stof commented Aug 7, 2013

FYI, OpenSky contributed their Doctrine extension doing such stuff into the Gedmo extensions a while ago as it is not specific to Symfony (there is no support for referencing an ORM entity inside a PHPCR document yet but it is a matter of writing 1 class. the opposite is already possible)

@hacfi
Copy link
Owner

hacfi commented Aug 7, 2013

Thanks for the suggestions @dbu, @sjopet & @GromNaN !

@stof I checked out the Gedmo reference extension when I started working with PHPCR ODM but didn’t know how to extend it to support PHPCR ODM back then. I went with netvlies DoctrineBridgeBundle instead which worked fine but had a few limitations and was a bit outdated so I started this bundle.

I wouldn’t mind closing this repo and work on gedmo instead. I just have to check it out and talk to @l3pp4rd if he would merge PHPCR support if implemented properly. As described over at #2 I would like to make it work between all the Doctrine data mappers.

@l3pp4rd
Copy link

l3pp4rd commented Aug 7, 2013

Hi @hacfi , I do not mind concerning phpcr odm support in extensions if someone is willing to support it.

If you are willing to add PHPCR ODM support for one or more extensions I will add you to collaborators and you will be able to maintain it properly.

extensions are currently being refactored for version 2.4.0 but it should not be hard to migrate it later if there will be some tests available in 2.3.x version. Ping me about your plans

@markri
Copy link

markri commented Aug 7, 2013

I never expected to see other people picking up my old code, that was never fully functional :-). I also had had this problem when picking a name, it would be confusing to others having 2 DoctrineBridge bundles, that would feel kind of redundant. So I thought about ORMPHPCRBridge, because it only focusses on a bridge between ORM and PHPCR ( in which PHPCR is a kind of ODM with some extras).

On the other hadn ORMPHCR or maybe ORMODM is kind of unreadable beause of excessive use of abbreviation, which makes it a kind of Frankenstein name.

So back to basic, what exactly are we bridgin'? I guess it is always an object and always a model. So I'm thinking of the name "ObjectModelBridge" which can optionally be placed under the Doctrine namespace, and gives some room for bridgin other ODM's (like mongoDB) as well.

@markri
Copy link

markri commented Aug 7, 2013

altough my previous statement just should be ignored when embedding into gedmo extensions naturally :-)

@sjopet
Copy link

sjopet commented Aug 7, 2013

Just took a brief look at the references code in the extensions lib and I think adding support for phpcr is going to bring some problems. Currently it only supports ORM and MongoDB-ODM the type attribute on the annotations accepts either entity or document which needs to be changed when you add support for different ODM backends like phpcr or couchDB.

Looking at the extensions lib and the features is contains I doubt if it's really the right place for this bridge or references feature. As far as I can see all the other functions add some automated logic to a single class and have no real coherence or relation to each other. The 'DoctrineBridgeBundle' is in my eyes something that solves a very different but common problem. Adding support for more doctrine data mappers is bound to add extra configuration options etc.
Most importantly the code for bridging different doctrine backends should be in one place, whatever is decided on what repository that should be.

@l3pp4rd
Copy link

l3pp4rd commented Aug 7, 2013

yes, I agree, extensions are extensions, not a bridge library. The reference extension will serve as long as there is no better solution available.

@hacfi
Copy link
Owner

hacfi commented Aug 7, 2013

I’ll have a closer look in a couple of days. My bundle isn’t very flexible yet either.

@hacfi hacfi added the 1 - Ready label Oct 7, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants