Book Availability Logic

[9gix]

The existing logic is based on the Count of Loan and LoanReturn.

• if Count(Loan) == Count(LoanReturn) then available
• else: unavailable

The correct logic should be:

• if Book has no loan or Book last loan has been returned, then available.
• else unavailable

[digawp]

If I don't trace it wrongly, this logic is now implemented here. Seems like it is already using the correct logic. @9gix can you verify?
I would try to verify it myself, but I am not sure how do I create a new account for testing, or if there is a default one

[9gix]

I believe you're pointing to the correct section for the current master branch.
However, just to share some historical reason, the master used to have the available queryset, which was removed at PR #30. I haven't read in details about the changes yet. But If I'm not wrong, I remove it to decouple catalog and circulation. Since catalog shouldn't know about circulation or availability.