? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
1 Nov 2016

Pull Request for fix regression from #12051 .

Summary of Changes

fixed wrong group by when multilanguages enabled

Testing Instructions

  • latest staging fresh multilanguage install
  • Go to Menu -> All menu items
    menugroupby

  • postgresql - got sql error wrong group by

  • mysql - should work as before
avatar alikon alikon - open - 1 Nov 2016
avatar alikon alikon - change - 1 Nov 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 1 Nov 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 1 Nov 2016
Category Administration Components
avatar brianteeman
brianteeman - comment - 14 Nov 2016

Looking at the code i dont see why this is just for postgresql

Also your test instructions dont describe the issue so its hard to test

avatar andrepereiradasilva
andrepereiradasilva - comment - 14 Nov 2016

if mysql was in sql strict mode ONLY_FULL_GROUP_BY it would give that error too.

See #12494 for more info

avatar joomla-cms-bot joomla-cms-bot - change - 21 Nov 2016
Category Administration Components Administration com_menus Components
avatar alikon
alikon - comment - 21 Nov 2016

test instructions updated, conflicts resolved

avatar alikon alikon - change - 21 Nov 2016
The description was changed
avatar alikon alikon - edited - 21 Nov 2016
avatar jeckodevelopment
jeckodevelopment - comment - 21 Nov 2016

@alikon still see conflicts!

avatar alikon
alikon - comment - 24 Nov 2016

closing this one in favour of #12999 wich does'nt have conflicts hopefully

avatar alikon alikon - change - 24 Nov 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-11-24 07:48:14
Closed_By alikon
avatar alikon alikon - close - 24 Nov 2016

Add a Comment

Login with GitHub to post a comment