From 46eb2c01947c4c075ac33c7bbda065bc5417dc44 Mon Sep 17 00:00:00 2001
From: Stephan de Wit <stephan.de.wit@deciso.com>
Date: Thu, 2 Nov 2023 12:04:32 +0100
Subject: [PATCH] Gateways: virtualize dpinger defaults during migration

---
 .../mvc/app/models/OPNsense/Routing/Migrations/M1_0_0.php | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/src/opnsense/mvc/app/models/OPNsense/Routing/Migrations/M1_0_0.php b/src/opnsense/mvc/app/models/OPNsense/Routing/Migrations/M1_0_0.php
index 52aed0777..5252fa08f 100644
--- a/src/opnsense/mvc/app/models/OPNsense/Routing/Migrations/M1_0_0.php
+++ b/src/opnsense/mvc/app/models/OPNsense/Routing/Migrations/M1_0_0.php
@@ -74,6 +74,14 @@ class M1_0_0 extends BaseModelMigration
                     $node->$key = (string)$value;
                 }
 
+                // set, but do not persist default values
+                foreach (Gateways::getDpingerDefaults() as $key => $value) {
+                    if (empty((string)$node->$key)) {
+                        $node->$key = $value;
+                        $node->$key->setInternalIsVirtual(true);
+                    }
+                }
+
                 // increase time period if old model had it set too low
                 $min_time_period = 2 * (intval((string)$node->interval) + intval((string)$node->loss_interval));
                 if ((string)$node->time_period < $min_time_period) {