update comments on $primary_*_map use.

use $curproc->{ credential } not call "use FML::Credential ...".

1 files changed, 8 insertions, 4 deletions

diff --git a/fml/lib/FML/Command/User/chaddr.pm b/fml/lib/FML/Command/User/chaddr.pm
index 66cf9f55..7126672e 100644
--- a/fml/lib/FML/Command/User/chaddr.pm
+++ b/fml/lib/FML/Command/User/chaddr.pm
@@ -4,7 +4,7 @@
 #   All rights reserved. This program is free software; you can
 #   redistribute it and/or modify it under the same terms as Perl itself.
 #
-# $FML: chaddr.pm,v 1.23 2003/10/17 14:00:52 fukachan Exp $
+# $FML: chaddr.pm,v 1.24 2003/11/22 05:41:51 fukachan Exp $
 #
 
 package FML::Command::User::chaddr;
@@ -75,9 +75,13 @@ sub process
     my ($self, $curproc, $command_args) = @_;
     my $config        = $curproc->config();
 
-    #
-    # XXX-TODO: correct to use *_maps not primary_*_map for chaddr?
-    #
+    # XXX We should always add/rewrite only $primary_*_map maps via 
+    # XXX command mail, CUI and GUI.
+    # XXX Rewriting of maps not $primary_*_map is
+    # XXX 1) may be not writable.
+    # XXX 2) ambigous and dangerous 
+    # XXX    since the map is under controlled by other module.
+    # XXX    for example, one of member_maps is under admin_member_maps. 
     my $member_map    = $config->{ primary_member_map };
     my $recipient_map = $config->{ primary_recipient_map };
     my $cache_dir     = $config->{ db_dir };