fix warning

umbrellio · Dec 25, 2023 · 54879bd · 54879bd
1 parent 519e2ac
commit 54879bd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/lib/simplecov-html.rb b/lib/simplecov-html.rb
@@ -95,6 +95,10 @@ def assets_path(name)
         File.join("./assets", SimpleCov::Formatter::HTMLFormatter::VERSION, name)
       end
 
+      def to_id(value)
+        value.gsub(/^[^a-zA-Z]+/, "").gsub(/[^a-zA-Z0-9\-_]/, "")
+      end
+
       def asset_inline(name)
         path = File.join(@public_assets_dir, name)
 
@@ -119,7 +123,6 @@ def formatted_source_file(source_file)
 
       # Returns a table containing the given source files
       def formatted_file_list(title, source_files)
-        title_id = title.gsub(/^[^a-zA-Z]+/, "").gsub(/[^a-zA-Z0-9\-_]/, "")
         template("file_list").result(binding)
       end
 

diff --git a/views/file_list.erb b/views/file_list.erb
@@ -1,4 +1,4 @@
-<div class="file_list_container" id="<%= title_id %>">
+<div class="file_list_container" id="<%= to_id(title) %>">
   <h2>
     <span class="group_name"><%= title %></span>
     (<span class="covered_percent">
@@ -13,7 +13,7 @@
     )
   </h2>
 
-  <a name="<%= title_id %>"></a>
+  <a name="<%= to_id(title) %>"></a>
 
   <div>
     <b><%= source_files.length %></b> files in total.