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.

Script sources with querystrings can't be loaded

#60

Hi

e.g.

(script type="text/envjs" src="../../include.js?param1,param2" /)

The engine ends up trying to load a file at file:/path/to/../../include.js?param1,param2

I've resolved this by .split('?')[0]-ing the src passed to load(). I have no idea if I have broken anything else as a result of this change.

Cheers L

Reported by Lee Henson · March 19th, 2009 @ 07:47 AM

State: resolved
Milestone: 1.0 Release
Assigned to: Thatcher Thatcher

Activity

  1. Lee Henson
  2. Thatcher
    Thatcher
    • Milestone set to 1.0 Release
    • Tag set to load, script, url
    • State changed from new to open
    • Assigned user set to Thatcher

    justin meyer submitted a patch for this. keep an eye out I'm going to push out env.0.9.0.js which has this patch

    March 24th, 2009 @ 04:00 PM

  3. Lee Henson
    Lee Henson

    Yeah, mine was pretty much a placeholder for jmvc until Justin could work on it. As far as I am concerned this is now closed!

    March 25th, 2009 @ 12:35 AM

  4. Thatcher
    Thatcher
    • State changed from open to resolved

    September 9th, 2009 @ 05:49 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.