Notification Meta

[johnjamesjacoby]

Notifications do not currently have a metadata table, which could be useful for storing additional data outside the available notification table's schema.

With the work that's gone into metadata interoperability and caching, this should be a fairly straightforward task.

[johnjamesjacoby]

First pass - no unit tests

[johnjamesjacoby]

In 9572:

Notifications: Introduce notification metadata table.

• Bump DB version
• Introduce DB upgrade function & routine for 2.3.0
• Add metadata table to Notifications installation function
• Add metadata CRUD functions to Notifications component

See #6257.

[johnjamesjacoby]

In 9573:

Notifications: Add table_name_meta to global tables in loader. See #6257.

[johnjamesjacoby]

In 9574:

Notifications: Wire up meta queries so they return accurate results, largely lifted from Activity component. More to fine-tune here. See #6257.

[johnjamesjacoby]

In 9613:

Notifications: incorporate meta query logic into get_total_count() method. See #6257.

[johnjamesjacoby]

In 9614:

Always 80. See #6257.

[johnjamesjacoby]

In 9615:

Notifications. Bump database version to current revision, and add some inline docs to version updaters. See #6257.

[johnjamesjacoby]

Feature is in. Test coverage will get committed to this ticket as well.

[DJPaul]

@johnjamesjacoby Did unit tests ever happen, or shall we open a new ticket for that part, and aim them for 2.4?

[johnjamesjacoby]

No tests yet. I started writing them and noticed that Activity and Notifications work a little differently now, and got side tracked.

I'm fine with closing this and making a new ticket for tests in 2.4.

[r-a-y]

In 10789:

Notifications: Update meta cache in BP_Notifications_Notification::get().

We update the meta cache if update_meta_cache is true, which it is by
default. Commit also updates the DocBlock to outline a few missing
parameters.

See #6257, #7067.

[r-a-y]

In 11840:

Notifications: Fix date querying in bp_has_notifications().

Previously, we introduced the ability to query notifications by date with
the 'date_query' parameter in r9574. However, the DB column used for the
date query is incorrect.

See #6257.

Fixes #7682.