Actions
Patch #32813
closed
Clean up toggleMultiSelect js function
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)
Files
Updated by Go MAEDA almost 5 years ago
- Target version set to Candidate for next major release
Updated by Go MAEDA almost 5 years 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?
Updated by Mizuki ISHIKAWA almost 5 years ago
Go MAEDA wrote:
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?
I created a issue reporting the bug and attached a patch. #33059
Updated by Go MAEDA almost 5 years ago
- Target version changed from Candidate for next major release to 4.2.0
Setting the target version to 4.2.0.
Since 0002-Fix-roles.patch has already been committed in r19538, the patch to be committed is only 0001-Unify-processing-for-switching-multiple-selections.patch.
Updated by Go MAEDA almost 5 years ago
- Status changed from New to Closed
- Assignee set to Go MAEDA
Committed the patch. Thank you.
Updated by Go MAEDA almost 5 years ago
- Subject changed from Clean up toggleMultiSelect function to Clean up toggleMultiSelect js function
Actions