JILI fun Casino,BetBoom slots is legit or not Philippines.REGISTER NOW GET FREE 888 PESOS REWARDS!

doog2229
(@doog2229)

6 months, 4 weeks ago

Hi,

I think there is a bug in the last version of plugin. When i load the url with this parameter /?sum=1

i has a critical error. Here the trace of my log : 2024/09/02 11:43:26 [error] 780#780: *603687 FastCGI sent in stderr: "PHP message: PHP Fatal error: Uncaught ValueError: Unknown format specifier "S" in /var/aegir/platforms/wordpress-dev/sites/crm-dev.oudinot.symbiodev.xyz/wp-content/plugins/buddypress-group-email-subscription/bp-activity-subscription-digest.php:206

in the line 206 i have a problem with sprint_f function. I resolved in my website with this correction

// Might be nice here to link to anchor tags in the message. if ( 'dig' === $type ) { // translators: number of items. $summary .= apply_filters( 'ass_digest_summary', "<li class=\"digest-group-summary\" {$ass_email_css['summary']}><a href='{$group_permalink}'>$group_name</a> " . sprintf( __( '(%1$s items)', 'buddypress-group-email-subscription' ), count( $activity_ids ) ) . "</li>\n", $ass_email_css['summary'], $group_slug, $group_name, $activity_ids ); }

Viewing 4 replies - 1 through 4 (of 4 total)

Thread Starter doog2229
(@doog2229)

6 months, 4 weeks ago

My version of PHP is 8.2.23

Plugin Author Boone Gorges
(@boonebgorges)

6 months, 4 weeks ago

Hello and thanks for the report. Your error message has a capital S when it indicates the unsupported format. When you made your edit, did it look like it was capital (%S items) or was it lowercase (%s items)? See https://plugins.trac.www.ads-software.com/browser/buddypress-group-email-subscription/tags/4.2.3/bp-activity-subscription-digest.php?marks=206#L200

Can you confirm that you downloaded the plugin in the normal way, ie by installing via Dashboard > Plugins?

Thread Starter doog2229
(@doog2229)

6 months, 3 weeks ago

in the code i had a lowercase %s items , i changed this code with this %1$s items . With this modification, my critical error has disappeared

i had a old version of your plugin. I just upgrade the plugin in the specific screen of WordPress (plugin menu)

Plugin Author Boone Gorges
(@boonebgorges)

6 months, 3 weeks ago

Thanks for confirming. I don’t understand why PHP would complain that %s is an invalid format operator for sprintf(). It’s not invalid; see https://www.php.net/manual/en/function.sprintf.php. And I’m unable to reproduce the issue on similar versions of PHP. Replacing with %1$s items probably won’t hurt anything, but I’m wary to make the change without understanding the underlying source of the error. (If I make the change to this use of %s, do I have to do it everywhere in the plugin?)

I’ll leave this ticket open in case others chime in with a similar issue.

Viewing 4 replies - 1 through 4 (of 4 total)

The topic ‘bug with the new version of plugin’ is closed to new replies.