search

thescrat / contao-loginlink

GNU Lesser General Public License v3.0
2 stars 2 forks source link

Exception wenn man Benutzer nicht für den FrontendLogin berechtigt #7

Closed akroii closed 2 years ago

akroii commented 2 years ago

Hallo, nicht wirklich ein Fehler, mehr ein Bedienungsproblem. Wenn man ein Mitglied den Haken für den FrontendLogin (und Logindaten nicht setzt) nicht setzt, dann erscheint eine Exception wenn man den LoginKey im Frontend aufruft. Praktisch fehlt ihm die Erlaubnis sich im Frontend einzuloggen.

Im Debug Modus steht dann folgendes:

TypeError:
Argument 1 passed to thescrat\LoginLinkBundle\EventListener\GeneratePageListener::loginUser() must be of the type string, null given, called in /var/www/vhosts/h18828.web140.dogado.net/public_html/mammographie/vendor/thescrat/contao-loginlink/src/EventListener/GeneratePageListener.php on line 132

  at vendor/thescrat/contao-loginlink/src/EventListener/GeneratePageListener.php:177
  at thescrat\LoginLinkBundle\EventListener\GeneratePageListener->loginUser(null)
     (vendor/thescrat/contao-loginlink/src/EventListener/GeneratePageListener.php:132)
  at thescrat\LoginLinkBundle\EventListener\GeneratePageListener->onGeneratePage(object(PageModel), object(LayoutModel), object(PageRegular))
     (vendor/contao/core-bundle/src/Resources/contao/pages/PageRegular.php:196)
  at Contao\PageRegular->prepare(object(PageModel))
     (vendor/contao/core-bundle/src/Resources/contao/pages/PageRegular.php:53)
  at Contao\PageRegular->getResponse(object(PageModel), true)
     (vendor/contao/core-bundle/src/Resources/contao/controllers/FrontendIndex.php:339)
  at Contao\FrontendIndex->renderPage(object(PageModel))
     (vendor/symfony/http-kernel/HttpKernel.php:158)
  at Symfony\Component\HttpKernel\HttpKernel->handleRaw(object(Request), 1)
     (vendor/symfony/http-kernel/HttpKernel.php:80)
  at Symfony\Component\HttpKernel\HttpKernel->handle(object(Request), 1, true)
     (vendor/symfony/http-kernel/Kernel.php:201)
  at Symfony\Component\HttpKernel\Kernel->handle(object(Request))
     (web/index.php:31)
  at require('/var/www/vhosts/h18828.web140.dogado.net/public_html/mammographie/web/index.php')
     (web/app.php:4)

Evtl kann man hier die readme ergänzen? Ich würde das auch tuen. C 4.9.28

thescrat commented 2 years ago

Danke für den Hinweis. Hab dir gerade ein neues Release erstellt wo genau das geprüft wird. Ich hoffe das funktioniert jetzt für die "alte" Version. Schaust du mal ob dir die Version im Manager angezeigt wird 1.0.4.1 müsste es eigentlich sein.
Wenn es passt und der Fehler weg ist, übernehm ich ihn gleich für das 4.13er Release.

akroii commented 2 years ago

Das passt so! Danke!