Context Navigation

← Previous Ticket
Next Ticket →

#44896 closed enhancement (fixed)

Introduce new `populate_site_meta()` function

Reported by:	flixos90	Owned by:	flixos90
Milestone:	5.1	Priority:	normal
Severity:	normal	Version:
Component:	Upgrade/Install	Keywords:	has-patch has-unit-tests
Focuses:	multisite	Cc:

Description

Similar to the outsourcing of network meta population into a new populate_network_meta() function (see #44895), a populate_site_meta() function that works similarly should be introduced. This came up during work on #41333.

Attachments (1)

44896.diff (4.0 KB) - added by flixos90 6 years ago.

Download all attachments as: .zip

Change History (8)

@flixos90
6 years ago

Attachment 44896.diff added

#1 @flixos90
6 years ago

Keywords has-patch has-unit-tests added; needs-patch needs-unit-tests removed

#2 @flixos90
6 years ago

Resolution set to fixed
Status changed from assigned to closed

In 43629:

Upgrade/Install: Introduce populate_site_meta().

Fixes #44896. See #41333.

#3 @flixos90
6 years ago

In 43630:

Upgrade/Install: Fix coding standard errors after [43629].

See #44896.

#4 @flixos90
6 years ago

In 43641:

Upgrade/Install: Properly exclude the admin/includesSchema.php tests file for the WordPress.DB.PreparedSQL.NotPrepared sniff.

The way this was implemented in [43630] caused the file to be excluded for all sniffs, since PHPCS does not support file-wide ignores of a specific sniff at this point.

Props jrf.
See #44896.