?
avatar brianteeman
brianteeman
28 Feb 2016

Steps to reproduce the issue

Go to extensions->install languages
Search for french

Expected result

two languages are found

Actual result

With Mysql you see two languages
With PGSQL you see zero languages

Additional comments

Repeat pgsql test with searching for French and you get matches

@alikon can you take a look please

avatar brianteeman brianteeman - open - 28 Feb 2016
avatar brianteeman
brianteeman - comment - 28 Feb 2016

Looks like it is only the search filter on the install languages page - the other search filters I tested were all ok


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Feb 2016

just a question, did you test with this PR #9089

avatar brianteeman
brianteeman - comment - 28 Feb 2016

No - I will apply that now and see if it is resolved by the changes you
made there

On 28 February 2016 at 12:48, andrepereiradasilva notifications@github.com
wrote:

just a question, did you test with this PR #9089
#9089


Reply to this email directly or view it on GitHub
#9242 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman
brianteeman - comment - 28 Feb 2016

Sorry to say that #9089 doesnt resolve it

On 28 February 2016 at 12:49, Brian Teeman brian@teeman.net wrote:

No - I will apply that now and see if it is resolved by the changes you
made there

On 28 February 2016 at 12:48, andrepereiradasilva <
notifications@github.com> wrote:

just a question, did you test with this PR #9089
#9089


Reply to this email directly or view it on GitHub
#9242 (comment)
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Feb 2016

sure, it was just a wild guess.

I dont know much about PGSQL neither have the means to test, but maybe the problem should be in this lines? https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_installer/models/languages.php#L128-L129

See this example in com_modules https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_modules/models/modules.php#L373-L374

I think in postgresql we have to use LOWER() but @alikon certainly knows better than me.

avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Feb 2016

if i'm right the same problem exists in other views (ex: newsfeeds, contacts).

avatar brianteeman
brianteeman - comment - 28 Feb 2016

Will check after lunch. If you are on a Mac then you can easily use
postgres with http://postgres.app

if i'm right the same problem exists in other views (ex: newsfeeds).


Reply to this email directly or view it on GitHub
#9242 (comment).

avatar andrepereiradasilva
andrepereiradasilva - comment - 28 Feb 2016

no mac here i leave postgresql to others.
my interrest is just learn the more noticleble differences in queries between mysql and postgreSQL so when modfiying some code don't make regressions to postgresql.

avatar brianteeman
brianteeman - comment - 28 Feb 2016

News feeds is fine

On 28 February 2016 at 13:34, andrepereiradasilva notifications@github.com
wrote:

no mac here i leave postgresql to others.
my interrest in just learn the more noticleble differences in queries
between mysql and postgreSQL so when modfiying some code don't make
regressions to postgresql.


Reply to this email directly or view it on GitHub
#9242 (comment).

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman
brianteeman - comment - 28 Feb 2016

Looks like this might work http://alvinalexander.com/sql/sql-select-case-insensitive-query-queries-upper-lower but not sure
@alikon should have the answer so I will wait for the maestro


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

avatar alikon alikon - reference | e31d85f - 28 Feb 16
avatar alikon
alikon - comment - 28 Feb 2016

:laughing:

avatar brianteeman brianteeman - change - 28 Feb 2016
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-02-28 17:27:21
Closed_By brianteeman
avatar brianteeman
brianteeman - comment - 28 Feb 2016

Closed as we have a PR #9244


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

avatar brianteeman brianteeman - close - 28 Feb 2016

Add a Comment

Login with GitHub to post a comment