(model) skip save on migration when defaults lead to inconsistent data, closes...

(model) skip save on migration when defaults lead to inconsistent data, closes https://github.com/opnsense/core/issues/1149

src/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php

@@ -559,7 +559,11 @@ abstract class BaseModel
             // serialize to config after last migration step, keep the config data static as long as not all
             // migrations have completed.
             if ($upgradePerfomed) {
+                try {
                     $this->serializeToConfig();
+                } catch (\Exception $e) {
+                    $logger->error("Model ".$class_info->getName() ." can't be saved, skip ( " .$e . " )");
+                }
             }
         }
     }