?
avatar C-Lodder
C-Lodder
10 Mar 2017

Steps to reproduce the issue

In the backend, go to:

  • Menus >> All menu items
  • Then click Menu Items on the subnav (sidebar)

Expected result

  • A normal view

Actual result

Undefined variable: saveOrderingUrl in \administrator\components\com_menus\views\items\tmpl\default.php on line 104

System information (as much as possible)

  • Joomla 4.0-nightly
  • PHP 7.0.10
avatar C-Lodder C-Lodder - open - 10 Mar 2017
avatar joomla-cms-bot joomla-cms-bot - change - 10 Mar 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 10 Mar 2017
avatar C-Lodder C-Lodder - edited - 10 Mar 2017
avatar C-Lodder C-Lodder - change - 10 Mar 2017
The description was changed
avatar C-Lodder C-Lodder - edited - 10 Mar 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 11 Mar 2017

Can't confirm, Error Reporting is set on Maximum, also tested on Development

avatar joomdonation
joomdonation - comment - 11 Mar 2017

@franz-wohlkoenig Make sure you don't select a Menu in Select Menu dropdown, see this screenshot

menus- all menu items - joomla cms 4 0 - administration

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 11 Mar 2017

Select Menu Items, no Error shown:
bildschirmfoto 2017-03-11 um 08 50 58

avatar joomdonation
joomdonation - comment - 11 Mar 2017

@franz-wohlkoenig At the right, there is a a dropdown Sort Table By. Select Ordering ascending and you should see the notice

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 11 Mar 2017

Changed ordering in Sort Table By:, no Notice.

avatar joomdonation
joomdonation - comment - 11 Mar 2017

Don't know why you don't see this error :(

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 11 Mar 2017

Tried Safari instead Firefox, same Result. But other see Notice, PR works, all fine.

avatar yasirunilan
yasirunilan - comment - 12 Mar 2017

I also tried, can't confirm

avatar joomdonation
joomdonation - comment - 12 Mar 2017

That's so strange to me. Could you please:

  1. Try to check it using a nightly build installation https://developer.joomla.org/nightly-builds.html

  2. Access to Menus -> All Menu Items (the URL looks like this http://localhost/cms40/administrator/index.php?option=com_menus&view=items&menutype=)

  3. In the Sort Table By dropdown, set it to Ordering ascending

(See this screenshot, the selected value of the two dropdowns is important)

menus- all menu items - joomla cms 4 0 - administration 1

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Mar 2017

@joomdonation, done like you wrote, same Result (no Notice):
bildschirmfoto 2017-03-12 um 07 19 40

avatar joomdonation
joomdonation - comment - 12 Mar 2017

Final try: Go to System -> Global Configuration, look at Server tab, set the config option Error Reporting to Maximum, then try again

If still no notices, I don't have any other clues

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Mar 2017

"Error Reporting" is set on Maximum or on Development: No Notice.

avatar joomdonation
joomdonation - comment - 12 Mar 2017

Strange. I can see the error, both from the test and from reading the source code. We will have to wait for others to test it, then

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 12 Mar 2017

i tried also with branch https://github.com/zero-24/joomla-cms/archive/indexphp-40.zip – no Notice. Wait on other Testers.

avatar C-Lodder
C-Lodder - comment - 12 Mar 2017

I'll try reinstalling J4 later on.

avatar mbabker
mbabker - comment - 12 Mar 2017

$saveOrderingUrl is only defined if both $saveOrder and $menuType evaluate to true, but is only used if $saveOrder is true. Just on reading the code, @joomdonation instructions are correct for reproducing the error. If nothing else, it shows there's a logic error in the layout.

avatar wilsonge wilsonge - change - 18 Mar 2017
The description was changed
Status New Closed
Closed_Date 0000-00-00 00:00:00 2017-03-18 10:03:02
Closed_By wilsonge
avatar wilsonge wilsonge - close - 18 Mar 2017
avatar wilsonge
wilsonge - comment - 18 Mar 2017

Fxied

Add a Comment

Login with GitHub to post a comment