myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matthias Wessendorf" <mat...@apache.org>
Subject Re: Exporter comes finally to the scene :)
Date Sat, 19 Apr 2008 08:11:25 GMT
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

Mime
View raw message