Problem attaching image to invoice

[Geert Janssens]

On Wednesday 10 February 2016 10:42:39 Derek Atkins wrote:
> Geert Janssens <geert.gnucash at kobaltwit.be> writes:
> > On Tuesday 09 February 2016 20:03:00 David Goodenough wrote:
> >> That is disappointing.  I would have thought that it should be a
> >> strong requirement in the business world as well.  Would it be
> >> difficult to implement for the business module as well?
> > 
> > It would certainly be useful for the business features. There is
> > currently an enhancement request for this already [1]
> > 
> > Whether it would be difficult to implement depends partly on the
> > skills of the person willing to look into it. It is a fairly broad
> > change in that it affects many layers:
> > the storage system, the invoice logic and the invoice gui. And
> > probably it requires
> > some tweaks to the existing feature to make it more generally
> > usable.
> 
> It would probably be easiest to modify the function to bypass the
> "read-only" nature of Invoice transactions and allow attachment,
> without allowing any other changes.  It would be a small breach of
> abstraction, but probably the easiest path.
> 
True. However, as already mentioned in the bug [1] doing so would mean the external 
document is linked to the transaction. Unposting the invoice, deletes the transaction and hence 
the reference to the external document. I think linking the reference to the invoice object 
would be cleaner.

Regards,

Geert

[1] https://bugzilla.gnome.org/show_bug.cgi?id=735408