✅ Optimized in 3.3.2

AbstractDecoder::extractExifData() now uses either file paths or binary data as direct input for exif_read_data(), instead of always creating a file pointer and reading from it as before.

Old Implementation

image/src/Drivers/AbstractDecoder.php

Lines 76 to 91 in fd09f59

    
           protected function extractExifData(string $image_data): CollectionInterface 
        
           { 
        
               if (!function_exists('exif_read_data')) { 
        
                   return new Collection(); 
        
               } 
        
               try { 
        
                   $pointer = $this->buildFilePointer($image_data); 
        
                   $data = @exif_read_data($pointer, null, true); 
        
                   fclose($pointer); 
        
               } catch (Exception) { 
        
                   $data = []; 
        
               } 
        
               return new Collection(is_array($data) ? $data : []); 
        
           }

	protected function extractExifData(string $image_data): CollectionInterface
	{
	if (!function_exists('exif_read_data')) {
	return new Collection();
	}

	try {
	$pointer = $this->buildFilePointer($image_data);
	$data = @exif_read_data($pointer, null, true);
	fclose($pointer);
	} catch (Exception) {
	$data = [];
	}

	return new Collection(is_array($data) ? $data : []);
	}

Uh oh!

Improve Memory Usage #1269

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions