Vote #81803
完了Query Links on Issues and Time Logs Import Sidebars broken
0%
説明
The import pages for issues and time logs have sidebars with links to saved issue queries and time log queries. These links are currently broken and just render as @the_current_page?query_id=3@, for example @http://localhost:3000/issues/imports/new?query_id=7@
journals
We have tracked down the issue to source:/trunk/app/helpers/queries_helper.rb@21387#L457, which is not aware of the issues and time logs imports, and thus can't correctly generate the query links.
Instead of trying to fix this we would like to suggest simply removing the sidebars entirely for the imports. While doing the import the query links are not of a great use because clicking on one of those links would interrupt the import steps. Furthermore, the user import for example does not have a sidebar, i.e. removing the sidebar in the other cases would be consistent. Last but not least, the new issue view for example does not have a sidebar either, probably because having links to the queries while creating something is not very helpful.
Do you think just removing the sidebar from the CSV import would be a good option here? If yes we can prepare a patch accordingly.
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
The attached patch removes the aforementioned sidebars.
--------------------------------------------------------------------------------
Patch committed, thanks!
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
Thank you!
--------------------------------------------------------------------------------