Skip to content

Commit 4f88fa1

Browse files
committed
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.
1 parent 7cbe2d3 commit 4f88fa1

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lib/rails/html/scrubbers.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -137,11 +137,11 @@ def validate!(var, name)
137137
# If set, attributes included will be removed.
138138
class TargetScrubber < PermitScrubber
139139
def allowed_node?(node)
140-
!@tags.include?(node.name)
140+
!super
141141
end
142142

143143
def scrub_attribute?(name)
144-
@attributes.include?(name)
144+
!super
145145
end
146146
end
147147
end

0 commit comments

Comments
 (0)