SimplePie in S9Y - Hinweis für S9Y 1.7!

[hagenroewer]

Moin,

ich las gerade in serendipity_plugin_remoterss kann kein atom-feed

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

[garvinhicking]

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

[hagenroewer]

Moin,

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

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 ..

[hagenroewer]

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:

Code: Select all

DEBUG: cache_authors['realname'] = authorid has 13 entries DEBUG: cache_entries['title']['authorid']['timestamp'] = entryid has 393 entries. DEBUG: cache_md5['md5'] = entryid has 1 entries. Read http://www.igmetall.de/cps/rde/xchg/internet/style.xsl/igm.rss. DEBUG: Current authorid = 3
== SERENDIPITY ERROR ==

Please correct:

Favicon handling has been removed, please use your own handling in /home/igmetall-abde/public_html/aktuell/plugins/serendipity_event_aggregator/simplepie/simplepie.inc on line 2946

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

spuckt ähnliche Fehlermeldungen aus:

Code: Select all

DEBUG: cache_authors['realname'] = authorid has 13 entries DEBUG: cache_entries['title']['authorid']['timestamp'] = entryid has 393 entries. DEBUG: cache_md5['md5'] = entryid has 1 entries. Read http://www.igmetall.de/cps/rde/xchg/internet/style.xsl/igm.rss. DEBUG: Current authorid = 3
== SERENDIPITY ERROR ==

Please correct:

Favicon handling has been removed, please use your own handling in /home/igmetall-abde/public_html/aktuell/plugins/serendipity_event_aggregator/simplepie/simplepie.inc on line 1802

Version 1.2.2-dev
https://github.com/simplepie/simplepie/ ... plepie.inc
hingegen folgendes:

Code: Select all

== SERENDIPITY ERROR ==

Please correct:

Assigning the return value of new by reference is deprecated in /home/igmetall-abde/public_html/aktuell/plugins/serendipity_event_aggregator/simplepie/simplepie.inc on line 738

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.

[garvinhicking]

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

[Timbalu]

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?

[hagenroewer]

Hi Ian,

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.

[Timbalu]

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?

[hagenroewer]

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.

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.