Conversation
supportHello. 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.
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.