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