SemanticMediaWiki/SemanticResultFormats

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

Aperta

#979 aperta il 15 nov 2025

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (87 fork)auto 404
good first issue

Metriche repository

Star
 (52 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina le righe 187-201 di SRF Math.php e il file di test math 01.json per capire come viene utilizzato array count values. Correggi la logica per restituire il valore modale effettivo invece del conteggio.
Tech stack
php
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitPHP
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70