Exporter new syntax discussion

View: New views
11 Messages — Rating Filter:   Alert me  

Exporter new syntax discussion

by Hazem Saleh :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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

Re: Exporter new syntax discussion

by Hazem Saleh :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Thu, May 8, 2008 at 9: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

(-1) for :).

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

Re: Exporter new syntax discussion

by zwadia :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

-1

On Thu, May 8, 2008 at 2:02 PM, Hazem Saleh <hazem.saleh@...> wrote:
On Thu, May 8, 2008 at 9: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

(-1) for :).


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


Re: Exporter new syntax discussion

by Gerald Müllan-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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

Re: Exporter new syntax discussion

by Hazem Saleh :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi Gerald,

No special reason, I just asked people who use this component and they disliked the new syntax.

On Thu, May 8, 2008 at 9:29 PM, Gerald Müllan <gmuellan@...> wrote:
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

Re: Exporter new syntax discussion

by Cagatay Civici :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

+1 for the newer one.

On Thu, May 8, 2008 at 9:52 PM, Hazem Saleh <hazem.saleh@...> wrote:
Hi Gerald,

No special reason, I just asked people who use this component and they disliked the new syntax.


On Thu, May 8, 2008 at 9:29 PM, Gerald Müllan <gmuellan@...> wrote:
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



--


Re: Exporter new syntax discussion

by Glauco P. Gomes :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

+1

I'm using the old component in one project (with the old sintax), but I prefer the new sintax.

Glauco P. Gomes

Cagatay Civici escreveu:
+1 for the newer one.

On Thu, May 8, 2008 at 9:52 PM, Hazem Saleh <hazem.saleh@...> wrote:
Hi Gerald,

No special reason, I just asked people who use this component and they disliked the new syntax.


On Thu, May 8, 2008 at 9:29 PM, Gerald Müllan <gmuellan@...> wrote:
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



Re: Exporter new syntax discussion

by Bernd Bohmann :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

+1 for the new syntax

Glauco P. Gomes schrieb:

> +1
>
> I'm using the old component in one project (with the old sintax), but I prefer
> the new sintax.
>
> Glauco P. Gomes
>
> Cagatay Civici escreveu:
>> +1 for the newer one.
>>
>> On Thu, May 8, 2008 at 9:52 PM, Hazem Saleh <hazem.saleh@...
>> <mailto:hazem.saleh@...>> wrote:
>>
>>     Hi Gerald,
>>
>>     No special reason, I just asked people who use this component and they
>>     disliked the new syntax.
>>
>>
>>     On Thu, May 8, 2008 at 9:29 PM, Gerald Müllan <gmuellan@...
>>     <mailto:gmuellan@...>> wrote:
>>
>>         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@...
>>         <mailto: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
>>
>>
>

Re: Exporter new syntax discussion

by Volker Weber-5 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

+1 for the new syntax.


Regards,
    Volker

2008/5/9 Bernd Bohmann <bernd.bohmann@...>:

> +1 for the new syntax
>
> Glauco P. Gomes schrieb:
>> +1
>>
>> I'm using the old component in one project (with the old sintax), but I prefer
>> the new sintax.
>>
>> Glauco P. Gomes
>>
>> Cagatay Civici escreveu:
>>> +1 for the newer one.
>>>
>>> On Thu, May 8, 2008 at 9:52 PM, Hazem Saleh <hazem.saleh@...
>>> <mailto:hazem.saleh@...>> wrote:
>>>
>>>     Hi Gerald,
>>>
>>>     No special reason, I just asked people who use this component and they
>>>     disliked the new syntax.
>>>
>>>
>>>     On Thu, May 8, 2008 at 9:29 PM, Gerald Müllan <gmuellan@...
>>>     <mailto:gmuellan@...>> wrote:
>>>
>>>         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@...
>>>         <mailto: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
>>>
>>>
>>
>



--
inexso - information exchange solutions GmbH
Bismarckstraße 13 | 26122 Oldenburg
Tel.: +49 441 4082 356 |
FAX: +49 441 4082 355 | www.inexso.de

Re: Exporter new syntax discussion

by Matthias Wessendorf-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

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

Re: Exporter new syntax discussion

by Hazem Saleh :: Rate this Message:

Reply to Author | View Threaded | Show Only 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