Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #440 from ZogStriP/composer-count-only-significant-whitespaces 

the composer will now count only significant whitespaces
This commit is contained in:
Sam 2013-03-12 16:57:52 -07:00
parent cde9d8c3a9 439ac95b73
commit 259827d14b
2 changed files with 9 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/assets/javascripts/discourse/models/composer.js
View File

@ -478,7 +478,7 @@ Discourse.Composer = Discourse.Model.extend({
},
/**
Computes the length of the reply minus the quote(s).
Computes the length of the reply minus the quote(s) and non-significant whitespaces
@property replyLength
**/
@ -486,7 +486,7 @@ Discourse.Composer = Discourse.Model.extend({
var reply = this.get('reply');
if(!reply) reply = "";
while (Discourse.BBCode.QUOTE_REGEXP.test(reply)) { reply = reply.replace(Discourse.BBCode.QUOTE_REGEXP, ""); }
return reply.trim().length;
return reply.replace(/\s+/img, " ").trim().length;
}.property('reply')
});

8
spec/javascripts/models/composer_spec.js
View File

@ -10,10 +10,16 @@ describe("Discourse.Composer", function() {
});
it("trims whitespaces", function() {
var composer = Discourse.Composer.create({ reply: "\nbasic reply\t" });
var composer = Discourse.Composer.create({ reply: " \nbasic reply\t" });
expect(composer.get('replyLength')).toBe(11);
});
it("count only significant whitespaces", function() {
// this will count the '\n' only once
var composer = Discourse.Composer.create({ reply: "ba sic\n\nreply" });
expect(composer.get('replyLength')).toBe(12);
});
it("removes quotes", function() {
var composer = Discourse.Composer.create({ reply: "1[quote=]not counted[/quote]2[quote=]at all[/quote]3" });
expect(composer.get('replyLength')).toBe(3);