Bugzilla – Attachment 17838 Details for
Bug 10104
koha-disable error handling
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 10104 - Followup: fix param check
Bug-10104---Followup-fix-param-check.patch (text/plain), 1.42 KB, created by
Mason James
on 2013-04-30 07:19:11 UTC
(
hide
)
Description:
Bug 10104 - Followup: fix param check
Filename:
MIME Type:
Creator:
Mason James
Created:
2013-04-30 07:19:11 UTC
Size:
1.42 KB
patch
obsolete
>From 75e9b51a778004bf158e4c146fca34308df1b9b4 Mon Sep 17 00:00:00 2001 >From: Tomas Cohen Arazi <tomascohen@gmail.com> >Date: Mon, 29 Apr 2013 15:22:09 -0300 >Subject: [PATCH] Bug 10104 - Followup: fix param check >MIME-Version: 1.0 >Content-Type: text/plain; charset=UTF-8 >Content-Transfer-Encoding: 8bit >Content-Type: text/plain; charset="utf-8" > >As noted by Mason on bug 10101, the check was not working for zero arguments. > >Sponsored-by: Universidad Nacional de Córdoba > >Signed-off-by: Mason James <mtj@kohaaloha.com> >--- > debian/scripts/koha-disable | 16 +++++++++++++--- > 1 files changed, 13 insertions(+), 3 deletions(-) > >diff --git a/debian/scripts/koha-disable b/debian/scripts/koha-disable >index 7affe5e..75e3997 100755 >--- a/debian/scripts/koha-disable >+++ b/debian/scripts/koha-disable >@@ -1,6 +1,6 @@ > #!/bin/sh > # >-# koha-disable -- disable Koha instances. >+# koha-disable - disable Koha instances. > # Copyright 2010 Catalyst IT, Ltd > # > # This program is free software: you can redistribute it and/or modify >@@ -73,8 +73,19 @@ disable_instance() > fi > } > >+usage() >+{ >+ local scriptname=$0 >+ cat <<EOF >+Disables Koha instances. >+ >+Usage: $scriptname instancename1 instancename2... >+ >+EOF >+} >+ > # Parse command line. >-[ "$#" > 1 ] || die "Usage: $0 instancename..." >+[ $# -ge 1 ] || ( usage ; die "Missing instance name..." ) > > restart_apache="no" > >@@ -96,4 +107,3 @@ if [ "$restart_apache" = "yes" ]; then > fi > > exit 0 >- >-- >1.7.2.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 10104
:
17626
|
17775
|
17812
|
17819
|
17837
| 17838