From 43a8724d3a707118f63bf2e7db2ffd344ed42ec0 Mon Sep 17 00:00:00 2001
From: Valentin Lorentz <progval+git@progval.net>
Date: Fri, 8 May 2020 22:01:50 +0200
Subject: [PATCH] Don't strip spaces when parsing registry values.

It breaks plugins.Messageparser.listSeparator, and probably others.
---
 src/registry.py       | 4 ++--
 test/test_registry.py | 8 ++++++++
 2 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/registry.py b/src/registry.py
index 1753aaf4b..4cb1f8bc3 100644
--- a/src/registry.py
+++ b/src/registry.py
@@ -103,9 +103,9 @@ def open_registry(filename, clear=False):
         else:
             acc += line
         try:
-            (key, value) = re.split(r'(?<!\\):', acc, 1)
+            (key, value) = re.split(r'(?<!\\): ', acc, 1)
             key = key.strip()
-            value = value.strip()
+            value = value.strip('\r\n')
             value = decoder(value)[0]
             acc = ''
         except ValueError:
diff --git a/test/test_registry.py b/test/test_registry.py
index dfd4e97c8..70ccba87e 100644
--- a/test/test_registry.py
+++ b/test/test_registry.py
@@ -200,6 +200,14 @@ class ValuesTestCase(SupyTestCase):
         registry.open_registry(filename)
         self.assertEqual(conf.supybot.reply.whenAddressedBy.chars(), '\\')
 
+    def testSpacesValues(self):
+        with conf.supybot.networks.test.password.context(' foo '):
+            self.assertEqual(conf.supybot.networks.test.password(), ' foo ')
+            filename = conf.supybot.directories.conf.dirize('spaces.conf')
+            registry.close(conf.supybot, filename)
+            registry.open_registry(filename)
+            self.assertEqual(conf.supybot.networks.test.password(), ' foo ')
+
     def testWith(self):
         v = registry.String('foo', 'help')
         self.assertEqual(v(), 'foo')