User tests: Successful: Unsuccessful:
Pull Request for Issue #17582 .
Add m.language and l.image in the "GROUP BY" clause :
sql_group="a.id, m.language, l.image"
Login as administrator.
Click on Menus
All menus are displayed as expected.
All menus are displayed as expected.
Category | ⇒ | Administration com_menus |
Status | New | ⇒ | Pending |
I have tested this item
Teststack:
MAMP 4.2 on macOS 10.1.2
PostgreSQL 9.6.4 on Docker
Tested:
Without patch:
Error page with title "Error: 0 ERROR: column "m.language" must appear in the GROUP BY clause or be used in an aggregate function LINE 1: ...E(SUM(m.home), 0) WHEN 0 THEN '' WHEN 1 THEN CASE m.language... ^"
With patch:
All menus are displayed as expected.
Tested @icampus
Please also test the following condition:
Go to #__menu table and set two or more record in a menutype to home=1 manually. Then check if the globe icon is displayed in the admin menu for that menu type.
Labels |
Added:
?
|
@izharaazmi Could you specify further what I need to do, to test your scenario? I manually set some lines to home=1, but I don't know where and when I should see that globe icon.
I've set two records to home=1 in the #__menu table, and the globe icon is well displayed.
Just log in to administrator, and click on "Menus".
I have tested this item
When testing Joomla 3.7.5 with PostgreSQL, Administrator page displays noting but the error description: "Error page with title "Error: 0 ERROR: column "m.language" must appear in the GROUP BY clause or be used in an aggregate function LINE 1"
After applying the patch it worked as expected and all menus were displayed correctly.
@icampus
@SamuelSchepp can you please retest?
I have tested this item
Still works with patch as described earlier. @icampus
Status | Pending | ⇒ | Ready to Commit |
RTC after two successful tests.
Category | Administration com_menus | ⇒ | Administration com_menus Modules Front End |
Labels |
Added:
?
|
Category | Administration com_menus Modules Front End | ⇒ | Administration com_menus Language & Strings Modules Front End |
@twister65 Last 2 commits are in wrong branch - this should by a separate PR.
Labels |
Added:
?
|
Category | Administration com_menus Modules Front End Language & Strings | ⇒ | Administration com_menus |
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-08-21 23:46:52 |
Closed_By | ⇒ | mbabker | |
Labels |
Removed:
?
|
@izharaazmi @alikon
Please test