<div dir="ltr"><div>Hi all,</div><div><br></div>I have moved out of C4::Members the following subroutines:<br><div><div>  - GetMemberAccountRecords<br>  - GetMemberAccountBalance<br>  - Check_userid<br>  - Generate_Userid<br>  - GetPendingIssues</div><div>  - <span id="inbox-inbox-summary_container"><span id="inbox-inbox-short_desc_nonedit_display">GetOverduesForPatron</span></span></div><div>  - patronflags</div><div><br></div><div>See the remote branch <a href="https://github.com/joubu/Koha/tree/clean_c4_members">https://github.com/joubu/Koha/tree/clean_c4_members</a></div><div>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).<br></div><div><br></div><div>Most of these subroutines were ugly and impacted performances (too many useless fetches from the DB).</div><div>This whole stack will be hard to rebase and maintain, so it would be great to get it in soon.</div><div><br></div><div>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.</div><div>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.<br></div><div><br></div><div>Let me know if you have any questions.</div><div><br></div><div>Note that patronflags is not completely removed as I need help to remove it from C4::SIP<br></div><div><br></div><div>Cheers,</div><div>Jonathan<br></div></div></div>