? Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
24 May 2018

Alternative PR for issue #17114. (Original PR: #17345)

When casting the weight as float, it seems PHP is using a locale aware method, which results in the number sometimes having the wrong decimal-separator and then the query fails. The weight already is formatted as a float with a dot as a decimal-separator, so it should be enough to simply quote this, as @csthomas wrote in the original PR.

avatar Hackwar Hackwar - open - 24 May 2018
avatar Hackwar Hackwar - change - 24 May 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 May 2018
Category Administration com_finder
avatar csthomas
csthomas - comment - 25 May 2018

I have tested this item successfully on 6834fc0

Code review


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

avatar csthomas
csthomas - comment - 25 May 2018

I have tested this item successfully on 6834fc0

Code review


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

avatar csthomas csthomas - test_item - 25 May 2018 - Tested successfully
avatar wilsonge
wilsonge - comment - 7 Jun 2018

In non-mysql does the mis-match (this will now be a string) cause an issue?

avatar csthomas
csthomas - comment - 7 Jun 2018

This is a common problem for all databases.

Take a look at joomla-framework/database#127 (comment) and these links.

avatar wilsonge wilsonge - change - 14 Jun 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-06-14 23:08:34
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 14 Jun 2018
avatar wilsonge wilsonge - merge - 14 Jun 2018

Add a Comment

Login with GitHub to post a comment