Skip to content

Single choice csv export bug fix#330

Open
rezeau wants to merge 1 commit into
PoetOS:MOODLE_400_STABLEfrom
rezeau:single_choice_csv_export_bug_fix
Open

Single choice csv export bug fix#330
rezeau wants to merge 1 commit into
PoetOS:MOODLE_400_STABLEfrom
rezeau:single_choice_csv_export_bug_fix

Conversation

@rezeau

@rezeau rezeau commented Jan 16, 2021

Copy link
Copy Markdown
Contributor

Here's a suggested fix

Signed-off-by: rezeau <joseph@rezeau.org>
@mchurchward mchurchward changed the base branch from master to MOODLE_310_STABLE August 4, 2021 17:35
@mchurchward mchurchward force-pushed the MOODLE_310_STABLE branch 3 times, most recently from 7a9b599 to a5d2b59 Compare August 5, 2021 12:32
@mchurchward

Copy link
Copy Markdown
Contributor

Confirming... The bug this is intended to fix is for CSV export only. If a value like:
"2::coffee" is specified for a choice, the CSV export for that choice is expected to be "2". Instead, it is the entire string "2::coffee".

@mchurchward mchurchward changed the base branch from MOODLE_310_STABLE to MOODLE_400_STABLE December 21, 2022 15:05
@mchurchward

Copy link
Copy Markdown
Contributor

Currently, the CSV export displays "2 : 2::coffee". The code provided changes that to "2 : 2". That doesn't seem right either?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants