Commit b3c4c4e6 authored by IKEDA Soji's avatar IKEDA Soji
Browse files

Fixed a flaw on suboptions similar to previous commit.

parent 5b690d23
......@@ -61,7 +61,10 @@
</a>
[%~ END %]
[% IF current_subscriber.email == user.email ~%]
<p><a href="[% 'suboptions' | url_rel([list]) %]" title="">[%|loc%]Changing your picture for this list[%END%]</a></p>
<p>
<a href="[% 'suboptions' | url_rel([list],{previous_action=>action}) %]"
title="">[%|loc%]Changing your picture for this list[%END%]</a>
</p>
[%~ END %]
[%~ END %]
......
......@@ -5701,33 +5701,20 @@ sub do_subscribe {
sub do_suboptions {
wwslog('info', '');
 
unless ($param->{'is_subscriber'}) {
Sympa::Report::reject_report_web('user', 'not_subscriber',
{'list' => $list->{'name'}},
$param->{'action'}, $list);
wwslog(
'info',
'%s not subscribed to %s',
$param->{'user'}{'email'},
$param->{'list'}
);
return undef;
}
my ($s, $m);
 
unless ($s = $list->get_list_member($param->{'user'}{'email'})) {
Sympa::Report::reject_report_web(
'intern',
'subscriber_not_found',
{'email' => $param->{'user'}{'email'}},
'user',
'not_subscriber',
{list => $list->{'name'}},
$param->{'action'},
$list,
$param->{'user'}{'email'},
$robot
);
wwslog('info', 'Subscriber %s not found', $param->{'user'}{'email'});
return undef;
return $in{'previous_action'} || 'info';
}
 
$s->{'reception'} ||= 'mail';
......
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