Comment 40 for bug 1846354

I've pushed a rebased version to collab/phasefx/consolidate_patron_notes3

4dbea99f75d62e928d59477212a89e4bafc93482 lp1859502 fix A/T ApplyPatronPenalty reactor
3cb9a5c70c3b0bb3457c0823bbc4990110b4d558 lp1846354 toward consolidated patron notes
ed276069c2ebe8a6c9ca36e3dfa1d27234280bad LP#1846354 various speed improvements
4d9a562d4ae32b157bf55170bc5b84c5923ae73d lp1846354 revisions to upgrade script
a5925a9708f12734957950f5b9f77dd150e27b62 lp1846354 misc fixes

The last commit includes these changes:

    * don't hide referenced deleted messages from staff
    * don't exclude archived messages from unread Messages count in OPAC
    * migrate unmatched public notes as deleted user messages
    * don't use 'Penalty Note' as a message title

The very first commit has its own LP entry, bug 1859502. If committed here, we should close it out there, but this branch doesn't depend on it; I was just using the functionality for testing.