Seite 2 von 3

Re: phpBB Gallery

Verfasst: 26.06.2017 23:28
von HJW
canonknipser hat geschrieben:($config['num_images'] + 0)
Ich habe unter php 7 bei so einer Konstruktion aber auch schon eine Fehlermeldung erhalten (bei mir war es *1) bei php <7 war das kein Problem.
ware das nicht dann richtig:

Code: Alles auswählen

intval($config['num_images'])

Re: phpBB Gallery

Verfasst: 27.06.2017 06:14
von canonknipser
Typumwandlung ist immer eine Krücke ;) Kann sein, das php 7.x da stärker prüft.

intvalfunktioniert nur dann, wenn schon eine Zahl oder ein numerischer String vorliegt; hier ist es aber "Zufall", was in den Daten steht.

Die Daten sollten normalerweise ordentlich initialisiert werden, das scheint hier bei der Extension zu fehlen.

Re: phpBB Gallery

Verfasst: 27.06.2017 07:12
von oxpus
canonknipser hat geschrieben:[...]
intvalfunktioniert nur dann, wenn schon eine Zahl oder ein numerischer String vorliegt; hier ist es aber "Zufall", was in den Daten steht.

Die Daten sollten normalerweise ordentlich initialisiert werden, das scheint hier bei der Extension zu fehlen.
Das ist so nicht richtig.
intval() liefert die in einer Variable erste gefundene Integer-Zahl zurück ansonsten im Normalfall immer 0.
Ausser die Variable wäre eh schon nicht korrekt, also z. B. leer oder null.
Die Migration der Gallery setzt den Wert für num_images initial auf 0, von daher scheint php 7(.1) hierbei etwas sehr empfindlich zu sein.
Dennoch wäre intval() hierzu die beste und sauberste Methode, den Wert umzuwandeln.

BTW:
Alle numerischen Werte in der config-Tabelle werden durchgängig als Strings abgelegt und ohne Probleme im gesamten Forum je nach Anwendungsfall auch einfach als numerische Werte verwendet. Da wandelt php (auch 7.1) automatisch von String in Zahl um.
Vielleicht war aber hier num_images doch leer (null) und dann macht auch die mathematische Operation unter php immer Fehler. Und das bereits seit Urzeiten von php.

Re: phpBB Gallery

Verfasst: 27.06.2017 11:02
von CoolCheffe
Hallo,
ich möchte die phpBB Gallery installieren.
Habe die Ext von https://github.com/satanasov/phpbbgallery/tree/1.2.1 in mein Testforum hochgeladen.

Nun sehe ich im ACP 4 deaktivierte Erweiterungen.
In welcher Reihenfokge müssen sie aktiviert werden ?

Gruß CC

Re: phpBB Gallery

Verfasst: 27.06.2017 14:37
von oxpus
Am besten erst die Gallery und dann die Add ons.

Re: phpBB Gallery

Verfasst: 27.06.2017 20:38
von sv3n
canonknipser hat geschrieben:Ich würde das:

Code: Alles auswählen

$images_per_day = sprintf('%.2f', $config['num_images'] / $boarddays);
in das ändern:

Code: Alles auswählen

$images_per_day = sprintf('%.2f', ($config['num_images'] + 0) / $boarddays);
- damit wird auch ein nicht existenter Wert in eine Zahl umgewandelt und die Ergebnisse werden nicht verfälscht ;)
Danke, und in welcher Datei änder ich das?

Re: phpBB Gallery

Verfasst: 27.06.2017 21:05
von HJW
sv3n hat geschrieben:
canonknipser hat geschrieben:Ich würde das:

Code: Alles auswählen

$images_per_day = sprintf('%.2f', $config['num_images'] / $boarddays);
in das ändern:

Code: Alles auswählen

$images_per_day = sprintf('%.2f', ($config['num_images'] + 0) / $boarddays);
- damit wird auch ein nicht existenter Wert in eine Zahl umgewandelt und die Ergebnisse werden nicht verfälscht ;)
Danke, und in welcher Datei änder ich das?
Mhhhm, hast du dir deine Fehlermeldung mal durchgelesen?
[phpBB Debug] PHP Warning: in file [ROOT]/ext/phpbbgallery/core/acp/main_module.php on line 520: A non-numeric value encountered

Re: phpBB Gallery

Verfasst: 27.06.2017 21:20
von sv3n
HJW hat geschrieben:
sv3n hat geschrieben:
canonknipser hat geschrieben:Ich würde das:

Code: Alles auswählen

$images_per_day = sprintf('%.2f', $config['num_images'] / $boarddays);
in das ändern:

Code: Alles auswählen

$images_per_day = sprintf('%.2f', ($config['num_images'] + 0) / $boarddays);
- damit wird auch ein nicht existenter Wert in eine Zahl umgewandelt und die Ergebnisse werden nicht verfälscht ;)
Danke, und in welcher Datei änder ich das?
Mhhhm, hast du dir deine Fehlermeldung mal durchgelesen?
[phpBB Debug] PHP Warning: in file [ROOT]/ext/phpbbgallery/core/acp/main_module.php on line 520: A non-numeric value encountered
immt garnicht mehr dran gedacht so auf die schnelle. Aber geht bei mir nicht mit der Änderung von canonknipser

Änder ich es so: viewtopic.php?p=1370100#p1370100 geht es .!

Re: phpBB Gallery

Verfasst: 30.06.2017 00:54
von CoolCheffe
Ausgezeichnete Ext !!! Mein Lob dem(n) Author(en)

Installiert und in Betrieb genommen, ohne Fehler. Keine Probleme, gut gemacht. :grin: :grin: :grin:

Gruß CC

Re: phpBB Gallery

Verfasst: 30.06.2017 11:46
von CoolCheffe
Hallo,
nachdem ich nun fast alles in der Gallery getestet habe ist mir folgendes aufgefallen.

Wenn ich bei Bilder bearbeiten den " Vollständigen Editor" anwähle erscheint folgende Seite.

Externes Bild : [ externes Bild ]

Wenn ich nun den Bildtitel oben anklicke (grüne Pfeile im Bild) erscheint eine weiße Seite. Zurück geht dann nur über den Browser zurück Button.

Frage: Wie kann das geändert werden ?

Gruß CC


Edit: Vielleicht hilft das noch weiter, der Link in der Browser Eingabezeile sieht ja nachdem wo ich anklicke wie folgt aus:

Code: Alles auswählen

https://meine domain.de/..../gallery/image/1/medium
oder

Code: Alles auswählen

https://meine domain.de/..../gallery/image/1