Serendipity Forums

Hallo Garvin,

und wieder ein Verbesserungsvorschlag aus dem Hause Falk .

Diesmal geht es um die Pear-Klassen im bundled-libs-Verzeichnis. In diesen sind überall die Pfade bei den require_once nachgearbeitet worden. Ich war mal so frei und habe in der aktuellen Alpha auch aktuelle Pear-Pakete reingebaut und die require_once nicht angepasst. Es funktioniert. Also hier mein aktueller Vorschlag:

Die Bibliotheken im bundled-libs aktualisieren, ohne die Pfade der Pear-Klassen anzupassen. Diese werden durch korrektes setzen des include_path bereits geladen und brauchen ergo keine Anpassungen. Abgesehen davon ist das schlechter Programmierstiel .

Hi!

Es gibt Fälle bei denen s9y das lokale PEAR eines Server nutzt, und Fälle wo man es konfigurieren kann das das nicht passiert. Ohne die Anpassung der Libraries wäre dieser "DualUse" nicht möglich.

Grüße,
Garvin

Wenn der include_path ordentlich konfiguriert ist, wird Serendipity entweder die Pear-Klassen aus dem bundled-libs oder aus dem Pear nehmen. Es kommt auf die Reihenfolge an, in der die Klassen eingebunden werden. Eine Anpassung der Pear-Klassen in S9y ist nicht notwendig.

falk wrote:Wenn der include_path ordentlich konfiguriert ist, wird Serendipity entweder die Pear-Klassen aus dem bundled-libs oder aus dem Pear nehmen. Es kommt auf die Reihenfolge an, in der die Klassen eingebunden werden. Eine Anpassung der Pear-Klassen in S9y ist nicht notwendig.

Also es gab bisher mit PEAR in den vergangenne Jahren immer Probleme, kannst du ja mal die diversen Sonderfälle im Forum anschauen (einfach nach PEAR und file oder so suchen). Die jetzige Lösung klappt überall, auch in shared installations, PEAR-installationen und No-PEAR. Ich möchte da ungerne rütteln und dann schon wieder zig Supportrequest kriegen.

Also bei einem etwaigen Patch muss gewährleistet sein, dass alle bisher hier im Forum aufgetretenen Probleme nicht wieder auftreten, dann committe ich das gerne der Sauberkeit halber.

Bis dahin halte ich es eher so mit dem "never run a changing system "

Viele Grüße,
Garvin