From: <gi...@ma...> - 2010-11-30 15:34:41
|
The branch, master has been updated via dd45535140881e5cd303c2a385264e9f2779ef09 (commit) from 6baa60165a018619ab6c0f0c3fd7cf7c94b2fc65 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit dd45535140881e5cd303c2a385264e9f2779ef09 Author: Daryn Warriner <da...@ii...> Date: Tue Nov 30 09:33:14 2010 -0600 Fix Bug #12489: resolution is not updated on re-open The reopen resolution must be set for everyone with permission to reopen an issue. Not just the reporter. ----------------------------------------------------------------------- Summary of changes: bug_update.php | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) ----------------------------------------------------------------------- commit dd45535140881e5cd303c2a385264e9f2779ef09 Author: Daryn Warriner <da...@ii...> Date: Tue Nov 30 09:33:14 2010 -0600 Fix Bug #12489: resolution is not updated on re-open The reopen resolution must be set for everyone with permission to reopen an issue. Not just the reporter. diff --git a/bug_update.php b/bug_update.php index eaa4669..6431d42 100644 --- a/bug_update.php +++ b/bug_update.php @@ -172,12 +172,15 @@ if ( $t_existing_bug->status !== $t_updated_bug->status ) { $t_existing_bug->reporter_id === auth_get_current_user_id() && config_get( 'allow_reporter_reopen' ) ) { $t_can_bypass_status_access_thresholds = true; - $t_updated_bug->resolution = config_get( 'bug_reopen_resolution' ); } if ( !$t_can_bypass_status_access_thresholds ) { trigger_error( ERROR_ACCESS_DENIED, ERROR ); } } + if( $t_reopen_issue ) { + # for everyone allowed to reopen an issue, set the reopen resolution + $t_updated_bug->resolution = config_get( 'bug_reopen_resolution' ); + } } # Validate any change to the handler of an issue. ----------------------------------------------------------------------- -- Mantis Bug Tracker |