Skip to content
Extraits de code Groupes Projets

fix: Correction dans l'affichage du message d'erreur en cas d'adresse incorrect du dépôt

Fusionnées Maïeul requested to merge issue_4908_message_erreur into master
1 fil de conversation non résolu

Rapports de requête de fusion

Approuvée par

Fusionnée par marcimatmarcimat il y a 10 mois (mars 20, 2024 11:00am UTC)

Détails de la fusion

Activité

Filtrer l'activité
  • Approbations
  • Assignés et relecteurs
  • Commentaires (des bots)
  • Commentaires (des utilisateurs)
  • Branches et validations
  • Modifications
  • Labels
  • État de verrouillage
  • Mentions
  • État de la demande de fusion
  • Suivi
  • RastaPopoulos approved this merge request

    approved this merge request

    • Maintainer

      Avant de valider la PR j'aimerai reproduire le bug pour être certain que le fix est bon, tu peux nous dire comment tu obtiens cette erreur ?

    • Auteur Maintainer

      Ca a été compliqué à trouver car pour le coup l'erreur affichée n'était pas correcte.

      Il s'agit d'un bug plus en amont dans recuperer_url_distante() sur un cas très particulier avec un cite dont la base de donnée avait été remis à neuf, mais qui avait encore des fichiers tmp/cache/rid correspondant au fichier à récuprer. Bref le genre de choses quasiment irreproductible IRL, et pour lequel je n'ai même pas eu le courage d'ouvrir un ticket et de faire une PR.

      Cela étant, même si l'erreur affichée n'était pas correcte, son affichage lui même était bugué.

      Et donc j'ai tenté de faire comme si je reproduis le premier bug.

      1. Site en local, sans dépot configuré.
      2. Dossier tmp/cache/rid vide
      3. Pas de connexion internet, et donc pas de possibilité de récuperer le fichier -> alors tu as ce message rouge qui apparait.
    • Effectivement.

    • Veuillez vous inscrire ou vous connecter pour répondre
  • marcimat approved this merge request

    approved this merge request

  • merged

Veuillez vous inscrire ou vous connecter pour répondre
Chargement en cours