myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hazem Saleh" <hazem.sa...@gmail.com>
Subject Re: Exporter comes finally to the scene :)
Date Sat, 19 Apr 2008 08:55:25 GMT
OK, I will apply this soon with Cagatay!

On 4/19/08, Cagatay Civici <cagatay.civici@gmail.com> wrote:
>
> Agree with Matzew, action listener way looks better.
>
> It'll also help making it work with client side state saving.
>
> On Sat, Apr 19, 2008 at 11:11 AM, Matthias Wessendorf <matzew@apache.org>
> wrote:
>
> > On Sat, Apr 19, 2008 at 9:22 AM, Hazem Saleh <hazem.saleh@gmail.com>
> > wrote:
> > > Hi Matzew,
> > > I followed the previous pattern of the excelExport component.
> > > So IMO, I think it is not odd for the component users.
> >
> >
> > well, perhaps personal choice.
> >
> > Take this:
> >
> > http://myfaces.apache.org/trinidad/trinidad-api/tagdoc/tr_fileDownloadActionListener.html
> >
> > that is (somewhat) similar and other libs do it in a similar fashion.
> > I don't care that strong,
> > but looks like the "old" component did it already... well not correct
> > ;-)
> > In sandbox there is a way to change this, outside of sandbox not, but
> > do what you want.
> > Was just a suggestion ;-)
> >
> > -M
> >
> > >
> > >
> > >
> > > On Fri, Apr 18, 2008 at 7:41 PM, Matthias Wessendorf <
> > matzew@apache.org>
> > > wrote:
> > >
> > > >
> > > > On Fri, Apr 18, 2008 at 7:02 PM, Hazem Saleh <hazem.saleh@gmail.com>
> > > wrote:
> > > > > Hi Matzew,
> > > > > What is the problem with the previous syntax :-
> > > > >
> > > > > <s:exporter for="<your dataTable id>" fileType="<EXCEL
or PDF>">
> > > > >     <h:commandButton action="" value="Export"/>
> > > > >  </s:exporter>
> > > >
> > > > I thought that the exporter would be an ActionListener.
> > > > Therefore embedding it inside an ActionSource(2) component
> > > > would make sense.
> > > >
> > > > I think it is odd, the nest the button inside a comp, to do the
> > export.
> > > > Nesting the exporter inside the button makes more sense, since
> > > > the button performs the action. So IMO it should be ActionListener.
> > > >
> > > > -M
> > > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > >
> > > > >
> > > > >
> > > > > On Fri, Apr 18, 2008 at 6:57 PM, Matthias Wessendorf <
> > matzew@apache.org>
> > > > > wrote:
> > > > > > Hi,
> > > > > >
> > > > > > I would  instead of:
> > > > > >
> > > > > > <s:exporter for="<your dataTable id>" fileType="<EXCEL
or PDF>">
> > > > > >    <h:commandButton action="" value="Export"/>
> > > > > > </s:exporter>
> > > > > >
> > > > > > expect something like:
> > > > > >
> > > > > > <commandButton...>
> > > > > >    <s:exporter/>
> > > > > > </commandButton>
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > On Fri, Apr 18, 2008 at 6:49 PM, Hazem Saleh <
> > hazem.saleh@gmail.com>
> > > > > wrote:
> > > > > > > Hi Team,
> > > > > > > Generated file is now depending on the mime type :) .
> > > > > > >
> > > > > > >  Here is an example of usage)
> > > > > > >  <s:exporter for="<your dataTable id>" fileType="<XLS
or
> > PDF>">
> > > > > > >      <h:commandButton action="" value="Export"/>
> > > > > > >  </s:exporter>
> > > > > > > Thanks all for your suggestions.
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > On Fri, Apr 18, 2008 at 5:45 AM, Hazem Saleh <
> > hazem.saleh@gmail.com>
> > > > > wrote:
> > > > > > > > Hi Team,
> > > > > > > > Iam pleased to tell you that I integrated both the
> > excelExport and
> > > the
> > > > > > > pdfExport components into
> > > > > > > > one single component called (exporter).
> > > > > > > >
> > > > > > > > Till now, it exports the contents of DataTables to
EXCEL or
> > PDF
> > > files.
> > > > > > > >
> > > > > > > > Here is an example of usage)
> > > > > > > > <s:exporter for="<your dataTable id>" fileType="<EXCEL
or
> > PDF>">
> > > > > > > >     <h:commandButton action="" value="Export"/>
> > > > > > > > </s:exporter>
> > > > > > > >
> > > > > > > > Here is the patch URL:
> > > > > > > > https://issues.apache.org/jira/browse/TOMAHAWK-1231
> > > > > > > >
> > > > > > > > Thanks all very much.
> > > > > > > > --
> > > > > > > > Hazem Ahmed Saleh Ahmed
> > > > > > > > http://www.jroller.com/page/HazemBlog
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Hazem Ahmed Saleh Ahmed
> > > > > > > http://www.jroller.com/page/HazemBlog
> > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Matthias Wessendorf
> > > > > >
> > > > > > further stuff:
> > > > > > blog: http://matthiaswessendorf.wordpress.com/
> > > > > > sessions: http://www.slideshare.net/mwessendorf
> > > > > > mail: matzew-at-apache-dot-org
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > >
> > > > > Hazem Ahmed Saleh Ahmed
> > > > > http://www.jroller.com/page/HazemBlog
> > > >
> > > >
> > > >
> > > > --
> > > >
> > > >
> > > >
> > > > Matthias Wessendorf
> > > >
> > > > further stuff:
> > > > blog: http://matthiaswessendorf.wordpress.com/
> > > > sessions: http://www.slideshare.net/mwessendorf
> > > > mail: matzew-at-apache-dot-org
> > > >
> > >
> > >
> > >
> > > --
> > >
> > > Hazem Ahmed Saleh Ahmed
> > > http://www.jroller.com/page/HazemBlog
> >
> >
> >
> >
> > --
> >  Matthias Wessendorf
> >
> > further stuff:
> > blog: http://matthiaswessendorf.wordpress.com/
> > sessions: http://www.slideshare.net/mwessendorf
> > mail: matzew-at-apache-dot-org
> >
> >
>
>


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

Mime
View raw message