Projet

Général

Profil

Actions

Anomalie #1363

fermé

PB de paramètre dans lib/Galette/repository/Members.php

Ajouté par Daniel BABAULT il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
Core
Version cible:
Début:
26/11/2019
Echéance:
% réalisé:

100%

Temps estimé:
Version utilisée:

Description

Dans la fonction getList de lib/Galette/repository/Members.php, le paramètre $full n'est pas pris en compte.

Le paramètre correspondant à passer à getMembersList est $limit qui est l'inverse de $full.

/**
   * Get members list
   *
   * @param boolean $as_members return the results as an array of
   *                            Member object.
   * @param array   $fields     field(s) name(s) to get. Should be a string or
   *                            an array. If null, all fields will be
   *                            returned
   * @param boolean $full       Whether to return full list
   *
   * @return Adherent[]|ResultSet
*/
public function getList($as_members = false, $fields = null, $full = true) {
return $this->getMembersList(
$as_members,
$fields,
false,
false,
false,
true,
false
);
}

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Catégorie mis à Core
  • Assigné à mis à Johan Cwiklinski
  • Version cible mis à 0.9.3
  • Version utilisée changé de 0.9.3 à 0.9.2.1

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

En effet, il y a un delta ; mais ce code semble ne pas être utilisé (bon, j'ai peu de temps, je n'ai pas eu l'occasion d'investiguer plus en profondeur).

Ce problème est-il apparu sur une action particulière qui n'aurait pas fonctionné comme escompté ? Sinon, le plus simple serait sûrement de virer ce paramètre.

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de Nouveau à Commentaire

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de Commentaire à In Progress

Johan Cwiklinski a écrit :

En effet, il y a un delta ; mais ce code semble ne pas être utilisé (bon, j'ai peu de temps, je n'ai pas eu l'occasion d'investiguer plus en profondeur).

Okay, j'avais mal regardé. Ce paramètre ne doit pas être utilisé, je vais juste le supprimer.

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de In Progress à Résolu
  • % réalisé changé de 0 à 100

Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Statut changé de Résolu à Fermé
Actions

Formats disponibles : Atom PDF