[pve-devel] r5027 - pve-common/trunk

[svn-commits at proxmox.com]

Author: dietmar
Date: 2010-08-17 14:00:00 +0000 (Tue, 17 Aug 2010)
New Revision: 5027

Modified:
   pve-common/trunk/ChangeLog
   pve-common/trunk/JSONSchema.pm
Log:
 use option type 's' for boolean



Modified: pve-common/trunk/ChangeLog
===================================================================
--- pve-common/trunk/ChangeLog	2010-08-17 10:24:41 UTC (rev 5026)
+++ pve-common/trunk/ChangeLog	2010-08-17 14:00:00 UTC (rev 5027)
@@ -1,5 +1,9 @@
 2010-08-17  Proxmox Support Team  <support at proxmox.com>
 
+	* JSONSchema.pm (get_options): we use option type 's' for boolean
+	values - that way we can pass true and false (and any alias for
+	them)
+
 	* Exception.pm (raise_param_exc): new helper function
 
 2010-08-16  Proxmox Support Team  <support at proxmox.com>

Modified: pve-common/trunk/JSONSchema.pm
===================================================================
--- pve-common/trunk/JSONSchema.pm	2010-08-17 10:24:41 UTC (rev 5026)
+++ pve-common/trunk/JSONSchema.pm	2010-08-17 14:00:00 UTC (rev 5027)
@@ -657,8 +657,6 @@
 	    # we turn this into a boolean option and ask for password below
 	    # using $pwcallback() (for security reasons).
 	    push @getopt, "$prop";
-	} elsif ($pd->{type} eq 'boolean') {
-	    push @getopt, "$prop";
 	} else {
 	    push @getopt, "$prop=s";
 	}