From 2b33c3c303a252744275f3d4fee84e7e76f16976 Mon Sep 17 00:00:00 2001
From: Sadie Powell
Date: Tue, 20 Dec 2022 10:51:37 +0000
Subject: Don't automatically fall back to ToNetwork from ToInternal.

Only one thing used this and its really unclear especially when
SimpleExtItem does the opposite.
---
 src/modules/m_sslinfo.cpp | 10 ++++++++++
 1 file changed, 10 insertions(+)

(limited to 'src/modules')

diff --git a/src/modules/m_sslinfo.cpp b/src/modules/m_sslinfo.cpp
index 99f51beb4..93df069aa 100644
--- a/src/modules/m_sslinfo.cpp
+++ b/src/modules/m_sslinfo.cpp
@@ -64,6 +64,11 @@ public:
 		Delete(user, UnsetRaw(user));
 	}
 
+	std::string ToInternal(const Extensible* container, void* item) const noexcept override
+	{
+		return ToNetwork(container, item);
+	}
+
 	std::string ToNetwork(const Extensible* container, void* item) const noexcept override
 	{
 		const ssl_cert* cert = static_cast<ssl_cert*>(item);
@@ -84,6 +89,11 @@ public:
 		return value.str();
 	}
 
+	void FromInternal(Extensible* container, const std::string& value) noexcept override
+	{
+		FromNetwork(container, value);
+	}
+
 	void FromNetwork(Extensible* container, const std::string& value) noexcept override
 	{
 		if (container->extype != this->extype)
-- 
cgit v1.3.1-10-gc9f91