Project

General

Profile

Development #27407

ajouter une période prédéfinie "depuis le 1er janvier de l'année précédente"

Added by Benjamin Dauvergne 8 months ago. Updated 5 months ago.

Status:
Fermé
Priority:
Normal
Start date:
18 Oct 2018
Due date:
% Done:

100%

Patch proposed:
Yes
Planning:
No

Description

Pour éviter une période vide au 1er janvier de l'année en cours.

0001-add-a-predefined-date-range-from-1st-january-of-prev.patch View (3.29 KB) Benjamin Dauvergne, 18 Oct 2018 07:47 PM

0001-engine-make-Dimension.order_by-a-list-fixes-28175.patch View (18.8 KB) Benjamin Dauvergne, 14 Jan 2019 11:01 AM

0002-visualization-add-setting-to-disable-cache-27407.patch View (2.07 KB) Benjamin Dauvergne, 14 Jan 2019 12:32 PM

0001-engine-only-use-join-in-filter-if-there-are-27407.patch View (935 Bytes) Benjamin Dauvergne, 14 Jan 2019 12:32 PM

0003-visualization-add-truncated-previous-year-range-fixe.patch View (5.66 KB) Benjamin Dauvergne, 14 Jan 2019 12:32 PM

0002-visualization-add-setting-to-disable-cache-27407.patch View (2.07 KB) Benjamin Dauvergne, 15 Jan 2019 01:01 PM

0001-engine-when-filtering-only-use-joins-if-defined-2740.patch View (939 Bytes) Benjamin Dauvergne, 15 Jan 2019 01:01 PM

0003-visualization-add-truncated-previous-year-range-fixe.patch View (5.66 KB) Benjamin Dauvergne, 15 Jan 2019 01:01 PM

Associated revisions

Revision 6451f27c (diff)
Added by Benjamin Dauvergne 5 months ago

engine: when filtering only use joins if defined (#27407)

Revision 86b169bd (diff)
Added by Benjamin Dauvergne 5 months ago

visualization: add setting to disable cache (#27407)

Revision 8d28b501 (diff)
Added by Benjamin Dauvergne 5 months ago

visualization: add truncated previous year range (fixes #27407)

It goes from 01/01 to 31/12 of previous year, i.e. if we are the
01/01/2019 it matches 01/01/2018-31/12/2018.

History

#1 Updated by Benjamin Dauvergne 8 months ago

#2 Updated by Benjamin Dauvergne 6 months ago

  • Tracker changed from Bug to Development

#5 Updated by Benjamin Dauvergne 5 months ago

Avec des tests et au passage:
  • une correction pour le cas, général en fait, où une dimension déclare avoir besoin de jointures pour ses filtres mais n'en a pas
  • un setting pour désactiver le cache qui est gênant pendant les tests, surtout si on jour avec freezegun

#6 Updated by Christophe Siraut 5 months ago

dans le premier patch le commentaire n'est pas complet: "if there are what"

un setting pour désactiver le cache qui est gênant pendant les tests, surtout si on jour avec freezegun

si c'est uniquement pour les tests, peut-être privilégier un monkeypatch que d'ajouter un paramètre de configuration?

#7 Updated by Christophe Siraut 5 months ago

  • Status changed from Solution proposée to Solution validée

#8 Updated by Benjamin Dauvergne 5 months ago

Christophe Siraut a écrit :

dans le premier patch le commentaire n'est pas complet: "if there are what"

Ben des "joins", "only use joins in filter if there are joins".. Je vais réecrire.

un setting pour désactiver le cache qui est gênant pendant les tests, surtout si on jour avec freezegun

si c'est uniquement pour les tests, peut-être privilégier un monkeypatch que d'ajouter un paramètre de configuration?

Je trouve beaucoup plus propre d'avoir un setting (les tests de django ont ce genre de choses).

#10 Updated by Benjamin Dauvergne 5 months ago

  • % Done changed from 0 to 100
  • Status changed from Solution proposée to Résolu (à déployer)

#11 Updated by Benjamin Dauvergne 5 months ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF