Patch #32813
Clean up toggleMultiSelect function
| Status: | New | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | - | % Done: | 0% | |
| Category: | Code cleanup/refactoring | |||
| Target version: | Candidate for next major release |
Description
The attached patch cleans up toggleMultiSelect function(toggle_multiple_select.gif).
- 0001. Unify the process to toggleMultiSelect written in workflows/permissions, workflows/edit, and query_filter respectively
- 0002. Fix app/controllers/workflows_controller.rb that should have changed along with #15988
(It works like toggle_role_selectbox.gif due to the problem)
0002-Fix-roles.patch 
(1.66 KB)
0001-Unify-processing-for-switching-multiple-selections.patch
(7.09 KB)
toggle_multiple_select.gif (88.2 KB)
toggle_role_selectbox.gif (491 KB)
History
#1
Updated by Go MAEDA about 1 month ago
- Target version set to Candidate for next major release
#2
Updated by Go MAEDA about 9 hours ago
Mizuki ISHIKAWA wrote:
- 0002. Fix app/controllers/workflows_controller.rb that should have changed along with #15988
(It works like toggle_role_selectbox.gif due to the problem)
I think it is another issue. It is not a code cleanup but a bug. I think it should be handled by another new issue. What do you think?