BUG/MINOR: redispatch: allow interval of 0

Author: mjuraga

configuration/backend_test.go

@@ -827,7 +827,7 @@ func TestCreateEditDeleteBackend(t *testing.T) {
 		Disabled: true,
 		Redispatch: &models.Redispatch{
 			Enabled:  misc.StringP("enabled"),
-			Interval: 0,
+			Interval: misc.Int64P(0),
 		},
 	}
 
@@ -843,9 +843,6 @@ func TestCreateEditDeleteBackend(t *testing.T) {
 		t.Error(err.Error())
 	}
 
-	// A redispatch with a zero interval is automatically removed from configuration.
-	b.Redispatch = nil
-
 	var givenJSONB []byte
 	givenJSONB, err = b.MarshalBinary()
 	if err != nil {

configuration/configuration.go

@@ -968,9 +968,8 @@ func (s *SectionParser) redispatch() interface{} {
 		return nil
 	}
 	d := data.(*types.OptionRedispatch)
-	br := &models.Redispatch{}
-	if d.Interval != nil {
-		br.Interval = *d.Interval
+	br := &models.Redispatch{
+		Interval: d.Interval,
 	}
 	if d.NoOption {
 		d := "disabled"
@@ -2437,15 +2436,12 @@ func (s *SectionObject) redispatch(field reflect.Value) error {
 			return misc.CreateTypeAssertError("option redispatch")
 		}
 		d := &types.OptionRedispatch{
-			Interval: &br.Interval,
+			Interval: br.Interval,
 			NoOption: false,
 		}
 		if *br.Enabled == "disabled" {
 			d.NoOption = true
 		}
-		if br.Interval == 0 {
-			d = nil
-		}
 		if err := s.set("option redispatch", d); err != nil {
 			return err
 		}

models/redispatch.go

@@ -9338,7 +9338,7 @@ definitions:
               type: string
           interval:
               type: integer
-              x-nullable: false
+              x-nullable: true
       required:
           - enabled
       type: object

models/redispatch_compare.go

@@ -149,7 +149,7 @@ redispatch:
       enum: [enabled, disabled]
     interval:
       type: integer
-      x-nullable: false
+      x-nullable: true
 errorfile:
   type: object
   x-display-name: Error File