markocupic/calendar-event-booking-bundle: Fehler bei Event Kopie

[tintifax]

Hi,

ich habe nach Update ein Problem beim Kopieren von Events:

Wenn ich für ein neues Event ein altes Event kopiere mit angemeldeten Mitgliedern, dann werden auch die Mitglieder mitkopiert.

Das war vor dem Update nicht so und macht natürlich keinen Sinn, weil ich ja ein neus Event ohne angemeldete Mitglieder haben möchte.

markocupic/calendar-event-booking-bundle: Version 5.6.2

Contao: 4.13.52

Gibt es dafür einen schnellen Workaround oder soll das so sein, weil streng genommen ist es ja eine Kopie ?

Bitte um Tipps

Vielen Dank

Grüße Tintifax

[Spooky]

Man müsste im DCA doNotCopyRecords aktivieren für diese Child Table.

[tintifax]

d.h. in der

tl_calender_events_member.php

könnte ich es so machen für alle Felder (Beispiel lastname) die eingetragen/kopiert werden:

        'lastname'     => [
            'eval'      => [
                'mandatory' => true,
                'maxlength' => 255,
                'tl_class'  => 'w50',
            ],
            'filter'    => true,
            'inputType' => 'text',
            'search'    => true,
            'sorting'   => true,
            'doNotCopyRecords' => true,
            'sql'       => "varchar(255) NOT NULL default ''",
        ], 

[Spooky]

Nein, das ist eine config Einstellung des DCAs (siehe https://docs.contao.org/dev/referenc...fig/#reference).

[tintifax]

in die tl_calendar_events_member.php => 'doNotCopyRecords' => true, eintragen ?

$GLOBALS['TL_DCA']['tl_calendar_events_member'] = [
    // Config
    'config'   => [
        'dataContainer'     => DC_Table::class,
        'ptable'            => 'tl_calendar_events',
        'doNotCopyRecords' => true,
        'enableVersioning'  => true,
        'onsubmit_callback' => [],
        'ondelete_callback' => [],
        'sql'               => [
            'keys' => [
                'id'        => 'primary',
                'email,pid' => 'index',
            ],
        ],
    ], 

[Spooky]

Ja, wenn du's als Applikationsanpassung machst aber nicht so - weil dann löschst du das gesamte DCA . Also zB:

// contao/dca/tl_calendar_events_member.php
$GLOBALS['TL_DCA']['tl_calendar_events_member']['config']['doNotCopyRecords'] = true; 

Könntest du auch als PR hier vorschlagen.

[tintifax]

Ja, wenn du's als Applikationsanpassung machst aber nicht so - weil dann löschst du das gesamte DCA . Also zB:

// contao/dca/tl_calendar_events_member.php
$GLOBALS['TL_DCA']['tl_calendar_events_member']['config']['doNotCopyRecords'] = true; 

Könntest du auch als PR hier vorschlagen.

ugga agga ^^, ja löschen will ich es nicht

na ich werd das mal in einer Test-Installation durchspielen,

Danke mal soweit, github hab ich schon geguckt, denn ich denke, die aktuelle Handhabung kann nicht so gewollt sein. war ja vorher auch nicht so

[tintifax]

Habs nun so gemacht:

/src/Resources/contao/dca/tl_calender_events.php

<?php
$GLOBALS['TL_DCA']['tl_calendar_events']['config']['oncopy_callback'][] = ['MyEventCopy', 'removeEventMembers'];

class MyEventCopy
{
    public function removeEventMembers($newEventId, $dc, $oldEventId = null)
    {
        if (!$newEventId) {
            return;
        }

        // Lösche alle Mitglieder des neuen Events
        \Database::getInstance()->prepare("DELETE FROM tl_calendar_events_member WHERE pid=?")
                                ->execute($newEventId);
    }
}

funktioniert einwandfrei

[Spooky]

Das brauchst du nicht... die DCA Anpassung reicht.

[tintifax]

Das brauchst du nicht... die DCA Anpassung reicht.

Aber Nachteile habe ich ja so auch nicht, oder ?

Dachte, falls ich mehr zum Manipulieren hätte, müßte ich nur noch per Update Befehl hier ran

[Spooky]

Aber Nachteile habe ich ja so auch nicht, oder ?

Naja doch, da werden zuerst alle Records kopiert und dann wieder gelöscht, das ist ja unnötig.

[tintifax]

Naja doch, da werden zuerst alle Records kopiert und dann wieder gelöscht, das ist ja unnötig.

ok das überzeugt mich, Danke