? Pending

User tests: Successful: Unsuccessful:

avatar cheryeong
cheryeong
30 May 2018

There is a missing argument for sprintf() when calling InvalidArgumentException().

avatar cheryeong cheryeong - open - 30 May 2018
avatar cheryeong cheryeong - change - 30 May 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 30 May 2018
Category Libraries
avatar infograf768
infograf768 - comment - 30 May 2018

Agree. I even wonder why we use %1$s instead of %s here as we have a single variable.

avatar mbabker
mbabker - comment - 30 May 2018

Personal preference. I pretty much always use that syntax on my sprintfs, even on single variable uses, as a means of consistency in my projects.

avatar joomla-cms-bot joomla-cms-bot - edited - 30 May 2018
avatar Quy Quy - change - 30 May 2018
Title
Missing argument in sprintf().
[4.0] Missing argument in sprintf().
avatar infograf768 infograf768 - change - 30 May 2018
Labels Added: ?
avatar wilsonge wilsonge - close - 30 May 2018
avatar wilsonge wilsonge - merge - 30 May 2018
avatar wilsonge wilsonge - change - 30 May 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-05-30 22:57:28
Closed_By wilsonge
avatar wilsonge
wilsonge - comment - 30 May 2018

Thankyou!

Add a Comment

Login with GitHub to post a comment