#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.

New-ticket 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