Probleme mit PHP 7 und Tag

Probleme mit PHP 7 und Tag_News

Hallo,

habe heute eine Seite auf einen neuen Server umgesiedelt und im gleichen Atemzug auf php7 gebracht doch sobald ich Tagcloud News aktiviere kommt nur noch eine weiße seite (zumindest auf den seiten in welchen das Modul eingebunden ist)
Im Log kommt folgende Fehlermeldung:

Code:

Warning: Declaration of Contao\TagListNews::getRelatedTagList($for_tags) should be compatible with Contao\TagList::getRelatedTagList($for_tags, $blnExcludeUnpublishedItems = true) in system/modules/tags_news/classes/TagListNews.php on line 127 #0 system/modules/core/library/Contao/ClassLoader.php(148): __error(2, 'Declaration of ...', '/var/www/web...', 127, Array) #1 system/modules/core/library/Contao/ClassLoader.php(148): include() #2 [internal function]: Contao\ClassLoader::load('Contao\\TagListN...') #3 system/modules/tags_news/modules/ModuleTagCloudNews.php(39): spl_autoload_call('Contao\\TagListN...') #4 system/modules/core/elements/ContentModule.php(59): Contao\ModuleTagCloudNews->generate() #5 system/modules/core/library/Contao/Controller.php(484): Contao\ContentModule->generate() #6 system/modules/core/modules/ModuleArticle.php(213): Contao\Controller::getContentElement(Object(Contao\ContentModel), 'main') #7 system/modules/tags/modules/ModuleArticle.php(26): Contao\ModuleArticle->compile() #8 system/modules/core/modules/Module.php(287): Aurealis\ModuleArticle->compile() #9 system/modules/core/modules/ModuleArticle.php(67): Contao\Module->generate() #10 system/modules/core/library/Contao/Controller.php(417): Contao\ModuleArticle->generate(true) #11 system/modules/core/elements/ContentArticle.php(29): Contao\Controller::getArticle('69', false, true) #12 system/modules/core/library/Contao/Controller.php(484): Contao\ContentArticle->generate() #13 system/modules/core/modules/ModuleArticle.php(213): Contao\Controller::getContentElement(Object(Contao\ContentModel), 'main') #14 system/modules/tags/modules/ModuleArticle.php(26): Contao\ModuleArticle->compile() #15 system/modules/core/modules/Module.php(287): Aurealis\ModuleArticle->compile() #16 system/modules/core/modules/ModuleArticle.php(67): Contao\Module->generate() #17 system/modules/core/library/Contao/Controller.php(417): Contao\ModuleArticle->generate(false) #18 system/modules/core/library/Contao/Controller.php(277): Contao\Controller::getArticle(Object(Contao\ArticleModel), false, false, 'main') #19 system/modules/core/pages/PageRegular.php(133): Contao\Controller::getFrontendModule('0', 'main') #20 system/modules/core/controllers/FrontendIndex.php(285): Contao\PageRegular->generate(Object(Contao\PageModel), true) #21 index.php(20): Contao\FrontendIndex->run() #22 {main} Warning: Declaration of Contao\TagListNews::getTagList() should be compatible with Contao\TagList::getTagList($blnExcludeUnpublishedItems = true) in system/modules/tags_news/classes/TagListNews.php on line 127 #0 system/modules/core/library/Contao/ClassLoader.php(148): __error(2, 'Declaration of ...', '/var/www/web...', 127, Array) #1 system/modules/core/library/Contao/ClassLoader.php(148): include() #2 [internal function]: Contao\ClassLoader::load('Contao\\TagListN...') #3 system/modules/tags_news/modules/ModuleTagCloudNews.php(39): spl_autoload_call('Contao\\TagListN...') #4 system/modules/core/elements/ContentModule.php(59): Contao\ModuleTagCloudNews->generate() #5 system/modules/core/library/Contao/Controller.php(484): Contao\ContentModule->generate() #6 system/modules/core/modules/ModuleArticle.php(213): Contao\Controller::getContentElement(Object(Contao\ContentModel), 'main') #7 system/modules/tags/modules/ModuleArticle.php(26): Contao\ModuleArticle->compile() #8 system/modules/core/modules/Module.php(287): Aurealis\ModuleArticle->compile() #9 system/modules/core/modules/ModuleArticle.php(67): Contao\Module->generate() #10 system/modules/core/library/Contao/Controller.php(417): Contao\ModuleArticle->generate(true) #11 system/modules/core/elements/ContentArticle.php(29): Contao\Controller::getArticle('69', false, true) #12 system/modules/core/library/Contao/Controller.php(484): Contao\ContentArticle->generate() #13 system/modules/core/modules/ModuleArticle.php(213): Contao\Controller::getContentElement(Object(Contao\ContentModel), 'main') #14 system/modules/tags/modules/ModuleArticle.php(26): Contao\ModuleArticle->compile() #15 system/modules/core/modules/Module.php(287): Aurealis\ModuleArticle->compile() #16 system/modules/core/modules/ModuleArticle.php(67): Contao\Module->generate() #17 system/modules/core/library/Contao/Controller.php(417): Contao\ModuleArticle->generate(false) #18 system/modules/core/library/Contao/Controller.php(277): Contao\Controller::getArticle(Object(Contao\ArticleModel), false, false, 'main') #19 system/modules/core/pages/PageRegular.php(133): Contao\Controller::getFrontendModule('0', 'main') #20 system/modules/core/controllers/FrontendIndex.php(285): Contao\PageRegular->generate(Object(Contao\PageModel), true) #21 index.php(20): Contao\FrontendIndex->run() #22 {main} Fatal error: Cannot use 'String' as class name as it is reserved in /var/www/web/html/system/modules/core/library/Contao/String.php on line 28

habe contao einem check unterzogen (alles i.O.) und die Extention neu eingespielt
Bin hier etwas mit meinem Latein am Ende :(

Die Erweiterung ist schlicht nicht mit PHP 7 kompatibel.

nun die Frage aller Fragen: wird sie es noch? bzw. welche Anpassungen müssen gemacht werden? Die Tags Erweiterung scheint ja mit php 7 Kompatibel zu sein.

Öffne ein Ticket hier: https://github.com/hschottm/tags_news/issues

Erstellt - ich hoffe das ich das auch im richtigen erstellt hab, ist mein erstes Ticket bei Github ;)

Ich habe dort mal kommentiert. Sind je ein Fehler in zwei Erweiterungen, "tag_news" benötigt ja "tags".

Dank dir, hab ich gesehen und hat auch geholfen. Habe nur den Fehler mit der String Klasse etwas anders gelöst, hab ich aber schon mit reingeschrieben

Das man StringUtil nun nimmt weiß ich, aber wenn man es so importiert wie du nun geändert hast, dann müsste man die String Funktionen beispielsweise nun statt String::toHtml5() nun mit StringUtil::toHtml5() ansprechen.
Das würde aber weitere Änderungen in der Erweiterung bedeuten. Daher hatte ich ja gesucht, aber keine Nutzung der Klasse gefunden. Dann kann der Import auch weggelassen werden.
Will man das kompatibel machen, dann könnte man es versuchen so umzubauen:

Code:

$this->import('StringUtil','String');

Auf diese Weise kann man weiterhin die StringUtil Klasse über ~~String::~~ $this->String ansprechen.

Eher $this->String, oder? ;)

Äh ja richtig. Den Import bräucht man gar nicht, weil das ja alles static Methoden sind. (wenn man diese dann so ansprechen würde)

Ja, aber ohne Import musst du auf jeden Fall \StringUtil:: verwenden.