search

hschottm / tags

Contao tags extension
11 stars 16 forks source link

Contao 4.8: Auflistung dazugehöriger News funktioniert nicht #51

Closed jankout closed 4 years ago

jankout commented 4 years ago

Hallo,

seit 4.8.x habe ich gemerkt, dass die Auflistung dazugehöriger News nicht funktioniert. Wie kann ich diesen Fehler beheben?

Siehe https://www.allesbrille-berlin.com/tags/tag/KOMONO

Schon jetzt danke ich dir für deine Antwort und wünsche dir noch einen schönen Tag.

jankout commented 4 years ago

Und wenn ich auf das Modul für den Tag-Cloud klicke, um ihn im Backend zu bearbeiten, erscheint folgende Fehlermeldung:

Argument 2 passed to Contao\Controller::getTemplateGroup() must be of the type array, string given, called in /homepages/35/d432404500/htdocs/contao4/var/cache/prod/contao/dca/tl_module.php on line 2570

fritzmg commented 4 years ago

Selbes Problem wie hier und [hier](https://community.contao.org/de/showthread.php?76082-Contao-Controller-getTemplateGroup()-erwartet-Array-als-2-Parameter).

jankout commented 4 years ago

@fritzmg Danke dir. Leider kenne ich mich bei dieser Erweiterung nicht so gut aus, um es selber zu korrigieren. Ich vermute, dass ich im Ordner classes/ schauen müsste. Weißt du vielleicht, um welche Stelle es sich handelt?

fritzmg commented 4 years ago

https://github.com/hschottm/tags/blob/45464754c441b60031ff54ebafeffd5624aaa405/dca/tl_module.php#L58

https://github.com/hschottm/tags/blob/45464754c441b60031ff54ebafeffd5624aaa405/dca/tl_module.php#L68

https://github.com/hschottm/tags/blob/45464754c441b60031ff54ebafeffd5624aaa405/dca/tl_module.php#L78

https://github.com/hschottm/tags/blob/45464754c441b60031ff54ebafeffd5624aaa405/dca/tl_module.php#L88

jankout commented 4 years ago

@fritzmg Vielen Dank. Dann einfach ohne , $dc->activeRecord->pid? D.h. return $this->getTemplateGroup('mod_');?

fritzmg commented 4 years ago

Genau, ja. Den zweiten Parameter einfach entfernen.

jankout commented 4 years ago

@fritzmg Wunderbar. Deine Hilfe war wieder perfekt. Mit dem Modul hat es nun geklappt.

Allerdings geht noch nicht die Auflistung der zugehörigen News, wenn ich auf eines der Schlagwörter klicke. @fritzmg Hast du da vielleicht eine Idee?

hschottm commented 4 years ago

Fixed in 762071795f9c2675a709c425bc79a539e3fdd557

jankout commented 4 years ago

Hallo,

Kamil already fixed it because we didn’t have any respond from you. See his work.

    1. 2019 v 12:04, Helmut Schottmüller notifications@github.com:

Closed #51 https://github.com/hschottm/tags/issues/51.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/hschottm/tags/issues/51?email_source=notifications&email_token=AA4K72SCNTCJAXPYM6TK2ALQVT7DFA5CNFSM4IZ7ORN2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOVDCM5GQ#event-2831470234, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA4K72XB6UHNGA7BPEJPBOTQVT7DFANCNFSM4IZ7ORNQ.