dev/core#2073 just tweak a memory leak
authoreileen <emcnaughton@wikimedia.org>
Tue, 29 Sep 2020 04:03:48 +0000 (17:03 +1300)
committereileen <emcnaughton@wikimedia.org>
Tue, 29 Sep 2020 04:03:59 +0000 (17:03 +1300)
This one is like https://github.com/civicrm/civicrm-core/pull/18632 but only
seems to have switched us from an xml leakage to a string leakage - it still
seems slightly better from a code quality point of view

CRM/Core/CodeGen/Main.php

index 1b754b959fb4274020eb631489e845d4b0ee9fcc..d108102259617250657d3a13a654d5bb4e78d99d 100644 (file)
@@ -5,6 +5,10 @@
  */
 class CRM_Core_CodeGen_Main {
   public $buildVersion;
+
+  /**
+   * @var string
+   */
   public $db_version;
   /**
    * drupal, joomla, wordpress
@@ -76,11 +80,11 @@ class CRM_Core_CodeGen_Main {
 
     $versionFile = $this->phpCodePath . "/xml/version.xml";
     $versionXML = CRM_Core_CodeGen_Util_Xml::parse($versionFile);
-    $this->db_version = $versionXML->version_no;
+    $this->db_version = (string) $versionXML->version_no;
     $this->buildVersion = preg_replace('/^(\d{1,2}\.\d{1,2})\.(\d{1,2}|\w{4,7})$/i', '$1', $this->db_version);
     if (isset($argVersion)) {
       // change the version to that explicitly passed, if any
-      $this->db_version = $argVersion;
+      $this->db_version = (string) $argVersion;
     }
 
     $this->schemaPath = $schemaPath;