koha-email-disable should check stuff like if the instance exists, if the email is already disabled.
Created attachment 17800 [details] [review] Bug 10150 - koha-email-disable error handling koha-email-disable now - Checks the instance exists. - Checks if email is already disabled. Regards To+ Sponsored-by: Universidad Nacional de Córdoba
./debian/scripts/koha-email-disable Disables the email for Koha instances. Usage: ./debian/scripts/koha-email-disable instancename1 instancename2... Missing instance name... ./debian/scripts/koha-email-disable persona Email already disabled for instance persona. ./debian/scripts/koha-email-disable per Unknown instance per. ./debian/scripts/koha-email-disable per persona Unknown instance per. Email already disabled for instance persona.
Created attachment 17826 [details] [review] Signed off patch
Created attachment 17877 [details] [review] Bug 10150 - koha-email-disable error handling koha-email-disable now - Checks the instance exists. - Checks if email is already disabled. Regards To+ Sponsored-by: Universidad Nacional de Córdoba Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Mason James <mtj@kohaaloha.com>
> Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> > Signed-off-by: Mason James <mtj@kohaaloha.com> passing QA on this patch...
Pushed to master. Thanks, Tomás!
Pushed to 3.10.x will be in 3.10.6