Remove CGI.unescape duplication in Store#get

Shopify · Aug 15, 2023 · 4336333 · 4336333
1 parent 5a24001
commit 4336333
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/lib/ruby_lsp/store.rb b/lib/ruby_lsp/store.rb
@@ -22,13 +22,11 @@ def initialize
 
     sig { params(uri: URI::Generic).returns(Document) }
     def get(uri)
-      path = uri.to_standardized_path
-      return T.must(@state[T.must(uri.opaque)]) unless path
-
-      document = @state[path]
+      document = @state[uri.storage_key]
       return document unless document.nil?
 
-      set(uri: uri, source: File.binread(CGI.unescape(path)), version: 0)
+      path = T.must(uri.to_standardized_path)
+      set(uri: uri, source: File.binread(path), version: 0)
       T.must(@state[path])
     end