disable uplink dynamic validation in server side

commit diff contains comment + explanation
inverse-inc · May 15, 2019 · b206ec4 · b206ec4
1 parent 0091086
commit b206ec4
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/html/pfappserver/lib/pfappserver/Form/Config/Switch.pm b/html/pfappserver/lib/pfappserver/Form/Config/Switch.pm
@@ -816,11 +816,13 @@ sub validate {
         }
     }
 
-    if ($self->value->{uplink_dynamic} ne 'dynamic') {
-        unless ($self->value->{uplink} && $self->value->{uplink} =~ m/^(\d(,\s*)?)*$/) {
-            $self->field('uplink')->add_error("The uplinks must be a list of ports numbers.");
-        }
-    }
+    # Temporarily disabled as new admin sends uplink_dynamic as undef which has this evaluated everytime although the inherited value might be 'dynamic'
+    # The frontend does a validation of this requirement
+    #if ($self->value->{uplink_dynamic} ne 'dynamic') {
+    #    unless ($self->value->{uplink} && $self->value->{uplink} =~ m/^(\d(,\s*)?)*$/) {
+    #        $self->field('uplink')->add_error("The uplinks must be a list of ports numbers.");
+    #    }
+    #}
 }
 
 =head1 COPYRIGHT