No Code Attached Yet
avatar brianteeman
brianteeman
9 Jan 2025

The filesystemhelper is marked as deprecated

* File system helper
*
* Holds support functions for the filesystem, particularly the stream
*
* @since 1.7.0
* @deprecated 4.4 will be removed in 6.0
* Use Joomla\Filesystem\Helper instead.

However the function fileUploadMaxSize is not marked as deprecated AND is not present in Joomla\Filesystem\Helper

/**
* Calculates the maximum upload file size and returns string with unit or the size in bytes
*
* Call it with JFilesystemHelper::fileUploadMaxSize();
*
* @param bool $unitOutput This parameter determines whether the return value should be a string with a unit
*
* @return float|string The maximum upload size of files with the appropriate unit or in bytes
*
* @since 3.4
*/
public static function fileUploadMaxSize($unitOutput = true)

avatar brianteeman brianteeman - open - 9 Jan 2025
avatar joomla-cms-bot joomla-cms-bot - change - 9 Jan 2025
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 9 Jan 2025
avatar brianteeman brianteeman - change - 9 Jan 2025
The description was changed
avatar brianteeman brianteeman - edited - 9 Jan 2025
avatar brianteeman brianteeman - change - 10 Jan 2025
Title
FilesystemHelper
[6.0] FilesystemHelper
avatar brianteeman brianteeman - edited - 10 Jan 2025
avatar Hackwar Hackwar - change - 11 Jan 2025
Status New Closed
Closed_Date 0000-00-00 00:00:00 2025-01-11 18:14:25
Closed_By Hackwar
avatar Hackwar Hackwar - close - 11 Jan 2025
avatar Hackwar
Hackwar - comment - 11 Jan 2025
avatar brianteeman
brianteeman - comment - 11 Jan 2025

Does it not need to be specifically deprecated? All the other functions are together with info on their replacement

avatar Hackwar
Hackwar - comment - 11 Jan 2025

The class itself is deprecated and can be replaced with the class from the framework package.

avatar brianteeman
brianteeman - comment - 11 Jan 2025

strange as thats not consistent with all the other functions in that file.

avatar Hackwar
Hackwar - comment - 11 Jan 2025

I can guarantee you, that that function will also be deprecated as soon as the filesystem PR is merged.

avatar brianteeman
brianteeman - comment - 11 Jan 2025

Thank you

Add a Comment

Login with GitHub to post a comment