https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8000

--- Comment #108 from Marcel de Rooy <[email protected]> ---
Functionality
I have my doubts if some user will ever want to use this in a production
system.
In a test system, I run one sql statement to achieve the same:  UPDATE
borrowers SET email=, emailpro = .. etc
For testing a notice on a test patron, you do not need this address. 

Data::Validate::Email
It seems that you add a new dependency and then not use it? Can you remove it
and not change PerlDependencies?

sentto_address
Do we really need that? If you choose to override the to_address, why not
update the existing column?

sendbasket/sendshelf
I do not see the advantage of overriding an email address that you entered on
that form. I would recommend not to change these scripts.
Similar (but maybe little bit less strong) for runreport.pl. Just update the to
argument for the cronjob.

I will not change the status of this report. Another QAer may add his opinion.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to