[pve-devel] applied: [Patch V2 common 2/3] Reuse id parse code.

Fabian Grünbichler f.gruenbichler at proxmox.com
Wed Apr 1 15:25:44 CEST 2020


On March 31, 2020 12:08 pm, Wolfgang Link wrote:
> The storage_id is the same as the plugin_id.
> 
> Signed-off-by: Wolfgang Link <w.link at proxmox.com>
> ---
>  src/PVE/JSONSchema.pm | 13 +++++++++----
>  1 file changed, 9 insertions(+), 4 deletions(-)
> 
> diff --git a/src/PVE/JSONSchema.pm b/src/PVE/JSONSchema.pm
> index fa405ac..3eb38eb 100644
> --- a/src/PVE/JSONSchema.pm
> +++ b/src/PVE/JSONSchema.pm
> @@ -180,14 +180,19 @@ PVE::JSONSchema::register_format('pve-storage-id', \&parse_storage_id);
>  sub parse_storage_id {
>      my ($storeid, $noerr) = @_;
>  
> -    if ($storeid !~ m/^[a-z][a-z0-9\-\_\.]*[a-z0-9]$/i) {
> +    return parse_id($storeid, 'storage', $noerr);
> +}
> +
> +sub parse_id {
> +    my ($id, $msg, $noerr) = @_;
> +
> +     if ($id !~ m/^[a-z][a-z0-9\-\_\.]*[a-z0-9]$/i) {
>  	return undef if $noerr;
> -	die "storage ID '$storeid' contains illegal characters\n";
> +	die "$msg ID '$id' contains illegal characters\n";
>      }
> -    return $storeid;
> +    return $id;
>  }
>  
> -
>  register_format('pve-vmid', \&pve_verify_vmid);
>  sub pve_verify_vmid {
>      my ($vmid, $noerr) = @_;
> -- 
> 2.20.1
> 
> 
> _______________________________________________
> pve-devel mailing list
> pve-devel at pve.proxmox.com
> https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
> 
> 




More information about the pve-devel mailing list