Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

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

#89

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 + }

Reported by Alexandre Gomes · January 29th, 2010 @ 08:39 AM

State: invalid
Milestone: 1.2.x
Assigned to: Thatcher Thatcher

Activity

  1. Alexandre Gomes
    Alexandre Gomes

    January 29th, 2010 @ 08:42 AM

  2. Thatcher
    Thatcher
    • 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

    February 3rd, 2010 @ 04:25 PM

  3. Thatcher
    Thatcher
    • 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.

    February 5th, 2010 @ 05:37 PM

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.