tec3/five-for-the-future
tec3/five-for-the-future
1
0
Fork 0
mirror of https://github.com/WordPress/five-for-the-future.git synced 2025-04-22 11:03:43 +03:00
Code Issues Actions Packages Projects Releases Wiki Activity

Email: Check first name exists before using.

Browse source 
The previous check worked when the field existed as an empty string, but threw a notice when it didn't exist.
This commit is contained in:
Ian Dunn 2022-08-10 11:41:14 -07:00
parent 4dc03ada83
commit 993bd53765
No known key found for this signature in database
GPG key ID: 99B971B50343CBCB
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
plugins/wporg-5ftf/includes/email.php
View file

@ -215,7 +215,7 @@ function send_contributor_inactive_email( array $contributor ) : bool {
* Their first name is ideal, but their username is the best fallback because `nickname`, `display_name`,
* etc are too formal.
*/
$name = $contributor['first_name'] ? $contributor['first_name'] : '@' . $contributor['user_nicename'];
$name = empty( $contributor['first_name'] ) ? '@' . $contributor['user_nicename'] : $contributor['first_name'];
$subject = 'Please update your Five for the Future pledge';
$short_team_names = array_map( function( $team ) {