Skip to content
GitLab
Commit 10fa8b07 authored by IKEDA Soji's avatar IKEDA Soji
Browse files

[change] Like the other parameters, defaults of list scenarios would be... 

[change] Like the other parameters, defaults of list scenarios would be specified by corresponding robot/site parameters, instead of creating file or link named "*.default".
Defaults by older manner will be detected and written in sympa.conf/robot.conf during upgrading process.
parent 7fbb263d
Hide whitespace changes
Inline Side-by-side
src/lib/Sympa/ConfDef.pm
View file @ 10fa8b07
......@@ -585,6 +585,109 @@ our @params = (
'Minimum number of owners for each list must satisfy the owner_domain restriction. The default of zero (0) means *all* list owners must match. Setting to 1 requires only one list owner to match owner_domain; all other owners can be from any domain. This setting can be used to ensure that there is always at least one known contact point for any mailing list.',
},
{'gettext_id' => 'Default privileges for the lists'},
# List definition
{ 'name' => 'visibility',
'gettext_id' => "Visibility of the list",
'scenario' => 1,
'vhost' => 1,
'default' => 'conceal',
},
# Sending
{ 'name' => 'send',
'gettext_id' => "Who can send messages",
'scenario' => 1,
'vhost' => 1,
'default' => 'private',
},
# Privileges
{ 'name' => 'info',
'gettext_id' => "Who can view list information",
'scenario' => 1,
'vhost' => 1,
'default' => 'open',
},
{ 'name' => 'subscribe',
'gettext_id' => "Who can subscribe to the list",
'scenario' => 1,
'vhost' => 1,
'default' => 'open',
},
{ 'name' => 'add',
'gettext_id' => "Who can add subscribers",
'scenario' => 1,
'vhost' => 1,
'default' => 'owner',
},
{ 'name' => 'unsubscribe',
'gettext_id' => "Who can unsubscribe",
'scenario' => 1,
'vhost' => 1,
'default' => 'open',
},
{ 'name' => 'del',
'gettext_id' => "Who can delete subscribers",
'scenario' => 1,
'vhost' => 1,
'default' => 'owner',
},
{ 'name' => 'invite',
'gettext_id' => "Who can invite people",
'scenario' => 1,
'vhost' => 1,
'default' => 'private',
},
{ 'name' => 'remind',
'gettext_id' => "Who can start a remind process",
'scenario' => 1,
'vhost' => 1,
'default' => 'owner',
},
{ 'name' => 'review',
'gettext_id' => "Who can review subscribers",
'scenario' => 1,
'vhost' => 1,
'default' => 'owner',
},
{ 'name' => 'd_read',
'gettext_id' => "Who can view",
'scenario' => 1,
'vhost' => 1,
'default' => 'private',
},
{ 'name' => 'd_edit',
'gettext_id' => "Who can edit",
'scenario' => 1,
'vhost' => 1,
'default' => 'owner',
},
# Archives
{ 'name' => 'archive_web_access',
'gettext_id' => "access right",
'scenario' => 1,
'vhost' => 1,
'default' => 'closed',
},
{ 'name' => 'archive_mail_access',
'gettext_id' => "access right by mail commands",
'scenario' => 1,
'vhost' => 1,
'default' => 'closed',
},
# Bounces
{ 'name' => 'tracking',
'gettext_id' => "who can view message tracking",
'scenario' => 1,
'vhost' => 1,
'default' => 'owner',
},
{'gettext_id' => 'Archives'},
{ 'name' => 'process_archive',
......
src/lib/Sympa/ListDef.pm
View file @ 10fa8b07
......@@ -68,11 +68,10 @@ our %pinfo = (
'This parameter indicates whether the list should feature in the output generated in response to a LISTS command or should be shown in the list overview of the web-interface.',
'scenario' => 'visibility',
'synonym' => {
'default' => 'conceal', # Compat. <= 6.2.40
'public' => 'noconceal',
'private' => 'conceal'
},
'default' => 'conceal',
'default' => {'conf' => 'visibility'},
},
'owner' => {
......@@ -292,10 +291,7 @@ our %pinfo = (
'gettext_comment' =>
'This parameter specifies who can send messages to the list.',
'scenario' => 'send',
'synonym' => {
'default' => 'private', # Compat. <= 6.2.40
},
'default' => 'private',
'default' => {'conf' => 'send'},
},
'delivery_time' => {
......@@ -646,10 +642,7 @@ our %pinfo = (
'group' => 'command',
'gettext_id' => "Who can view list information",
'scenario' => 'info',
'synonym' => {
'default' => 'open', # Compat. <= 6.2.40
},
'default' => 'open',
'default' => {'conf' => 'info'},
},
'subscribe' => {
......@@ -659,10 +652,7 @@ our %pinfo = (
'gettext_comment' =>
'The subscribe parameter defines the rules for subscribing to the list.',
'scenario' => 'subscribe',
'synonym' => {
'default' => 'open', # Compat. <= 6.2.40
},
'default' => 'open',
'default' => {'conf' => 'subscribe'},
},
'subscription' => {'obsolete' => 'subscribe'},
......@@ -673,10 +663,7 @@ our %pinfo = (
'gettext_comment' =>
'Privilege for adding (ADD command) a subscriber to the list',
'scenario' => 'add',
'synonym' => {
'default' => 'owner', # Compat. <= 6.2.40
},
'default' => 'owner',
'default' => {'conf' => 'add'},
},
'unsubscribe' => {
......@@ -686,10 +673,7 @@ our %pinfo = (
'gettext_comment' =>
'This parameter specifies the unsubscription method for the list. Use open_notify or auth_notify to allow owner notification of each unsubscribe command.',
'scenario' => 'unsubscribe',
'synonym' => {
'default' => 'open', # Compat. <= 6.2.40
},
'default' => 'open',
'default' => {'conf' => 'unsubscribe'},
},
'unsubscription' => {'obsolete' => 'unsubscribe'},
......@@ -698,10 +682,7 @@ our %pinfo = (
'group' => 'command',
'gettext_id' => "Who can delete subscribers",
'scenario' => 'del',
'synonym' => {
'default' => 'owner', # Compat. <= 6.2.40
},
'default' => 'owner',
'default' => {'conf' => 'del'},
},
'invite' => {
......@@ -709,10 +690,7 @@ our %pinfo = (
'group' => 'command',
'gettext_id' => "Who can invite people",
'scenario' => 'invite',
'synonym' => {
'default' => 'private', # Compat. <= 6.2.40
},
'default' => 'private',
'default' => {'conf' => 'invite'},
},
'remind' => {
......@@ -722,10 +700,7 @@ our %pinfo = (
'gettext_comment' =>
'This parameter specifies who is authorized to use the remind command.',
'scenario' => 'remind',
'synonym' => {
'default' => 'owner', # Compat. <= 6.2.40
},
'default' => 'owner',
'default' => {'conf' => 'remind'},
},
'review' => {
......@@ -735,11 +710,8 @@ our %pinfo = (
'gettext_comment' =>
'This parameter specifies who can access the list of members. Since subscriber addresses can be abused by spammers, it is strongly recommended that you only authorize owners or subscribers to access the subscriber list. ',
'scenario' => 'review',
'synonym' => {
'default' => 'owner', # Compat. <= 6.2.40
'open' => 'public',
},
'default' => 'owner',
'synonym' => {'open' => 'public',},
'default' => {'conf' => 'review'},
},
'owner_domain' => {
......@@ -778,19 +750,13 @@ our %pinfo = (
'order' => 1,
'gettext_id' => "Who can view",
'scenario' => 'd_read',
'synonym' => {
'default' => 'private', # Compat. <= 6.2.40
},
'default' => 'private',
'default' => {'conf' => 'd_read'},
},
'd_edit' => {
'order' => 2,
'gettext_id' => "Who can edit",
'scenario' => 'd_edit',
'synonym' => {
'default' => 'owner', # Compat. <= 6.2.40
},
'default' => 'owner',
'default' => {'conf' => 'd_edit'},
},
'quota' => {
'order' => 3,
......@@ -822,7 +788,8 @@ our %pinfo = (
'order' => 1,
'gettext_id' => "access right",
'scenario' => 'archive_web_access',
'obsolete' => 1, # Use archive.web_access
'default' => {'conf' => 'archive_web_access'},
'obsolete' => 1, # Use archive.web_access
},
'quota' => {
'order' => 2,
......@@ -831,14 +798,14 @@ our %pinfo = (
'format' => '\d+',
'default' => {'conf' => 'default_archive_quota'},
'length' => 8,
'obsolete' => 1, # Use archive.quota
'obsolete' => 1, # Use archive.quota
},
'max_month' => {
'order' => 3,
'gettext_id' => "Maximum number of month archived",
'format' => '\d+',
'length' => 3,
'obsolete' => 1, # Use archive.max_month
'obsolete' => 1, # Use archive.max_month
}
}
},
......@@ -867,17 +834,16 @@ our %pinfo = (
'order' => 3,
'gettext_id' => "access right",
'scenario' => 'archive_web_access',
'default' => 'closed',
'default' => {'conf' => 'archive_web_access'},
},
'mail_access' => {
'order' => 4,
'gettext_id' => "access right by mail commands",
'scenario' => 'archive_mail_access',
'synonym' => {
'default' => 'closed', # Compat. <= 6.2.40
'open' => 'public', # Compat. with <=6.2b.3.
'open' => 'public', # Compat. with <=6.2b.3.
},
'default' => 'closed',
'default' => {'conf' => 'archive_mail_access'},
},
'quota' => {
'order' => 5,
......@@ -1059,10 +1025,7 @@ our %pinfo = (
'order' => 3,
'gettext_id' => "who can view message tracking",
'scenario' => 'tracking',
'synonym' => {
'default' => 'owner', # Compat. <= 6.2.40
},
'default' => 'owner',
'default' => {'conf' => 'tracking'},
},
'retention_period' => {
'order' => 4,
......
src/lib/Sympa/Upgrade.pm
View file @ 10fa8b07
......@@ -29,6 +29,7 @@ package Sympa::Upgrade;
use strict;
use warnings;
use Cwd qw();
use Encode qw();
use English qw(-no_match_vars);
use MIME::Base64 qw();
......@@ -1900,6 +1901,67 @@ sub upgrade {
}
}
# Default list scenario names will be specified in robot.conf/sympa.conf
# instead of creating symbolic links named "*.default".
if (lower_version($previous_version, '6.2.41b.1')) {
my @scenarios = qw(visibility
send info subscribe add unsubscribe del invite remind review
d_read d_edit
archive_web_access archive_mail_access
tracking);
my @dirs = ($Conf::Conf{'etc'});
push @dirs,
map { $Conf::Conf{'etc'} . '/' . $_ } Sympa::List::get_robots();
foreach my $dir (@dirs) {
my %scenario_names;
next unless $dir eq $Conf::Conf{'etc'} or -e $dir . '/robot.conf';
next unless -d $dir . '/scenari';
foreach my $scenario (@scenarios) {
my $path = $dir . '/scenari/' . $scenario . '.default';
next unless -e $path;
my $name = '';
if (-l $path) {
$path = Cwd::abs_path($path);
next unless $path and -e $path;
if ($path =~ m{/$scenario\.([-\w]+)\z}) {
$name = $1;
}
}
$scenario_names{$scenario} = $name;
}
next unless %scenario_names;
my $ofh;
if ($dir eq $Conf::Conf{'etc'}) {
open $ofh, '>>', Conf::get_sympa_conf() or next;
} else {
open $ofh, '>>', $dir . '/robot.conf' or next;
}
print $ofh "\n\n";
print $ofh
"# Following parameters were added during upgrade to %s\n\n",
$new_version;
foreach my $scenario (@scenarios) {
unless (exists $scenario_names{$scenario}) {
next;
} elsif ($scenario_names{$scenario}) {
printf $ofh "%s %s\n", $scenario,
$scenario_names{$scenario};
} else {
printf $ofh
"#%s (Assign default manually, or distribution default will be used.)\n",
$scenario;
}
}
close $ofh;
}
}
return 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