From 4f88fa1712e1e6513aef19384ba2aa9fee6808ef Mon Sep 17 00:00:00 2001 From: yui-knk Date: Wed, 17 Jun 2015 20:14:36 +0900 Subject: [PATCH] Refactor TargetScrubber TargetScrubber asts as reversal to PermitScrubber. Two methods `allowed_node?` and `scrub_attribute?` is now too much to know about implementaion of PermitScrubber. It should simply reverse a result of parent method. --- lib/rails/html/scrubbers.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/rails/html/scrubbers.rb b/lib/rails/html/scrubbers.rb index b3942a7..1384a2f 100644 --- a/lib/rails/html/scrubbers.rb +++ b/lib/rails/html/scrubbers.rb @@ -137,11 +137,11 @@ def validate!(var, name) # If set, attributes included will be removed. class TargetScrubber < PermitScrubber def allowed_node?(node) - !@tags.include?(node.name) + !super end def scrub_attribute?(name) - @attributes.include?(name) + !super end end end