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ána jednotná kategorie vratek účastnických poplatků pro vzdělávačky #2360

Merged
merged 3 commits into from
Oct 8, 2023

Conversation

marekdedic
Copy link
Collaborator

Closes #2317

Obsoletes #2332

@marekdedic
Copy link
Collaborator Author

Hele jediná věc, co moc nechápu, je co se děje tady:

$categories = array_filter($categories, function (ICategory $category) {
return ! in_array($category->getId(), [ICategory::CATEGORY_REFUND_CHILD_ID, ICategory::CATEGORY_REFUND_ADULT_ID]);
});

Tam se z toho fltrují ty kategorie pro tábory, ale ne ta obecná pro vratky u akcí - proč?

@sinacek
Copy link
Member

sinacek commented Oct 8, 2023

Hele jediná věc, co moc nechápu, je co se děje tady:

$categories = array_filter($categories, function (ICategory $category) {
return ! in_array($category->getId(), [ICategory::CATEGORY_REFUND_CHILD_ID, ICategory::CATEGORY_REFUND_ADULT_ID]);
});

Tam se z toho fltrují ty kategorie pro tábory, ale ne ta obecná pro vratky u akcí - proč?

Protože categoryId=20 je k dispozici jen u general a camp má všechny kategorie převzaté ze skautisu od konkrétního tábora.

@marekdedic
Copy link
Collaborator Author

Protože categoryId=20 je k dispozici jen u general a camp má všechny kategorie převzaté ze skautisu od konkrétního tábora.

Ok, takže pro vzdělávačky to možná taky bude potřeba sem přidat, protože ty taky berou kategorie z ISu - přidávám si TODO do #2309

@marekdedic marekdedic merged commit 5e40475 into master Oct 8, 2023
13 checks passed
@marekdedic marekdedic deleted the education-returns-unified branch October 8, 2023 19:11
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.

Vzdělávačky nemají kategorie výdajů pro vratky úč. poplatků
2 participants