Alert Improvements by Xon 2.13.1

• Require php 7.2+
• Require XenForo 2.2+, remove XF2.1 support
• Fix sticky unread alerts could unexpectedly occur due to (effectively undocumented) 'auto read' functionality in XF2.2
• Fix confusion over view_date vs read_date not being correctly used in the expected places
• Fix alerts pop-up not respecting "Days to retain viewed alerts in popup" option in all cases
• Fix 'mark read' functionality could result in inconsistent view/read date for older alerts near the expiry period
• Fix unstaller not correctly removing columns added to xf_user_option
• Fix alert summarization/grouping may not trigger when expected
• Ensure xf_user_alert.summerize_id column has the same column size as xf_user_alert.alert_id. If alert_id has been to a bigint, this may be an expensive operation.
  • The following SQL schema change is executed if the alert_id column has been manually changed:
    SQL:

    alter table xf_user_alert modify column summerize_id bigint unsigned  default null;

• Use index hints to control poor index selection on some installs
• Attempt to improve unviewed alert cleanup which can have issues when dealing with very large number of alerts
• Improve UI.X compatibility
• In Alerts pop-up, do not prevent marking alerts as recently read when they are read in that request
• Add various styling options to control alerts styling
• Add styling option to control if just-read-alerts from the alerts pop-up are styled as read or unread, and if to fade from unread to read.
• Visiting the full alerts page no longer marks alerts as read by default, and removes the relevant per-user option ("Alerts page skips mark read").
• Display an font-awesome icon (star with exclamation) on alerts which are unread with auto-read off, rather than a star for recent-ish alerts. With more correct hint text on both

• Another attempt to reduce deadlocks during alert pruning
• When editing a post, mark related alerts as read, as this displays changes in post contents to the visitor
• If Live Content add-on is installed; only try to mark alerts as read if the visitor has unread alerts

• Reduce possibility of deadlocks when pruning alert cleanup table
• Fix summary alert pagination

• Add CLI command "xf-rebuild:sv-alert-totals" to expose rebuilding alert totals job
• Avoid updating alerts which are condinates to be pruned, this should reduce the possibility of the alerts cleanup tasks generating deadlocks.

• Fix "Call to a member function keys() on array"
• Fix that very old alerts could be shown in the alerts pop-up
• Increase thickness of border between unread and read alerts
• Force unread alerts to be preferred when fetching alerts in the alerts pop-up

• Rework alert pruning to better scale for large & busy forums

• Fix that alerts css would be injected after extra.less, which made styling difficult

• Fix that mark read/unread would remove the checkbox on the alerts page
• Push fauxBlockLink to be just around the per-template render to not overlap the minor info row

• Adjust lock ordering to reduce/prevent deadlocks between the mark read/unread feature and inserting alerts
  • Note; this affects stock alerts as well.
• Fix that mark read/unread would silently fail if an alert went away
• Fix bulk alert toggle could be inconsistent compared to the individual alert toggler with how it manipulates is-read/is-recently-read/is-unread classes
• Fix alert/unread/unsummarize alert did not displaying a UI on a GET request
• Fix that clicking 'unsummarize' would reload page, now it removes the alert and updates alert totals
• Add public account/alert route, allowing nicer URLs for alert specific actions
• Rename various templates & phrases to be consistently named
• Improve handling of alerts which are not viewable but are still counting a user's read count

• Adjust locking ordering in markUserAlertsRead to hopefully be less deadlock prone
• Add sanity checking to unread alerts page to automatically rebuild alert counts on no unread alerts being seen