Update Image class, remove PNG -> JPEG conversion and use orientate() method to detect proper orientation in Exif

pull/86/merge
Daniel Supernault 2018-06-02 15:29:33 -06:00
rodzic ae6a59e782
commit b437b62780
1 zmienionych plików z 10 dodań i 31 usunięć

Wyświetl plik

@ -105,42 +105,25 @@ class Image {
$orientation = $ratio['orientation'];
try {
$img = Intervention::make($file);
$img = Intervention::make($file)->orientate();
$img->resize($aspect['width'], $aspect['height'], function ($constraint) {
$constraint->aspectRatio();
});
$converted = $this->convertPngToJpeg($path, $thumbnail, $img->extension);
$converted = $this->setBaseName($path, $thumbnail, $img->extension);
$newPath = storage_path('app/'.$converted['path']);
$is_png = false;
if($img->extension == 'png' || $converted['png'] == true) {
\Log::info('PNG detected, ' . json_encode([$img, $converted]));
$is_png = true;
$newPath = str_replace('.png', '.jpeg', $newPath);
$img->encode('jpg', 80)->save($newPath);
if(!$thumbnail) {
@unlink($file);
}
\Log::info('PNG SAVED, ' . json_encode([$img, $newPath]));
} else {
\Log::info('PNG not detected, ' . json_encode([$img, $converted]));
$img->save($newPath, 75);
}
$img->save($newPath, 75);
if(!$thumbnail) {
$media->orientation = $orientation;
}
if($is_png == true) {
if($thumbnail == false) {
if($thumbnail == true) {
$media->thumbnail_path = $converted['path'];
$media->thumbnail_url = url(Storage::url($converted['path']));
} else {
$media->media_path = $converted['path'];
$media->mime = $img->mime;
}
}
if($thumbnail == true) {
$media->thumbnail_path = $converted['path'];
$media->thumbnail_url = url(Storage::url($converted['path']));
}
$media->save();
@ -150,18 +133,14 @@ class Image {
}
}
public function convertPngToJpeg($basePath, $thumbnail = false, $extension)
public function setBaseName($basePath, $thumbnail = false, $extension)
{
$png = false;
$path = explode('.', $basePath);
$name = ($thumbnail == true) ? $path[0] . '_thumb' : $path[0];
$ext = last($path);
$basePath = "{$name}.{$ext}";
if($extension == 'png' || $ext == 'png') {
$ext = 'jpeg';
$basePath = "{$name}.{$ext}";
$png = true;
}
return ['path' => $basePath, 'png' => $png];
}