-
-
Notifications
You must be signed in to change notification settings - Fork 326
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 ServiceEntityRepository #3395
Changes from 16 commits
96e3c62
4941898
78e99e5
0f4d2e3
17169e4
896255f
6519ff8
b59989e
8e21d99
b5b8c21
d1df50f
0abfa76
c7be020
da8af01
144d243
81f00da
2e470cd
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -52,15 +52,11 @@ doctrine_orm_bridge: | |
connection: default | ||
|
||
services: | ||
fork.entity.create_schema: | ||
class: Common\Doctrine\Entity\CreateSchema | ||
_defaults: | ||
autowire: true | ||
autoconfigure: true | ||
public: true | ||
arguments: | ||
- "@doctrine.orm.entity_manager" | ||
|
||
fork.repository.meta: | ||
class: Common\Doctrine\Repository\MetaRepository | ||
public: true | ||
factory: ["@doctrine.orm.entity_manager", getRepository] | ||
arguments: | ||
- Common\Doctrine\Entity\Meta | ||
Common\Doctrine\Entity\CreateSchema: | ||
|
||
Common\Doctrine\Repository\MetaRepository: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we define an alias so modules can still use There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would do a break here. |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -10,14 +10,14 @@ services: | |
- "80:80" | ||
- "443:443" | ||
depends_on: | ||
- db | ||
- db6 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. WIP? 👀 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. c7be020 nope, did this on purpose. |
||
env_file: ./var/docker/.env | ||
volumes: | ||
- .:/var/www/html:cached | ||
- ./src/Frontend/Files:/var/www/html/src/Frontend/Files:cached | ||
- fork-cms-var:/var/www/html/var | ||
- fork-cms6-var:/var/www/html/var | ||
|
||
db: | ||
db6: | ||
image: "mysql:5.7" | ||
restart: always | ||
ports: | ||
|
@@ -32,10 +32,10 @@ services: | |
# Uncomment the next line if you want to keep mysql in a data container | ||
# - db-data:/var/lib/mysql:rw | ||
# By default, use a bind-mounted host directory instead. It's harder to accidentally lose all your db data! | ||
- ./var/docker/db/data:/var/lib/mysql:rw | ||
- ./var/docker/db/data6:/var/lib/mysql:rw | ||
- ./tests/data/test_db.sql:/test_db.sql:ro | ||
- ./var/docker/db/scripts:/docker-entrypoint-initdb.d:ro | ||
|
||
volumes: | ||
fork-cms-var: {} | ||
db-data: {} | ||
fork-cms6-var: {} | ||
db6-data: {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'
or"
? Should we decide on one format? (the line above is inconsistent- "@fork.settings"
)There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doing that in the symfony 5 branch