Skip to content

Conversation

@kwizer15
Copy link
Contributor

Description

Cette PR corrige un problème détecté par PHPStan : une variable $market était utilisée sans avoir été définie au préalable dans le processus de suppression de dépôt via AJAX.

En analysant le code, il semble que la variable $repo était initialement utilisée mais a été renommée en $market à un moment donné, probablement lors d'une évolution du système. Cependant, l'assignation de la variable n'avait pas été mise à jour en conséquence.

Le correctif renomme $repo en $market dans l'assignation pour maintenir la cohérence avec le reste du code qui utilise cette variable.

Suggested changelog entry

Correction d'une variable non définie détectée par PHPStan lors de la suppression de dépôt via AJAX

Related issues/external references

N/A

Types of changes

  • Bug fix (non-breaking change which fixes)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
    • This change is only breaking for integrators, not for external standards or end-users.
  • Documentation improvement

PR checklist

@zoic21
Copy link
Contributor

zoic21 commented Jan 30, 2026

Je suis pas sur l'idée était d'avoir plusieurs source repo (market) possible. Aujourd'hui on a que market mais demain on pourrait en avoir plus.

@kwizer15
Copy link
Contributor Author

Je suis pas sur l'idée était d'avoir plusieurs source repo (market) possible. Aujourd'hui on a que market mais demain on pourrait en avoir plus.

Je comprend tes doutes. L'objectif ici c'est de fixer en l'état. Pour gérer plusieurs repo tu pourras créer une interface avec la méthode static byId, et dans ce code tu vérifies que ta classe implemente bien la nouvelle inerface. Ca prend 5 minutes à faire. Seulement, encore une fois, ce n'était pas l'objectif ici.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants