ScoDoc/ScoDoc
ScoDoc
/
ScoDoc
6
2
Fork 15
Code Issues 143 Pull Requests Projects 3 Releases 128 Wiki Activity

add parameter and fix parameter effects #610

Closed
jmplace wants to merge 1 commits from jmplace/ScoDoc-Lille:filter_none_groups_where_export_admission_data into master
pull from: jmplace/ScoDoc-Lille:filter_none_groups_where_export_admission_data
merge into: ScoDoc:master
Conversation 1 Commits 1 Files Changed 2 +3 -5
jmplace commented 2023-03-01 13:46:36 +01:00
Collaborator
Copy Link

Proposition de résultion d'un bug lors de l'export/uimport des données d'admission.
Lorsqu'on exporte puis reimporte de suite des données alors que des groupes sont constitués, on obtie un exception ValueError sur un groupe invalide.

En fait lors de l export le groupe vide (interne est pris en compte et ceci génère un ;: en fin des données de groupe.

Proposition:
mettre le parametre only_to_show lors de l appel de la fonction etud_add_group_infos qui inventorie les groupe des étudiants pour export.

Pour que ce parametre soit complètement fonctionnel, il a falu changer aussi le test à l intérieur de la fonction (le nom associé au groupe est '' et non None).

J'avoue ne pas être capable de déterminer si cei à un impact ailleurs que dans l export (autres appels avec ce parametre à True dans FicheEtud)

Proposition de résultion d'un bug lors de l'export/uimport des données d'admission. Lorsqu'on exporte puis reimporte de suite des données alors que des groupes sont constitués, on obtie un exception ValueError sur un groupe invalide. En fait lors de l export le groupe vide (interne est pris en compte et ceci génère un ;: en fin des données de groupe. Proposition: mettre le parametre only_to_show lors de l appel de la fonction `etud_add_group_infos` qui inventorie les groupe des étudiants pour export. Pour que ce parametre soit complètement fonctionnel, il a falu changer aussi le test à l intérieur de la fonction (le nom associé au groupe est '' et non None). J'avoue ne pas être capable de déterminer si cei à un impact ailleurs que dans l export (autres appels avec ce parametre à True dans FicheEtud)
jmplace added 1 commit 2023-03-01 13:46:37 +01:00
jmplace commented 2023-03-01 14:25:20 +01:00
Author
Collaborator
Copy Link

reformulation pour éviter des régression -> #611

reformulation pour éviter des régression -> https://scodoc.org/git/ScoDoc/ScoDoc/pulls/611
jmplace closed this pull request 2023-03-01 14:25:22 +01:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
ABS

Gestion des absences

  
à confirmer

   
API

API json

  
Apogée

Interfaces Apogée

  
bug

Something is not working

  
BUT

Bachelor IUT

  
Dev

Enhance code/refactoring/tools

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
Entreprises

App Relations Entreprises

  
frontend

HTML/CSS/JS

  
help wanted

Need some help

  
invalid

Something is wrong

  
Jury

Gestion des jurys

  
PE

Poursuites études

  
prio

prioritaire

  
question

More information is needed

  
RGPD

   
Users

Gestion utilisateurs

  
wontfix

This won't be fixed

No Label
ABS
à confirmer
API
Apogée
bug
BUT
Dev
duplicate
enhancement
Entreprises
frontend
help wanted
invalid
Jury
PE
prio
question
RGPD
Users
wontfix
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: ScoDoc/ScoDoc#610
No description provided.
Delete Branch "jmplace/ScoDoc-Lille:filter_none_groups_where_export_admission_data"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?