codeninjasllc/discourse
1
0
Fork 0
mirror of https://github.com/codeninjasllc/discourse.git synced 2024-11-23 15:48:43 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

Allow TextSentinel#seems_unpretentious? to accept words joined with dashes or forward slashes. (Issue 1133)

Browse source
This commit is contained in:
Nathan Nontell 2013-09-16 09:45:57 -04:00
parent 406c29e79a
commit d95172cb5d
2 changed files with 9 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/text_sentinel.rb
View file

@ -67,7 +67,7 @@ class TextSentinel
def seems_unpretentious?
# Don't allow super long words if there is a word length maximum
@opts[:max_word_length].blank? || @text.split(/\s/).map(&:size).max <= @opts[:max_word_length]
@opts[:max_word_length].blank? || @text.split(/\s|\/|-/).map(&:size).max <= @opts[:max_word_length]
end

8
spec/components/text_sentinel_spec.rb
View file

@ -96,6 +96,14 @@ describe TextSentinel do
TextSentinel.new("{{$!").should_not be_valid
end
it "does allow a long alphanumeric string joined with slashes" do
TextSentinel.new("gdfgdfgdfg/fgdfgdfgdg/dfgdfgdfgd/dfgdfgdfgf", max_word_length: 30).should be_valid
end
it "does allow a long alphanumeric string joined with dashes" do
TextSentinel.new("gdfgdfgdfg-fgdfgdfgdg-dfgdfgdfgd-dfgdfgdfgf", max_word_length: 30).should be_valid
end
end
context 'title_sentinel' do