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

Use Service factories in Doctrine MongoDB ODM module #24

Open
rieschl opened this issue May 29, 2020 · 5 comments
Open

Use Service factories in Doctrine MongoDB ODM module #24

rieschl opened this issue May 29, 2020 · 5 comments

Comments

@rieschl
Copy link
Contributor

rieschl commented May 29, 2020

Due to the latest license incidents in PHP-land, @alcaeus asked me to reach out to you and ask if it's ok to use parts of your service factories in the DoctrineMongoODMModule package.

Most of the factories are the same for ORM and ODM and your AbstractFactory is written quite nicely so I used them to instantiate the services there while upgrading the library to be compatible with mongodb-odm 2.0

Is it ok for you to put a license header to each file stating that it's origin or should I mention it somewhere else (LICENCE, README)?

Thanks!

@Ocramius
Copy link
Member

In theory, you'd need to reference the original source and license in the location where you copy the code to.

IANAL, but you'll need to copy the license somewhere too.

@rieschl
Copy link
Contributor Author

rieschl commented May 29, 2020

In theory, you'd need to reference the original source and license in the location where you copy the code to.

It's in the files, so that should be satisfied., right?

IANAL, but you'll need to copy the license somewhere too.

So, a notice in the LICENSE file?
Symfony has a copyright notice in one of its files, but not a separate notice in the LICENSE file.

So, what should I do? :)

@rieschl
Copy link
Contributor Author

rieschl commented May 29, 2020

Hm, @Ocramius if you're bored, you could extract all non-ORM stuff (basically everything except EntityManager- and Migration-stuff) into a separate base package which I can use directly in the DoctrineMongoODMModule. That would be much DRYer and no license-issue 😄

@Ocramius
Copy link
Member

Ocramius commented May 29, 2020 via email

@rieschl
Copy link
Contributor Author

rieschl commented Nov 16, 2020

Update:
I went another way with DoctrineMongoODMModule (not using this factories), so it's not an issue, there.
But as the DoctrineMongoODMModule needs laminas-mvc I wrote a separate package like this one to be used with Mezzio (or any other PSR container).

In that package I used some files from this library.

I added an attribution in the main LICENSE file and the files which I copied: AbstractOdmFactory, ConfigurationFactory, ConnectionFactory, DocumentManagerFactory

Is that okay for you?
Thanks!

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

No branches or pull requests

2 participants