? ? Success

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
9 Sep 2020

Pull Request for Issue # .

Summary of Changes

Changes second argument of call_user_func_array() from associative to indexed array. Before PHP 8 both types of arrays work the same but on PHP 8 associative arrays are interpreted as "named parameters". In our case this causes code to break.

Testing Instructions

Use PHP 8.
Open article form.

Actual result BEFORE applying this Pull Request

Error:

Unknown named parameter $name

Expected result AFTER applying this Pull Request

No errors, editor and editor buttons appear correctly.

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 9 Sep 2020
avatar SharkyKZ SharkyKZ - change - 9 Sep 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Sep 2020
Category Libraries
avatar SharkyKZ SharkyKZ - change - 10 Sep 2020
Labels Added: ?
avatar alikon
alikon - comment - 24 Sep 2020

unable to replicate on PHP Version 8.0.0-dev

avatar SharkyKZ
SharkyKZ - comment - 24 Sep 2020

@alikon How recent is your PHP build? I'm using Beta 4.

avatar alikon
alikon - comment - 24 Sep 2020
avatar SharkyKZ
SharkyKZ - comment - 2 Oct 2020

Just tried with RC1. Issue still exists.

avatar alikon alikon - test_item - 7 Nov 2020 - Tested successfully
avatar alikon
alikon - comment - 7 Nov 2020

I have tested this item successfully on 4ca4641


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30608.

avatar HLeithner HLeithner - change - 14 Nov 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-11-14 17:59:38
Closed_By HLeithner
avatar HLeithner HLeithner - close - 14 Nov 2020
avatar HLeithner HLeithner - merge - 14 Nov 2020
avatar HLeithner
HLeithner - comment - 14 Nov 2020

Tested successfully, thanks

Add a Comment

Login with GitHub to post a comment