Difference between revisions of "Talk:Files/EditBufHacks"
| Line 7: | Line 7: | ||
b- The check: | b- The check: | ||
| − | my $start_work = EPrints::Utils::tree_to_utf8( $user->render_value("onlinemail") ); | + | my $start_work = EPrints::Utils::tree_to_utf8( $user->render_value("onlinemail") ); |
| − | if ($start_work eq 'Yes') | + | if ($start_work eq 'Yes') |
| − | + | is language dependant and works only for English repositories. I suggest to change it in something like this: | |
| − | my $start_work = $user->{"onlinemail"}; | + | my $start_work = $user->{"onlinemail"}; |
| − | if ($start_work eq 'True') { | + | if ($start_work eq 'True') { |
Revision as of 09:23, 3 June 2007
It works.
A couple of improvments:
a - We should add a config to select which archive has this feature and which not, otherwise in multiarchive conf we send email even if not required. Luckly, modifind the global cgi edit_eprints don't create problems .
b- The check:
my $start_work = EPrints::Utils::tree_to_utf8( $user->render_value("onlinemail") );
if ($start_work eq 'Yes')
is language dependant and works only for English repositories. I suggest to change it in something like this:
my $start_work = $user->{"onlinemail"};
if ($start_work eq 'True') {
