[Koha-bugs] [Bug 8007] Discharge management

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Apr 28 09:41:39 CEST 2015


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8007

Jonathan Druart <jonathan.druart at biblibre.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Failed QA                   |Passed QA

--- Comment #108 from Jonathan Druart <jonathan.druart at biblibre.com> ---
(In reply to Tomás Cohen Arazi from comment #106)
(In reply to Robin Sheat from comment #107)

Hi Tomás and Robin,

As you can see, this bug report has been opened a while ago. And I rewrote the
original patch (from Matthias) to make it cleaner.

The wiki page is outdated, I will put a warning on it. Sorry about that (I have
forgotten the existence of this page...)

Almost all of your concerns have been addressed/answered in the comments, but I
admit 100 comments is a lot to read :)

First, the dependencies. Have a look at comment 64 and comment 65, the packaged
perl modules don't seem to work. python-pisa works as needed but as it's a
python dependency I preferred to remove it (so python vs non packaged perl
module).

Then the paths, the pdf file is generated on the fly (see Katrin's concerns on
comment 64). The html and pdf files are generated using tmpnam.

> * the RFC talks about mailing things, but I can't see in the patches where
> that happens, does it exist? I was trying to verify it was using the
> mailqueue, as the RFC implies (but doesn't say) that it doesn't.

This has been abandoned.

The most recent "doc" is the commit messages of the last (non-obsoleted)
patches.

I spent days on this development, starting 2 years ago, and I have rewritten it
several times. It starts to make me crazy...

Back to Passed QA to let you see my comment.

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list