search

markocupic / calendar-event-booking-bundle

Contao 4 Extension
13 stars 8 forks source link

Begleitpersonen für maximale Teilnehmeranzahl berücksichtigen #10

Closed ghost closed 5 years ago

ghost commented 5 years ago

Hi @markocupic

Diese Erweiterung ist für meinen Anwendungsfall fast perfekt!

Nur eines ist mir aufgefallen: Bei der maximalen Teilnehmeranzahl werden die Begleitpersonen nicht berücksichtigt, oder? Also, im Extremfall, kann z.B. die max. Teilnehmeranzahl 10 sein, und zehn Anmeldungen mit z.B. je zwei Begleitpersonen wären möglich, oder? Zumindest konnte ich es machen.

Für meinen Anwendungsfall sollten die Begleitpersonen für die maximale Teilnehmeranzahl auch berücksichtigt werden. Irgendeine Chance, dies umzusetzen?

Danke dir für die Rückmeldung!

markocupic commented 5 years ago

Hallo DanEi75

Das Modul habe ich damals für einen Kunden erstellt. Dieser wollte das eben nicht so.

Ich könnte dir das schon anpassen, aber würde dafür etwas Entschädigung verlangen.

Sorry, ich muss mich da etwas abgrenzen. Anfragen wie diese kommen bei 90 Erweiterungen auf github fast täglich.

Liebe Grüsse

 

Marko  

Marko Cupic Surenweidstr. 2 CH-6208 Oberkirch

http://4ae-racing-team.ch http://kletterkader.com

   

Gesendet: Dienstag, 05. Februar 2019 um 17:38 Uhr Von: "DanEi75" notifications@github.com An: markocupic/calendar-event-booking-bundle calendar-event-booking-bundle@noreply.github.com Cc: "Marko Cupic" m.cupic@gmx.ch, Mention mention@noreply.github.com Betreff: [markocupic/calendar-event-booking-bundle] Begleitpersonen für maximale Teilnehmeranzahl berücksichtigen (#10)

Hi @markocupic

Diese Erweiterung ist für meinen Anwendungsfall fast perfekt!

Nur eines ist mir aufgefallen: Bei der maximalen Teilnehmeranzahl werden die Begleitpersonen nicht berücksichtigt, oder? Also, im Extremfall, kann z.B. die max. Teilnehmeranzahl 10 sein, und zehn Anmeldungen mit z.B. je zwei Begleitpersonen wären möglich, oder? Zumindest konnte ich es machen.

Für meinen Anwendungsfall sollten die Begleitpersonen für die maximale Teilnehmeranzahl auch berücksichtigt werden. Irgendeine Chance, dies umzusetzen?

Danke dir für die Rückmeldung!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

ghost commented 5 years ago

Hallo @markocupic

Kein Thema! Ich versteh dich völlig und Arbeit soll vergütet werden ;-)!

Ich habe mir den Quellcode etwas angeschaut und ich meine, das lässt sich prima als Option umsetzen. Ich möchte zuerst noch etwas selber basteln und komme dann gerne auf dich zu. Eventuell nehme ich dir etwas Arbeit ab, damit du es in den Core einbauen kannst, wenn du möchtest.

LG

markocupic commented 5 years ago

Alles klar. Nur zu. Am besten als Option in tl_calendar_events. LG Marko

ghost commented 5 years ago

Hallo Marko

So, bin erst gestern dazu gekommen. Ich habe das mal wie folgt eingebaut und getestet. Ich meine, das klappt ziemlich gut und korrekt. Je nach Optionswahl wird die Teilnehmeranzahl in der Liste verändert und bei der FE-Buchung berücksichtigt. Aber ich bin jetzt definitiv nicht der Profi-Entwickler, deshalb auf eigene Gefahr, wenn du es in den Core einbetten möchtest ;-).

Ich hab das auf Github jetzt noch nie so dokumentiert und ich möchte jetzt auch nicht einen riesen Kommentar generieren. Hoffe, es passt, wenn ich dir die angepassten Dateien als zip-Datei einfüge.

countEscorts.zip

Hier für jede Datei noch die Anpassungen:

LG Dani