?
avatar gitlae
gitlae
13 Jul 2017

Steps to reproduce the issue

Run Joomla on a host system with a non US locale, for example 'swedish'.
Create/Open an article and save it.

Expected result

Silently save article.

Actual result

com_finder will generate "Column count doesn't match value count at row 1" (Error Code: 1136.)

System information (as much as possible)

Database Version 5.5.55-cll
Database Collation latin1_swedish_ci
Database Connection Collation utf8mb4_general_ci
PHP Version 5.6.16
Web Server Apache
WebServer to PHP Interface cgi-fcgi
Joomla! Version Joomla! 3.7.3 Stable [ Amani ] 4-July-2017 08:03 GMT
Joomla! Platform Version Joomla Platform 13.1.0 Stable [ Curiosity ] 24-Apr-2013 00:00 GMT

Additional comments

Reason is that just casting a value to (float) will generate a locale aware floating-point number.
In Swedish ',' is used for decimal separation (3,14) and that will not work in an SQL statement.

com_finder_sql_float_cast.patch.txtApplied patch will at least fix the problem, might not be the perfect solution but it works for me :)

avatar gitlae gitlae - open - 13 Jul 2017
avatar joomla-cms-bot joomla-cms-bot - labeled - 13 Jul 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 28 Jul 2017

@maintainers is this a Solution?

@gitlae can you make a PR (how to make a Pull Request)?

avatar franz-wohlkoenig franz-wohlkoenig - change - 28 Jul 2017
Status New Information Required
avatar gitlae
gitlae - comment - 30 Jul 2017

I have created a pull request #17345.
The patch differ from the original due to recent code refactoring.

This is my first PR so bare with me if something is not perfect :)


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17114.
avatar franz-wohlkoenig franz-wohlkoenig - change - 30 Jul 2017
Status Information Required Closed
Closed_Date 0000-00-00 00:00:00 2017-07-30 06:56:27
Closed_By franz-wohlkoenig
avatar joomla-cms-bot joomla-cms-bot - change - 30 Jul 2017
Closed_Date 2017-07-30 06:56:27 2017-07-30 06:56:28
Closed_By franz-wohlkoenig joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 30 Jul 2017
avatar joomla-cms-bot
joomla-cms-bot - comment - 30 Jul 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 30 Jul 2017

closed as having PR #17345.

@gitlae Thanks for contributing.


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

Add a Comment

Login with GitHub to post a comment