?
avatar chmst
chmst
16 May 2020

Steps to reproduce the issue

On a fresh installation try to install for example the patchtester component.

Notice: on line 148 of libraries\src\Installer\InstallerScript.php screen shot 2020-05-16 at 07 50 35

Additional comments

avatar chmst chmst - open - 16 May 2020
avatar joomla-cms-bot joomla-cms-bot - labeled - 16 May 2020
avatar zero-24
zero-24 - comment - 16 May 2020

What notice? Seems this is not the complet error message

avatar chmst
chmst - comment - 16 May 2020

Trying to access array offset on value of type null in line 148 of libraries\src\Installer\InstallerScript.php

Cannot be reproduced, once the installation is done.

avatar zero-24
zero-24 - comment - 16 May 2020

Hmm can you add a debugger here:
https://github.com/joomla/joomla-cms/blob/4.0-dev/libraries/src/Installer/InstallerScript.php#L147-L148

And check what is in the $manifest variable? Maybe a var_dump ?

avatar chmst
chmst - comment - 16 May 2020

I have to set up a clean installation, as this happens only once. The problem seems to be the manifest cache.

avatar zero-24
zero-24 - comment - 16 May 2020

sounds like a valid issue so we need to take care of that and check wehether the old version is set. On a new install this can never be the case. So an if check should do the trick i guess.

avatar SharkyKZ
SharkyKZ - comment - 16 May 2020

I can reproduce this on staging. Happens with extensions that can be discovered (files exist but extension is not installed).

To reproduce:

  1. Download Patch Tester.
  2. Drop files from admin folder of the package to administrator/components/com_patchtester.
  3. Install Patch Tester.
avatar zero-24
zero-24 - comment - 16 May 2020

Ok so this should be fixed in staging with the if check mention above right

avatar zero-24 zero-24 - close - 23 May 2020
avatar zero-24 zero-24 - change - 23 May 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-05-23 21:59:55
Closed_By zero-24
avatar zero-24
zero-24 - comment - 23 May 2020

Closing here is a PR for staging that should than be merged up to 4.x; #29207

Add a Comment

Login with GitHub to post a comment