Skip to content

Conversation

@donnapep
Copy link
Collaborator

@donnapep donnapep commented Aug 6, 2025

Fixes an issue reported by Team51:

When using Safety Net on a site with the Sensei plugin, the following warning was encountered:

Warning: Attempt to read property "display_name" on bool in /plugins/sensei-lms/includes/class-sensei-modules.php on line 2653

Proposed Changes

Check that $author is a WP_User object before trying to access the display_name property.

Though I wasn't able to reproduce the reported issue, this seems like a safe change.

Pre-Merge Checklist

  • PR title and description contain sufficient detail and accurately describe the changes
  • Adheres to coding standards (PHP, JavaScript, CSS, HTML)
  • All strings are translatable (without concatenation, handles plurals)
  • Follows our naming conventions (P6rkRX-4oA-p2)
  • Hooks (p6rkRX-1uS-p2) and functions are documented
  • New UIs are responsive and use a mobile-first approach
  • Code is tested on the minimum supported PHP and WordPress versions

@donnapep donnapep added this to the 4.25.1 milestone Aug 6, 2025
@donnapep donnapep self-assigned this Aug 6, 2025
@m1r0 m1r0 modified the milestones: 4.25.1, 4.25.2 Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants