In addition when I add "?method=foo" to the src url, where it should
parse out and set the querystring, it doesn't behave correctly either.  

Its almost seems like the setSrc function is being called twice; the
first time when it gets parsed correctly and the "?method=foo" gets
stripped from the src attribute by:

        url.query = null;
        this.src = url.toString();

then it gets run again and the querystring gets blanked out, just like
when you set the querystring attribute like I have.  

Just a thought.
David

-----Original Message-----
From: Henry Minsky [mailto:[EMAIL PROTECTED] 
Sent: Thursday, September 06, 2007 3:02 PM
To: David Buckler
Cc: [email protected]
Subject: Re: [Laszlo-user] Querystring Dataset Attribute

Yes, that is a bug. I will file a bug report.

The setter for the "src" attribute in LzDataset.lzs is nulling out the
querystring arg, because it tries to parse it out of the src url.

LzDataset.lzs:
function setSrc( src ) {
    var url = new LzURL(src);
    this.querystring = url.query;
    url.query = null;
    this.src = url.toString();
    if ( this.autorequest ){
        this.doRequest() ;
    }
}

I guess it ought to check if the user supplied the argument during
init. The question is what to do if there is a conflict between the
src url and the querystring, like

<dataset src="http:foo.php?method=foo" querystring="method=bar"/>

I guess a runtime warning would be the right thing to do.

The workaround for you would be to call setQueryString manually before
making the request.





On 9/6/07, David Buckler <[EMAIL PROTECTED]> wrote:
>
> Thanks for checking this out Henry, here is a test line, obviously the
> cfc that this points to isn't real.  Also in our situation we add
query
> parameters with the setQueryParam method. When I watch the HTTP
requests
> I notice that the querystring parameter is never added but the
> parameters that are with setQueryParam are.  I also tested by adding
the
> method=testme to the end of the src and that didn't work either.
> However from what I understand that is just pulled off the end of the
> src and is stored in the querystring attribute.  Note that other
> datasets we are using where we set the method by means of the
> setQueryParam work fine.
>
> David
>
>
> <dataset name="ds_test" request="false" queuerequests="true"
type="http"
> src="https://testing.laszlo.com/test.cfc"; secure="true"
> querystring="method=testme"/>
>
> -----Original Message-----
> From: Henry Minsky [mailto:[EMAIL PROTECTED]
> Sent: Thursday, September 06, 2007 1:42 PM
> To: David Buckler
> Cc: [email protected]
> Subject: Re: [Laszlo-user] Querystring Dataset Attribute
>
> You might want to compare the behavior in one of the nightly builds of
> LPS 4, to see if there is an issue that may have been recently fixed,
> as well.
>
>
>
> On 9/6/07, Henry Minsky <[EMAIL PROTECTED]> wrote:
> > Do you have a small test case you can send me?
> >
> >
> >
> > On 9/6/07, David Buckler <[EMAIL PROTECTED]> wrote:
> > >
> > >
> > >
> > >
> > > I just installed 4.0.3 and have begun moving my code base from
3.3.3
> over to
> > > the new version.  I am having an issue with the querystring
> attribute in the
> > > dataset object.  It appears that Laszlo isn't even paying
attention
> to it.
> > > Has anyone had a similar problem or know anything about this?
> >
> >
> > --
> > Henry Minsky
> > Software Architect
> > [EMAIL PROTECTED]
> >
>
>
> --
> Henry Minsky
> Software Architect
> [EMAIL PROTECTED]
>


-- 
Henry Minsky
Software Architect
[EMAIL PROTECTED]

Reply via email to