#89 ✓invalid
Alexandre Gomes

trouble loading this script in env.js because of the ampersand (&) symbol

Reported by Alexandre Gomes | January 29th, 2010 @ 08:39 AM | in 1.2.x

Hi all,

I'm having trouble loading this script in env.js because of the
ampersand (&) symbol.

When I change to & it seems to work but I really need it to be a pure ampersand. Here's the code for a patch I made: 3136 + if( this._needToReplaceEntities(strV) ) { 3139 3138 iRet = this._replaceEntities(strV); 3140 3139 3141 3140 if(iRet == XMLP._ERROR) { ... 3376 3375 3377 3376 } 3378 3377 3378 + XMLP.prototype._needToReplaceEntities = function(strV){ 3379 + var entities = $w.$entityDefinitions; 3380 + for(var entity in entities){ 3381 + if( strV.match( "&"+entity+";" ) ) return true; 3382 + } 3383 + return false; 3384 + }

Comments and changes to this ticket

  • Alexandre Gomes

    Alexandre Gomes January 29th, 2010 @ 08:42 AM

    • no changes were found...
  • Thatcher

    Thatcher February 3rd, 2010 @ 04:25 PM

    • State changed from “new” to “open”
    • Milestone set to “1.2.x”

    if possible can you supply a test case that breaks? I get the gist so I'll make the patch (it will be different becase of the refactor but can still be applied to Envjs.location). The example will help me assign a test case to the ticket number and be confident it's correct from here on out.

    Thatcher

  • Thatcher

    Thatcher February 5th, 2010 @ 05:37 PM

    • State changed from “open” to “invalid”

    Hi Alexandre, when I went to create a test case I could not get ff to honor a parameter with

    &
    for example
    http://www.google.com/search?hl=en&site=&q=xml&btnG=Search&num=100
    return 100 results but
    http://www.google.com/search?hl=en&site=&q=xml&btnG=Search&num=100
    will only return 20.

    Can you help me understand a test case that works in firefoc but not in Envjs? I'll mark this as invalid just to help flag that i cant produce a valid test case as I understand the issue, though I might be misunderstanding the issue.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป

a javascript browser environment

People watching this ticket

Attachments

Tags

Pages