?
avatar svenbluege
svenbluege
7 Dec 2016

Steps to reproduce the issue

  1. open Joomla! lanugage manager
  2. click "install languages" button in the toolbar
  3. select two random languages
  4. click the install button in the toolbar

Expected result

The two selected languages are installed.

Actual result

An error message appears:

image

Fatal error: Nesting level too deep - recursive dependency? in /var/www/eventgallery/tests/joomla-cms3/libraries/joomla/event/dispatcher.php on line 197

System information (as much as possible)

  • PHP 7
  • Joomla 3.7.0-dev
  • LAMP

Additional comments

Installing those two languages one after another works. I use the JoomlaBrowser Codeception class to create a multilanguage environment. This is where I encountered this error.

avatar svenbluege svenbluege - open - 7 Dec 2016
avatar joomla-cms-bot joomla-cms-bot - change - 7 Dec 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 7 Dec 2016
avatar svenbluege svenbluege - change - 7 Dec 2016
The description was changed
avatar svenbluege svenbluege - edited - 7 Dec 2016
avatar brianteeman
brianteeman - comment - 7 Dec 2016

I used to do this many times during testing without any issues but today I can confirm the issue with the same error message. It must be a recent commit

avatar brianteeman brianteeman - change - 7 Dec 2016
Title
Install multiple language at once fails
REGRESSION Install multiple language at once fails
Status New Confirmed
Labels Added: ?
avatar brianteeman brianteeman - edited - 7 Dec 2016
avatar brianteeman brianteeman - labeled - 7 Dec 2016
avatar brianteeman brianteeman - change - 7 Dec 2016
Category Multilanguage
avatar Bakual
Bakual - comment - 7 Dec 2016

@mbabker Could that be related to #12977 as it has been merged yesterday and deals with installing packages.

avatar brianteeman
brianteeman - comment - 7 Dec 2016

Also #13122

On 7 December 2016 at 20:55, Thomas Hunziker notifications@github.com
wrote:

@mbabker https://github.com/mbabker Could that be related to #12977
#12977 as it has been merged
yesterday and deals with installing packages.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#13124 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABPH8TCgFwnfHB5vsjVmIeNIVey_-aPiks5rFx0ogaJpZM4LHEq_
.

--
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
https://brian.teeman.net/ http://brian.teeman.net/

avatar andrepereiradasilva
andrepereiradasilva - comment - 7 Dec 2016

this issue also happens on joomla installation if one try install two or more languages at same time (but we get no error there thanks to the "excelent" error handling of installation app ? ).

but in chrome dev tools one can see the error:

Fatal error: Nesting level too deep - recursive dependency? in /path/to/joomla-staging/libraries/joomla/event/dispatcher.php on line 197
avatar mbabker
mbabker - comment - 7 Dec 2016

The only thing I can think of is moving that line loading the in-class
event listener to some place where it'll only run once.
On Wed, Dec 7, 2016 at 4:47 PM andrepereiradasilva notifications@github.com
wrote:

this issue also happens on joomla installation if one try install two or
more languages at same time (but we get no error there thanks to the
"excelent" error handling of installation app ? ).

but in chrome dev tools one can see the error:

Fatal error: Nesting level too deep - recursive dependency? in /path/to/joomla-staging/libraries/joomla/event/dispatcher.php on line 197


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#13124 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWfoTpUpJcAD_qFigkT_TY1ExZmsFcxks5rFzd-gaJpZM4LHEq_
.

--

  • Michael Please pardon any errors, this message was sent from my iPhone.
avatar infograf768
infograf768 - comment - 8 Dec 2016

#13122 is not at stake.
It is indeed related to #12977 as I tested reverting it here.

avatar mbabker
mbabker - comment - 8 Dec 2016

Just like I said

The only thing I can think of is moving that line loading the in-class event listener to some place where it'll only run once.

What's probably happening is the event is getting registered on each install and that's throwing it for a loop. Let's not get trigger happy on the revert button now ?

avatar Bakual
Bakual - comment - 8 Dec 2016

Let's not get trigger happy on the revert button now

Nah, I'm sure you can fix it. I have just no clue about which line you're talking ?

avatar alikon
alikon - comment - 8 Dec 2016

any feedback #13134

avatar alikon
alikon - comment - 8 Dec 2016

sorry don't consider previuos attempt

avatar mbabker
mbabker - comment - 9 Dec 2016

See #13147

avatar mbabker mbabker - change - 9 Dec 2016
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2016-12-09 13:51:21
Closed_By mbabker
avatar mbabker mbabker - close - 9 Dec 2016
avatar zero-24 zero-24 - change - 16 Oct 2017
Labels Removed: ?
avatar zero-24 zero-24 - unlabeled - 16 Oct 2017

Add a Comment

Login with GitHub to post a comment