PR-5.0-dev Pending

User tests: Successful: Unsuccessful:

avatar Denitz
Denitz
28 Sep 2023

Summary of Changes

ActionsDropdown::$dropDownList is declared as string while it's string[]

Once rendered, the array is casted to null (I guess to free memory), but it's an array actually and all any next further call to add a button will produce warning on adding element to null.

Testing Instructions

Apply patch.

Actual result BEFORE applying this Pull Request

See IDE warning for static::$dropDownList[] = ... code.

Expected result AFTER applying this Pull Request

No IDE warnings.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

669668f 28 Sep 2023 avatar Denitz fix
avatar joomla-cms-bot joomla-cms-bot - change - 28 Sep 2023
Category Libraries
avatar Denitz Denitz - open - 28 Sep 2023
avatar Denitz Denitz - change - 28 Sep 2023
Status New Pending
avatar HLeithner HLeithner - change - 30 Sep 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-09-30 20:07:32
Closed_By HLeithner
Labels Added: PR-5.0-dev
avatar HLeithner HLeithner - close - 30 Sep 2023
avatar HLeithner HLeithner - merge - 30 Sep 2023

Add a Comment

Login with GitHub to post a comment