Unverified Commit 0c743721 authored by IKEDA Soji's avatar IKEDA Soji Committed by GitHub
Browse files

Merge pull request #214 from ikedas/issue-212

Issue #212: Notification not sent to the owner when a list is confirmed by a listmaster
parents 2f526f5e f22b1392
...@@ -11113,7 +11113,8 @@ sub do_purge_list { ...@@ -11113,7 +11113,8 @@ sub do_purge_list {
   
sub do_close_list { sub do_close_list {
wwslog('info', '(%s, mode=%s)', $list, $in{'mode'}); wwslog('info', '(%s, mode=%s)', $list, $in{'mode'});
my $mode = $in{'mode'}; my $mode = $in{'mode'};
my $notify = !!$in{'notify'};
   
# Sanitize parameter: non-listmasters are allowed "close" mode only. # Sanitize parameter: non-listmasters are allowed "close" mode only.
$mode = 'close' $mode = 'close'
...@@ -11123,6 +11124,7 @@ sub do_close_list { ...@@ -11123,6 +11124,7 @@ sub do_close_list {
   
$param->{'mode'} = $mode; $param->{'mode'} = $mode;
$param->{'previous_action'} = $in{'previous_action'} || 'admin'; $param->{'previous_action'} = $in{'previous_action'} || 'admin';
$param->{'notify'} = $notify;
   
# Action confirmed? # Action confirmed?
my $next_action = $session->confirm_action( my $next_action = $session->confirm_action(
...@@ -11137,6 +11139,7 @@ sub do_close_list { ...@@ -11137,6 +11139,7 @@ sub do_close_list {
action => 'close_list', action => 'close_list',
current_list => $list, current_list => $list,
mode => $mode, mode => $mode,
notify => $notify,
sender => $param->{'user'}{'email'}, sender => $param->{'user'}{'email'},
( $param->{'user'}{'email'} ( $param->{'user'}{'email'}
? (md5_check => 1) ? (md5_check => 1)
...@@ -11184,7 +11187,8 @@ sub do_close_list { ...@@ -11184,7 +11187,8 @@ sub do_close_list {
# Old name: do_restore_list(). # Old name: do_restore_list().
sub do_open_list { sub do_open_list {
wwslog('info', '(mode=%s)', $in{'mode'}); wwslog('info', '(mode=%s)', $in{'mode'});
my $mode = $in{'mode'}; my $mode = $in{'mode'};
my $notify = !!$in{'notify'};
   
# Sanitize parameter. # Sanitize parameter.
$mode = 'open' $mode = 'open'
...@@ -11192,6 +11196,7 @@ sub do_open_list { ...@@ -11192,6 +11196,7 @@ sub do_open_list {
   
$param->{'mode'} = $mode; $param->{'mode'} = $mode;
$param->{'previous_action'} = $in{'previous_action'} || 'admin'; $param->{'previous_action'} = $in{'previous_action'} || 'admin';
$param->{'notify'} = $notify;
   
# Action confirmed? # Action confirmed?
my $next_action = $session->confirm_action( my $next_action = $session->confirm_action(
...@@ -11206,6 +11211,7 @@ sub do_open_list { ...@@ -11206,6 +11211,7 @@ sub do_open_list {
action => 'open_list', action => 'open_list',
current_list => $list, current_list => $list,
mode => $mode, mode => $mode,
notify => $notify,
sender => $param->{'user'}{'email'}, sender => $param->{'user'}{'email'},
( $param->{'user'}{'email'} ( $param->{'user'}{'email'}
? (md5_check => 1) ? (md5_check => 1)
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment