Skip to content
Snippets Groups Projects
Jan Kuchař's avatar
commented on issue #31 "More ergonomical insert() and update() methods?" at grifart libs / tables

Souhlas a dovedu si představit, že changes budou brát i ty současné update() insert() metody.

Jiří Pudil's avatar
pushed new tag 0.11.2 at grifart libs / tables
Jiří Pudil's avatar
deleted branch new-table-methods at grifart libs / tables
Jiří Pudil's avatar
accepted merge request !60 "Table: add new methods" at grifart libs / tables
Jiří Pudil's avatar
pushed to branch master at grifart libs / tables
Jiří Pudil's avatar
pushed to branch new-table-methods at grifart libs / tables
Jiří Pudil's avatar
opened issue #31 "More ergonomical insert() and update() methods?" at grifart libs / tables
Jiří Pudil's avatar
commented on merge request !60 "Table: add new methods" at grifart libs / tables

Pokud je mi známo, primary key se nad ničím jiným než nad sloupečky vytvořit ani nedá 🤔 a pokud jo, můžeme doplnit později.

Jiří Pudil's avatar
commented on merge request !60 "Table: add new methods" at grifart libs / tables

Ano, je to zkratka pro...

Jan Kuchař's avatar
approved merge request !60 "Table: add new methods" at grifart libs / tables
Jan Kuchař's avatar
commented on merge request !60 "Table: add new methods" at grifart libs / tables

Skvělé, za mě ready to ship. Zmíněné věci mohou být jako follow up.

Jan Kuchař's avatar
commented on merge request !60 "Table: add new methods" at grifart libs / tables

Prosím ještě o test na samotný upsert()

Jan Kuchař's avatar
commented on merge request !60 "Table: add new methods" at grifart libs / tables

Safety: Malinko se tu bojím, jestli musí být vždy primary key složen ze sloupců. Němůže tam být agregace? Možná něco jako getConflictsOnDefintion()...

Jan Kuchař's avatar
commented on merge request !60 "Table: add new methods" at grifart libs / tables

Skvělé! To je fakt moc hezký API.

Jan Kuchař's avatar
commented on merge request !60 "Table: add new methods" at grifart libs / tables

Pokud se to používá často, dávalo by smysl mít tu v konstrukturo optional parametry přes pojmenované parametry -> pak je možné použít jako one-line...

Jan Kuchař's avatar
commented on merge request !60 "Table: add new methods" at grifart libs / tables

Já jsem překvapený, jak dobrý je ten návrh ChangeSetu. :+1::+1::+1: Ten koncept funguje i když je tu použitý úplně jinak, než jsem původně zamýšlel.

Jan Kuchař's avatar
commented on merge request !60 "Table: add new methods" at grifart libs / tables

Sice edge case, ale existuje. Chci s hodnotou ze smazaného řádku nějak pracovat po smazání.

Jan Kuchař's avatar
commented on merge request !60 "Table: add new methods" at grifart libs / tables

Nerozumím funkci deleteAndGet(), to jako vrátit to co to smazalo, takže na tom již nejde zavolat save()?

Jiří Pudil's avatar
pushed to branch new-table-methods at grifart libs / tables
  • 49f44e9d · add Table::deleteAndGet() for situations where you need to delete a...
Jiří Pudil's avatar
opened merge request !60 "Table: add new methods" at grifart libs / tables