Re: Exporter new syntax discussion

by Hazem Saleh :: Rate this Message:

Hi Team,

Iam pleased to tell you that the new syntax is applied to the Exporter component.

The syntax now is :
<h:commandButton value="Export">
        <s:exporterActionListener filename="xyz"
                                               fileType="<XLS or PDF>"
                                               for="<yourTableID>"/>
</h:commandButton>

Here is the patch url:
https://issues.apache.org/jira/browse/TOMAHAWK-1252

BTW, the exporter now supports the client state saving :).
Thanks all :).

On Fri, May 9, 2008 at 12:13 PM, Matthias Wessendorf <mwessendorf@...> wrote:

I totally agree with Gerald.
I think the old syntax is just wrong.
So... +1

Sent from my iPod.

Am 08.05.2008 um 20:29 schrieb "Gerald Müllan" <gmuellan@...>:

Hi,

i would prefer the new syntax since it seems to be more natural to me.

Similar to an updateActionListener; the command component fires the
listener embedded inside.

So, +1

Any special reason why breaking the common sense?

cheers,

Gerald

On Thu, May 8, 2008 at 8:01 PM, Hazem Saleh <hazem.saleh@...> wrote:

Hi Team,

After I started modifying of the Exporter component syntax,

To be like this :
<h:commandButton ...>
<s:exporter for="tbl_cars"
fileType="XLS"/>
</h:commandButton>

Some guys didnot like this new syntax, and they preferred the current one :
<s:exporter for="tbl_cars" fileType="XLS">
<h:commandButton .../>
</s:exporter>

I need your input regarding this.
(+1) for supporting the new syntax.
(-1) for supporting the old syntax.

Thanks!

--
Hazem Ahmed Saleh Ahmed
http://www.jroller.com/page/HazemBlog

--
http://www.irian.at

Your JSF powerhouse -
JSF Consulting, Development and
Courses in English and German

Professional Support for Apache MyFaces

--
Hazem Ahmed Saleh Ahmed
http://www.jroller.com/page/HazemBlog

« Return to Thread: Exporter new syntax discussion