TGMPA/TGM-Plugin-Activation

Too few arguments, still happening

MethylBromide opened this issue · 3 comments

Warning: sprintf(): Too few arguments in …/class-bulk-upgrader-skin.php on line 152

Fix is described here: https://gist.github.com/al5dy/965d0eefffab32df3b04badbc2274d08

Please make the indicated code change on line 24 of class-tgm-plugin-activation.php -- the string skin_update_successful is used in a context where sprintf is called with one data argument, and the formatting string you are assigning currently has two placeholders.

RupW commented

This probably best as a pull request, if you can make one please?

jrfnl commented

This has already been fixed in the code here. There are a number of other things though which a release is waiting on, pull requests for those are very welcome.

jrfnl commented

Marking this as a duplicate of #669 which was fixed by #670.