[Prototype-core] Re: Bug in the Prototype's String.unescapeHTML()?

[Vladimir V. Perepelitsa]

prototype-1.5.1.1:
line 406:

  unescapeHTML: function() {
    return this.replace(/&amp;/g,'&').replace(/&lt;/g,'<').replace(/&gt;/g,'>');
  }

but should be

  unescapeHTML: function() {
    return this.replace(/&lt;/g,'<').replace(/&gt;/g,'>').replace(/&amp;/g,'&');
  }

Compare the results of

  var string = '<';
  alert(string.escapeHTML().escapeHTML().unescapeHTML());
  alert(string.escapeHTML());

in Firefox and IE and see the difference

On 7/10/07, jdalton <[EMAIL PROTECTED]> wrote:
>
> Check Prototype 1.5.1 or 1.5.1.1 (the guts for that method don't seem
> to have that problem)
>

-- 
Best wishes,
Vladimir V. Perepelitsa aka Mons <[EMAIL PROTECTED]>

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Prototype: Core" group.
To post to this group, send email to prototype-core@googlegroups.com
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/prototype-core?hl=en
-~----------~----~----~----~------~----~------~--~---