No Code Attached Yet
avatar conseilgouz
conseilgouz
23 Mar 2023

Expected result

No Error

Actual result

ERROR 0 - Call to a member function getInput() on null in ...\libraries\src\Component\Router\Rules\NomenuRules.php:82
$this->app is not initialized

System information (as much as possible)

Joomla 4.3 RC1
PHP 8.1

Additional comments

I found this issue with PhocaDownload extension, but, I don't think it's related to this extension.
In version 4.2.9, $this->router->app is being used instead of $this->app.

avatar conseilgouz conseilgouz - open - 23 Mar 2023
avatar joomla-cms-bot joomla-cms-bot - change - 23 Mar 2023
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 23 Mar 2023
avatar conseilgouz conseilgouz - change - 23 Mar 2023
Title
[4.3 RC] : NoMenuRules.php : undefined $this->app
[4.3 RC1] : NoMenuRules.php : undefined $this->app
avatar conseilgouz conseilgouz - edited - 23 Mar 2023
avatar conseilgouz conseilgouz - change - 23 Mar 2023
The description was changed
avatar conseilgouz conseilgouz - edited - 23 Mar 2023
avatar joomdonation
joomdonation - comment - 24 Mar 2023

Confirm the issue and the propose code is right, too. @conseilgouz Do you want to make PR to solve this issue?

avatar joomdonation joomdonation - change - 24 Mar 2023
Labels Added: Release Blocker
avatar joomdonation joomdonation - labeled - 24 Mar 2023
avatar joomdonation joomdonation - close - 24 Mar 2023
avatar joomdonation
joomdonation - comment - 24 Mar 2023

Closing as we have PR #40184 .

avatar joomdonation joomdonation - change - 24 Mar 2023
Status New Closed
Closed_Date 0000-00-00 00:00:00 2023-03-24 07:43:34
Closed_By joomdonation
avatar conseilgouz
conseilgouz - comment - 24 Mar 2023

PR created. I just have a problem on testing instruction.

As stated, I found it using Phocadownload extension.

@joomdonation : is code review enough ?

avatar joomdonation
joomdonation - comment - 24 Mar 2023

@joomdonation : is code review enough ?

We need to have a way to test it using Joomla core if it is possible. No worry about testing instructions, I will provide it. Thanks for making the PR.

avatar richard67 richard67 - change - 24 Mar 2023
Labels Removed: Release Blocker
avatar richard67 richard67 - unlabeled - 24 Mar 2023

Add a Comment

Login with GitHub to post a comment