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

[RFC] Refactor media to be framework agnostic #50

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

[RFC] Refactor media to be framework agnostic #50

wants to merge 3 commits into from

Conversation

iampersistent
Copy link

This is a proposal to find a way to share media code between the CMF project and the Vespolina project. Basically, I have extracted the framework specific code out into its own library https://github.com/iampersistent/Media . (this is all untested at this point) This library would be used in the Vespolina project for handling media related to commerce. Media in the Vespolia project can be more than images, it can also be documents and spreadsheets.

Because there are additional needs with the Vespolina project, the media library would need more features. It would become more like a framework agnostic SonataMediaBundle, as far as the scope of the features. I did notice that the CMF Media seems to be intended to be simple, which may clash with the needs of Vespolina.

I can see a few different options.

  • we can leave things as they are and not share media code between the two projects
  • we can have a full featured media library shared between the projects
  • we can have a simple media library for CMF that is used as a base library for Vespolina media library

There may be other options I'm missing and I'm open to hearing them.

@iampersistent
Copy link
Author

@dantleech
Copy link
Member

I think this is a really good proposal, a Media component would make a lot of sense, and CMF/Vespolinaisms can then be pushed to the "Bundle layer". Also I really want us to share taxonomy logic - I might make a proposal for that.. :)

@lsmith77
Copy link
Member

lsmith77 commented Sep 2, 2013

I think its a great idea .. the question is the timing. I was about to go to RC1 with the MediaBundle this Wednesday :)

So either we:

  1. rush this in
  2. delay RC1
  3. delay this refactoring until 1.1 along with the BC break (not sure if we should sensible try to then do some BC hacks)

@dbu
Copy link
Member

dbu commented Sep 3, 2013

i think its a good proposal, and i am all for sharing code. i think we have to "get this right". there is already SonataMediaBundle, and also LiipImagineBundle that both touch on what we do.

i am quite against delaying RC1 of the CmfMediaBundle. we promised to be stable 2 months ago. so lets discuss out how this all should look, in which organization it should live and what it should and should not do. then we can start building the library and provide a migration path to that for cmf 1.1

@iampersistent
Copy link
Author

@dbu I'm open to that.

I don't care personally where the library should live. I don't think there is necessarily any advantage to having it as part of V over CMF.

From the perspective of the CMF project, what shouldn't the library do? I think defining the limits might help to understand the needs.

@rmsint
Copy link
Contributor

rmsint commented Sep 6, 2013

Great idea! (Just read this, got some internet now on my phone) I think it would be nice to have a base for media that is shared across several projects. This also means additional features can be build and shared between projects.

Regarding limits and needs, @iampersistent have you seen this wiki page: https://github.com/symfony-cmf/symfony-cmf/wiki/Media, it was used to build the current bundle.

@dbu dbu removed this from the 1.1 milestone Mar 2, 2014
@lsmith77 lsmith77 added ready and removed in progress labels May 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants