Defect #26785
closedWrong columns after CSV export
0%
Description
On the /issues
page, if we create a new CSV export with all columns, every columns are selected next time we visit the html page.
Selected columns should remain unchanged after the CSV export.
Files
Updated by Go MAEDA about 7 years ago
- Status changed from New to Needs feedback
I could not reproduce the problem. Could you describe the detailed steps to reproduce?
Updated by Vincent Robert about 7 years ago
- File step3.png step3.png added
- File step2.png step2.png added
- File step1.png step1.png added
- File step1bis.png step1bis.png added
Here are the steps to reproduce this bug:
1 - Go to global /issues page¶
2 - Clear current query¶
3 - Export all columns to CSV file¶
4 - Click on Issues
link in the menu (or go to /issues page without any parameter in the url)¶
All columns are now selected!
Here is my configuration:
Environment: Redmine version 3.4.2.stable Ruby version 2.3.3-p222 (2016-11-21) [x86_64-darwin16] Rails version 4.2.8 Environment development Database adapter PostgreSQL SCM: Subversion 1.9.4 Git 2.12.1 Filesystem Redmine plugins: no plugin installed
Updated by Go MAEDA about 7 years ago
- Status changed from Needs feedback to Confirmed
I have confirmed the problem in trunk and 3.4-stable.
Updated by Go MAEDA almost 6 years ago
- File fix-26785.diff fix-26785.diff added
- Target version set to Candidate for next minor release
You can fix this issue with the attached patch.
My customer experiences ActionDispatch::Cookies::CookieOverflow
exception due to this defect. The Redmine has 200+ custom fields. A list of all custom fields is set to session[:issue_query][:column_names]
after exporting all columns to CSV and the very long session value causes the exception.
The problem can be addressed by restoring the session value after sending CSV to the client.
Updated by Mizuki ISHIKAWA almost 6 years ago
- File fix-26785-v2.patch added
Go MAEDA wrote:
The problem can be addressed by restoring the session value after sending CSV to the client.
The patch attached to #26785#note-5 worked fine for me, but I think it would be better to use the argument use_session of retrieve_query.
I attached a patch of proposed code and test code.
Updated by Mizuki ISHIKAWA almost 6 years ago
- File fix-26785-v3.patch fix-26785-v3.patch added
I cleaned up the code a little.
Updated by Go MAEDA almost 6 years ago
- Target version changed from Candidate for next minor release to 3.4.7
LGTM. Setting the target version to 3.4.7.
Updated by Go MAEDA almost 6 years ago
- Status changed from Confirmed to Resolved
- Assignee set to Go MAEDA
- Resolution set to Fixed
Committed the patch with a slight change in the test.
Thank you for reporting and fixing the issue.
Updated by Go MAEDA almost 6 years ago
- Status changed from Resolved to Closed
Merged to 3.4-stable branch.