![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#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: | birgire | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.2 |
| Component: | Site Health | Version: | 5.2 |
| Severity: | normal | Keywords: | site-health good-first-bug has-patch |
| Cc: | Focuses: |
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.