search

Xirdion / contao-modal-bundle

Contao bundle to add a modal / popup (dialog) to the CMS.
3 stars 1 forks source link

Bei HTML Modals kommt es zu einem Typen-Problem #5

Closed fschmid-dev closed 2 years ago

fschmid-dev commented 2 years ago

In der src/Modal/Builder.php wird in der Funktion isModalPage() das PageModal und die Ziel-URL erwartet: https://github.com/sowieso-web/contao-modal-bundle/blob/9348d2fab8c46c800d7a0d2fdcbd16a40af534eb/src/Modal/Builder.php#L195

Wenn aber als Inhalt für das Modal "HTML" ausgewählt wird, anstatt Text + Bild oder nur Bild, gibt es beim FrontendModul keine Einstellungsmöglichkeiten für eine Weiterleitungs-URL. Es wird dann null als zweiter Parameter übergeben und es kommt zu einem TypeError.

Entweder sollte der Parameter nullable sein oder das Eingabefeld für die Weiterleitungs- bzw. Ziel-URL auch beim Typ "HTML" hinzugefügt werden. Je nachdem, was hier der gewünschte Weg ist :)

Xirdion commented 2 years ago

Thanks for the tip. With the new version 1.0.4 the error is now fixed.