diff --git a/frappe/core/doctype/document_naming_settings/document_naming_settings.py b/frappe/core/doctype/document_naming_settings/document_naming_settings.py
index e340f554f8..ba74b78732 100644
--- a/frappe/core/doctype/document_naming_settings/document_naming_settings.py
+++ b/frappe/core/doctype/document_naming_settings/document_naming_settings.py
@@ -144,6 +144,7 @@ class DocumentNamingSettings(Document):
 		if self.prefix:
 			prefix = NamingSeries(self.prefix).get_prefix()
 			self.current_value = frappe.db.get_value("Series", prefix, "current", order_by="name")
+		return self.current_value
 
 	@frappe.whitelist()
 	def update_series_start(self):
@@ -155,7 +156,7 @@ class DocumentNamingSettings(Document):
 		db_prefix = NamingSeries(self.prefix).get_prefix()
 
 		if frappe.db.get_value("Series", db_prefix, "name", order_by="name") is None:
-			series.insert(db_prefix, 0).columns(series.name, series.current).run()
+			frappe.db.sql("insert into `tabSeries` (`name`, `current`) values (%s, 0)", (db_prefix))
 
 		(
 			frappe.qb.update(series)
diff --git a/frappe/core/doctype/document_naming_settings/test_document_naming_settings.py b/frappe/core/doctype/document_naming_settings/test_document_naming_settings.py
index 2d6878e604..ce65478339 100644
--- a/frappe/core/doctype/document_naming_settings/test_document_naming_settings.py
+++ b/frappe/core/doctype/document_naming_settings/test_document_naming_settings.py
@@ -7,6 +7,7 @@ from frappe.core.doctype.document_naming_settings.document_naming_settings impor
 )
 from frappe.model.naming import get_default_naming_series
 from frappe.tests.utils import FrappeTestCase
+from frappe.utils import cint
 
 
 class TestNamingSeries(FrappeTestCase):
@@ -16,6 +17,11 @@ class TestNamingSeries(FrappeTestCase):
 	def tearDown(self):
 		frappe.db.rollback()
 
+	def get_valid_serieses(self):
+		VALID_SERIES = ["SINV-", "SI-.{field}.", "SI-#.###", ""]
+		exisiting_series = self.dns.get_transactions_and_prefixes()["prefixes"]
+		return VALID_SERIES + exisiting_series
+
 	def test_naming_preview(self):
 		self.dns.transaction_type = "Webhook"
 
@@ -39,3 +45,21 @@ class TestNamingSeries(FrappeTestCase):
 	def test_default_naming_series(self):
 		self.assertIn("HOOK", get_default_naming_series("Webhook"))
 		self.assertIsNone(get_default_naming_series("DocType"))
+
+	def test_updates_naming_options(self):
+		self.dns.transaction_type = "Webhook"
+		test_series = "KOOHBEW.###"
+		self.dns.naming_series_options = self.dns.get_options() + "\n" + test_series
+		self.dns.update_series()
+		self.assertIn(test_series, frappe.get_meta("Webhook").get_naming_series_options())
+
+	def test_update_series_counter(self):
+		for series in self.get_valid_serieses():
+			if not series:
+				continue
+			self.dns.prefix = series
+			current_count = cint(self.dns.get_current())
+			new_count = self.dns.current_value = current_count + 1
+			self.dns.update_series_start()
+
+			self.assertEqual(self.dns.get_current(), new_count)