-
-
Notifications
You must be signed in to change notification settings - Fork 70
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
2.0 Ideas #110
Comments
See #111 for a proposal to fix this |
Hello @kbond Are you willing to support |
@nikophil the library is written based on |
Nice to here that, I'm gonna try that on a project of mine which uses mongo |
all of the subjects raised here have been addressed |
@wouterj and I have been discussing some potential changes for 2.0. This is a list of potential ideas:
ProxyRepository
toRepositoryDecorator
and mark internal ("proxy" is a bit of a misnomer - it is actually a "decorator")Proxy
and changeFactory::create()
to return the real doctrine entity. While the proxy idea is clever and sort of creates a "quasi" AR system for your entities, it can be confusing (to use and document)ModelFactory
/RepositoryDecorator
and some stand alone helper functionsocramius/proxy-manager
could be used for auto-refresh as this would be lost with removal of theProxy
$user = UserFactory::new()->create()->object()
is unfortunate. The->object()
would be removed by removing theProxy
(above) but$user = UserFactory::new()->create()
is still confusing... (solved in Remove "visual duplication" of ModelFactory::new()->create() #111)ModelFactory::repository()
/AnonymousFactory::repository()
to return the "real" repository for the entityFOUNDRY_DISABLE_DATABASE_RESET
,FOUNDRY_RESET_CONNECTIONS
,FOUNDRY_RESET_OBJECT_MANAGERS
)? [feat] [remove bundleless usge] use config instead of environment variables #320ModelFactory::getClass()
public and removeModelFactory::getEntityClass()
? See Only list entities which do not have factories #162 for details (specifically Only list entities which do not have factories #162 (comment))The text was updated successfully, but these errors were encountered: