Skip to content

Add Migrator methods to create DB foreign keys#1019

Merged
mvorisek merged 19 commits intodevelopfrom
ref_integrity
Jul 19, 2022
Merged

Add Migrator methods to create DB foreign keys#1019
mvorisek merged 19 commits intodevelopfrom
ref_integrity

Conversation

@mvorisek
Copy link
Copy Markdown
Member

@mvorisek mvorisek commented Jul 7, 2022 •
edited
Loading

tested with all tests, fix various related integrity issues

FK can be setup from test like:

$this->createMigrator()->createForeignKey($join);
$this->createMigrator()->createForeignKey($hasOneOrHasManyReference);

@mvorisek mvorisek force-pushed the ref_integrity branch 30 times, most recently from 19d282c to 1c57657 Compare July 11, 2022 23:43
@mvorisek mvorisek force-pushed the ref_integrity branch 10 times, most recently from cb842bc to 99ff67a Compare July 14, 2022 14:00
mvorisek added 19 commits July 17, 2022 15:48
@mvorisek
format queries before dump
1c8e8d2
@mvorisek
unify Reference/Join getters
abf4117
@mvorisek
fix MSSQL/Oracle DBAL unique index issues
37105a8
@mvorisek
TestCase::setDb must not drop any table
6f4d844
@mvorisek
fix not-unsigned "uid" and "cid" for MySQL FK
205abb5
@mvorisek
fix wrong data in JoinSqlTest
628fb60
@mvorisek
simplify old comment for joins
b51c44f
@mvorisek
Add Migrator methods to setup FK
ae302b7
@mvorisek
DEBUG dropIfExists before create
30d0955
@mvorisek
DEBUG run all tests with FKs from model relations
274f010
@mvorisek
add Join getters to match Reference
f6e219f
@mvorisek
test some tests with FK
e7b2531
@mvorisek
Revert "DEBUG dropIfExists before create"
b177646 
This reverts commit 30d0955.
@mvorisek
Revert "DEBUG run all tests with FKs from model relations"
c44076c 
This reverts commit 4187929.
@mvorisek
fix FKs added twice
1e79384
@mvorisek
test has/getRef and has/getJoin
e52af2b
@mvorisek
no JoinLinkTrait::setJoin method
091db4b
@mvorisek
unify Join/Ref internal names
aecdd82
@mvorisek
unify debug SQL dump with Expression::getDebugQuery()
9528687
@mvorisek mvorisek mentioned this pull request Dec 20, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@georgehristov georgehristov Awaiting requested review from georgehristov

@DarkSide666 DarkSide666 Awaiting requested review from DarkSide666

@abbadon1334 abbadon1334 Awaiting requested review from abbadon1334

@mkrecek234 mkrecek234 Awaiting requested review from mkrecek234

Assignees

No one assigned

Labels

MAJOR

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mvorisek