Project

General

Profile

Tâche #35420

Scénario #35316: Impossibilité d'éditer les fichiers à supprimer sur le client : fichier non éditable (binaire)

Ne pas considérer un fichier absent comme un fichier binaire

Added by Benjamin Bohard 12 months ago. Updated 12 months ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Start date:
10/01/2022
Due date:
% Done:

100%

Remaining (hours):
0.0

Associated revisions

Revision fbe9fdc8 (diff)
Added by Benjamin Bohard 12 months ago

L’erreur en cas de fichier absent est légitime.

Ref #35420

Revision 221aa8ee (diff)
Added by Benjamin Bohard 12 months ago

Déplacer la gestion de l’erreur dans les fonctions appelant istextfile

Retour sur la gestion des erreurs liées au contenu d’un fichier.
Le formulaire s’attend à ce que le backend retourne une erreur pour
savoir si le fichier removed est disponible ou non.

Propagation d’une erreur si le fichier n’existe pas :
- plus de gestion d’erreur dans la fonction istextfile ;
- gestion explicite des cas d’existence ou non dans les fonctions
appelantes de istextfile ;
- plus d’affichage du traceback dans la fonction appelant istextfile ;
- gestion de l’erreur BackendError spécifiquement côté web.

Ref #35420

History

#1 Updated by Benjamin Bohard 12 months ago

  • Status changed from Nouveau to En cours

#2 Updated by Benjamin Bohard 12 months ago

La modification apportée dans le cadre de la tâche #33514 a amené à considérer un fichier comme binaire si il n’existe pas.
Il est peut-être nécessaire de revenir sur le problème de traceback.
Il y a un appel de traceback.print_exc explicite dans la fonction qui appelle istextfile qui pourrait être conditionné à un mode debug ou simplement supprimé si il est vraiment la cause de l’inconvénient remonté dans le scénario #33446.

#3 Updated by Benjamin Bohard 12 months ago

  • Status changed from En cours to À valider
  • % Done changed from 0 to 100

#4 Updated by Laurent Gourvenec 12 months ago

  • Status changed from À valider to Résolu

#5 Updated by Joël Cuissinat 12 months ago

  • Status changed from Résolu to Fermé
  • Remaining (hours) set to 0.0
zephir-parc$ git branch -r --contains fbe9fdc8
  origin/2.8.0/master
  origin/2.8.1/master
  origin/HEAD -> origin/master
  origin/dist/eole/2.8.0/master
  origin/dist/eole/2.8.1/master
  origin/dist/eole/2.9.0/master
  origin/master
zephir-parc$ git branch -r --contains 221aa8ee
  origin/2.8.0/master
  origin/2.8.1/master
  origin/HEAD -> origin/master
  origin/dist/eole/2.8.0/master
  origin/dist/eole/2.8.1/master
  origin/dist/eole/2.9.0/master
  origin/master

Also available in: Atom PDF