grifart libs issueshttps://gitlab.grifart.cz/groups/grifart/-/issues2018-08-03T13:28:33Zhttps://gitlab.grifart.cz/grifart/scaffolder/-/issues/2Do not regenerate all the time all file as it is not needed2018-08-03T13:28:33ZJan Kuchařhonza.kuchar@grifart.czDo not regenerate all the time all file as it is not needed- checksum in header?- checksum in header?https://gitlab.grifart.cz/grifart/stateful/-/issues/3Support for anonymous classes2021-11-16T08:20:00ZJan Kuchařhonza.kuchar@grifart.czSupport for anonymous classesDnes jsme na to znovu narazili s @dkurowski . A přišel jsem na několik pozorování, které by to celé mohly vyřešit:
- nedává smysl serializovat anonymní třídu samotnou; ani taková věc nejde zaregistrovat do mapperu (s tímto není problém)...Dnes jsme na to znovu narazili s @dkurowski . A přišel jsem na několik pozorování, které by to celé mohly vyřešit:
- nedává smysl serializovat anonymní třídu samotnou; ani taková věc nejde zaregistrovat do mapperu (s tímto není problém)
- dává smysl podporovat jen serializaci anonymních tříd, které dědí z nějaké pojmenované třídy. Pokud bude pojmenovaná třída podporovat `_fromState`, který místo `createNewWithoutConstructor` vytvoří novou instanci třídy dle potřeb a obsah obnoví ze state, problém je vyřešen.
- serializační funkce může být rozdělena i mezi jednotlivé potomky, jen musí být zaručeno, že jejich rodič (který bude mít _fromState) bude umět všechny stavy deserializovat
cc @jpudil, @kerlebachovahttps://gitlab.grifart.cz/grifart/stateful/-/issues/4Add tests for !14, !162020-04-20T14:12:36ZJan Kuchařhonza.kuchar@grifart.czAdd tests for !14, !16https://gitlab.grifart.cz/grifart/tables/-/issues/6Support for explicit connection factory2021-12-29T13:43:34ZJan Kuchařhonza.kuchar@grifart.czSupport for explicit connection factoryfor cases like this, where code depends on the same connection to be used in code above AND in tables
![image](/uploads/bcca91e7d095161aaae45f5951c9e471/image.png)for cases like this, where code depends on the same connection to be used in code above AND in tables
![image](/uploads/bcca91e7d095161aaae45f5951c9e471/image.png)https://gitlab.grifart.cz/grifart/scaffolder/-/issues/12Union types & list of is not possible to compose with nullable type2020-09-01T15:12:42ZJan Kuchařhonza.kuchar@grifart.czUnion types & list of is not possible to compose with nullable typehttps://gitlab.grifart.cz/grifart/stateful/-/issues/6Readme: elaborate mappers2020-10-06T12:32:48ZDaniel Kurowskidaniel.kurowski@grifart.czReadme: elaborate mappersCurrently, CampApp config is as crazy as:
![image](/uploads/55fca0c21785735fb0bc37961eaced7e/image.png)
But there is almost no word about how to use these `*Mapper` things even they are part of this library.Currently, CampApp config is as crazy as:
![image](/uploads/55fca0c21785735fb0bc37961eaced7e/image.png)
But there is almost no word about how to use these `*Mapper` things even they are part of this library.https://gitlab.grifart.cz/grifart/stateful/-/issues/8Generický State2020-12-01T15:17:00ZJiří Pudilme@jiripudil.czGenerický StateThe following discussion from !20 should be addressed:
- [ ] @jkuchar1 started a [discussion](https://gitlab.grifart.cz/grifart-internal/stateful/-/merge_requests/20#note_74913):
> Možná raději genericky?
@jpudil replied:
...The following discussion from !20 should be addressed:
- [ ] @jkuchar1 started a [discussion](https://gitlab.grifart.cz/grifart-internal/stateful/-/merge_requests/20#note_74913):
> Možná raději genericky?
@jpudil replied:
> U interního Payloadu v tom nevidím přínos; když už generika, tak u `State`, ale to nebude tak jednoduché, klidně bych z toho udělal samostatné issuehttps://gitlab.grifart.cz/grifart/stateful/-/issues/10MatchMapper: throw an exception on abstract class or interface2020-10-19T12:34:25ZDaniel Kurowskidaniel.kurowski@grifart.czMatchMapper: throw an exception on abstract class or interfaceWhen I use `VersionIdentifier::class` which is an abstract class, mapper does not tell me I should not do it and I have a bug as inheritors of such class do not get serialized.When I use `VersionIdentifier::class` which is an abstract class, mapper does not tell me I should not do it and I have a bug as inheritors of such class do not get serialized.https://gitlab.grifart.cz/grifart/tables/-/issues/11Check type's default value in addition to column's default?2021-01-11T16:46:44ZJiří Pudilme@jiripudil.czCheck type's default value in addition to column's default?Follow-up from https://gitlab.grifart.cz/grifart/tables/-/merge_requests/8#note_77803Follow-up from https://gitlab.grifart.cz/grifart/tables/-/merge_requests/8#note_77803https://gitlab.grifart.cz/grifart/scaffolder/-/issues/21Nullable values & named constructors2021-07-16T08:42:47ZJan Kuchařhonza.kuchar@grifart.czNullable values & named constructorse.g.
```php
PrefilledValues::from(
emailAddress: null,
name: null,
language: 'cs',
gender: null,
);
```
Jde o to, aby bylo možné vyplnit jen některé parametry. Tj. jedná se o to, že chybí `null` parametrů výchozí hodnota `= nu...e.g.
```php
PrefilledValues::from(
emailAddress: null,
name: null,
language: 'cs',
gender: null,
);
```
Jde o to, aby bylo možné vyplnit jen některé parametry. Tj. jedná se o to, že chybí `null` parametrů výchozí hodnota `= null`, takže je nelze přeskočit.
Navrhuji `MakeNullableParametersOptionalDecorator($methodName)`.
---------------
The following discussion from CampApp/CampApp!1023 should be addressed:
- [ ] @jkuchar1 started a [discussion](https://gitlab.grifart.cz/CampApp/CampApp/-/merge_requests/1023#note_82362): (+5 comments)
> Toto nechápu, vždyť jsou nullable již každý parametr samostatně, takže by měly být nullable i konstruktor parametry. Ne?https://gitlab.grifart.cz/grifart/tables/-/issues/14API pro kopírování?2023-06-02T16:08:57ZJiří Pudilme@jiripudil.czAPI pro kopírování?Viz https://gitlab.grifart.cz/ivy/server/-/merge_requests/338#note_82517, zakládám issue, ať to v ivy můžu zavřít, aniž by to zapadlo.
@jkuchar1:
> Tady na ty kopírování mezi tabulkami by asi dávalo smysl časem něco přidat do tables, c...Viz https://gitlab.grifart.cz/ivy/server/-/merge_requests/338#note_82517, zakládám issue, ať to v ivy můžu zavřít, aniž by to zapadlo.
@jkuchar1:
> Tady na ty kopírování mezi tabulkami by asi dávalo smysl časem něco přidat do tables, co myslíš? :-) Že by člověk jen nahodil co na co a WHERE a hotovo. I když přidaná hodnota, není zase tak velká. Chtělo by to udělat nějak šikovně tak, aby to vyřešil ten typový problém, že třeba chybí povinný sloupec.
@jpudil
> Hm, to bude obtížné navrhnout dostatečně obecně - kupříkladu při publikaci se vkládá naopak než tady a přidává se tam napevno versionNumber právě publikované verze.https://gitlab.grifart.cz/grifart/scaffolder/-/issues/24Docs: add ref to grifart/tables once it is public2021-10-05T08:57:18ZDaniel Kurowskidaniel.kurowski@grifart.czDocs: add ref to grifart/tables once it is publicAs an example of advanced scaffoldingAs an example of advanced scaffoldinghttps://gitlab.grifart.cz/grifart/stateful/-/issues/13Missing exceptions annotation on serialization (toPayload)2021-11-15T10:23:45ZDaniel Kurowskidaniel.kurowski@grifart.czMissing exceptions annotation on serialization (toPayload)There are plenty exceptions that can be thrown but none are annotated.There are plenty exceptions that can be thrown but none are annotated.https://gitlab.grifart.cz/grifart/scaffolder/-/issues/25Release 1.0.0-beta12022-08-30T09:55:36ZJan Kuchařhonza.kuchar@grifart.czRelease 1.0.0-beta1Jiří Pudilme@jiripudil.czJiří Pudilme@jiripudil.czhttps://gitlab.grifart.cz/grifart/tables/-/issues/19Better error handling2021-12-29T13:48:50ZJan Kuchařhonza.kuchar@grifart.czBetter error handlingPokrýt možnost chytat výjimky pro:
- checks
- (unique) constraints
- domain types
Nápad: Co kdybychom tyto výjimky anotovali je nu tabulek, kde tyto chyby opravdu mohou nastat?Pokrýt možnost chytat výjimky pro:
- checks
- (unique) constraints
- domain types
Nápad: Co kdybychom tyto výjimky anotovali je nu tabulek, kde tyto chyby opravdu mohou nastat?https://gitlab.grifart.cz/grifart/stateful/-/issues/14State#getClassName() should return class-string instead of string2022-03-17T16:35:47ZDaniel Kurowskidaniel.kurowski@grifart.czState#getClassName() should return class-string instead of stringhttps://gitlab.grifart.cz/grifart/tables/-/issues/23Generating read models for arbitrary queries / views2023-01-16T09:54:57ZJan Kuchařhonza.kuchar@grifart.czGenerating read models for arbitrary queries / viewsPostgreSQL database is very powerful. Why not to use full power of SQL queries?
...but nodody wants to map each time all columns. We want to get proper data types, right from database...
Let's say we have complex SQL read query. What i...PostgreSQL database is very powerful. Why not to use full power of SQL queries?
...but nodody wants to map each time all columns. We want to get proper data types, right from database...
Let's say we have complex SQL read query. What is the missing parts is strictly typed representation of row of that query and some kind of data mapper.
----
Stretch goal would be to make support also for parameters. This would make `.sql` files more powerful ten using views.
----
Current workaround could be to build view from query we want to read from and continue with standard table generation procedure.https://gitlab.grifart.cz/grifart/scaffolder/-/issues/28ConstantStringType2022-06-22T11:55:08ZJan Kuchařhonza.kuchar@grifart.czConstantStringTypeUse-case: https://gitlab.grifart.cz/CampApp/CampApp/-/merge_requests/1194#note_93313Use-case: https://gitlab.grifart.cz/CampApp/CampApp/-/merge_requests/1194#note_93313Jiří Pudilme@jiripudil.czJiří Pudilme@jiripudil.czhttps://gitlab.grifart.cz/grifart/tables/-/issues/24Table::save() does not expose GivenSearchCriteriaHaveNotMatchedAnyRows except...2024-03-21T15:24:00ZJan Kuchařhonza.kuchar@grifart.czTable::save() does not expose GivenSearchCriteriaHaveNotMatchedAnyRows exception in generated classhttps://gitlab.grifart.cz/grifart/tables/-/issues/27TypeSystem: There should be explicitly written that composite/array types acc...2022-11-11T18:27:29ZJan Kuchařhonza.kuchar@grifart.czTypeSystem: There should be explicitly written that composite/array types accepts NULLs everywhereThe following discussion from !40 should be addressed:
- [ ] @jkuchar1, @jpudil started a [discussion](https://gitlab.grifart.cz/grifart/tables/-/merge_requests/40#note_97018): (+1 comment)
> There is implicit support for nullabl...The following discussion from !40 should be addressed:
- [ ] @jkuchar1, @jpudil started a [discussion](https://gitlab.grifart.cz/grifart/tables/-/merge_requests/40#note_97018): (+1 comment)
> There is implicit support for nullable types, shouldn't it be explicit instead?