1up-lab / contao-full-background-images

Full background images extension for the Contao Open Source CMS
MIT License
4 stars 3 forks source link

Contao 5 compatibility #38

Closed rustykowski closed 5 months ago

rustykowski commented 5 months ago

Ich würde deine Erweiterung auch sehr gerne zusammen mit Contao 5 verwenden. Die composer.json verbietet das allerdings zur Zeit. Wie hoch wäre der Aufwand dafür?

bytehead commented 5 months ago

Ich habe dev-main für Contao 5 freigegeben, kannst du damit mal testen?

rustykowski commented 5 months ago

Danke! Kann sein, dass ich es erst morgen schaffe …

bytehead commented 5 months ago

Kein Stress 🙃

rustykowski commented 5 months ago

Diesen Fehler erhalte ich in Contao 5.3.7 und PHP 8.3:

request.CRITICAL: Uncaught PHP Exception Error: "Undefined constant "Oneup\ContaoFullBackgroundImagesBundle\TL_ROOT"" at Runner.php line 81 {"exception":"[object] (Error(code: 0): Undefined constant \"Oneup\\ContaoFullBackgroundImagesBundle\\TL_ROOT\" at /var/www/vhosts/.../vendor/oneup/contao-full-background-images/src/Runner.php:81)"}

Außerdem erhalte ich noch einen Fehler beim Bearbeiten einer Root-Page, aber der kommt aus dem Core.

bytehead commented 5 months ago

Kannst du updaten und erneut testen?

rustykowski commented 5 months ago

das sieht jetzt teilweise gut aus! Script läuft. :) Aber: Der Fehler beim Bearbeiten der root-Page hatte offenbar doch mit contao-full-background-images zu tun. Ich hatte nämlich eine Hintergrundgalerie in der root-Page definiert, alle anderen Seiten erben. Sobald ich den Modus "individuelle Bildauswahl" einstellen möchte, erhalte ich System::import() failed because class "BackendUser" is not a valid class name or does not exist.

request.CRITICAL: Uncaught PHP Exception RuntimeException: "System::import() failed because class "BackendUser" is not a valid class name or does not exist." at System.php line 165 {"exception":"[object] (RuntimeException(code: 0): System::import() failed because class \"BackendUser\" is not a valid class name or does not exist. at .../vendor/contao/core-bundle/contao/library/Contao/System.php:165)"} {"request_uri":"https://example.de/contao?act=edit&do=page&id=19&ref=1wDK9dkD&rt=xxx","request_method":"POST"}

bytehead commented 5 months ago

Kannst du nochmals updaten und testen?

rustykowski commented 5 months ago

Jetzt läuft alles einwandfrei, super!

bytehead commented 5 months ago

Top, dann tagge ich noch ein Release.

bytehead commented 5 months ago

See https://github.com/1up-lab/contao-full-background-images/releases/tag/3.1.0

rustykowski commented 5 months ago

Vielen Dank, David!