Serendipity Forums

Moin,

ich las gerade in serendipity_plugin_remoterss kann kein atom-feed

garvinhicking wrote:Hi!

Ich habe im s9y Kern jetzt simplepie hinzugefügt mit dem man auch Atom parsen kann; theoretisch sollte das klappen. Praktisch hatte ich jetzt keine Zeit mehr zu testen.

Gibt jedoch das Problem, dass man dafür den s9y core updaten muss, also am besten per github mal die aktuelle Version ziehen.

Heute sah ich dazu folgendes:
SimplePie 1.3.1 is now available!
Ryan hat anscheinend die Klasse erheblich aufgebohrt.

Zu beachten ist dabei folgendes - aus seiner Readme

### Where's `simplepie.inc`?
For SimplePie 1.3, we've split the classes into separate files to make it easier
to maintain and use.

If you'd like a single monolithic file, you can run `php build/compile.php` to
generate `SimplePie.compiled.php`, or grab a copy from
[dev.simplepie.org][dev_compiled] (this is kept up-to-date with the latest
code from Git).

[dev_compiled]: http://dev.simplepie.org/SimplePie.compiled.php

Wenn diese Klasse sowieso gebundelt wird künftig im Kern ab 1.7, macht es dann nicht Sinn, die Klasse direkt aus dem Plugin serendipity_event_aggregator herauszunehmen, und direkt auf die Lib im Kern hinzuweisen?

Ich teste jedenfalls gerade die Klasse im besagten Plugin - und zumindestens scheinen sich meine alten quot-Probleme zu lösen.

Herzliche Grüße von der Ruhr

Hi!

Danke für die Info!

OK; solange da noch keine derzeit offenen Bugs mit der alten Lib entstehen fehlt mir gerade die Zeit, mich in die neue Lib einzuarbeiten.

Da das aggregator plugin ja auch in älteren s9y versionen klappen soll würde ich das simplepie dort vorerst nicht entfernen.

Grüße,
GArvin

Moin,

garvinhicking wrote: OK; solange da noch keine derzeit offenen Bugs mit der alten Lib entstehen fehlt mir gerade die Zeit, mich in die neue Lib einzuarbeiten.

Wobei Ryan die Umstellung uns wiederum relativ einfach macht. Wir brauchen im Grunde den Inhalt der simplepie.inc mit der aus http://dev.simplepie.org/SimplePie.compiled.php zu tauschen. Alles andere, was noch im Plugin bzw, im Core ab 1.7 dabei ist, ist soweit geblieben, das hatte ich gestern Abend noch geprüft. siehe weiter oben mein Zitat

garvinhicking wrote: Da das aggregator plugin ja auch in älteren s9y versionen klappen soll würde ich das simplepie dort vorerst nicht entfernen.

Ja, das macht Sinn ..

Ich hatte bis vor kurzen SimplePie in der Version '1.4-dev
http://dev.simplepie.org/SimplePie.compiled.php im Plugin serendipity_event_aggregator laufen lassen.

Das klappt anscheinend mit der Version 1.7 von S9Y nicht mehr! Vorher lief die Klasse problemlos mit der Version 1.6.
1. Symptom: Die RSS-Feeds werden nicht mehr aktualisiert eingelesen.

Nach ich im Plugin die Login-Funktion eingeschaltet habe, spuckte mir das Plugin folgendes aus:


Ich bin dann einfach in den Versionsnummern zurückgegangen:
Version 1.3.1
http://simplepie.org/downloads/simplepi ... mpiled.php

spuckt ähnliche Fehlermeldungen aus: Version 1.2.2-dev
https://github.com/simplepie/simplepie/ ... plepie.inc
hingegen folgendes:
Die im Plugin eingebaute Klasse version 1.2
https://github.com/s9y/additional_plugi ... plepie.inc macht keine Probleme ebenso die Version 1.2.1, die im Kern integriert ist.

Ich lasse die letztgenannte Version als Klasse im Plugin weiterlaufen.

Hi!

Simplepie wirft da eine PHP Fehlermeldung, und weil s9y ein eigenes fehlerhandling hat seit 1.7, fängt es die Fehlermeldung und "stirbt".

Man müsste also Simplepie so konfigurieren/verändern, dass es das favicon-handling überspringt oder keine Fehlermeldung derart wirft bzw. unterdrückt. Das muss man innerhalb von Simplepie machen.

Die andere Fehlermeldung ("reference variable") ist auch eine Sache die mit modernem PHP5 so einen Fehler erzeugt.

Grüße,
Garvin

Darf ich mal fragen, warum man denn jetzt noch ein extra simplepie fahren will/muss, wenn doch ein bereits von uns zusätzlich PHP5 fehlerbereinigtes 1.2.1-dev in den core bundled libs mitgeliefert wird?

Hi Ian,

Timbalu wrote:Darf ich mal fragen, warum man denn jetzt noch ein extra simplepie fahren will/muss, wenn doch ein bereits von uns zusätzlich PHP5 fehlerbereinigtes 1.2.1-dev in den core bundled libs mitgeliefert wird?

Ja, die wird im Kern mitgeliefert, aber nicht in dem von mir angesprochenen Plugin. Da läuft noch die Version 1.2. Da kann man sich direkt in Github anschauen.

Ich habe die eigentliche Frage in deinem Posting gar nicht heraus gehört...
Ist dann die Frage, ob man einen S9y-version switch in serendipity_event_aggregator einbauen sollte, der im Falle von 1.7+up auf das core simplepie zurückgreift?

Timbalu wrote:Ich habe die eigentliche Frage in deinem Posting gar nicht heraus gehört...

Meine Ankündigung von gestern war auch nicht als Frage gedacht, sondern als Hinweis, wenn irgendwann mal auf die aktuelle Klasse ein Update erfolgen soll, dann muss in der Klasse, wie Garvin es schon beschrieben hat, noch was geändert werden.

Timbalu wrote:Ist dann die Frage, ob man einen S9y-version switch in serendipity_event_aggregator einbauen sollte, der im Falle von 1.7+up auf das core simplepie zurückgreift?

Die Frage zum Plugin hat Garvin ja schon in diesem Thread vor einem dreiviertel Jahr beantwortet, d. h. die Klasse soll da drin bleiben wegen Abwärtskompatibilität. Aus meiner Sicht wäre es sinnvoll zumindestens die Versionen zu harmonisieren, also auf gleichen Stand zu bringen.