[twitterreader] Uncaught exception 'Exception'

[QCO]

Ich bekomme in den letzten Tagen hin und wieder nicht näher bestimmten Fehler einen Fehler im Frontend. Das ist ziemlich problematisch, weil dann die ganze Seite nicht angezeigt wird.
Beim nächsten Aufruf der Seite ist wieder alles normal.

Im Error-Log des Webservers finde ich nur folgende Zeile:

PHP Fatal error: Uncaught exception 'Exception' with message 'String could not be parsed as XML' thrown in /var/www/vhosts/xxx.de/httpdocs/system/modules/twitterreader/FrontendTwitterReader.php on line 55

Hat jemand eine Ahnung, was hier das Problem ist?

[Stephan]

Das Problem tritt immer dann auf, wenn Twitter gerade down ist bzw. Probleme mit der Last hat. Das gleiche passiert in dem Modul twitterreader von lindesbs.

[QCO]

Oh, ok.

Könnte in diesem Fall der Autor der Erweiterung nicht irgendetwas einbauen, dass so ein Fehler besser abgefangen wird?
Im Moment ist die Erweiterung damit für mich leider unbrauchbar.

[lindesbs]

Habe mal ein Update gemacht. Wenn keine Verbindung aufgebaut werden konnte, wird keine Ausgabe erzeugt.

Bitte mal testen und berichten.

[QCO]

Erstmal danke für deine Mühe.
Habe gerade das Update installiert, allerdings schon das erste Problem im Backend: Als ich das entsprechende Modul wieder veröffentlichen wollte, kam wieder die bekannte Fehlermeldung - Twitter hat in den letzten Tagen, wie ich das beobachtet habe fast ständig Lastprobleme.
Nach zwei Versuchen ging es dann aber. Die entsprechende Änderung wäre also auch im Backend schön.

Ansonsten besteht der Fehler leider weiter. Mit der Fehlerbehandlung in PHP kenne ich mich nicht so aus, aber folgendes ist passiert:
Auch im Frontend kommt weiterhin die allgemeine Fehlermeldung, allerdings taucht im Error-Log von PHP kein Eintrag mehr auf.
Allerdings dafür bei Typolight unter system/logs/error.log ist wieder eine Zeile:

[09-Sep-2009 03:19:49] PHP Fatal error: Uncaught exception 'Exception' with message 'String could not be parsed as XML' thrown in /var/www/vhosts/xxx.de/httpdocs/system/modules/twitterreader/FrontendTwitterReader.php on line 71

Man beachte, dass der Fehler diesmal in Zeile 71 passiert.

[lindesbs]

Da gerade ein massiver AUsfall von Twitter war. habe ich gut testen koennen.
Habe meiner Meinung nach alle Fehlerfaelle abgefangen. Der Feed wird nun zwischengespeichert und im Problemfall wird quasi das "letzte Backup" dargestellt.

Klappt bei mir gerade wunderbar. Also bitte ein Update aus dem ER machen.

Damit keine Last durch hohe Anfrage bei Twitter entsteht, wird der Feed nur aktualisiert, wenn mind. 5 Minuten vergangen sind, seit dem letzten Update.
Hatte naemlich gestern abend das Problem, das Twitter mich abgelehnt hat, weil ich mehr als 150 Anfragen pro Stunde gemacht habe. Durch diese Abfragereduzierung sollten somit weniger Probleme entstehen.

Bitte ausprobieren, ob es etwas gebracht hat.

[Stephan]

Klingt super, tolle Lösung mit dem Zwischenspeichern! Ich habe nun auch das Update installiert und werde es beobachten. Danke Stefan.

[lindesbs]

Bitte nochmals ein Update machen. Ich musste die Passworteingabe mit aufnehmen, da es sonst zu Problemen fuehrt.

Also nach einem Update der Extension nochmal ein DB Update machen, und in der Modulkonfiguration das Passwort des TwitterBenutzers mit angeben.

Dann laeufts.

[QCO]

Noch ein Testbericht: Fehlermeldungen habe ich jetzt keine mehr, aber irgendetwas ist noch mit der Zählung der Tweets faul.
Ich hab zwei twitterreader-Module angelegt; das erste soll 3 Tweets anzeigen, das zweite 5. Beim ersten Laden der Seiten stimmte das auch noch, aber jetzt zeigen alle beide Module alle(!) Tweets an, die mit dem Twitteraccount gepostet wurden.

[lindesbs]

Ist gerade gefixt worden. Ich fordere zwar nur die eingestellte Anzahl an, geliefert werden aber alle. Ist in der aktuelle Version im ER gefixt.

Twitter hat offensichtlich momentan groessere Probleme....

[philsniff]

passwort-eingabe?

hallo lindesbs,

bis jetzt war es ne supiendgute erweiterung, die ich auch gleich auf einer hauptseite bei einem kunden einbauen wollte. ist noch in der entwicklung, deshalb verfolgte ich diesen threat wegen den fehlermeldungen in letzter zeit.

und tata, du als entwickler, zack gleich reagiert und ich mich gefreut.
aber....

nun brauche ich eine passwort-abfrage zum account?
ich habe aber bis jetzt (und da bin ich icht allein) den reader wirklich als reader eines anderen accounts (hier DATEV) genutzt.

nun kann ich die tweets nicht mehr einbauen, da ich ja das passwort für diesen account gar nicht weiß, und auch nicht wissen will, denn ich will ja nur lesen!

gibt es da noch ein workaround von dir, ansonsten bin ich echt aufgeschmissen...

gruß
philipp

[lindesbs]

Upps, hast recht. An diesen Fall hab ich nicht gedacht.

Werd ich noch anpassen, kurzfristif im laufe des VOrmittags

[Stephan]

Hallo Stefan,

wenn du schonmal dabei bist, wie wäre es noch zwei kleine Fixes mit einzubauen, damit der HTML-Validator auch keine Fehlermeldung mehr ausspuckt?

FrontendTwitterReader.php:
Da es bei HTML strict keine taget geben darf, sollte man auf target=_blank im <a>-Tag verzichten. Anstelle dessen könnte man evtl. onclick="window.open(this.href); return false;" mit einbauen, in Analogie zu den von TYPOlight erzeugten Links. Außerdem könnte man dem Link ein title-Attribut hinzuzufügen. Dieses könnte dann so title="$0" bzw. so title="$1" aussehen.

if ($this->twitterEnableHTTPLinks)
{
$textOutput = preg_replace('(http://(\S+))','<a title="$0" onclick="window.open(this.href); return false;" href="$0">$0</a>',$textOutput);            
}
 
if ($this->twitterEnableUserProfileLink)                
{
$textOutput = preg_replace('(@(\S+))','<a title="$1" onclick="window.open(this.href); return false;" href="http://www.twitter.com/$1">$1</a>',$textOutput);
} 

FrontendTwitterReader.tpl:
In der Template-Datei wird ein <div>- in ein <ul>-Element eingebettet. Dies wird ebenfalls vom Validator angemeckert, denn ein <ul>-Element darf nur <li>-Elemente enthalten. Ich würde das <div> einfach löschen und vorher die Klassennamen dem <li>-Tag mitgeben.

<ul id="<?php echo $this->class; ?>">
<?php if ($this->TwitterData): ?>
<?php foreach ($this->TwitterData as $item): ?>
 
<li class="<?php echo $item->EvenOdd; ?> <?php echo $item->First; ?> <?php echo $item->Last; ?>">
<span class="date"><?php echo date($GLOBALS['TL_CONFIG']['dateFormat'],strtotime($item->created_at)); ?></span>
<?php echo $item->text; ?>
</li>
<?php endforeach; ?>
<?php endif; ?>    
</ul>

[lindesbs]

Danke fuer die Tips. SInd eingepflegt, sowie das passwortlose Abfragen der Tweets.

Klappt hier nun wieder alles.

Allerdings muss man seine CSS wieder anpassen.

[Stephan]

Super, danke!
Wärst du so nett und könntest mir noch erklären, worin der Unterschied nun besteht, ob ich ein Passwort eingebe oder nicht?