Mr. Pedantic sez:

 if (slash == 0 || (colon > 0 && (slash == -1 || colon < slash)) {

But wait, then there is:

  http:foo

which is a relative url with an explicit scheme. Shouldn't that also prepend the load directory? It seems you really have to parse the url and then decide whether the path of that url is relative or not. Don't we already have an url parser utility?

On 2007-04-03, at 09:42 EDT, Max Carlson wrote:

Change 20070403-maxcarlson-s by [EMAIL PROTECTED] on 2007-04-03 06:40:38 PDT
    in /Users/maxcarlson/openlaszlo/legals-clean
    for http://svn.openlaszlo.org/openlaszlo/branches/legals

Summary: Fix DHTML proxied loads in DHTML for apps loaded from urls beginning with any protocol

New Features:

Bugs Fixed:

Technical Reviewer: promanik
QA Reviewer: ptw
Doc Reviewer: (pending)

Documentation:

Release Notes:

Details: LzBrowser.js - Leave urls beginning with slash or with a colon before the slash alone, 'specially for Mr. Pedantic :)


Tests:


Files:
M      WEB-INF/lps/lfc/services/platform/dhtml/LzBrowser.js

Changeset: http://svn.openlaszlo.org/openlaszlo/patches/20070403- maxcarlson-s.tar

Reply via email to