#46783 closed defect (bug) (fixed)
Site Health: Notice: Undefined property: WP_Site_Health::$mariadb in /wp-admin/includes/class-wp-site-health.php
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 5.2 | Priority: | normal |
| Severity: | normal | Version: | 5.2 |
| Component: | Site Health | Keywords: | site-health good-first-bug has-patch |
| Focuses: | Cc: |
Description
When viewing /wp-admin/site-health.php from trunk, I get:
Notice: Undefined property: WP_Site_Health::$mariadb in /wp-admin/includes/class-wp-site-health.php
Here
https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/class-wp-site-health.php#L931
https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/class-wp-site-health.php#L947
https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/class-wp-site-health.php#L961
we have
( $this->mariadb ? 'MariaDB' : 'MySQL' ),
But it should be:
( $this->is_mariadb ? 'MariaDB' : 'MySQL' ),
Maybe this is already fixed, but I couldn't find it after a quick search here on Trac.
Attachments (1)
Change History (6)
#2
@
7 years ago
- Keywords has-patch added; needs-patch removed
@birgire Added patch.
In https://core.trac.wordpress.org/browser/trunk/src/wp-admin/includes/class-wp-site-health.php#L961 there is no error.
Note: See
TracTickets for help on using
tickets.
Patch.