cropper icon indicating copy to clipboard operation
cropper copied to clipboard

Gecropptes Medium behält alte Abmaße Höhe/Breite statt zugeschnittene Maße

Open AWqxKAWERbXo opened this issue 3 years ago • 3 comments

Description / Beschreibung Korrekt wäre, neue Breite/Höhe auszulesen und zu hinterlegen.

Siehe https://friendsofredaxo.slack.com/archives/C1BAXLN2F/p1668599007789409

AWqxKAWERbXo avatar Nov 16 '22 12:11 AWqxKAWERbXo

Und die Dateigröße

eaCe avatar Nov 16 '22 14:11 eaCe

Schneller Fix: im if-Block: https://github.com/FriendsOfREDAXO/cropper/blob/master/lib/Cropper/CropperExecutor.php#L200

        if (isset($result['ok']) && $result['ok'] == 1 && isset($result['filename'])) {
            $media = rex_media::get($result['filename']);
            $msg = 'cropper_successful_' . $msgType;
            $ok = true;

            $size = @getimagesize($this->zebraImage->target_path);

            $sql = rex_sql::factory();
            $sql->setTable(rex::getTable('media'));
            $sql->setWhere(['filename' => $result['filename']]);
            $sql->setValue('filesize', filesize($this->zebraImage->target_path));

            if ($size) {
                $sql->setValue('width', $size[0]);
                $sql->setValue('height', $size[1]);
            }

            $sql->update();
        } else {
            $msg = 'cropper_failed_' . $msgType;
            $ok = false;
        }

eaCe avatar Nov 16 '22 15:11 eaCe

Dabei können auch gleich die rex_mediapool-Methoden ausgetauscht werden

eaCe avatar Nov 16 '22 15:11 eaCe

Vielen Dank @eaCe: ist so übernommen.

alexwenz avatar Apr 12 '23 15:04 alexwenz