psi-4ward / news4ward

Contao Extensions: News/Blog based on contentelements
13 stars 12 forks source link

Fragen zur Weiterentwicklung #93

Closed fenepedia closed 5 years ago

fenepedia commented 7 years ago

Hallo PSI,

könntest du kurz sagen, wie die Pläne bezüglich der Weiterentwicklung von news4ward aussieht?

frontendschlampe commented 7 years ago

@psi-4ward wird an der Erweiterung sicherlich nicht mehr viel machen, da er leider nicht mehr viel mit Contao zu tun hat.

Aktuell pflegen wir die Erweiterung soweit wir Bedarf/Budget haben.

Das Problem ist, dass die Erweiterung eigentlich nicht mehr so wirklich benötigt wird, da ja Contao die Hauptfunktion (CE innerhalb einer News) mittlerweile als Corefunktion mitbringt.

Aber wir sind für alle Ideen offen und zu Gesprächen bereit! :-)

psi-4ward commented 7 years ago

Genau so ist es. Auftragsbezogen würde ich schon noch was machen. PHP7 Kompatibel müsste der News4ward Stack sein.

fenepedia commented 7 years ago

Danke für die Info.

PHP7 funktioniert leider nicht, hier kommt es zu Fehlermeldungen.

PHP Fatal error: Uncaught exception 'Error' with message 'Function name must be a string' thrown in /system/modules/news4ward/Module/Listing.php on line 95

Ich nutze die Erweiterung wegen folgenden Features:

Hab mich gerade mal nach Alternativen umgesehen. Ich werde das erst mal testen:

Damit können wir das Thema wohl beenden.

psi-4ward commented 7 years ago

Hast du nen Stacktrace zu deinem Error? Klingt mir nicht nach nem PHP7 Problem! Iwo wird blödsinn in $GLOBALS['TL_HOOKS']['News4wardListFilter'] geschrieben

fenepedia commented 7 years ago

Wie kann ich einen Stacktrace erstellen? Oder meinst du die komplette error.log?

bibib commented 5 years ago

Gibt's zum Stacktrace schon neue Erkenntnisse? Bzw. wie hoch wären denn die Kosten für eine Anpassung für C4? Ich hätte schon einige Kunden, die das brauchen könnten und wenn es insgesamt günstiger wäre als ein Umstieg auf das News-Modul, wäre das vielleicht eine Option.

psi-4ward commented 5 years ago

Nen Stacktrace hab ich nie erhalten und auch keine weiteren Logs o.Ä.

Anpassung für Contao 4: Siehe meinen Kommentar von table4ward.

fenepedia commented 5 years ago

Aufgrund der fehlenden Weiterentwicklung habe ich alle aktuellen news4ward Seiten auf das normale Contao News-Modul geändert.

@bibib Im Moment ist die Erweiterung nicht PHP7 kompatibel. Das "Problem Function name must be a string".

Evtl. reicht es, wenn du den Code komplett nach diesem Aufruf durchsuchst und änderst. https://community.contao.org/de/showthread.php?72297-PHP-7-Function-name-must-be-a-string-gt-call_user_func_array&p=484745&viewfull=1#post484745

bibib commented 5 years ago

Evtl. reicht es, wenn du den Code komplett nach diesem Aufruf durchsuchst und änderst. https://community.contao.org/de/showthread.php?72297-PHP-7-Function-name-must-be-a-string-gt-call_user_func_array&p=484745&viewfull=1#post484745

Danke für den Hinweis - scheinbar braucht es wirklich nur wenige Änderungen: https://community.contao.org/de/showthread.php?72234-news4ward-und-Contao-4-4&p=493144#post493144

psi-4ward commented 5 years ago

Hab mal nen commit 12c5577 gemacht, bitte testen ;)

bibib commented 5 years ago

Sieht gut aus, nur die Icons aus dem html-Ordner fehlen noch. :-) Lokal habe ich den Ordner html in assets umbenannt und die Pfade entsprechend angepasst.

psi-4ward commented 5 years ago

Okay? Mir nicht klar warum das so muss aber habs mal umbenannt.

bibib commented 5 years ago

Mir auch nicht, aber Spooky wird's schon wissen. ;-) Die Icons sind da, ich musste aber das Teaserbild neu zuordnen? Lässt sich das irgendwie verhindern?

xchs commented 5 years ago

Ich war auch der Meinung, dass eigentlich beide Ordnernamen funktionieren sollten.

psi-4ward commented 5 years ago

Teaserbild?? Hier hat sich nichts verändert.

bibib commented 5 years ago

Na, passt schon. In Contao 3.5 ist auch alles paletti. Vielen Dank für die Anpassung!

bibib commented 5 years ago

Sorry, muss nochmal öffnen. Spooky fragt, ob du die Erweiterung bitte auch noch für Contao 4 freigeben könntest?

Bzw. natürlich dann auch analog archive_menu, categories, comments, newsletter, related und tags.

psi-4ward commented 5 years ago

erledicht

bibib commented 5 years ago

Muss leider nochmal aufschnüren ... im Contao-Manager wird die Erweiterung immer noch als inkompatibel angezeigt. Könntest du das bitte auch noch richten? Bzw. ist die Anpassung auch unter Contao 3.5 im Composer nicht auswählbar. Synchronisation mit Packagist scheint nicht zu funktionieren.

psi-4ward commented 5 years ago

Normal updated sich doch pakagist selbst? Hab mal auf update gedrückt

frontendschlampe commented 5 years ago

Normal updated sich doch pakagist selbst? Hab mal auf update gedrückt

Die Einbindung hat sich doch geändert! ;-)

psi-4ward commented 5 years ago

Aja? Meine Docker-Registry und Travis Hooks funzen auch noch.