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

Pokladní knihy vzdělávaček a jejich kategorie specifické pro rok #2414

Merged
merged 6 commits into from
Oct 26, 2023

Conversation

marekdedic
Copy link
Collaborator

@marekdedic marekdedic commented Oct 25, 2023

fixes H-SKAUTING-CZ-1GT

@sinacek
Copy link
Member

sinacek commented Oct 26, 2023

Poslední rok vzdelavacky si myslim, ze se muze zmenit, kdyz se nestihne vikend nebo se prida, tak preteces do dalsiho roku. Proc nepouzit prvni? Proc je potreba to fixovat na rok?

@marekdedic
Copy link
Collaborator Author

Hele jakmile se ti akce rozprostře do více kalendářních let, tak se musí celé účetnictví rozdělit a řešit mnohem komplikovaněji - viz #2331. To v tomhle PR rozhodně řešit nechci, byť se snažím ty úpravy dělat tak, aby stím v budoucnu mohly být kompatibilní.

Takže jestli použít začátek nebo konec akce je za mně jedno, jakmile ji máš ve více letech, tak máš problém a h.skauting ti (v tuhle chvíli) nepomůže.

@sinacek
Copy link
Member

sinacek commented Oct 26, 2023

Pokud je to jedno, delal bych to dle zacatku, to mi prijde, ze se bude hýbat spis méně často, ne?

@marekdedic marekdedic force-pushed the education-cashbook-years branch from 97156b3 to 68c00a1 Compare October 26, 2023 12:29
@marekdedic
Copy link
Collaborator Author

Ok, updatováno :)

@sinacek
Copy link
Member

sinacek commented Oct 26, 2023

Muzes mi jen jeste vysvetlit, proc u education je potreba drzet ten rok?

@marekdedic
Copy link
Collaborator Author

Myslíš proč je to potřeba v DB?

Já tady v zásadě řeším problém, že pro získání kategorií VzA je potřeba znát rok (viz EducationCategoryRepository). No a jako mohl bych ho tam nějak protáhnout z CategoryRepository jinak, ale vzhledem k tomu, že tuším, že někdy v budoucnu třeba budeme chtít zprovoznit i vzdělávačky ve více letech (a ty budou potřebovat pro každý rok různý cashbook), tak mi přišlo čistší si rovnou v té databázi uložit, pro který rok ten cashbook je, i když zatím více let pro jednu akci nepodporujeme. Tím pádem všechna místa, kde říkám, že se má vzít $event->startDate->year jsou až v presenterech a implementace #2331 pak už do tohohle nebude muset šahat...

Copy link
Member

@sinacek sinacek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, napsal bych to do te entity, proc potrebujeme rok a tim bych to zde uzavrel

@marekdedic marekdedic merged commit 7a697b8 into master Oct 26, 2023
13 checks passed
@marekdedic marekdedic deleted the education-cashbook-years branch October 26, 2023 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Education PR - Bugfix PR s opravou chyby
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants