Update Contao 3 -> 4 : Modul wird nicht berücksichtigt

[Thomas Pyrin]

ja:

php public/contao-manager.php composer install

auch gerade nochmal, aber es bleibt beim

CRITICAL  [console] Error thrown while running command "debug:container AppExampleController". Message: "No services found that match "AppExampleController"." ["exception" => Symfony\Component\Console\Exception\InvalidArgumentException^ { …},"command" => "debug:container AppExampleController","message" => "No services found that match "AppExampleController"."]

                                                        
  No services found that match "AppExampleController". 

[Spooky]

Ohne deine restliche Installation zu kennen können wir jetzt nur Schüsses ins blaue machen. Hast du evt. eigene Autoloading Angaben in der composer.json der Contao Installation? Eine eigene config/services.yaml?

[Thomas Pyrin]

Nein, da hab ich gar nichts zusätzlich gemacht. Wie ich das verstanden habe, sollte das auch doch Autoloading damit funktionieren, oder?
Ich würde halt ungern in den Core-Dateien Änderungen machen und die ganze Installation nicht mehr updatefähig zu machen.

[Thomas Pyrin]

Im Notfall versuche ich das alles über ein hartes Ändern einer anderen Erweiterung (calendar_extented_bundle) zu lösen, dann kann diese halt nicht aktualisiert werden

[Spooky]

Wie ich das verstanden habe, sollte das auch doch Autoloading damit funktionieren, oder?

Wenn du in deiner composer.json kein eigenes Autoloading definierst hast, dann konfiguriert das contao/manager-bundle automatisch eins für dich für den App\ Namespace. Auch die Services werden automatisch geladen, ohne eigene services.yaml. Es sei denn in deinem Code in src/ befindet sich ein Fehler, dann werden die Services unter Umständen nicht automatisch geladen.

[Thomas Pyrin]

Wenn du in deiner composer.json kein eigenes Autoloading definierst hast, dann konfiguriert das contao/manager-bundle automatisch eins für dich für den App\ Namespace. Auch die Services werden automatisch geladen, ohne eigene services.yaml. Es sei denn in deinem Code in src/ befindet sich ein Fehler, dann werden die Services unter Umständen nicht automatisch geladen.

hab gerade nochmal das Script von

https://docs.contao.org/dev/guides/f...existing-class

in die Datei src/Controller/FrontendModule/AppExampleController.php

geschrieben.

Nochmal install .... die composer.json sieht so aus:

{
    "type": "project",
    "require": {
        "contao/calendar-bundle": "4.13.*",
        "contao/comments-bundle": "4.13.*",
        "contao/conflicts": "*@dev",
        "contao/faq-bundle": "4.13.*",
        "contao/listing-bundle": "4.13.*",
        "contao/manager-bundle": "4.13.*",
        "contao/news-bundle": "4.13.*",
        "contao/newsletter-bundle": "4.13.*",
        "hofff/contao-calendarfield": "^4.0",
        "kmielke/calendar-extended-bundle": "^1.0",
        "madeyourday/contao-rocksolid-antispam": "^2.1",
        "madeyourday/contao-rocksolid-columns": "^2.1",
        "madeyourday/contao-rocksolid-custom-elements": "^2.4",
        "madeyourday/contao-rocksolid-icon-picker": "^2.1",
        "madeyourday/contao-rocksolid-mega-menu": "^2.2",
        "madeyourday/contao-rocksolid-slider": "^2.2",
        "madeyourday/contao-rocksolid-theme-assistant": "^2.1",
        "marebe1602/mae_event_categories": "^1.5",
        "terminal42/notification_center": "1.6.14"
    },
    "extra": {
        "public-dir": "public",
        "contao-component-dir": "assets"
    },
    "scripts": {
        "post-install-cmd": [
            "@php vendor/bin/contao-setup"
        ],
        "post-update-cmd": [
            "@php vendor/bin/contao-setup"
        ]
    }
} 

aber leider keine Änderung

[Spooky]

Ah, für diesen Fall musst du die Services tatsächlich selbst registrieren. Siehe die Notiz darunter:

Since we are extending classes from the legacy Contao framework here, these controllers will not be automatically registered as a service by Contao. Therefore you will need to specifically register these controllers as services in your own config/services.yaml. See this article for more information.

[Thomas Pyrin]

ok, versuche ich, was ist bei mir der Unterschied, dass es nicht automatisch klappt?

[Spooky]

Das, was in der Notiz steht ist der Unterschied:

Since we are extending classes from the legacy Contao framework here, these controllers will not be automatically registered as a service by Contao.

[Thomas Pyrin]

Ok, das hat funktioniert. Danke sehr.
Ich teste das mal, wie weit ich damit komme, das ursprüngliche Problem mit den Events zu lösen.

Ich würde dann wie beim AppExampleController all die Klassen zu überschreiben oder zu erweitern, ist das der Ansatz?

[Spooky]

Ich würde dann wie beim AppExampleController all die Klassen zu überschreiben oder zu erweitern, ist das der Ansatz?

Für Legacy Content Elements und Frontend Module kannst du das so machen, ja.

[Thomas Pyrin]

Gibt es für diese "Art" der Anpassungen noch weitere ausführliche Tutorials usw?
Z.B. wo man dem Modul dann auch einen Namen geben kann.

Wie man Ausgaben manipuliert usw?

[Spooky]

Z.B. wo man dem Modul dann auch einen Namen geben kann.

Das wäre hier dokumentiert: https://docs.contao.org/dev/framewor.../#translations

Wie man Ausgaben manipuliert usw?

Was meinst du damit?

[Thomas Pyrin]

Mh, das sind noch so viele Fragen ....

Wenn ich das FE Modul so überschreibe und das neue Modul "app_example" wähle, verliert man im BE jegliche Einstellungen am Modul.
Ich hatte gehofft, dass es dahingehend eine Erweiterung ist, die von dem eigentlichen Modul ausgeht.

Und wenn ich eine public function() übernehmen und anpassen möchte, dann geht das so auch nicht, ich hatte mir vorgestellt, hier dann z.B. eine public function generate() aus der Klasse zu nehmen und in das neue FE Modul zu kopieren und da dann anpassungen zu machen.

das habe ich gerade schon gelöst, das klappt.

[Thomas Pyrin]

Bzw. wie überschreibe ich eine Klasse, die kein FE Modul ist?

(es geht Schritt für Schritt .... danke für die Hilfe - wenn ich das an diesem Beispiel hab, dann hoffe ich in Zukunft besser klar zu kommen)


Thomas

[Spooky]

Bzw. wie überschreibe ich eine Klasse, die kein FE Modul ist?

"Klassen überschreiben" geht nicht, wie schon mehrmals im Thread hier erwähnt. Was genau möchtest du erreichen?

[Thomas Pyrin]

Momentan nutze ich die Erweiterung calendar_extended_bundle für die Ausgabe der Termine.

Hier muss das Holen der Events angepasst werden -> ich benötige für einige Listen eine Gruppierung der Events nach dem Title.
So gesehen könnte man auch eine Option im BE im Modul der Event-Liste hinzufügen: "Gruppieren nach Title" und dann greift man im Hintergrund auf eine
Methode zu, die sich entsprechend die Events zieht.

Die Erweiterung calendar_extended_bundle verwendet dafür grundlegend die Funktion "getAllEvents()" aus der Event Klasse von Contao und überschreibt die mit einer eigenen Klasse / Funktion:

class EventsExt extends Events {

....

    protected function getAllEvents($arrCalendars, $intStart, $intEnd, $blnFeatured = null)
    {
        return $this->getAllEventsExt($arrCalendars, $intStart, $intEnd, array(null, true), $blnFeatured);
    }


    protected function getAllEventsExt($arrCalendars, $intStart, $intEnd, $arrParam = null, $blnFeatured = null) {

    }

....

} 

Das soll auch alles so bleiben, im BE für das Module EventListe sollen die Einstellungen für die EventListe gemacht werden.
Aber die Funktion getAllEventsExt() muss angepasst werden, dass bei Bedarf die Events gruppiert nach Title geholt werden.


Oder als Idee: ich schaue mir die Erweiterung Kmielke\CalendarExtendedBundle an und erweiteres diese nach dem Muster, wie diese Erweiterung die Contao-Events erweitert hat?

[Spooky]

Wenn ich das FE Modul so überschreibe und das neue Modul "app_example" wähle, verliert man im BE jegliche Einstellungen am Modul.

Für dein neues Modul musst du auch noch eine eigene Palette anlegen (https://docs.contao.org/dev/framewor...dules/#example).

Ich hatte gehofft, dass es dahingehend eine Erweiterung ist, die von dem eigentlichen Modul ausgeht.

Alternativ könntest du das Legacy Modul, von dem du erweiterst, auch komplett ersetzen, also sodass du dann keine Auswahlmöglichkeit mehr zwischen Original-Modul und deinem neuen Modul hast. Dazu gibst du als Type und Category bei deinem Modul den selben Type und Category an, wie das Modul, dass du ersetzen willst. Bei der Nachrichtenliste wäre das zB:

#[AsFrontendModule(type: 'newslist', category: 'news')] 

[Thomas Pyrin]

Momentan nutze ich die Erweiterung calendar_extended_bundle für die Ausgabe der Termine.

Hier muss das Holen der Events angepasst werden -> ich benötige für einige Listen eine Gruppierung der Events nach dem Title.
So gesehen könnte man auch eine Option im BE im Modul der Event-Liste hinzufügen: "Gruppieren nach Title" und dann greift man im Hintergrund auf eine
Methode zu, die sich entsprechend die Events zieht.

Die Erweiterung calendar_extended_bundle verwendet dafür grundlegend die Funktion "getAllEvents()" aus der Event Klasse von Contao und überschreibt die mit einer eigenen Klasse / Funktion:

class EventsExt extends Events {

....

    protected function getAllEvents($arrCalendars, $intStart, $intEnd, $blnFeatured = null)
    {
        return $this->getAllEventsExt($arrCalendars, $intStart, $intEnd, array(null, true), $blnFeatured);
    }


    protected function getAllEventsExt($arrCalendars, $intStart, $intEnd, $arrParam = null, $blnFeatured = null) {

    }

....

} 

Das soll auch alles so bleiben, im BE für das Module EventListe sollen die Einstellungen für die EventListe gemacht werden.
Aber die Funktion getAllEventsExt() muss angepasst werden, dass bei Bedarf die Events gruppiert nach Title geholt werden.


Oder als Idee: ich schaue mir die Erweiterung Kmielke\CalendarExtendedBundle an und erweiteres diese nach dem Muster, wie diese Erweiterung die Contao-Events erweitert hat?

----------

#[AsFrontendModule(type: 'newslist', category: 'news')]

Bei den Events müsste das sein: #[AsFrontendModule(type: 'eventlist', category: 'events')]

aber dann wird das Modul nicht mehr erkannt

[Spooky]

Poste deinen Code.

[Thomas Pyrin]

<?php

// src/Controller/FrontendModule/AppExampleController.php
namespace App\Controller\FrontendModule;

use Contao\CoreBundle\DependencyInjection\Attribute\AsFrontendModule;
use Contao\System;
use Contao\ModuleModel;
use Kmielke\CalendarExtendedBundle\EventsExt;
use Kmielke\CalendarExtendedBundle\ModuleEventlist;
use Kmielke\CalendarExtendedBundle\CalendarLeadsModel;

use Symfony\Component\HttpFoundation\Response;

#[AsFrontendModule(type: 'eventlist', category: 'events')]

class AppExampleController extends ModuleEventlist
{

    public function __construct() {}


    public function generate()
    {
        if (TL_MODE == 'BE') {
            /** @var \BackendTemplate|object $objTemplate */
            $objTemplate = new \BackendTemplate('be_wildcard');

            $objTemplate->wildcard = '### ' . utf8_strtoupper($GLOBALS['TL_LANG']['FMD']['eventlist'][0]) . ' ###';
            $objTemplate->title = $this->headline;
            $objTemplate->id = $this->id;
            $objTemplate->link = $this->name;
            $objTemplate->href = 'contao/main.php?do=themes&amp;table=tl_module&amp;act=edit&amp;id=' . $this->id;

            return $objTemplate->parse();
        }



        ..........

die public function generate() überschreibt schon die, aus der Erweiterung calendar_extended_bundle

Aber mit

#[AsFrontendModule(type: 'eventlist', category: 'events')] 

wird das Module im Backend nicht mehr angezeigt.

Oder als Idee: ich schaue mir die Erweiterung Kmielke\CalendarExtendedBundle an und erweiteres diese nach dem Muster, wie diese Erweiterung die Contao-Events erweitert hat?

Ist das vielleicht dann doch sinnvoller?

[Spooky]

Aber mit

#[AsFrontendModule(type: 'eventlist', category: 'events')] 

wird das Module im Backend nicht mehr angezeigt.

Ja, damit ersetzt du ja das bereits bestehende Eventlist modul.

[Thomas Pyrin]

Stimmt, mein Fehler, das war zu voreilig.

Aber nur mit meinem FE Modul kann ich noch nicht die Methode für das Holen aus der DB anpassen, richtig?

Wie wäre dann der sinnvollste Weg Deiner Meinung nach? Ich würde es halt gerne verstehen und nachvollziehen

[Spooky]

Aber nur mit meinem FE Modul kann ich noch nicht die Methode für das Holen aus der DB anpassen, richtig?

Welche Methode möchtest du anpassen bzw. was genau möchtest du anpassen?

[Thomas Pyrin]

In der Erweiterung calendar_extended_bundle gibt eine EventsExt.php, die die Contao Klasse Events.php überschreibt.

class EventsExt extends Events { 

In dieser gibt es einen Funktionsaufruf:

public static function findCurrentByPid($intPid, $intStart, $intEnd, array $arrOptions = array()) 

Diese Funktion greift auf die Klasse

class CalendarEventsModel extends \Model 

in der Datei CalendarEventsModelExt.php aus der Erweiterung calendar_extended_bundle zu.
Hier wurde also das Contao-Model schon überschrieben.

Die Funktion findCurrentByPid() muss optional eine Option mitbekommen können:

$arrOptions['group'] = "$t.title"; 

und macht dann letztendlich

return static::findBy($arrColumns, $intPid, $arrOptions); 

Damit sollten die EventListen optional auch gruppiert nach Title aus der DB geholt werden können.
Eine generelle Umschreibung ist nicht sinnvoll, da es auch kalendarische Listen gibt, wo nicht gruppiert nach Namen werden soll.
(Wo die Optionen für die EventListen aus dem BE verarbeitet werden, muss ich noch rausfinden)

------------------------------------

Dann gibt es noch das Module "ModuleEventReader.php", was ebenfalls von der Erweitwerung calendar_extended_bundle überschrieben wird.
Die Detailansicht bringt automatisch auch die Anzahl von Wiederholungen mit über findPublishedByParentAndIdOrAlias() - eine neue Funktion, die von der Erweitwerung calendar_extended_bundle aus der CalendarEventsModelExt.php kommt.

Ich selber erweitere dann ModuleEventReader.php um eine weitere Suche findByTitle() und führe die Wiederholungen zusammen, damit die Detailansicht eines Events einerseits die eigenen Wiederholungen ausgibt und die von namensgleichen Events.


------------------------------------


Also, die Theorie ist klar und rein theoretisch kann ich das sicher auch alles in der Erweitwerung calendar_extended_bundle ändern, weil die bestimmt nicht mehr erweitert wird.

Die CalendarEventsModel.php bzw. CalendarEventsModelExt.php kann ich dann offenbar nur Anpassen, wenn ich ganz eine eigene Erweiterung erstelle?
Kann ich da die calendar_extended_bundle kopieren und die Namespaces und Klassen-Namen einfach ändern?

[Spooky]

Hier wurde also das Contao-Model schon überschrieben.

Nein, die Extension erbt einfach nur von dieser Klasse und ruft dann diese statische Funktion selbst auf.

[Thomas Pyrin]

Ich brauch also eine eigene Extension, die sich an der calendar_extended_bundle orientiert und von da wieder erbt?

[Spooky]

Ob du eine "echte Extension" brauchst oder nicht hängt davon ab, was du machen willst - also ob deine Anpassungen bspw. in mehrere Contao Applikationen wieder verwendet werden können sollen. Wenn nicht reichen Applikationsanpassungen.

Zu deinem Vorhaben bzgl. Gruppiertung/Sortierung nach Titel: ich denke das musst du sowieso per PHP machen, nicht über das Model (also nicht über die Datenbank). Denn in der Datenbank befinden sich ja nicht alle Termine eines Events. Die werden erst dynamisch per PHP erzeugt ggf. (bei Verwendung von Wiederholungen und/oder bei den komplexeren Regeln der calendar_extended Extension).

[Thomas Pyrin]

PHP-seitig würde sicher auch gehen ...

$arrAllEvents = $this->getAllEventsExt($this->cal_calendar, $strBegin, $strEnd, array($this->cal_holiday, $showRecurrences));

das ist allerdings ein riesiges Array und auf 4. Ebene in diesem multidimensionalem Array müsste man das Feld 'title' uniquen .... das kann u.U. langsamer sein, als die Manipulation der Datenbank-Abfrage, gerade weil auch pro Event auch eine grosse Menge an Wiederholungen vorhanden sind.

(aber ich schau mal, wie ich das $arrAllEvents auf doppelte 'title' in der vierten Ebene überprüfen kann.

[Spooky]

PHP-seitig würde sicher auch gehen ...

$arrAllEvents = $this->getAllEventsExt($this->cal_calendar, $strBegin, $strEnd, array($this->cal_holiday, $showRecurrences));

das ist allerdings ein riesiges Array und auf 4. Ebene in diesem multidimensionalem Array müsste man das Feld 'title' uniquen .... das kann u.U. langsamer sein, als die Manipulation der Datenbank-Abfrage, gerade weil auch pro Event auch eine grosse Menge an Wiederholungen vorhanden sind.

Wie gesagt - du bekommst ja nicht alle Events rein über die Datenbank. Andere Extensions spielen ja auch zusätzliche Events (bzw. Event-Termine) über den getAllEvents Hook rein.

[Thomas Pyrin]

Wie gesagt - du bekommst ja nicht alle Events rein über die Datenbank. Andere Extensions spielen ja auch zusätzliche Events (bzw. Event-Termine) über den getAllEvents Hook rein.

Muss ich nochmal drüber schauen, die Eventliste (so sieht es für mich gerade aus) bekommt alle Events aus der Funktion getAllEventsExt (das ist die von calendar_extended_bundle) und die wiederum greift auf die getAllEvents von Contao zurück.

Eine Frage für den Moment habe ich noch:

Aber mit

#[AsFrontendModule(type: 'eventlist', category: 'events')] 

wird das Module im Backend nicht mehr angezeigt.

Ginge es über eine kleine Abwandlung auch, dass das Module im Backend angezeigt wird und alle Eigenschaften vom type "eventlist" erhält, an dem ich dann seperat ändern kann?

Vielen Dank auch nochmals, es hat auf jeden Fall etwas Licht für mich gebracht.

Thomas

[Spooky]

Ginge es über eine kleine Abwandlung auch, dass das Module im Backend angezeigt wird und alle Eigenschaften vom type "eventlist" erhält, an dem ich dann seperat ändern kann?

Ja, du musst dann einfach einen eigenen Typ festlegen - und für deinen Typ dann auch eine Palette definieren (dazu kannst du die Original Palette kopieren).