Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#35132] - [4] Allow methods to be called statically (as they are) without warnings

? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
15 Aug 2021

The only ever calls to these methods are made in a static way, therefore the methods should be marked as static

ERROR: InvalidStaticInvocation - libraries/src/Filesystem/Streams/StreamString.php:148:14 - Method Joomla\CMS\Filesystem\Support\StringController::getRef is not static, but is called statically (see https://psalm.dev/014)
		$string = &StringController::getRef(str_replace('string://', '', $path));
avatar PhilETaylor PhilETaylor - open - 15 Aug 2021
avatar PhilETaylor PhilETaylor - change - 15 Aug 2021
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Aug 2021
Category ⇒ Libraries
avatar ricardo1709 ricardo1709 - test_item - 17 Aug 2021 - Tested successfully
avatar ricardo1709
ricardo1709 - comment - 17 Aug 2021

I have tested this item ✅ successfully on 6fc2b8b

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

avatar PhilETaylor
PhilETaylor - comment - 18 Dec 2021

This is still an issue and needs more people to test to get merged.

avatar chmst chmst - change - 31 Jan 2022
Labels Added: ?
avatar PhilETaylor PhilETaylor - change - 6 Mar 2022
Labels Added: ?
Removed: ?
avatar PhilETaylor PhilETaylor - change - 7 Mar 2022
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2022-03-07 18:33:43
Closed_By ⇒ PhilETaylor
avatar PhilETaylor PhilETaylor - close - 7 Mar 2022

Add a Comment

Login with GitHub to post a comment