[PATCH] dialog-payment fixes

Matthijs Kooijman matthijs at stdin.nl
Sat Nov 20 13:04:42 EST 2010


On Sat, Nov 20, 2010 at 09:41:04AM -0800, John Ralls wrote:
> 
> On Nov 20, 2010, at 5:28 AM, Matthijs Kooijman wrote:
> 
> > Hi folks,
> > 
> > Here's a patch to fix a pet peeve I've had for quite some time with the
> > invoice payment dialog: Make it possible to enter just the invoice
> > (number), instead of always needing to select the owner manually. This
> > is especially useful when processing bank statements, when the invoice
> > number is listed on the statement.
> > 
> > I've added a second patch, for a bug I noticed while digging through the
> > code. Note that the inconsistent use of spaces in between a function
> > name and the opening brace for the arguments is to remain consistent
> > with the surrounding code (but dialog-payment.c uses different
> > styles...).
> > 
> > Patches are against current SVN trunk.
> > 
> 
> Thanks, but could you resubmit this as an enhancement request in
> bugzilla? Patches in the mailing list have a bad habit of getting
> forgotten.
Sure. I got my queue from the README, though, so you might want to
update that:

      and send /tmp/mypatch to gnucash-devel at gnucash.org with `[PATCH]'
      prefixed to the Subject: of the email.  Note that this ONLY WORKS
      if you have NOT added any new files to the source tree.

I think there was another spot where it was documented like this, but I
can't find that right now. The Development page on the wiki has it right
(but I missed the instructions there the first time).

Gr.

Matthijs
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.gnucash.org/pipermail/gnucash-devel/attachments/20101120/aa2460e9/attachment.bin>


More information about the gnucash-devel mailing list