From fd8d58ce36961c91eea710292053f37baeb14ca3 Mon Sep 17 00:00:00 2001 From: Jeremy Mitchell Date: Wed, 30 Nov 2016 13:44:52 -0700 Subject: [PATCH] dnssecEnabled is required on a create and it should come across as a true/false --- traffic_ops/app/lib/API/Cdn.pm | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/traffic_ops/app/lib/API/Cdn.pm b/traffic_ops/app/lib/API/Cdn.pm index 14503bdd3b..c20243ef0a 100644 --- a/traffic_ops/app/lib/API/Cdn.pm +++ b/traffic_ops/app/lib/API/Cdn.pm @@ -100,18 +100,22 @@ sub create { return $self->alert("CDN 'name' is required."); } + if ( !defined( $params->{dnssecEnabled} ) ) { + return $self->alert("dnssecEnabled is required."); + } + my $existing = $self->db->resultset('Cdn')->search( { name => $params->{name} } )->single(); if ($existing) { $self->app->log->error( "a cdn with name '" . $params->{name} . "' already exists." ); return $self->alert( "a cdn with name " . $params->{name} . " already exists." ); } - my $value = { name => $params->{name}, }; - if ( defined( $params->{dnssecEnabled} ) ) { - $value->{dnssec_enabled} = lc( $params->{dnssecEnabled} ) ne 'false' ? 1 : 0; - } + my $values = { + name => $params->{name}, + dnssec_enabled => $params->{dnssecEnabled}, + }; - my $insert = $self->db->resultset('Cdn')->create($value); + my $insert = $self->db->resultset('Cdn')->create($values); $insert->insert(); my $rs = $self->db->resultset('Cdn')->find( { id => $insert->id } );