Same cause and fix as bug 19928
Sorry, won't have the time to work on this (there will be always more problematic stuff for me to fix). But it's a reasonably low hanging fruit/beginner friendly bug and I'll be glad to provide advice if needed.
Created attachment 84169 [details] [review] Bug 20171: Fixed CSV output to use syspref 'delimiter' in export files To Find Fix: 1. Go to sysprefs and change delimiter to slashes 2. Under the Search drop down click Item search 3. Set Output as CSV 4. Click Search 5. A file should be downloaded and when opened the set delimiter should appear as the delimiter
Created attachment 84216 [details] [review] Bug 20171: Fixed CSV output to use syspref 'delimiter' in export files To Find Fix: 1. Go to sysprefs and change delimiter to slashes 2. Under the Search drop down click Item search 3. Set Output as CSV 4. Click Search 5. A file should be downloaded and when opened the set delimiter should appear as the delimiter Signed-off-by: Mikael Olangcay Brisebois <mikael.olangcay-brisebois@inlibro.com>
Signed-off by Mikaël, but status not changed. So have updated. Also changed assignee to Olivia. (Can confirm the patch works as expected).
Hi all, I tested, but found 2 issues that we need to fix: - The QA script complains about missing filters - While most values of the system preference translate directly, tabulation does not: ;|tabulation|,|/|\|#||. So when tab is selected, the file exported is not usable as it has "tabulation" printed as a separator instead of the actual 'tab'.
I believe this is already fixed in master. Setting to resolved