SemanticMediaWiki/SemanticResultFormats

[mode] Mode format returns mode count, not the actual mode

Ouverte

#979 ouverte le 15 nov. 2025

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (0 personne assignée)JavaScript (87 forks)auto 404
good first issue

Métriques du dépôt

Stars
 (52 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

It seems that the mode format returns how many times the mode appears, not the actual mode itself https://github.com/SemanticMediaWiki/SemanticResultFormats/blob/42c26905f5d9f25a321a0c3b286d5b0d239a8083/formats/math/SRF_Math.php#L187 https://github.com/SemanticMediaWiki/SemanticResultFormats/blob/42c26905f5d9f25a321a0c3b286d5b0d239a8083/formats/math/SRF_Math.php#L189 https://github.com/SemanticMediaWiki/SemanticResultFormats/blob/42c26905f5d9f25a321a0c3b286d5b0d239a8083/formats/math/SRF_Math.php#L201 array_count_values gives counts as values, and max gets the max value, meaning $max is a count, not the most common property value

It seems the unit test even expects the count instead of the actual mode

https://github.com/SemanticMediaWiki/SemanticResultFormats/blob/42c26905f5d9f25a321a0c3b286d5b0d239a8083/tests/phpunit/Integration/JSONScript/TestCases/math-01.json#L309-L313

https://github.com/SemanticMediaWiki/SemanticResultFormats/blob/42c26905f5d9f25a321a0c3b286d5b0d239a8083/tests/phpunit/Integration/JSONScript/TestCases/math-01.json#L94-L95

https://github.com/SemanticMediaWiki/SemanticResultFormats/blob/42c26905f5d9f25a321a0c3b286d5b0d239a8083/tests/phpunit/Integration/JSONScript/TestCases/math-01.json#L14-L15

The mode should be 333, the only number, but instead it expects and gets how many times 333 occurs

Guide contributeur

Direction de recherche
Inspectez les lignes 187-201 de SRF Math.php et le fichier de test math 01.json pour comprendre comment array count values est utilisé. Corrigez la logique pour retourner la valeur modale réelle au lieu du décompte.
Stack technique
php
Domaine
backendapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitPHP
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70