fixed ldap helper setting compress option wrong, standardize on options

2 files changed, 7 insertions, 5 deletions

diff --git a/ChangeLog b/ChangeLog
index 19548f6..8cd85c6 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -21,6 +21,8 @@ version 0.9.5 -- unreleased
 	 . Fixed shell command quoting issues, missing 'then' clauses, cleaned up
 	   compress=yes to be less redundant and not create empty uncompressed
 	   file (Closes: #394935)
+	 . Fixed ninjahelper to properly set compress option, standardized on yes/no
+	   instead of on/off
 	maildir:
 	 . Added an examples file (Closes: Trac#23)
         mysql:
diff --git a/handlers/ldap.helper.in b/handlers/ldap.helper.in
index 697c720..5ccbe7e 100644
--- a/handlers/ldap.helper.in
+++ b/handlers/ldap.helper.in
@@ -5,11 +5,11 @@ HELPERS="$HELPERS ldap:ldap_database_backup"
 ldap_create_file() {
 while true; do
       checkBox "ldap action wizard" "check options (slapcat OR ldapsearch)" \
-         "slapcat" "export ldif using slapcat" on \
-         "ldapsearch" "export ldif using ldapsearch" off \
-         "compress" "compress the ldif output files" on
+         "slapcat" "export ldif using slapcat" yes \
+         "ldapsearch" "export ldif using ldapsearch" no \
+         "compress" "compress the ldif output files" yes
       status=$?
-      compress="compress = off"
+      compress="compress = no"
       method="method = <unset>"
       restart="restart = no"
       binddn=""
@@ -18,7 +18,7 @@ while true; do
       result="$REPLY"
       for opt in $result; do
          case $opt in
-           '"compress"') compress="compress = on";;
+           '"compress"') compress="compress = yes";;
            '"slapcat"')
               method="method = slapcat"
               [ "$_RESTART" == "yes" ] && restart="restart = yes"