Warning d'inspection dans une fonction
Dans la méthode suivante, la variable $retour n'est pas utilisée. Soit elle est inutile car on se fout du retour, soit il manque un traitement conditionnel au retour qui influerait sur la sortie de la fonction retirer().
Si quelqu'un a une idée pour qu'on corrige cela et si possible qu'on évite des return partout si le $retour est bien la variable de retour de la fonction retirer().
/**
* {@inheritDoc}
*/
public function retirer(array $fichiers = []): bool {
$archive = $this->archiveEnEcriture();
if ($archive) {
if (1 === $archive->open($this->fichier_archive, 'edition')) {
// Vérifier qu'on ne cherche pas à vider l'archive
$reste = $this->informer();
$fichiers_restants = array_column($reste['fichiers'], 'filename');
if (0 === count(array_diff($fichiers_restants, $fichiers))) {
$this->setErreur(8);
return false;
}
$retour = $archive->remove($fichiers);
$archive->close();
}
$this->setErreur(0);
return true;
}
return false;
}