[Tickets #3700] Proforma - invoice manager
bugs@bugs.horde.org
bugs at bugs.horde.org
Thu Mar 30 03:57:46 PST 2006
DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.
Ticket URL: http://bugs.horde.org/ticket/?id=3700
-----------------------------------------------------------------------
Ticket | 3700
Updated By | thomas at gelf.net
Summary | Proforma - invoice manager
Queue | Horde Base
Version | HEAD
State | Feedback
Priority | 1. Low
Type | Enhancement
Owners |
-----------------------------------------------------------------------
thomas at gelf.net (2006-03-30 03:57) wrote:
Great work!!!
Some comments (Please don't take them too serious if I'm totally wrong on
some points. I installed proforma yesterday evening and just played around
a
little bit - but I didn't dig deep enough to be sure about what I'm
writing):
- using AJAX for tax calculation while updating fields seems a useless
over-
head - it should be worth the effort to write some client-side JavaScript
to do
this job. While finally saving an invoice I wouldn't trust these values
and do
the same calculation once again on server side.
This may be seen as a duplication of code (same thing in PHP and JS) -
but it saves us a lot of HTTP traffic. Especially if you're going to use
HTTPS
this shouldn't be underestimated.
- show tax sums only for taxes used in the current invoice (there are
countries
with 5 and more tax variants - and only one of them is used in probably
99%
of all invoices)
- what will the SQL layout look like? I would suggest something like:
* invoices: containing customer details (address... - it's important to
duplicate them and not to save just an ID), billing date, ctime, mtime,
document type (offer, invoice, proforma invoice...), state (sent,
payed)
* invoice details: invoice id, amount, price, tax, currency
I'm in a hurry right now - but maybe this weekend I can find some time to
go on playing with "proforma" ;-)
Kind regards,
Thomas Gelf
NB: Please keep this ticket up to date!
More information about the bugs
mailing list