FriendsOfREDAXO / focuspoint

Erweitert den Medienpool um die Fähigkeit, den Fokuspunkt eines Bildes zu bestimmen.
MIT License
58 stars 7 forks source link

Deprecated Meldung #118

Closed olien closed 1 year ago

olien commented 1 year ago

Core und AddOns aktuell (aus dem Installer) / PHP 8.1.13

image

https://github.com/FriendsOfREDAXO/focuspoint/commit/3d4c11387e1a6f757d88f69e97209dff1f80c525

half leider nicht

christophboecker commented 1 year ago

@olien Bei mir kommt der Fehler nicht. Trotzdem: wenn ich (1) den Cache lösche und (2) nur die Bild-Url-Abrufe und (3) vor Zeile 221 ein dd(get_defined_vars()); setze, kommt beim Testbild diese Ausgabe:

[^]array:16 [▼
    "gdimage" => GdImage {#70 ▶}
    "sw" => 1291
    "sh" => 855
    "sr" => 1.5099415204678
    "fx" => 648
    "fy" => 447
    "dw" => 500
    "dh" => 500
    "zoom" => 0
    "dr" => 1
    "too_wide" => true
    "cw" => 500
    "ch" => 500
    "cx" => 398.0
    "cy" => 197.0
    "des" => GdImage {#72 ▶}

Die Nachkommastelle bei cx und cy verschwindet, wenn ich Zeile 197/198 auf int caste:

            $cx = (int)($fx - floor( $cw/2 ));
            $cy = (int)($fy - floor( $ch/2 ));

Könntest Du bitte die beiden Zeilen bei Dir einbauen und schauen, was sich ändert. Bzw. ob der Fehler dann weg ist.

Danke vorab.

olien commented 1 year ago

Hm. Die Zeilennummern passen nicht 100%ig. Wo genau sollte ich das DD machen? Hab keine dump hinbekommen.

Wenn ich $cx & $cy ändere bleibt der Fehler trotzdem

image

Interessanterweise kommt der nur beim ersten Aufruf des Bildes...

christophboecker commented 1 year ago

Cache löschen! Wenn das Bild einmal (falsch) drin ist .... DD verhindert dann, das der Cache neu geschrieben wird.

grafik

olien commented 1 year ago

image

christophboecker commented 1 year ago

Mit Skalierung in % konnte ich es nachstellen.
Ich lös das mal radikal und caste die Werte für imagecopyresampled auf int. Am Wochenende.

olien commented 1 year ago

Danke sehr 😘

christophboecker commented 1 year ago

@olien Du köntest mal mit der Git-Version testen, bitte, bevor ich das Release erstelle. Danke.

christophboecker commented 1 year ago

Hab einfach mal ein Release daraus gemacht.

olien commented 1 year ago

Klappt. DANKE SEHR!!!!!