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

Pridat check constraints na vsechny sloupce ktere referuji na heslar #15

Closed
Najiva opened this issue Feb 5, 2021 · 9 comments
Closed
Labels
data migration Issues concerning data migration from old to new DB. enhancement New feature or request

Comments

@Najiva
Copy link
Contributor

Najiva commented Feb 5, 2021

Check constraint by mel byt zalozen na nazvu heslare ne na id-ckach.

EDIT @motyc: podrobněji viz: #15 (comment)

@pesikj
Copy link
Collaborator

pesikj commented Sep 6, 2021

Zde úplně nerozmím zadání. Contraint musí být nastavený na foreign key. Znamená to, že je potřeba foreign key předělat?
@Najiva @motyc

@Najiva
Copy link
Contributor Author

Najiva commented Sep 6, 2021

@pesikj Protoze jsou vsechny heslare v jedne tabulce, cizy klic nedokaze zamezit tomu aby vazba na zaznam z jineho heslare(podmnoziny zaznamu z tabulky heslar) nez na ktery ma ukazovat byla nevalidni. Je potreba pridat novy constraint (check) ktery bude omezovat reference na tabulku heslar pouze na jeji podmnoziny.

@pesikj
Copy link
Collaborator

pesikj commented Sep 7, 2021

@Najiva To lze řešit parametrem limit_choices_to u pole ForeignKey, jde tedy o přidání těchto parametrů do polí, kde chybí?

@motyc
Copy link
Member

motyc commented Sep 7, 2021

@pesikj Myslím, že když jsme to s @Najiva řešili, tak jsme to chtěli právě mít na úrovni databáze. Jestli jsem dobře hledal, tak "limit_choices_to" je funkce Djanga, nikoli PostgreSQL.

@Najiva
Copy link
Contributor Author

Najiva commented Sep 7, 2021

@pesikj Ne. Myslenka byla mit to jak na aplikacni urovni tak na urovni databaze (protoze se s daty manipuluje i mimo aplikace). Jesli to ale David nevyzaduje tak to neni kriticke. Taky proto jsem to issue nedodelal. Zakladal jsem ho podle sveho uvazeni.

@motyc
Copy link
Member

motyc commented Sep 7, 2021

Souhlasím, že toto můžeme řešit později, není to pro stávající fázi kritické.

@motyc motyc added this to the Podzim milestone Sep 7, 2021
@pesikj
Copy link
Collaborator

pesikj commented Sep 7, 2021

Dobře, mně se k tomu moc informací zatím nepodařilo dohledat. Ale přinejhorším se na nějakém testovacím projektu dá vytvořit testovací pole, zmigrovat ho a podívat se create skript databáze.

@motyc motyc removed this from the Podzim milestone Jan 25, 2022
@motyc motyc added the enhancement New feature or request label Jan 25, 2022
@motyc motyc added this to the Testing DB milestone Jul 27, 2022
@pesikj pesikj added the data migration Issues concerning data migration from old to new DB. label Jul 29, 2022
@motyc motyc removed this from the Testing DB milestone Sep 8, 2022
@motyc
Copy link
Member

motyc commented Oct 27, 2022

Cílem úpravy je na úrovni DB zamezit tomu, aby došlo užití heslar.id ve špatném kontextu. Přehled správného přiřazení hesel podle obsahu heslar_nazev.nazev je zde: https://docs.google.com/spreadsheets/d/1mwz3N31idVVX0yDKXVTtk161wddK3oWSASuN24TwCd4/edit#gid=0

Je třeba omezit (pravděpodobně pomocí check constraint) rozsah přípustných hodnot pro daná pole, aby nemohlo dojít k chybě.

@motyc
Copy link
Member

motyc commented Dec 29, 2022

Vyřešeno v 2ec64ab

@motyc motyc closed this as completed Dec 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data migration Issues concerning data migration from old to new DB. enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants