Estoy teniendo problemas para extender un código existente que guarda imágenes.

Los usuarios pueden subir imagenes y también tienen que poder actualizarlas (eliminar y subir nuevas). Lo primero es lo que tengo terminado, pero lo segundo tengo dudas ya que no quiero guardar las imagenes con su nombre original y en su lugar uso el número del array:

$tmp_name   = $_FILES['file']['tmp_name'];
 
for($i = 0; $i < count($tmp_name); $i++) {
 
    $img_name   = '/' . ($i + 1) . '.jpg';
 
}

Por lo tanto si uso este código la segunda vez que suba imágenes, las que ya existen serán reemplazadas. No me importa si el orden se pierde (pero si es posible prefiero algo limpio):

1.jpg, 2.jpg, 3.jpg ---> 1.jpg, 2.jpg, 3.jpg, 4.jpg, 5.jpg

¿Conoceis alguna manera más limpia de hacer esto? No me importa el nombre que tengan, no es de ningún uso.

Podría usar la hora de subida, algo así, pero me gustaría saber vuestras opiniones y como lo haríais vosotros.

Esta es la función completa, ¿podria mejorarse para que fuera más eficiente?

public function UploadImages($files, $test = '') {
 
    $imagesPath     = '../static.website.com/images/property/' . $this->id;
    $thumbnailsPath = '../static.website.com/images/property/' . $this->id . '/thumbnails';
 
    $log = 'Logging Images Upload:' . "\n\n";
    $upload_err = FALSE;
 
    if ($test == 'new') {
 
        // Will be any problem with $thumbnailsPath?
        if (!mkdir($imagesPath) && !mkdir($thumbnailsPath)) {
 
            return array($upload_err, 'Upload Error: mkdir().');
 
        }
 
    }
 
    $imagesPath     = realpath($imagesPath);
    $thumbnailsPath = realpath($thumbnailsPath);
 
    // Is it ok to only check for the child directory?
    if ( FALSE !== $thumbnailsPath && is_dir($thumbnailsPath) ) {
 
        if (isset($files['file'])) {
 
            $tmp_name   = $files['file']['tmp_name'];
            $img_size   = $files['file']['size'];
 
            // At least 1 image
            if ($test == 'new' && $tmp_name[0] == '') {
 
                $upload_err = TRUE;
                return array($upload_err, '¡La primera imagen es necesaria!');
 
            }
 
            for($i = 0; $i < count($tmp_name); $i++) {
 
                if ($tmp_name[$i] != '') {
 
                    if ($img_size[$i] <= 7340032) {
 
                        $img_name       = '/' . ($i + 1) . '.jpg';
                        $thumbnail_name = '/thumbnail_' . ($i + 1) . '.jpg';
 
                        list($width, $height) = getimagesize($tmp_name[$i]);
 
                        if ($width >= 1100 && $height >= 650) {
 
                            $img    = imagecreatefromjpeg($tmp_name[$i]);
                            $img_p  = imagecreatetruecolor(1100, 650);
                            imagecopyresampled($img_p, $img, 0, 0, 0, 0, 1100, 650, $width, $height);
                            imagedestroy($img);
                            $img_t  = imagecreatetruecolor(180, 180);
                            imagecopyresampled($img_t, $img_p, 0, 0, 0, 0, 180, 180, 1100, 650);
 
                            if (!imagejpeg($img_p, $imagesPath . $img_name, 70) || !imagejpeg($img_t, $thumbnailsPath . $thumbnail_name, 70)) {
 
                                $upload_err = TRUE;
                                return array($upload_err, 'Error al subir la imagen (' . $files['file']['name'][$i] . ')');
 
                            } else $log .= 'IMG: ' . $img_name . ' (original name: ' . $files['file']['name'][$i] . ') uploaded successfully.' . "\n";
 
                            imagedestroy($img_p);
                            imagedestroy($img_t);
 
                        } else {
                            $upload_err = TRUE;
                            return array($upload_err, 'La imagen: ' . $files['file']['name'][$i] . ' debe tener como mínimo 1100px de ancho y 650px de alto.');
                        }
                    } else {
                        $upload_err = TRUE;
                        return array($upload_err, 'La imagen (' . $files['file']['name'][$i] . ') pesa más de 7 MB.');
                    }
                }
            }
        } else return array($upload_err, '$files[\'file\'] is not set:' . "\n" . var_dump($files['file']) . "\n" . 'Property ID: ' . $this->id);
    } else return array($upload_err, '$thumbnailsPath is false and/or not a dir: ' . $thumbnailsPath . '". Property ID: ' . $this->id);
 
    return $log;
 
}

Cualquier surgerencia es bienvenida. Gracias!

Si, ya habia leido en alguna parte lo de usar md5, mientras sea una opción segura. La única pega es que el output es un poco largo para mi gusto, pero bueno supongo que no es relevante (yo y mis manías  )

El array $_FILES con más de un archivo tiene la siguiente estructura:

array(1) {
  ["file"]=>
  array(5) {
    ["name"]=>
    array(3) {
      [0]=>
      string(6) "01.jpg"
      [1]=>
      string(6) "01.jpg"
      [2]=>
      string(6) "01.jpg"
    }
    ["type"]=>
    array(3) {
      [0]=>
      string(10) "image/jpeg"
      [1]=>
      string(10) "image/jpeg"
      [2]=>
      string(10) "image/jpeg"
    }
    ["tmp_name"]=>
    array(3) {
      [0]=>
      string(14) "/tmp/phpRBV68j"
      [1]=>
      string(14) "/tmp/phpjuvvG9"
      [2]=>
      string(14) "/tmp/phpJpxmoZ"
    }
    ["error"]=>
    array(3) {
      [0]=>
      int(0)
      [1]=>
      int(0)
      [2]=>
      int(0)
    }
    ["size"]=>
    array(3) {
      [0]=>
      int(74357)
      [1]=>
      int(74357)
      [2]=>
      int(74357)
    }
  }
}

Si, recibe varios a la vez.Estoy usando el plugin dropzone con la opción de multiples archivos.

Vale pero md5_file se lo hago al tmp_name y al nombre? Son dos cosas distintas. No sería md5 a name y md5_file a tmp_name?

Ah claro ya entiendo, si tiene sentido.

Gracias!