[seam-dev] Seam Reports

[George Gastaldi]

And what about the "Seam" prefix on the interfaces ? Should it be removed ?
I was in doubt with that also.

Regards,
George

Em 16/05/2011, às 07:50, Pete Muir <pmuir at redhat.com> escreveu:

>
> On 14 May 2011, at 06:01, George Gastaldi wrote:
>
>> Hello !
>>
>> I started some work on Seam Reports, but I would like some help from
>> you guys. I created some interfaces to abstract loading/compiling a
>> report, populating and rendering.
>> The source code is on https://github.com/gastaldi/reports (not merged
>> on official repos yet) - test case included (and passing) !.
>> I recommend read the api and impl code before.
>>
>> But the points I am in need of help are:
>>
>> 1) The rendering part is kinda obscure yet. It would be nice if the
>> developer could write something like:
>>
>> @Inject @PDF SeamReportRenderer renderer;
>>
>> and then call renderer.render(...);
>
> Agreed. I would try to spend more time finding an abstraction if you can...
>
>>
>> The fact is that: Every provider (JasperReports, Velocity, Birt, etc)
>> have their own way of doing the rendering part.
>> I created a SeamReportsOutputType enum, but that is not so cool, as
>> will certainly restrict the app developer. Jason Porter and Pete Royle
>> has pointed me that using Qualifiers (as the example above) should be
>> a good way to go.
>
> IOW an enum in Java describes a closed set, and the providers are an open set.
>
>>
>> 2)  Should the interfaces be rich enough to allow someone to create a
>> report by using a fluent interface (I believe that could be out of
>> scope of this module) or this module should just allow hiding the
>> implementation layer of existing reports framework as it does now (In
>> my iniitial conception, this is what I needed and wanted) ?
>>
>> I´ll love to hear your suggestions,
>>
>>
>> Regards,
>>
>> George Gastaldi
>>
>> _______________________________________________
>> seam-dev mailing list
>> seam-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/seam-dev
>