8000 Update sample sqlite path according to symfony 3 directory structure by chalasr · Pull Request #1032 · symfony/symfony-standard · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Nov 27, 2020. It is now read-only.

Update sample sqlite path according to symfony 3 directory structure #1032

Closed
wants to merge 1 commit into from

Conversation

chalasr
Copy link
Member
@chalasr chalasr commented Dec 14, 2016

I think sqlite databases should be stored in var/ as it is storage file

@xabbuh
Copy link
Member
xabbuh commented Dec 15, 2016

👍

xabbuh added a commit to symfony/symfony-docs that referenced this pull request Dec 15, 2016
This PR was merged into the 3.1 branch.

Discussion
----------

Fix sample sqlite database path in doctrine config

I think sqlite databases should be stored in `var/` as it is storage.
About the `%kernel.data_dir%` removed here, I never heard about such a parameter in the core, either I missed it or it's very old.
Please tell me if we should first change it on lower branches, replacing it by `%kernel.root_dir%/app/data/data.sqlite`.

See also symfony/symfony-standard#1032 on the same topic

Commits
-------

9126fdf Fix sample sqlite database path in doctrine config
@chalasr
Copy link
Member Author
chalasr commented Dec 19, 2016

The change has been merged in the docs.
ping @fabpot

@fabpot
Copy link
Member
fabpot commented Dec 20, 2016

Thank you @chalasr.

fabpot added a commit that referenced this pull request Dec 20, 2016
…y structure (chalasr)

This PR was submitted for the 3.0 branch but it was merged into the 3.1 branch instead (closes #1032).

Discussion
----------

Update sample sqlite path according to symfony 3 directory structure

I think sqlite databases should be stored in `var/` as it is storage file

Commits
-------

47a1a14 Update sample sqlite path according to symfony 3 directory structure
@fabpot fabpot closed this Dec 20, 2016
javiereguiluz added a commit to symfony/demo that referenced this pull request Dec 20, 2016
This PR was squashed before being merged into the master branch (closes #412).

Discussion
----------

Move SQLite DB to the var/ directory

According to the changes in Symfony SE: symfony/symfony-standard#1032

Commits
-------

80bb759 Move SQLite DB to the var/ directory
@chalasr chalasr deleted the sqlite-db-path branch March 15, 2017 09:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0