yiisoft / yiisoft/db-migration
Migration namespace and path bugs
Open
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 32
- Forks
- 18
- Avg merge
- 22h 18m
- Merged PRs (30d)
- 2
Description
I added tests with different ways how to specify migration paths and 3 bugs were discovered:
- 1. If a path is specified, the namespace is not substituted;
- 2. If one namespace matches multiple paths, only one path will be found;
- 3. If one path corresponds to several namespaces, the same migrations along this path with different namespaces will be found.
Originally posted by @Tigrov in https://github.com/yiisoft/db-migration/issues/333#issuecomment-4368373981
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the tests mentioned in the issue for specifying migration paths and namespaces, then trace the migration path and namespace resolution entry points they exercise. Confirm the three listed cases, including the unchecked multiple-path case; done means the tests pass with each namespace/path combination resolving the intended migrations without duplicates.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- databases
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100