J4 Issue ?
avatar ITPrism
ITPrism
18 Nov 2018

Steps to reproduce the issue

Open All Menu Items.

Actual result

Fatal error
You have an error in your SQL syntax;.... published,a.component_id,....

System information (as much as possible)

Apache/2.4.37
PHP/7.2.12
MariaDB/10.1.37

Additional comments

It will be fixed if you replace 'a.published' with null in the method getListQuery in administrator/components/com_menus/Model/ItemsModel.php

sql
line_267

avatar ITPrism ITPrism - open - 18 Nov 2018
avatar joomla-cms-bot joomla-cms-bot - change - 18 Nov 2018
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 18 Nov 2018
avatar infograf768 infograf768 - change - 18 Nov 2018
Title
Fatal error All Menu Items
[4.0] Fatal error All Menu Items
avatar infograf768 infograf768 - edited - 18 Nov 2018
avatar infograf768 infograf768 - change - 18 Nov 2018
Labels Added: J4 Issue
avatar infograf768 infograf768 - labeled - 18 Nov 2018
avatar infograf768
infograf768 - comment - 18 Nov 2018

On mysqli, I do not get the error, whether I keep present code or modify as suggested.
Don't have MariadB to test.
@alikon

avatar tonypartridge
tonypartridge - comment - 20 Nov 2018

I’m a bit puzzled why we are trying to load in the column as it’s self? Suggest replacing it as null like the rest? Can you do a PR @ITPrism ?

avatar csthomas
csthomas - comment - 20 Nov 2018

The last change was at https://github.com/joomla/joomla-cms/pull/10281/files

The null array is probably useless. Simply removing the whole array should help.

avatar tonypartridge
tonypartridge - comment - 20 Nov 2018

Agreed

Many thanks
On 20 Nov 2018, 07:58 +0000, Tomasz Narloch notifications@github.com, wrote:

The last change was at https://github.com/joomla/joomla-cms/pull/10281/files
The null array is probably useless. Simply removing the whole array should help.

You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

avatar Quy Quy - change - 17 Dec 2018
Status New Closed
Closed_Date 0000-00-00 00:00:00 2018-12-17 16:12:53
Closed_By Quy
avatar joomla-cms-bot joomla-cms-bot - change - 17 Dec 2018
Closed_By Quy joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 17 Dec 2018
avatar joomla-cms-bot
joomla-cms-bot - comment - 17 Dec 2018

Set to "closed" on behalf of @Quy by The JTracker Application at issues.joomla.org/joomla-cms/23108

avatar Quy
Quy - comment - 17 Dec 2018

Please test PR #23293


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

Add a Comment

Login with GitHub to post a comment