Model::escape() now coerces non-string (non-falsy) values to strings before escaping them.

This commit is contained in:
Jeremy Ashkenas
2011-01-11 12:58:15 -05:00
parent c933262555
commit 289d4e0066
2 changed files with 3 additions and 1 deletions

View File

@@ -160,7 +160,7 @@
var html;
if (html = this._escapedAttributes[attr]) return html;
var val = this.attributes[attr];
return this._escapedAttributes[attr] = escapeHTML(val == null ? '' : val);
return this._escapedAttributes[attr] = escapeHTML(val == null ? '' : '' + val);
},
// Returns `true` if the attribute contains a value that is not null

View File

@@ -110,6 +110,8 @@ $(document).ready(function() {
equals(doc.escape('audience'), 'Bill & Bob');
doc.set({audience: 'Tim > Joan'});
equals(doc.escape('audience'), 'Tim > Joan');
doc.set({audience: 10101});
equals(doc.escape('audience'), '10101');
doc.unset('audience');
equals(doc.escape('audience'), '');
});