Project

General

Profile

Actions

Patch #8757

open

Add hook for bulk edit post-save state

Added by Ramesh Nair over 12 years ago. Updated about 5 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Hook requests
Target version:
-
Start date:
2011-07-06
Due date:
% Done:

0%

Estimated time:

Description

There is a hook which gets invoked just after an individual issue gets updated, which allows for plugins to perform any post-save actions. There ought to be a similar hook for when a number of issues have been updated via the bulk-editing process. I've patched the core code to enable such a hook - `controller_issues_bulk_edit_after_save`.

Patch available at https://github.com/7citylearning/redmine/commit/f88c6ca01e89246b5bcd62fa26cc651ee33ba212


Files


Related issues

Has duplicate Redmine - Feature #25454: Add bulk edit after save hookClosed

Actions
Has duplicate Redmine - Feature #29897: hook: controller_issues_bulk_edit_after_saveClosed

Actions
Has duplicate Redmine - Patch #5714: Request for controller_issues_bulk_edit_after_save hookClosed2010-06-20

Actions
Actions #1

Updated by Etienne Massip over 12 years ago

  • Category changed from Issues to Plugin API
Actions #2

Updated by An Phan about 9 years ago

+1. Some of the webhook plugins out there is not complete because of this hook missing, for example Redmine Webhook.
I've no idea why this takes so long to be merged. There was even a patch 4 years ago: #5714. I mean, how hard can it be to add this one liner into issues_controller.rb?

call_hook(:controller_issues_bulk_edit_after_save, { :params => params, :issue => issue, :journal => journal })

Actions #3

Updated by Hiroshi Miura almost 9 years ago

+1. To improve user productivity with webhook or other plugins, and chat platforms such as hipchat, slack, kandan and hubot.

Actions #4

Updated by Adam Pfeiffer almost 8 years ago

I absolutely need this as well. It does seem to be a simple change that should have been there from the beginning. I will modify my code locally to make sure this works with my plugin.

THanks

Actions #5

Updated by Joel SCHAAL over 7 years ago

+1

I would like to see that change in the next version.

Is this change too risky or too dirty ?
Does it need to be adapted before acceptance ?

Actions #6

Updated by Naoki Yoshida over 7 years ago

+1

Actions #7

Updated by Toshi MARUYAMA over 7 years ago

  • Category changed from Plugin API to Hook requests
Actions #8

Updated by Tobias Böhm about 7 years ago

I just created a patch for this issue that applies to the current master code base. Please find it attached.

Actions #9

Updated by Toshi MARUYAMA almost 7 years ago

Actions #10

Updated by Serghei Zagorinyak almost 7 years ago

This patch is wrong.

Implemented this way, if an exception is raised within bulk_edit_after_save it will prevent succeeding issues from being saved while they should be processed and only the problematic issue should be rejected.

Actions #11

Updated by Tobias Böhm almost 7 years ago

You are right, exceptions raising from the hook are not handled. But this applies also to the already existing hook :controller_issues_bulk_edit_after_save. If an exception is raised there, all following issues will not be processed.

This should be addressed for the whole process (before-hook, save, after-hook) or loop at once, I guess, in a separate issue and patch.

Actions #12

Updated by Serghei Zagorinyak almost 7 years ago

Tobias Böhm wrote:

You are right, exceptions raising from the hook are not handled. But this applies also to the already existing hook :controller_issues_bulk_edit_after_save. If an exception is raised there, all following issues will not be processed.

This should be addressed for the whole process (before-hook, save, after-hook) or loop at once, I guess, in a separate issue and patch.

Yes, didn't notice that first. Hope this will be fixed someday.

Actions #13

Updated by Toshi MARUYAMA almost 7 years ago

Actions #14

Updated by Toshi MARUYAMA almost 7 years ago

Actions #15

Updated by Julián Maestri over 5 years ago

Patch based on redmine v3.4.6
To avoid the problems with stopping the bulk update if something fails on the hook, iterate over the saved issues after saving them.

Actions #16

Updated by Marius BĂLTEANU over 5 years ago

  • Has duplicate Feature #29897: hook: controller_issues_bulk_edit_after_save added
Actions #17

Updated by Go MAEDA over 5 years ago

  • Has duplicate Patch #5714: Request for controller_issues_bulk_edit_after_save hook added
Actions #18

Updated by Jethro Yu over 5 years ago

+1

I am modifying a plugin that reads journal of bulk edit issues, too.
https://github.com/jcppkkk/redmine_hipchat/commit/45d740de41bcaa68f64d5def16a9367e6291ed8b#diff-a7f255c397e668f358fc183f6802c3f3R25
Julián Maestri wrote:

Patch based on redmine v3.4.6
To avoid the problems with stopping the bulk update if something fails on the hook, iterate over the saved issues after saving them.

Actions #19

Updated by Julián Maestri about 5 years ago

Any chance for this to be included in the roadmap? at least as Unplanned?

Actions

Also available in: Atom PDF