RE: [displaytag-devel] Support for displaying results from JSTL SQL Tag

[Fabrizio Giustina]

Title: RE: [displaytag-devel] Support for displaying results from JSTL SQL Tag

ok, got the time to really look at it:

 

I think your patch is totally unneeded, Matt. The result of a jstl query tag is a Result object, and the row property (getRow()) returns an array of Maps. The array of maps is already supported as a standard source for the table tag, so there is no reason for an addictional attribute.

 

This will also work in the standard table tag, not only in the EL one. Actually the only missing thing is the automatical column discovery, as you noted in your example: I'm going to submit a patch (and a couple of test case) now so that the column list is generated from the map keys.

 

Please test it without patches and let us know if it works for you

 

fabrizio

 

 

From: Fabrizio Giustina
Sent: Wed 10/03/2004 15.43
To: [EMAIL PROTECTED]
Subject: RE: [displaytag-devel] Support for displaying results from JSTL SQL Tag

Hi matt,

I just had a look at your proposed patch...

 

I can't try it or check it accurately at the moment, so I'm asking you some (maybe stupid) questions:

- Why do you need to define an addictional "items" attribute?

- Why can't the list of items generated by jstl sql tag be handled like any other collection?

- What is the type of the jstl generated "items" object? Is it a sql RowSet?

 

 

fabrizio

 

 

 

From: [EMAIL PROTECTED] on behalf of Matt Raible
Sent: Tue 09/03/2004 22.12
To: [EMAIL PROTECTED]
Subject: RE: [displaytag-devel] Support for displaying results from JSTL SQL Tag

Should I assume that no reponse means "we don't care - go ahead and
commit it?". ;0)

Matt

> -----Original Message-----
> From: [EMAIL PROTECTED]
> [mailto:[EMAIL PROTECTED]] On
> Behalf Of Matt Raible
> Sent: Sunday, March 07, 2004 10:15 PM
> To: [EMAIL PROTECTED]
> Subject: [displaytag-devel] Support for displaying results
> from JSTL SQL Tag
>
>
> All,
>
> I did some hacking this afternoon and managed to add the ability for 
> the displaytag to render results from JSTL's SQL Tag.
>
> Demo here: http://demo.raibledesigns.com/appfuse/users.jsp
>
> Ehancement/Patch here: 
> https://sourceforge.net/tracker/?
> func=detail&atid=536613&aid=911760&group_id=73068
>
> This patch only effect the EL tag - so users must specify
> that URI/TLD 
> to use this feature.  Let me know what you think and I'll
> commit it if 
> everyone likes it.
>
> Matt
>
>
>
> -------------------------------------------------------
> This SF.Net email is sponsored by: IBM Linux Tutorials
> Free Linux tutorial presented by Daniel Robbins, President
> and CEO of GenToo technologies. Learn everything from
> fundamentals to system
> administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
> _______________________________________________
> displaytag-devel mailing list [EMAIL PROTECTED]
> https://lists.sourceforge.net/lists/listinfo/displaytag-devel
>




-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
_______________________________________________
displaytag-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

------------------------------------------------------- This SF.Net email is sponsored by: IBM Linux Tutorials Free Linux tutorial presented by Daniel Robbins, President and CEO of GenToo technologies. Learn everything from fundamentals to system administration.http://ads.osdn.com/?ad_id70&alloc_id638&op=click _______________________________________________ displaytag-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/displaytag-devel