search

m-vo / contao-facebook-import

Facebook posts and events for Contao Open Source CMS
Other
11 stars 5 forks source link

Update README.md #1

Closed stefansl closed 6 years ago

stefansl commented 6 years ago

Moin! Eine Agentur hat nun zum zweiten Mal deine Erweiterung nicht vollständig installiert. Der Cronjob war noch auf PoorMan eingestellt und das Modul produzierte Fehler auf der Webseite (Facebook gibt zwischendurch scheinbar nix zurück). Ich hab jetzt mal diesen Schritt mit in die Installation aufgenommen. Den Punkt mit der Standard Edition halte ich für obsolet, da max nur noch Entwickler diese Version benutzen. Die wissen, dass sie das Bundle registrieren müssen. Eine Erinnerung reicht, denke ich. Was meinst du?

Edit: das war der Fehler

[2018-02-14 09:56:28] app.CRITICAL: An exception occurred. {"exception":"[object] (Symfony\\Component\\Debug\\Exception\\FatalThrowableError(code: 0): Call to a member function get() on null at /var/www/aermax/htdocs/vendor/mvo/contao-facebook-import/src/Facebook/OpenGraph.php:141)"} []
m-vo commented 6 years ago

Kannst du kurz beschreiben wie sich der Fehler reproduzieren lässt? Und was hat das mit dem PeriodicCommandScheduler zu tun?

stefansl commented 6 years ago

Das war meine Vermutung, da du in der Readme einen serverbasierten Cronjob als wichtig dargestellt hast. Den Fehler konnte ich nicht reproduzieren, Ich habe kurz hintereinander reloaded und mal kam die Fehlermeldung, im Frontend, sowie auch im Backend. Zwischendurch war aber alles ok. So liegt die Vermutung nahe, dass Facebook nix zurückgegeben hat. Wenn der echte Cronjob angelegt wird, dann landet dieser Fehler ggf. nicht im Frontend sondern in /dev/null. Oder liege ich da komplett falsch? Aber ganz abgesehen davon dachte ich, die Gewichtung der Inhalte in der Readme müsste etwas anders. Deswegen der Pull Request. Der Fehler ansich ist ja nach korrekter Installation, wie von dir beschrieben, weg.

m-vo commented 6 years ago

Das Bundle sollte egal mit welcher Variante ('fake cron' oder richtigem) funktionieren. Der Import kann einen kurzen Moment dauern - u.U. bis hin zum Timeout, sollte die Verbindung zu Facebook nicht aufgebaut werden können - weshalb es besser ist, das nicht beim Seitenaufbau eines Besuchers zu erledigen (was beim 'fake cron' aber der Fall ist). Das betrifft aber nicht speziell diese Erweiterung, sondern ist ein allgemeines Problem von allen Jobs, die periodisch ausgeführt werden müssen.

Wenn der echte Cronjob angelegt wird, dann landet dieser Fehler ggf. nicht im Frontend sondern in /dev/null. Oder liege ich da komplett falsch?

Typischerweise ja, sofern du den Output des Cronjobs nicht loggst. Ich werde bei Gelegenheit schauen, was die Ursache sein könnte. Meinst du mit 'korrekter Installation' das Anlegen eines Cronjobs?

stefansl commented 6 years ago

Das Bundle sollte egal mit welcher Variante ('fake cron' oder richtigem) funktionieren

Tut's generell auch, aber es kann ja zu einem Timeout kommen. Deswegen schriebst du ja auch, man soll einen echten Cronjob einrichten. Was ja auch richtig ist.

was beim 'fake cron' aber der Fall ist

das hab ich mir schon gedacht.

Meinst du mit 'korrekter Installation' das Anlegen eines Cronjobs?

Ja, genau. :)

Alles in allem: danke für die super Erweiterung. :) Der PR ja keine Kritik, sondern nur eine Anregung.

m-vo commented 6 years ago

Ich glaube nicht, dass es ein Timeout-Problem ist, das den Fehler produziert. Will see!

Der PR ja keine Kritik, sondern nur eine Anregung.

Und beides ist stets willkommen! :-)