From 1437659a4e6334432840660e5ee4b996f15e7d24 Mon Sep 17 00:00:00 2001
From: Veronica Berglyd Olsen <1619840+vkbo@users.noreply.github.com>
Date: Sun, 29 Jan 2023 02:14:21 +0100
Subject: [PATCH] Drop the splitVersionNumber function

---
 novelwriter/common.py               | 27 ---------------------------
 tests/test_base/test_base_common.py | 25 +++----------------------
 2 files changed, 3 insertions(+), 49 deletions(-)

diff --git a/novelwriter/common.py b/novelwriter/common.py
index ea38d841..91058fd0 100644
--- a/novelwriter/common.py
+++ b/novelwriter/common.py
@@ -277,33 +277,6 @@ def yesNo(value):
     return "yes" if value else "no"
 
 
-def splitVersionNumber(value):
-    """Split a version string on the form aa.bb.cc into major, minor
-    and patch, and computes an integer value aabbcc.
-    """
-    if not isinstance(value, str):
-        return 0, 0, 0, 0
-
-    vMajor = 0
-    vMinor = 0
-    vPatch = 0
-    vInt = 0
-
-    vBits = value.split(".")
-    nBits = len(vBits)
-
-    if nBits > 0:
-        vMajor = checkInt(vBits[0], 0)
-    if nBits > 1:
-        vMinor = checkInt(vBits[1], 0)
-    if nBits > 2:
-        vPatch = checkInt(vBits[2], 0)
-
-    vInt = vMajor*10000 + vMinor*100 + vPatch
-
-    return vMajor, vMinor, vPatch, vInt
-
-
 def transferCase(source, target):
     """Transfers the case of the source word to the target word. This
     will consider all upper or lower, and first char capitalisation.
diff --git a/tests/test_base/test_base_common.py b/tests/test_base/test_base_common.py
index 0aec498a..5559439e 100644
--- a/tests/test_base/test_base_common.py
+++ b/tests/test_base/test_base_common.py
@@ -33,9 +33,9 @@ from novelwriter.common import (
     checkStringNone, checkString, checkInt, checkFloat, checkBool, checkHandle,
     checkUuid, checkPath, isHandle, isTitleTag, isItemClass, isItemType,
     isItemLayout, hexToInt, minmax, checkIntTuple, formatInt, formatTimeStamp,
-    formatTime, simplified, yesNo, splitVersionNumber, transferCase, fuzzyTime,
-    numberToRoman, jsonEncode, readTextFile, makeFileNameSafe, sha256sum,
-    getGuiItem, NWConfigParser
+    formatTime, simplified, yesNo, transferCase, fuzzyTime, numberToRoman,
+    jsonEncode, readTextFile, makeFileNameSafe, sha256sum, getGuiItem,
+    NWConfigParser
 )
 
 
@@ -398,25 +398,6 @@ def testBaseCommon_YesNo():
 # END Test testBaseCommon_YesNo
 
 
-@pytest.mark.base
-def testBaseCommon_SplitVersionNumber():
-    """Test the splitVersionNumber function.
-    """
-    # OK Values
-    assert splitVersionNumber("1") == (1, 0, 0, 10000)
-    assert splitVersionNumber("1.2") == (1, 2, 0, 10200)
-    assert splitVersionNumber("1.2.3") == (1, 2, 3, 10203)
-    assert splitVersionNumber("1.2.3.4") == (1, 2, 3, 10203)
-    assert splitVersionNumber("99.99.99") == (99, 99, 99, 999999)
-
-    # Failed Values
-    assert splitVersionNumber(None) == (0, 0, 0, 0)
-    assert splitVersionNumber(1234) == (0, 0, 0, 0)
-    assert splitVersionNumber("1.2abc") == (1, 0, 0, 10000)
-
-# END Test testBaseCommon_SplitVersionNumber
-
-
 @pytest.mark.base
 def testBaseCommon_FormatInt():
     """Test the formatInt function.