From: intrigeri <intrigeri@boum.org>
Date: Tue, 29 Nov 2005 10:11:04 +0000 (+0000)
Subject:  r3567@krups:  intrigeri | 2005-11-17 22:08:30 +0100
X-Git-Tag: backupninja-0.9.2~47
X-Git-Url: https://git.stderr.nl/gitweb?p=matthijs%2Fupstream%2Fbackupninja.git;a=commitdiff_plain;h=fbd3c3d84c89f4d4ec31525d1793e186ab9d1f94

 r3567@krups:  intrigeri | 2005-11-17 22:08:30 +0100
 pgsql.helper: forbid the user to choose an empty database set to backup
---

diff --git a/ChangeLog b/ChangeLog
index b3de985..5ca58ec 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -7,7 +7,10 @@ version 0.9.2 -- unreleased
 		moved to lib/ some code that has to be shared between
 		backupninja and ninjahelper
 	added duplicity ninjahelper
-	a bunch of Vserver-related functions for ninjahelper in lib/vservers.in
+	a bunch of Vserver-related functions for ninjahelper in lib/vservers.in,
+		used only in dup.helper and pgsql.helper yet
+	pgsql ninjahelper bugfixes:
+		forbid the user to choose an empty database set to backup
 
 version 0.9.1 -- November 05 2005
 	rearranged source so that it is relocatable with autotools
diff --git a/handlers/pgsql.helper b/handlers/pgsql.helper
index 1c8b4ef..aa807a5 100644
--- a/handlers/pgsql.helper
+++ b/handlers/pgsql.helper
@@ -7,23 +7,25 @@ do_pgsql_vserver() {
 }
 
 do_pgsql_databases() {
-   formBegin "$pgsql_title: databases"
-     formItem "Database:"
-     formItem "Database:"
-     formItem "Database:"
-     formItem "Database:"
-     formItem "Database:"
-     formItem "Database:"
-     formItem "Database:"
-     formItem "Database:"
-     formItem "Database:"
-     formItem "Database:"
-   formDisplay
-   [ $? = 1 ] && return
-   
-   pgsql_databases="databases = "
-   for i in $REPLY; do
-      [ "$i" != "" ] && pgsql_databases="$pgsql_databases $i"
+   REPLY=
+   while [ -z "$REPLY" ]; do
+      formBegin "$pgsql_title: databases"
+         formItem "Database:"
+	 formItem "Database:"
+	 formItem "Database:"
+	 formItem "Database:"
+	 formItem "Database:"
+	 formItem "Database:"
+	 formItem "Database:"
+	 formItem "Database:"
+	 formItem "Database:"
+	 formItem "Database:"
+      formDisplay
+      [ $? = 0 ] || return 1
+      pgsql_databases="databases = "
+      for i in $REPLY; do
+	 [ -n "$i" ] && pgsql_databases="$pgsql_databases $i"
+      done
    done
 }