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

Migration and new folders #7269

Closed
lynicidn opened this issue Feb 13, 2015 · 6 comments
Closed

Migration and new folders #7269

lynicidn opened this issue Feb 13, 2015 · 6 comments
Assignees
Labels
status:ready for adoption Feel free to implement this issue. type:enhancement
Milestone

Comments

@lynicidn
Copy link
Contributor

Where am run yii migrate/up --migrationPath="@yii\rbac\migration" or do typo in word migrations this create new folders - it not correct behavior if action not create

@lynicidn
Copy link
Contributor Author

i think true way throw invalid config (param) exception folder not exists. Now if migrations not found in folder - be created new folder and try apply migrations from default folder

@lynicidn
Copy link
Contributor Author

По русски:
Если применяем миграцию, и ошиблись в migrationPath то будет создана папка соответствующая алиасу migrationPath, но список миграций отобразиться из дефолтной папки app/migrations, но при применении выскочит ошибка, что в ново созданной папке нет этой миграции. Отсюда 2 предложения - создавать папку только при migrate/create и разобраться почему если миграции не найдены в папке (в новь созданной) выдается список из дефолтной папки - ведь это ошибка, сразу известно что пути не сойдутся. Вообщем там какоето неадекватное сейчас поведение

@samdark samdark added this to the 2.0.x milestone Feb 13, 2015
@samdark
Copy link
Member

samdark commented Feb 13, 2015

Agree. Can you work on a pull request?

@samdark
Copy link
Member

samdark commented Feb 13, 2015

Behavior for create action shold stay as it is now.

@samdark samdark added status:ready for adoption Feel free to implement this issue. type:enhancement and removed type:bug Bug labels Feb 13, 2015
@lynicidn
Copy link
Contributor Author

постараюсь, только на выходных, сейчас чуточку загружен

@lynicidn
Copy link
Contributor Author

#7344

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:ready for adoption Feel free to implement this issue. type:enhancement
Projects
None yet
Development

No branches or pull requests

3 participants