WPIntell

Source evidence

Do these three lines have a syntax error?

Presto Player · support · 2025-12-13T13:09:00+00:00

neutralsentiment
lowseverity
0.28relevance
2replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

5 / 27 rows with source links

18.5% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

22 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
itapress resolved
Hello. Do these three lines have a syntax error with sprintf’s placeholder format? ‘Rewind %1ss’ instead of ‘Rewind %1$s’ or ‘Rewind %1$ss’ : https://plugins.trac.wordpress.org/browser/presto-player/tags/4.0.5/inc/Services/Translation.php#L103 ‘Forward %1ss’ instead of ‘Forward %1$s’ or ‘Forward %1$ss’ : https://plugins.trac.wordpress.org/browser/presto-player/tags/4.0.5/inc/Services/Translation.php#L107 ‘%1$1s of %2$2s’ instead of ‘%1$s of %2$s’ : https://plugins.trac.wordpress.org/browser/presto-player/tags/4.0.5/inc/Services/Translation.php#L110 Thanks. Hi, Thank you for bringing this to our attention. We will look into the reported issue and verify the placeholders syntax. If needed, we will include a fix in the next version. We appreciate your feedback and will keep you updated. Best, Neeraj Thanks for flagging this, @itapress . As Neeraj mentioned, we’ve noted the potential sprintf placeholder issues and will review them carefully. If any corrections are needed, they’ll be included in an upcoming release. We appreciate you taking the time to point these out and helping us improve the code quality.

Comments

2 shown
Neeraj 2025-12-15T10:02:00+00:00

Hi, Thank you for bringing this to our attention. We will look into the reported issue and verify the placeholders syntax. If needed, we will include a fix in the next version. We appreciate your feedback and will keep you updated. Best, Neeraj

John Buena 2025-12-29T01:03:00+00:00

Thanks for flagging this, @itapress . As Neeraj mentioned, we’ve noted the potential sprintf placeholder issues and will review them carefully. If any corrections are needed, they’ll be included in an upcoming release. We appreciate you taking the time to point these out and helping us improve the code quality.