Skip to content

Conversation

@xxsimoxx
Copy link
Contributor

ClassicPress is a fork of WordPress, without blocks.

Since ClassicPress have it's own news system, this prevents a fatal error in the dashboard.
wp_print_community_events_markup() is not present in ClassicPress as the Community Events widget.

Before

image

After

image

xxsimoxx added 3 commits June 10, 2025 14:42
Signed-off-by: xxsimoxx <simone@gieffeedizioni.it>
Signed-off-by: xxsimoxx <simone@gieffeedizioni.it>
Signed-off-by: xxsimoxx <simone@gieffeedizioni.it>
Copy link
Member

@joedolson joedolson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be better to check ! function_exists( 'wp_print_community_events_markup' ), as that would support any case where this was removed, rather than being specific to Classic Press. Thoughts?

@timiwahalahti timiwahalahti added the compatibility-issue Compatibility issue with some other plugin, theme, or environment label Jun 10, 2025
Signed-off-by: xxsimoxx <simone@gieffeedizioni.it>
@xxsimoxx
Copy link
Contributor Author

Sure, I agree to the proposed change and just pushed a commit.

@xxsimoxx xxsimoxx requested a review from joedolson June 12, 2025 11:56
Signed-off-by: Joe Dolson <design@joedolson.com>
Copy link
Member

@joedolson joedolson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added a comment to provide context for this, as I can easily imagine future me saying "why would this function not exist?"

@joedolson joedolson merged commit e96b524 into fairpm:main Jun 12, 2025
1 check passed
@rmccue rmccue added this to the 0.3.0 milestone Jun 14, 2025
@xxsimoxx xxsimoxx deleted the classicpress branch June 16, 2025 05:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

compatibility-issue Compatibility issue with some other plugin, theme, or environment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants