From 0e2453243961fabbb2c08ba71f2f2648b2b17285 Mon Sep 17 00:00:00 2001
From: Sam Waechter <138798589+swektr@users.noreply.github.com>
Date: Wed, 18 Oct 2023 23:42:57 -0400
Subject: [PATCH] Trim extra colons on conversion to NativeDeckName (#2748)

This fixes issue #2692. Leading/trailing colons are remove along with whitespace so " : : :foo:bar: : : " becomes "foo:bar"
---
 rslib/src/decks/name.rs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/rslib/src/decks/name.rs b/rslib/src/decks/name.rs
index c1cb69eb6..c8b6c7439 100644
--- a/rslib/src/decks/name.rs
+++ b/rslib/src/decks/name.rs
@@ -196,7 +196,7 @@ fn normalized_deck_name_component(comp: &str) -> Cow<str> {
     if out.contains(invalid_char_for_deck_component) {
         out = out.replace(invalid_char_for_deck_component, "").into();
     }
-    let trimmed = out.trim();
+    let trimmed = out.trim_matches(|c: char| c.is_whitespace() || c == ':');
     if trimmed.is_empty() {
         "blank".to_string().into()
     } else if trimmed.len() != out.len() {
@@ -236,6 +236,8 @@ mod test {
         // implicitly normalize
         assert_eq!(native_name("fo\x1fo::ba\nr"), "foo\x1fbar");
         assert_eq!(native_name("fo\u{a}o\x1fbar"), "foobar");
+        assert_eq!(native_name("foo:::bar"), "foo\x1fbar");
+        assert_eq!(native_name("foo:::bar:baz: "), "foo\x1fbar:baz");
     }
 
     #[test]