[Koha-bugs] [Bug 11846] Correct database update for debarments with 9999-12-31

[bugzilla-daemon at bugs.koha-community.org]

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

Kyle M Hall <kyle at bywatersolutions.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |kyle at bywatersolutions.com

--- Comment #13 from Kyle M Hall <kyle at bywatersolutions.com> ---
> I tested both db revisions by resetting Version of my database (and
> adding a SetVersion after the revised older dbrev). This was possible since
> the debarred field still exists in borrowers. (I have my doubts about that,
> but that goes outside the scope of this report.)

Yes, borrower.debarred was retained for both speed and the fact that we
wouldn't need to modify all the code that checks this field.

> Also note that routine _UpdateBorrowerDebarmentFlags puts back 9999-12-31
> into borrowers for indefinite debarments (which looks not very consistent).

Agreed, but again this is due to the extended feature being "backwards
compatible" with the original debarment system.

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