transmit multiple occurence of an argument into a list #130

jmplace · 2021-09-21T06:56:08+02:00

jmplace commented

2021-09-21 06:56:08 +02:00

*** Modification importante *** à valider avec précaution
objet: reporduire le comportement de scodoc7 quand un même argument apparaît plusieurs fois.

requete: groups_view?group_ids=2007&groups_id=2008
python: groups_view(..group_ids..)

scodoc7 -> group_ids vaut ['2007', '2008']
scodoc 9 (avant patch) -> group_ids vaut '2007'

impacte au minimum group_views mais potentiellemenbt d'autres URL

un correctif alternatif (plus locale) aurait été de transformer 'group_ids' en 'group_id_list' avec la modification du javascript impliqué (choix des groupes)

*** Modification importante *** à valider avec précaution objet: reporduire le comportement de scodoc7 quand un même argument apparaît plusieurs fois. requete: groups_view?group_ids=2007&groups_id=2008 python: groups_view(..group_ids..) scodoc7 -> group_ids vaut ['2007', '2008'] scodoc 9 (avant patch) -> group_ids vaut '2007' impacte au minimum group_views mais potentiellemenbt d'autres URL un correctif alternatif (plus locale) aurait été de transformer 'group_ids' en 'group_id_list' avec la modification du javascript impliqué (choix des groupes)

jmplace added 1 commit 2021-09-21 06:56:08 +02:00

9dd8198c7b transmit multiple occurence of an argument into a list

viennet merged commit 7214627994 into master

2021-09-21 16:49:41 +02:00

viennet referenced this issue from a commit

2021-09-21 16:49:41 +02:00

Merge pull request 'transmit multiple occurence of an argument into a list' (#130) from jmplace/ScoDoc-Lille:fix_multivalued_args into master

Sign in to join this conversation.

No reviewers