Project

General

Profile

Actions

Anomalie #1363

closed

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

Added by Daniel BABAULT almost 5 years ago. Updated almost 5 years ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
Start date:
11/26/2019
Due date:
% Done:

100%

Estimated time:
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
);
}
Actions #1

Updated by Johan Cwiklinski almost 5 years ago

  • Category set to Core
  • Assignee set to Johan Cwiklinski
  • Target version set to 0.9.3
  • Version utilisée changed from 0.9.3 to 0.9.2.1
Actions #2

Updated by Johan Cwiklinski almost 5 years ago

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.

Actions #3

Updated by Johan Cwiklinski almost 5 years ago

  • Status changed from Nouveau to Commentaire
Actions #4

Updated by Johan Cwiklinski almost 5 years ago

  • Status changed from Commentaire to 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.

Actions #5

Updated by Johan Cwiklinski almost 5 years ago

  • Status changed from In Progress to Résolu
  • % Done changed from 0 to 100
Actions #6

Updated by Johan Cwiklinski almost 5 years ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF