? Success

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
10 Nov 2015

[#34056] indexer does not work properly in languages with different decimal notation

== Reproduce

Have a joomla site in a language that uses comma instead of a dot for decimal places (ex: Portuguese).

Turn on search indexer content plugin.

Edit a content item in the frontend and save.
It will appear an sql error like this

Column count doesn't match value count at row 1 SQL=INSERT INTO 29i8c_finder_tokens (term,stem,common,phrase,weight,context,language) VALUES ('a', 'a', 0, 0, 0,0667, 1, 'pt')

Note the "0,0667" it should be "0.0667" and the error is there.


Redo this PR by @chrisdavenport: #4116

avatar zero-24 zero-24 - open - 10 Nov 2015
avatar zero-24 zero-24 - change - 10 Nov 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Nov 2015
Labels Added: ?
avatar waader
waader - comment - 11 Nov 2015

I was unabled to reproduce the sql error. Here is what I did:

  • installed current staging with testing data and english, german and portuguese language
  • switched frontend language to portuguese
  • enabled smart search content plugin
  • went to the frontend
  • logged in and edited and saved an article

Have I missed anything?

avatar zero-24
zero-24 - comment - 11 Nov 2015

Can You run The indexer The First time bevor go to The frontend?

What Can You find in The findet tokens table?

But hey maye it is fixed else were i'm not Sure but i Can check it maybe tomorrow

avatar astridx
astridx - comment - 14 Nov 2015

I think here is a patch for this issue: [#5318] - Fixing finder to format weight as non-locale float.

But:
I could not reproduce this error, too.
1. ´I installed the current staging with English test data.
2. Then I installed the German language files and activated them in the backend and in the frontend.
3. After that I enabled smart search content plugin and build the index.
4. Then I edited a lot of articles in the frontend.
5. My tokens table was empty all the time.

avatar zero-24
zero-24 - comment - 14 Nov 2015

Ok maybe This is fixed Else where. Thanks for testing.

avatar zero-24 zero-24 - change - 14 Nov 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-11-14 21:21:41
Closed_By zero-24
avatar zero-24 zero-24 - close - 14 Nov 2015
avatar zero-24 zero-24 - close - 14 Nov 2015
avatar zero-24 zero-24 - head_ref_deleted - 14 Nov 2015

Add a Comment

Login with GitHub to post a comment