Move C4::Members to Koha::Patrons - another big step

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view

Move C4::Members to Koha::Patrons - another big step

Jonathan Druart
Hi all,

I have moved out of C4::Members the following subroutines:
  - GetMemberAccountRecords
  - GetMemberAccountBalance
  - Check_userid
  - Generate_Userid
  - GetPendingIssues
  - GetOverduesForPatron
  - patronflags

There is a bunch of ~35 commits but do not be scared, they are all small and I wrote them to make them easy to read/review (well, at least it is what I think).

Most of these subroutines were ugly and impacted performances (too many useless fetches from the DB).
This whole stack will be hard to rebase and maintain, so it would be great to get it in soon.

Several things can appear ugly or not perfect, but keep in mind that we are moving, improving and refactoring our code. The test coverage is better and this is just a step before the next one. I really want to start to take care of the very ugly code, but for now we need keep moving.
C4::Members is almost dead and we are going to manipulate Koha::Patrons everywhere, which will ease the readability, maintenance and write of new improvements.

Let me know if you have any questions.

Note that patronflags is not completely removed as I need help to remove it from C4::SIP


Koha-devel mailing list
[hidden email]
website :
git :
bugs :