Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#18527] - Corrects attribute order in implode() function

? Pending

User tests: Successful: Unsuccessful:

avatar regularlabs
regularlabs
8 Nov 2017

Ther php implode function takes 2 parameters, where the first is the glue:

implode ( string $glue , array $pieces )

So this is wrong:

$value = implode($value, ' ');

Should be:

$value = implode(' ', $value);
avatar regularlabs regularlabs - open - 8 Nov 2017
avatar regularlabs regularlabs - change - 8 Nov 2017
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Nov 2017
Category ⇒ Administration com_fields
avatar regularlabs regularlabs - change - 8 Nov 2017
The description was changed
avatar regularlabs regularlabs - edited - 8 Nov 2017
avatar Quy Quy - test_item - 8 Nov 2017 - Tested successfully
avatar Quy
Quy - comment - 8 Nov 2017

I have tested this item ✅ successfully on 2a530c9

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18527.

avatar alikon alikon - test_item - 8 Nov 2017 - Tested successfully
avatar alikon
alikon - comment - 8 Nov 2017

I have tested this item ✅ successfully on 2a530c9

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18527.

avatar franz-wohlkoenig franz-wohlkoenig - change - 8 Nov 2017
Status Pending ⇒ Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 8 Nov 2017

RTC after two successful tests.

avatar wilsonge wilsonge - change - 10 Nov 2017
Status Ready to Commit ⇒ Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 ⇒ 2017-11-10 09:37:09
Closed_By ⇒ wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 10 Nov 2017
avatar wilsonge wilsonge - merge - 10 Nov 2017
avatar wilsonge
wilsonge - comment - 10 Nov 2017

Thanks!

Add a Comment

Login with GitHub to post a comment