From 89fa64dbcbde4aba9704b501fa3fc487400fae7f Mon Sep 17 00:00:00 2001
From: Ad Schellevis <ad@opnsense.org>
Date: Fri, 12 Aug 2016 11:27:49 +0200
Subject: [PATCH] (mvc) BaseModel add getVersion() to retrieve the running
 config version, commit changes to in memory config after every migration
 step.

Errors will be send to syslog, current policy is to always upgrade and ignore migration errors.
(there's not really a way back, other then undo pkg install and revert the full config)
---
 .../mvc/app/models/OPNsense/Base/BaseModel.php         | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/src/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php b/src/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php
index 48b687967..6c49cb4cd 100644
--- a/src/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php
+++ b/src/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php
@@ -541,6 +541,7 @@ abstract class BaseModel
                         $migobj = $mig_class->newInstance();
                         try {
                             $migobj->run($this);
+                            $this->serializeToConfig();
                         } catch (\Exception $e) {
                             $logger->error("failed migrating from version " .
                                 $this->internal_current_model_version .
@@ -554,4 +555,13 @@ abstract class BaseModel
             }
         }
     }
+
+    /**
+     * return current version number
+     * @return null|string
+     */
+    public function getVersion()
+    {
+        return $this->internal_current_model_version;
+    }
 }