Hi Adrian, I'm not happy with the change of the Graphics2DImagePainter interface in XGCommons that you seem to have done locally. This interface is targeted at painting only. IMO it should not assume that every image also has a URI associated with it for whatever that URI will then be used. For example, foreign objects don't have an explicit URI. If you need the MIME type and the original URI of an image inside the AFPGraphics2DAdapter, you can pass those in through the RendererContext. For example, AFPRenderer.drawImage() already has the ImageInfo object and would simply need to put it in the RendererContext. Extending the Graphics2DPainter interface just for transporting some non-painting-related information that only the AFP renderer needs is not justification enough for this change if you ask me. Plus this breaks other applications that rely on this interface.
On 02.06.2008 16:30:40 acumiskey wrote: > Author: acumiskey > Date: Mon Jun 2 07:30:39 2008 > New Revision: 662448 > > URL: http://svn.apache.org/viewvc?rev=662448&view=rev > Log: > * getImageUri() implementation provided on Graphics2DAdaptor implementations. > * Refactored data object creation abilities from > AbstractResourceGroupContainer into a new DataObjectFactory class. > * Added mime type to ObjectType inner class in Registry for warn/error > logging purposes. > * Fixed problem with handling ObjectContainer objects in IncludeObject. > * Added documentation for configuring AFP resource group files. > Jeremias Maerki
