Fix undefined variable in AJAX remove after market→repo migration#3118
Open
kwizer15 wants to merge 1 commit intojeedom:alphafrom
Open
Fix undefined variable in AJAX remove after market→repo migration#3118kwizer15 wants to merge 1 commit intojeedom:alphafrom
kwizer15 wants to merge 1 commit intojeedom:alphafrom
Conversation
Contributor
|
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. |
Sekiro-kost
approved these changes
Jan 30, 2026
Contributor
Author
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. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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
$repoen$marketdans 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
PR checklist