Hi Martin,

I just discovered this bug and, after reading quickly the bug's comments and commit messages, I didn't find the reason of this big refactoring (why was it needed ? what do we gain ? is it only a rename or is it more than that ?). Is this explained somewhere ?

In my opinion changes should be summarized at least in the first commit message, so that people rebasing other patches can have an overview of what changes are needed in their patches.

Le 04/02/2020 à 11:01, Renvoize, Martin a écrit :
Hi All,

I just wanted to let you all know that I'll be pushing bug 18936 to the master branch this morning. It's the largest to date of the refactoring bugs and will affect any/all bugs related to the circulation rules handling.

It's been thoroughly tested and QA'd, but as a large piece I'm sure there will be a few pain points still buried, so keep an eye out and report any bugs you find.

Good luck with any rebases that may be required after this and my apologies, but the end result should give us a much cleaner foundation for circ related enhancements and cleanup in the future.

Many thanks to those who have spent many hours coding, testing, refining and qaing this piece,

*Martin Renvoize*

        

<https://www.ptfs-europe.com>

Development Team Manager

Community Release Manager (19.11, 20.05)


*Phone:* +44 (0) 1483 378728

        

*Mobile:* +44 (0) 7725 985 636

*Email:* martin.renvo...@ptfs-europe.com <mailto:martin.renvo...@ptfs-europe.com>

        

*Fax:* +44 (0) 800 756 6384


        

www.ptfs-europe.com <https://www.ptfs-europe.com>

Registered in the United Kingdom No. 06416372   VAT Reg No. 925 7211 30


The information contained in this email message may be privileged, confidential and protected from disclosure. If you are not the intended recipient, any dissemination, distribution or copying is strictly prohibited. If you think that you have received this email message in error, please email the sender at i...@ptfs-europe.com <mailto:i...@ptfs-europe.com>




_______________________________________________
Koha-devel mailing list
Koha-devel@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/


--
Julian Maurice
BibLibre
_______________________________________________
Koha-devel mailing list
Koha-devel@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to