Swift-Gallery

[bbs2]

Hallo,

ich benutze weiterhin unter WB 2.12.x die swift gallery. Immer nachts gegen 0 Uhr treten lt. error-log folgende Fehler auf

Thu, 21 Feb 2019 13:28:34 +0000 [E_NOTICE] /modules/swift/view.php:[83]  from /framework/frontend.functions.php:[225] require "A non well formed numeric value encountered"

Code: [Select]

$i = 0;
if( is_dir( $gallery_path ) ) {
if( $handle = opendir( $gallery_path ) ) {
$all_images = array();
while( ( $file = readdir( $handle ) ) !== false ) {
if( is_file($gallery_path.$file) && $file!='index.php') {
$image = WB_URL.$gallery.$file;
$all_images[$i] = $image;
$i++;
}
}
closedir( $handle );
}
}
if( isset( $_REQUEST['start'] ) ) {
$start = $_REQUEST['start'];
}
else {
$start = 0;
}
$images = array();
$a=intval($i);                     
if ($a>0) {
$images = array_slice( $all_images, $start, $settings['thumbs_per_page']);   ///Dies ist die Fehlerzeile 83
}
Für den user tritt kein Fehler auf. Alle galeries werden fehlerfrei angezeigt.
Kann es sein, dass der Fehler nur bei scans einer Suchmaschine auftritt?

Viele Grüße

Heinz

[dbs]

Hallo, da es keine Swift Gallery im Add-ons Repo gibt und du keine Modulversion nennst, könntest du das Modul hier mal als Anhang reinsetzen.

[bbs2]

hallo,

es ist ein sehr schönes, jedoch altes Modul aus der Zeit WB 2.8. Es läuft für den Normaluser
jedoch gut unter WB 2.12.x und PHP 7.2
Swift Version 0.6

Siehe Anlage

[hgs]

Das Modul, so wie du es hier anbietest, kann icht installiert werden.
Wo ist das Modul her?

[hgs]

Ok, scheint von Github zu sein
Konnte es nach "umformatieren" installieren.
Schmeißt ne Menge Fehler in php 7.3

Code: [Select]

Fri, 22 Feb 2019 11:04:11 +0000 [E_NOTICE] /modules/swift/save.php:[45] from /modules/swift/save.php:[45] bin\Exceptions\ErrorHandler::handler "Undefined index: default"
Fri, 22 Feb 2019 11:04:55 +0000 [E_NOTICE] /modules/swift/save.php:[45] from /modules/swift/save.php:[45] bin\Exceptions\ErrorHandler::handler "Undefined index: default"
Fri, 22 Feb 2019 11:06:38 +0000 [E_WARNING] /modules/swift/view.php:[80] from /templates/DefaultTemplate/index.php:[34] page_content "count(): Parameter must be an array or an object that implements Countable"
Fri, 22 Feb 2019 11:07:09 +0000 [E_NOTICE] /modules/swift/save.php:[45] from /modules/swift/save.php:[45] bin\Exceptions\ErrorHandler::handler "Undefined index: default"
Fri, 22 Feb 2019 11:07:12 +0000 [E_WARNING] /modules/swift/view.php:[80] from /templates/DefaultTemplate/index.php:[34] page_content "count(): Parameter must be an array or an object that implements Countable"
Fri, 22 Feb 2019 11:07:31 +0000 [E_WARNING] /modules/swift/view.php:[80] from /templates/DefaultTemplate/index.php:[34] page_content "count(): Parameter must be an array or an object that implements Countable"
Fri, 22 Feb 2019 11:07:33 +0000 [E_WARNING] /modules/swift/view.php:[80] from /templates/DefaultTemplate/index.php:[34] page_content "count(): Parameter must be an array or an object that implements Countable"
Fri, 22 Feb 2019 11:07:38 +0000 [E_WARNING] /modules/swift/view

[jacobi22]

es ist ein sehr schönes, jedoch altes Modul aus der Zeit WB 2.8. Es läuft für den Normaluser
jedoch gut unter WB 2.12.x und PHP 7.2

jau, mag sein, aber mehr darfst du dem Modul auch nicht mehr zumuten, PHP 7.3.x ist nicht drin und ein NeuInstall unter Normalbedingungen auch nicht mehr
vom Code her gibt es keine Zeile 83 im Zip und der Code stimmt dann auch nicht mit obigem Code überein, von daher schlecht zu sagen, was da passiert

Allerdings.... frontend.functions. php ~ Zeile 225 (wenn sie denn noch original ist) betrifft die Suche über das Frontend, hier speziell das Highlighting der Suchbegriffe. Die Kombination wäre im Normalfall mein Ansatzpunkt. Ich denke, da würde ich mir mal gezielt ein paar Messages in die error.log schreiben lassen, was denn von dieser Zeile 83 im Script auch ankommt

Code: [Select]

$images = array_slice( $all_images, $start, $settings['thumbs_per_page']);$all_images ist der Array mit den Bildern
$start eine numerische Zahl (bzw soll eine numerische Zahl sein),
$settings['thumbs_per_page'] ebenfalls eine numerische Zahl

lt der Fehlermeldung ist eine der beiden wohl keine Zahl
Gehe ich davon aus, das die Einstellung der settings auf jeden Fall richtig ist, bleibt mir dieses Stück Code als Fehlerquelle

if( isset( $_REQUEST['start'] ) ) {
        $start = $_REQUEST['start'];
}
else {
        $start = 0;
}

Hier fehlt jegliche Absicherung in Bezug auf numerische Werte. $_REQUEST['start'] kann alles mögliche sein, von einer echten Zahl bis hin zu Parametern, die du nie im Leben erwartest, um jetzt mal nix von möglichen Anweisungen per SQL- oder Code-Injections zu sagen. Du glaubst garnicht, was da alles möglich ist

[evaki]

Anhang:
swift_gallery_v0.6_ origin.zip
swift_gallery_v0.6_ BBS2.zip

[bbs2]

Hallo,

ich hatte das Modul vor ca. 10 Jahren installiert und von der damals offiziellen
WB_Seite geladen. Leider habe ich die damalige zip-Datei nicht mehr.

Die von mir im vorigen post gelieferte zip-Datei war lediglich der komprimierte Swift-Ordner
aus der Installation.

Ich habe jedoch nun noch folgenden Link gefunden:

http://www.websitebakers.com/pages/modules/image-galleries/swift-gallery.php

Viele Grüße

Heinz

[dbs]

Es ist schon erstaunlich, dass das alte Teil noch läuft und sich sogar installieren lässt (nicht strict).
Habe in der Adresszeile mal eine gallery_id eingegeben, die es nicht gibt. Da zeigt er statt einer Galerie die URL zum WB upgrade-script.

In der view.php Zeile 83 hab ich leider eine Leerzeile. Das scheint nicht die gleiche Version zu sein, die du benutzt, oder?

[evaki]

@bbs2
Der Link zeigt auf v0.6.6.0,
$module_name "Swift Gallery 2"
$module_directory   = "swift2";
Ist also eine andere Baustelle.

Betreff v0.6
Mit ein paar Korrekturen sind Fehlermeldungen wie "Parameter must be an array or an object that implements Countable" unter php7.2 nicht mehr zu sehen.

Ob unter v7.3 Lauffähig wurde noch nicht getestet.
Es gäbe auch noch Änderungen Funktionen betreffend zu tätigen. (Z.B. sectionID wegen mehreren Modulen auf einer Seite)
Da das Teil bis dato bei "uns" funktioniert, kommt der nächste Aufschrei evtl. mit dem Wechsel zur nächsten PHP-Version.

[bbs2]

Danke für die viele Hinweise.

Ich habe mal die Variabel $start generell auf 0 gesetzt.
$start=0;
Ob dies in allen Nutzungsumgebungen funktioniert kann ich leider nicht beurteilen.
Bisher läuft es bei mir fehlerfrei.

Gruß
Heinz

[evaki]

Was ist denn nun installiert, v0.6 oder v0.6.6.0?
Und "PHP Fehlerberichte: Development" unter Optionen eingestellt?
MfG. Evaki

[bbs2]

Es ist 0.6 installiert.

[evaki]

Und "PHP Fehlerberichte: Development" unter Optionen eingestellt?
Falls ja, mal die Galerie im Frontend aufrufen, und dann im error-log reinschauen.


ps.Ich muß jetzt erstmal an den Backofen: Käse mit Preiselbeeren in Blätterteig. Sind schön anzusehen und schmecken!

Sehe gerade, daß jacobi22 online ist. Der kann Dir dazu viel mehr sagen als ich.
MfG. Evaki

[bbs2]

Hallo,

der Hinweis im Forum auf den möglichen Fehler mit der Variable $start war wohl richtig.
Mit der festen Belegung $start=0 scheint es zu funktionieren.
Ich habe alle swift-Galleries im frontend aufgerufen. Die error-log bleibt bis jetzt leer.

Vielleicht hat jemand im Forum Interesse, das Modul, evtl. die Version Swift2, 0.66, auf den heutigen Stand zu ertüchtigen.
Meine php-Kenntnisse reichen dazu leider nicht aus. Das Modul wäre es wert.

Viele Grüße
Heinz

[jacobi22]

Ich stell das hier mal rein...

Auf Basis der letzt bekannten Module-Version SwiftGallery v0.6 hab ich das Modul mal neu aufgesetzt. Minimum WB-Version ist WB 2.12.0, eine zweite Version, die ebenfalls fertig ist, setzt auf die demnächst erscheinende WB 2.12.2 auf. Diese bietet u.a. andere Möglichkeiten im Dateihandling innerhalb der neuen Module.
Da diese WB-Version aber noch nicht offiziell ist, mußte ich hier noch einmal rückbauen. Die Nachfolgeversionen werden aber als Minimum WB 2.12.2 haben.
Bei den PHP-Versionen habe ich bis PHP 5.6.1 getestet, aufwärts bis PHP 7.3.1

Idee des Moduls
Mit diesem Modul können Verzeichnisse unter /media ausgelesen werden. Die ermittelten Grafiken werden in einer Galerie dargestellt, das jeweils erste Bild als großes Bild, alle weiteren als Thumbnails je nach CSS daneben oder darunter. Die Bildgrößen vom Hauptbild und den Thumbnails ist definierbar, ebenso die Anzahl der Bilder pro Seite. Möglich ist ein Galerie-Name und eine Galeriebeschreibung . Beides kann über die Moduleinstellungen ein- und ausgeschaltet werden.
Das Modul benutzt keine Third-Party-Scripte wie z.b. JQuery-Effekte

Unterschiede zur Vorgänger-Version 0.6:
 - Positionen der Galerien in der Reihenfolge veränderbar - damit wäre die Menü-Darstellung weiterer Galerien in dieser Sektion veränderbar
 - Möglichkeit zum Anlegen einer Bildtitelliste (kann auch als kurze Bildbeschreibung verwendet werden). Die Beschreibung erscheint im title-Attribute des großen Bildes
 - Galerien können aktiv/inaktiv geschaltet werden
 - das Modul legt beim Install einen Ordner "swift" unter media an, in dem neue Bilder abgelegt werden können, einen Zwang zur Nutzung gibt es aber nicht
 - bei der Galeriebeschreibung wird ein Wysiwyg-Editor eingesetzt
 - die Standard-Galerie kann in der Übersicht und auch in den Moduleinstellungen geändert werden

Die neue Moduleversion ist kompatibel zur Ausgangs-Version 0.6. Die vorhandenen Einstellungen werden übernommen

Zum Bild-Titel noch:
in den Gallery-Einstellungen gibt es ein Textfeld - pro Zeile ein Eintrag. Das Textfeld bezieht sich auf alle Bilder im gewählten Verzeichnis, also 100 Bilder = 100 mögliche Zeilen. Eine Leerzeile macht den Titel für dieses Bild leer - Ist dieser Bildtitel leer oder wurde überhaupt keiner angegeben, erscheint der Dateiname

Ein paar Bilder
Modulübersicht - Backend -> https://i.gyazo.com/511e2040a37e6f116e313ebd853eface.png
Galerie bearbeiten -> https://i.gyazo.com/baee10304b5d080e9dfa3f0b034dbc00.png
Galerie Einstellungen -> https://i.gyazo.com/cd5c6fef52ce1a7c2ca7ca9278b82dae.png
Frontend-Ausgabe -> https://i.gyazo.com/3408b8f31edf45f3e22d76aa0e881b9e.png

Wie gehts weiter?
Das Modul läuft z.Z. noch in ein paar privaten Test's auf verschiedenen Servern. Vergessen hab ich wohl die Absicherung auf Grafiken, war im alten Modul auch nicht drin, ist aber schnell gemacht. Nach Abschluß meiner Testerei geht es noch an ein paar andere Tester und je nach Rückmeldungen werde ich es dann zeitnah hier veröffentlichen. Wer Interesse hat, kann sich ja per PN melden. Ich bin mir zwar sicher, das alles so läuft, wie erwartet und habe mittlerweile auch schon diverse und auch komplexe Uprades gefahren, empfehle aber dennoch für den Test eine nicht öffentliche Version und keine Produktiv-Seite

Wie auch das Vorgänger-Modul, verwendet die neue Version keinerlei Fremdscripte, z.b. für JQuery-Effekte. Es kann aber, das nötige Wissen vorausgesetzt, mit wenig Aufwand und auch überschreibungssich er bei Upgrades mit wenig Aufwand erweitert werden, z.b. mit einer Colorbox.

Für Anregungen, Erweiterungsvorschl äge etc ruhig auch eine PN nutzen

[bbs2]

Hallo,

ich haben heute eine neue Swift-Gallery eingestellt. Diese läuft testweise mit der neuesten view_09dev und der
save-Version vom 9.3.2019 (1a)
Im frontend und backend zeigt sich bis jetzt kein Fehler. Siehe:

https://bbsiikl.de/pages/schulleben/unterrichtsprojekt-gemeinsam-lernen/bildergalerie-gemeinsam-lernen.php?gallery_id=91&start=10

Schön wäre es, wenn die Pfadangabe je nach Wunsch deaktiviert werden könnte.

Viele Grüße

Heinz Werner

[evaki]

gehst du in view.php
zeile: echo $SWIFTTEXT['PATH'].': ',$gallery_url;
echo "<br>";
machst du
//echo $SWIFTTEXT['PATH'].': ',$gallery_url;
//echo "<br>";

färtisch

MfG. Evaki

[evaki]

Anbei für'n Test das komplette Modul: swift_gallery_v0.7. 1-dev

Wäre nicht schlecht, wenn ich damit dann "durch wäre".
Unter sql strict wurde das Modul  von mir übrigens nicht getestet.
MfG. Evaki

[evaki]

Die  swift_gallery_v0.7. 1-dev hat gegenüber den bisherigen Fixes noch einen korrigierten Tabelleneintrag, der den fehlerhaften Zustand nach Installation/Unnamed Gallery betrifft.
(Das betraf das an anderer Stelle erwähnte Verhalten bei Ersterstellung einer Galerie, was mich irritierte.)
Sieht so aus, als wenn der Fehler schon länger durchgeschleppt wurde, was ja nicht so selten vorkommt.

Man könnte das default Verzeichnis /media auch auf /media/swift ändern. Wäre "eleganter", weil leer. Kann ja ein Nachfolger machen, falls gewünscht...
MfG. Evaki

[dbs]

Moin, hab 2 Galerien erstellt. Fehlermeldungen vorn oder hinten nicht zu sehen. Galerien funktionieren.
Aber der Pfad bleibt nicht erhalten, nach Speichern ist er leer. Nochmaliges Speichern würde also das vorher gewählte Verzeichnis entfernen.

[evaki]

Aber der Pfad bleibt nicht erhalten

Du meinst im BE? Falls ja, hätte es m.E. sich schon in der 0.6 so verhalten. Das wäre dann etwas für die Coder, zu schauen ob das so überhaupt programmiert wurde. Hatte ich schon unter anderen Topic mal erwähnt - glaub ich wenigstens...
Aber ansonsten funktionieren die Galerien? (Bei mir zumindest schon  )

Nochmaliges Speichern würde also das vorher gewählte Verzeichnis entfernen.

Hat mich auch irritiert wg. Reihenfolge. Wenn, dann immer neue Galerie und dann Vergabe. Mußte mal durchprobieren. So ganz konsequent in der Handhabung scheint das nicht an jeder Stelle durchdacht zu sein, oder es ist noch'n Fehler offen  - neu oder durchgeschleppt -   

MfG. Evaki

[dbs]

Du meinst im BE? Falls ja, hätte es m.E. sich schon in der 0.6 so verhalten.

In einer Vorversion fiel mir das nicht auf. Aber ich würde da nicht drauf wetten.

Aber ansonsten funktionieren die Galerien? (Bei mir zumindest schon  )

Moin, hab 2 Galerien erstellt. Fehlermeldungen vorn oder hinten nicht zu sehen. Galerien funktionieren.

Nochmaliges Speichern meint zB wenn der Galerie Titel geändert wird.

[evaki]

Nochmaliges Speichern meint zB wenn der Galerie Titel geändert wird.

Das habe ich aktuell zumindest nicht mehr gemacht, ::hu::
Meine mich aber zu erinnern, daß auch schon vorher.....
Da es nicht wirklich etwas kaputt macht, könnte man es so lassen - wenn schon auch vorher...
Manche Sachen entdeckt man erst, wenn man dran rumfummelt -ähm...
Die "eigentlichen" Funktionen - bis auf count() und ein zusätzlicher Tabelleneintrag - wurden nicht angetastet! Alles andere über "feindliche Übernahme" geregelt, also bestehende Variable in eine neue kopiert/übernommen, und damit eigenständig weitergearbeitet/ergänzt.

Hoffentlich wird jetzt kein Rattenschwanz draus.

Vielleicht kann BBS2 mal den Vergleich OLD/NEU starten.
MfG. Evaki

[evaki]

Falls nix besonderes mehr kommen sollte - hab' ich selbst noch 'ne Überraschung.
Anscheinend ist das Teil noch nie auf Sicherheit getestet worden, nun schon.
Mal schauen, ob ich dafür 'nen Fix hinbekomme. Falls nicht, werde ich 'nen DEV drum bitten.
Was nun konkret vorliegt, wird natürlich erst nach 'nem Fix bekannt gegeben.

Aber keine Panik bitte, wenn von 2006 bis heute nichts Aufregendes passiert ist...
MfG. Evaki