search

WPBuddy / largo

A WordPress framework for news websites. Finely-crafted by INN and expertly-honed and maintained by the technology team at WP Buddy.
http://largo.wpbuddy.co
GNU General Public License v2.0
171 stars 83 forks source link

1526 - Replace file get contents #1898

Closed joshdarby closed 3 years ago

joshdarby commented 3 years ago

Originally submitted by @omnisite as PR #1815.

Replace file_get_contents with WP_Filesystem

Changes

This pull request makes the following changes:

Why

Because you should never use @ and it is better to use native WP functions if they exist (over plain PHP).

For issue https://github.com/INN/largo/issues/1526

Steps to test this PR:

  1. Go to you personal page in wp-admin and upload/change an avatar (not the gravatar one, but the avatar found at the bottom of the page)