Skip to content
This repository has been archived by the owner on Sep 16, 2021. It is now read-only.

Ensure all bundles use XML mapping #159

Closed
dantleech opened this issue May 1, 2013 · 6 comments
Closed

Ensure all bundles use XML mapping #159

dantleech opened this issue May 1, 2013 · 6 comments
Milestone

Comments

@dantleech
Copy link
Member

All bundles should use XML mapping .. right? :)

@dbu
Copy link
Member

dbu commented May 1, 2013

yes, but we should finalize doctrine/phpcr-odm#279 first, otherwise its double work.

the other thing i would like to do at the same point is moving model classes to Model and if needed at all have Persistence/Doctrine/Phpcr subclasses for storage specific things. this would need something similar to doctrine/DoctrineBundle#177 for phpcr-odm.

@dantleech
Copy link
Member Author

Yeah, doing the Model thing would be awesome :) Is it feasable to do
this for 1.0? Or should we just start by switching Annotation mappings
to XML mappings?

On Wed, May 01, 2013 at 11:17:49AM -0700, David Buchmann wrote:

yes, but we should finalize [1]doctrine/phpcr-odm#279 first, otherwise its
double work.

the other thing i would like to do at the same point is moving model
classes to Model and if needed at all have Persistence/Doctrine/Phpcr
subclasses for storage specific things. this would need something similar
to [2]doctrine/DoctrineBundle#177 for phpcr-odm.


Reply to this email directly or [3]view it on GitHub.

References

Visible links

  1. cleanup xml mapping names
    cleanup xml mapping names doctrine/phpcr-odm#279
  2. [WIP] add compiler pass for bundles to register mappings
    [WIP] add compiler pass for bundles to register mappings doctrine/DoctrineBundle#177
  3. Ensure all bundles use XML mapping #159 (comment)

@dbu
Copy link
Member

dbu commented May 1, 2013

actually we can do one after the other. doing the mapping pass in phpcr-odm is doable. but i will focus on getting the xml mapping right first, that is quite a bc break there.

@lsmith77
Copy link
Member

lsmith77 commented Aug 9, 2013

i guess we have tickets in all bundles for this now as part of the bundle standards .. or will open such tickets. so closing this one.

@lsmith77 lsmith77 closed this as completed Aug 9, 2013
@dantleech
Copy link
Member Author

Which bundles still use non-XML mappings?

@dbu
Copy link
Member

dbu commented Oct 18, 2013

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants