@@ -604,7 +604,7 @@ Ajax.InPlaceEditor = Class.create({
this.triggerCallback('onEnterHover');
},
getText: function() {
- return this.element.innerHTML;
+ return this.element.innerHTML.unescapeHTML();
},
handleAJAXFailure: function(transport) {
this.triggerCallback('onFailure', transport);
src/controls.js
@@ -24,6 +24,8 @@
<p id="newtbe">New to be edited</p>
<p id="newtbe_external">External control for it</p>
+<p id="contains_ampersand">Me & Myself</p>
+
<div id="tobeeditedMultiLine">First line<br/>
Second line<br/>
Third line</div>
@@ -875,6 +877,14 @@ Third line</div>
assertEqual('#ff0000', ipe.options.highlightColor, 'Highlight color was not migrated');
assertEqual('#00ff00', ipe.options.highlightEndColor, 'Highlight end color was not migrated');
ipe.dispose();
+ }},
+
+
+ testShouldShowAmpersandsProperly: function() {with(this) {
+ var ipe = new Ajax.InPlaceEditor('contains_ampersand', '', {});
+ Event.simulateMouse('contains_ampersand', 'click');
+ assertEqual("Me & Myself", $$('form#contains_ampersand-inplaceeditor input.editor_field')[0].value);
+ ipe.dispose();
}}
// FIXME: add AC w/o key conflicts?
// FIXME: doc w/ full details on what's new, what's changed, etc. + deprecation layer info.
test/unit/ajax_inplaceeditor_test.html
8daa8d946527e1bc7030c69ca7677df12c602910
Sean Kirby
sskirby@Resartus.(none)
http://github.com/madrobby/scriptaculous/commit/f13b5c05fdc149558f9be6f9096d43c46abfd084
f13b5c05fdc149558f9be6f9096d43c46abfd084
2008-09-22T10:35:54-07:00
2008-09-22T10:35:54-07:00
InPlaceEditor will correctly show text with ampersands upon edit (no more &)
58b91e07e1fcfac9f3c07a8a085f333bd5c1658f
Sean Kirby
sskirby@Resartus.(none)