Browse Source

Glitch::FilterHelper -> Glitch::KeywordMuteHelper. #234.

The class helps out with keyword mutes, not just some general concept of
"filtering".
closed-social-glitch-2
David Yip 7 years ago
parent
commit
53c86b29f0
No known key found for this signature in database GPG Key ID: 7DA0036508FCC0CC
3 changed files with 5 additions and 5 deletions
  1. +1
    -1
      app/lib/feed_manager.rb
  2. +2
    -2
      app/models/glitch/keyword_mute_helper.rb
  3. +2
    -2
      spec/models/glitch/keyword_mute_helper_spec.rb

+ 1
- 1
app/lib/feed_manager.rb View File

@ -178,7 +178,7 @@ class FeedManager
end end
def keyword_filter?(status, receiver_id) def keyword_filter?(status, receiver_id)
Glitch::FilterHelper.new(receiver_id).matches?(status)
Glitch::KeywordMuteHelper.new(receiver_id).matches?(status)
end end
def filter_from_mentions?(status, receiver_id) def filter_from_mentions?(status, receiver_id)

app/models/glitch/filter_helper.rb → app/models/glitch/keyword_mute_helper.rb View File

@ -1,6 +1,6 @@
require 'htmlentities' require 'htmlentities'
class Glitch::FilterHelper
class Glitch::KeywordMuteHelper
include ActionView::Helpers::SanitizeHelper include ActionView::Helpers::SanitizeHelper
attr_reader :text_matcher attr_reader :text_matcher
@ -26,6 +26,6 @@ class Glitch::FilterHelper
end end
def prepare_text(text) def prepare_text(text)
entity_decoder.decode(strip_tags(text))
entity_decoder.decode(strip_tags(text)).tap { |x| puts x }
end end
end end

spec/models/glitch/filter_helper_spec.rb → spec/models/glitch/keyword_mute_helper_spec.rb View File

@ -1,9 +1,9 @@
require 'rails_helper' require 'rails_helper'
RSpec.describe Glitch::FilterHelper do
RSpec.describe Glitch::KeywordMuteHelper do
describe '#matches?' do describe '#matches?' do
let(:alice) { Fabricate(:account, username: 'alice').tap(&:save!) } let(:alice) { Fabricate(:account, username: 'alice').tap(&:save!) }
let(:helper) { Glitch::FilterHelper.new(alice) }
let(:helper) { Glitch::KeywordMuteHelper.new(alice) }
it 'ignores names of HTML tags in status text' do it 'ignores names of HTML tags in status text' do
status = Fabricate(:status, text: '<addr>uh example</addr>') status = Fabricate(:status, text: '<addr>uh example</addr>')

Loading…
Cancel
Save