Compléter les historiques de Déclarations #1482
Conversation
| ) | ||
| ) | ||
| ] | ||
| ) |
There was a problem hiding this comment.
J'ai du mal à comprendre ce code. Je pense que c'est surtout car je n'ai pas les infos sur ica/vrs, population cible vs population... mais j'ai une question et une suggestion:
Question: est-ce qu'il existe une risque de MultipleObjectsReturned pour les gets ?
Suggestion:
cible_populations = Ica...Declaree.objects.filter(...)
populations = [Population.objects.get(...) for population in cible_populations]
declaration.populations.set(populations)
et pour les conditions aussi. Après, je sais pas si cible_populations est le bon nom de variable
| ) | ||
| ) | ||
| ] | ||
| ) | ||
| declaration.save() |
There was a problem hiding this comment.
je crois que c'est pas necessaire de faire ça après un set, mais faut tester
| ) | ||
| ) | ||
| ] | ||
| ) | ||
| declaration.save() | ||
| nb_created_declarations += 1 |
There was a problem hiding this comment.
je me demande si c'est mieux de mettre ça au dessus de ligne 278 si jamais il y a une erreur pas attendu avec les sets ?
| declaration.save() | ||
| nb_created_declarations += 1 | ||
|
|
||
| except IntegrityError: | ||
| # cette Déclaration a déjà été créée |
There was a problem hiding this comment.
pour confirmer : si la déclaration est déjà créée, on veut pas MAJ les populations ni les conditions ?
There was a problem hiding this comment.
oui et non.
Pour le moment cet import est plutôt pensé comme un import One-Shot.
Et même si je vais l'effectuer 2 fois, je pensais d'abord supprimer toutes les déclarations historiques (car elles ne peuvent pour le moment pas avoir été modifiées) pour les réimporter entièrement.
Mais c'est vrai que je pourrais envisager de juste mettre à jour des champs ManyToMany
| @@ -68,7 +68,7 @@ class Meta: | |||
|
|
|||
| dcl_ident = factory.Sequence(lambda n: n + 1) | |||
| cplalim = factory.SubFactory(ComplementAlimentaireFactory) | |||
| tydcl_ident = factory.Faker("pyint", min_value=0, max_value=20) | |||
| tydcl_ident = factory.Faker("pyint", min_value=0, max_value=2) | |||
There was a problem hiding this comment.
Dans TeleIcare, ce champ ne peut avoir que 3 valeurs (0, 1, 2) car il n'y à que trois types (Art 15, Art 16, Simplifiée)
| class Meta: | ||
| managed = False | ||
| db_table = "ica_population_cible_declaree" | ||
| unique_together = (("vrsdecl_ident", "popcbl_ident"),) |
There was a problem hiding this comment.
Est-ce que c'est possible d'avoir plusieurs lignes avec le même vrsdecl_ident alors ? Je demande car "primary_key=True implies null=False and unique=True." - https://docs.djangoproject.com/en/5.2/ref/models/fields/#primary-key
alors si il y a plusieurs qui sont les mêmes faut pas ajouter ligne 104 et 117 ?
Sinon, cette ligne unique_together est superflu
| # TODO: ces champs proviennent de tables pas encore importées | ||
| # populations= | ||
| # conditions_not_recommended= | ||
| # other_conditions= |
There was a problem hiding this comment.
ces changements couvrent aussi other_conditions ?
There was a problem hiding this comment.
Pas encore, je le ferais dans une future PR
pletelli
force-pushed
the
1465-history
branch
from
2c1cfca to
b223c9c
Compare
This comment was marked as outdated.
This comment was marked as outdated.
pletelli
changed the title
pletelli
changed the title
Closes #1465
Cette PR permet d'importer certaines informations qui étaient manquantes dans les déclarations historiques, notamment :
Produit
Composition
TODO next :
Import de champs