diff options
| author | Yorick Peterse <yorickpeterse@gmail.com> | 2015-12-30 20:16:53 +0300 |
|---|---|---|
| committer | Yorick Peterse <yorickpeterse@gmail.com> | 2015-12-31 17:46:47 +0300 |
| commit | 054df415f94abe1e517a729e53cdb325d592d31b (patch) | |
| tree | b7e5d560e39f05fdae0e5ee0ba654739eab59094 /lib/banzai/filter/redactor_filter.rb | |
| parent | d3951dfaa13b9aaf11695ef10fa63456ac75cc48 (diff) | |
Optimize CSS expressions produced by Nokogiri
Nokogiri produces inefficient XPath expressions when given CSS
expressions such as "a.gfm". Luckily these expressions can be optimized
quite easily while still achieving the same results.
In the two cases where this optimization is applied the run time has
been reduced from around 170 ms to around 15 ms.
Diffstat (limited to 'lib/banzai/filter/redactor_filter.rb')
| -rw-r--r-- | lib/banzai/filter/redactor_filter.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/banzai/filter/redactor_filter.rb b/lib/banzai/filter/redactor_filter.rb index f01a32b5ae5..66f77902319 100644 --- a/lib/banzai/filter/redactor_filter.rb +++ b/lib/banzai/filter/redactor_filter.rb @@ -10,7 +10,7 @@ module Banzai # class RedactorFilter < HTML::Pipeline::Filter def call - doc.css('a.gfm').each do |node| + Querying.css(doc, 'a.gfm').each do |node| unless user_can_see_reference?(node) # The reference should be replaced by the original text, # which is not always the same as the rendered text. |