Project

General

Profile

Anomalie #601

Galette v0.7.4.1. Recherche avancée.. "Date de fin" ne fonctionne pas correctement.

Added by Roland Telle about 7 years ago. Updated about 7 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
05/01/2013
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

Description

Il semble que la requête qui joue sur la fourchette "Date de fin" de cotisation (entre ... et ...) ne soit pas correcte.

#1

Updated by Roland Telle about 7 years ago

Exemple : pour une recherche concernant la date de fin (de contribution) dans la fourchette comprise entre le 1/1/2013 et le 31/3/2013, le résultat est complètement farfelu. La requête affichable est la suivante :

SELECT DISTINCT `a`.*, `p`.* FROM `galette_adherents` AS `a`
 INNER JOIN `galette_statuts` AS `p` ON a.id_statut=p.id_statut
 LEFT JOIN `galette_cotisations` AS `ct` ON ct.id_adh=a.id_adh WHERE (ct.date_fin_cotis >= '2013-01-01') ORDER BY `nom_adh` ASC, `prenom_adh` ASC LIMIT 20

Il semble que le WHERE soit incomplet.

#2

Updated by Johan Cwiklinski about 7 years ago

  • Assignee set to Johan Cwiklinski

En effet, la requête est "incomplète", il manque ici la date de fin. Merci pour les détails :)

#3

Updated by Johan Cwiklinski about 7 years ago

À priori, le correctif suivant remet ça d'équerre :

diff --git a/galette/lib/Galette/Repository/Members.php b/galette/lib/Galette/Repository/Members.php
index f072fa9..861f98c 100644
--- a/galette/lib/Galette/Repository/Members.php
+++ b/galette/lib/Galette/Repository/Members.php
@@ -1141,7 +1141,7 @@ class Members
                             $d->format('Y-m-d')
                         );
                     }
-                    if ( $this->_filters->rcontrib_begin_date_end ) {
+                    if ( $this->_filters->rcontrib_end_date_end ) {
                         $d = new \DateTime(
                             $this->_filters->rcontrib_end_date_end
                         );
#4

Updated by Johan Cwiklinski about 7 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100
#5

Updated by Johan Cwiklinski about 7 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF