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

Přidáni instruktoři do seznamu lidí u vzdělávaček #2330

Closed
wants to merge 20 commits into from

Conversation

marekdedic
Copy link
Collaborator

  • Přidán seznam instruktorů do záložky lidí pro VzA
  • Přidána možnost načíst HPD pro účastnické poplatky od instruktorů

Closes #2318. Závisí na #2326

@@ -220,9 +220,11 @@ private function processParticipants(array $participantsSis, Event $event): arra
$participants[$p->ID] = ParticipantFactory::create($p, $payment);
}

/*
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Tady moc nevím, co s tím - protože je obdobný kód i v InstrucotRepository a oba šahají do stejné tabulky, musel jsem tohle zakomentovat - když jsem to neudělal, tak si ParticipantRepository a InstructorRepository vzájemně mazaly platby.

Asi nejčistší řešení by bylo instruktorské platby v databázi mít v jiné tabulce, ale to už je dost výrazná úprava a moc se mi do toho nechce. A vlastně nevím, k čemu tady tenhle kód je, jestli je vůbec potřeba. Napadá mě něco jako odebírání úćastníků, kteří mají vyplněné poplatky, ale to by se podle mě mělo spíš smazat při tom odebrání - tady se to maže až při dalším načtení seznamu účastníků...

Copy link
Member

Choose a reason for hiding this comment

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

kdyz nekdo smaze ucastnika pres skautis, tak nam by tu zustala vyset platba bez ucastnika. A tohle to je mechanismus jak to smazat.

@marekdedic marekdedic force-pushed the education-instructors branch from 653f2bf to d574bb7 Compare October 15, 2023 15:08
@marekdedic
Copy link
Collaborator Author

Ne, tohle je moc velká prasárna, celá ta abstrakce s ParticipatingPerson je špatně a pokud se to má udělat, tak odděleně.

@marekdedic marekdedic closed this Oct 15, 2023
@marekdedic marekdedic deleted the education-instructors branch October 15, 2023 20:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Přidání instruktorů do seznamu lidí u VzA
2 participants