From d4a7058cb1712d46f92fc9ba6bb2760e6c0b7488 Mon Sep 17 00:00:00 2001
From: Penar Musaraj <pmusaraj@gmail.com>
Date: Tue, 8 Sep 2020 15:00:16 -0400
Subject: [PATCH] DEV: Strip unicode from color scheme stylesheet filenames
 (#10628)

---
 lib/stylesheet/manager.rb                  |  6 +++++-
 spec/components/stylesheet/manager_spec.rb | 15 +++++++++++++++
 2 files changed, 20 insertions(+), 1 deletion(-)

diff --git a/lib/stylesheet/manager.rb b/lib/stylesheet/manager.rb
index 695eae1b53f..3332b7e98fc 100644
--- a/lib/stylesheet/manager.rb
+++ b/lib/stylesheet/manager.rb
@@ -313,7 +313,7 @@ class Stylesheet::Manager
     if is_theme?
       "#{@target}_#{theme.id}"
     elsif @color_scheme
-      "#{@target}_#{Slug.for(@color_scheme.name)}_#{@color_scheme&.id.to_s}"
+      "#{@target}_#{scheme_slug}_#{@color_scheme&.id.to_s}"
     else
       scheme_string = theme && theme.color_scheme ? "_#{theme.color_scheme.id}" : ""
       "#{@target}#{scheme_string}"
@@ -333,6 +333,10 @@ class Stylesheet::Manager
     !!(@target.to_s =~ THEME_REGEX)
   end
 
+  def scheme_slug
+    Slug.for(ActiveSupport::Inflector.transliterate(@color_scheme.name), 'scheme')
+  end
+
   # digest encodes the things that trigger a recompile
   def digest
     @digest ||= begin
diff --git a/spec/components/stylesheet/manager_spec.rb b/spec/components/stylesheet/manager_spec.rb
index 7ca60a24349..2ae86992b11 100644
--- a/spec/components/stylesheet/manager_spec.rb
+++ b/spec/components/stylesheet/manager_spec.rb
@@ -273,6 +273,21 @@ describe Stylesheet::Manager do
 
       expect(stylesheet).to include("--special: rebeccapurple")
     end
+
+    context 'encoded slugs' do
+      before { SiteSetting.slug_generation_method = 'encoded' }
+      after { SiteSetting.slug_generation_method = 'ascii' }
+
+      it "strips unicode in color scheme stylesheet filenames" do
+        cs = Fabricate(:color_scheme, name: 'Grün')
+        cs2 = Fabricate(:color_scheme, name: '어두운')
+
+        link = Stylesheet::Manager.color_scheme_stylesheet_link_tag(cs.id)
+        expect(link).to include("/stylesheets/color_definitions_grun_#{cs.id}_")
+        link2 = Stylesheet::Manager.color_scheme_stylesheet_link_tag(cs2.id)
+        expect(link2).to include("/stylesheets/color_definitions_scheme_#{cs2.id}_")
+      end
+    end
   end
 
   # this test takes too long, we don't run it by default