Murielle666 / eid-applet

Automatically exported from code.google.com/p/eid-applet
Other
0 stars 0 forks source link

Enhancement: export servlets into seperate (sub)package #31

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Refactoring:

be.fedict.applet.service.JSON/Kml/PDF/VcardServlet -> 
be.fedict.applet.export...

be.fedict.applet.service.impl.JSON/Kml/PDF/VcardGenerator -> 
be.fedict.applet.export.impl...

be.fedict.applet.service.util -> be.fedict.applet.export.util

Original issue reported on code.google.com by bart.han...@fedict.be on 24 Feb 2010 at 10:07

GoogleCodeExporter commented 9 years ago
Further thoughts:

- split JSONServlet into JSONServlet and JSONGenerator (like PDF/KML/...)

- probably the JSON/PDF/...Servlet can be replaced with a generic Servlet 
calling 
generator

- new Generator interface/superclass, methods byte[] generate(EidData data); 
and 
String getMIMEType();

- instead of 1 export-package combining all export formats, create a package 
be.fedict.applet.export.pdf..., a be.fedict.applet.export.vcard...  

Benefits: applet package is cleaner, would allow for selective adding/removing 
export 
formats

Original comment by bart.han...@fedict.be on 25 Feb 2010 at 12:51

GoogleCodeExporter commented 9 years ago
I cannot do this for 1.0.1.GA since I need to keep binary compatibility at all 
time
for 1.0.x.GA releases.

Original comment by frank.co...@gmail.com on 25 Feb 2010 at 7:53

GoogleCodeExporter commented 9 years ago
Scheduled this for 2.0.

Original comment by frank.co...@gmail.com on 14 Aug 2010 at 6:28