J4 Issue ?
avatar C-Lodder
C-Lodder
19 May 2019

Steps to reproduce the issue

The call() method inside the Cache Callback Contrroller is marked as deprecated:
https://github.com/joomla/joomla-cms/blob/staging/libraries/src/Cache/Controller/CallbackController.php#L37

However appear to have been removed in Joomla 4

Therefore using Factory::getCache()->call([xxx, xxx]);

will throw an error

avatar C-Lodder C-Lodder - open - 19 May 2019
avatar joomla-cms-bot joomla-cms-bot - change - 19 May 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 19 May 2019
avatar C-Lodder C-Lodder - change - 19 May 2019
The description was changed
avatar C-Lodder C-Lodder - edited - 19 May 2019
avatar C-Lodder C-Lodder - change - 19 May 2019
Title
Cache->call() supposed to be deprecated?
Cache call() method supposed to be deprecated?
avatar C-Lodder C-Lodder - edited - 19 May 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 19 May 2019
Labels Added: J4 Issue
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 19 May 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 19 May 2019
Title
Cache call() method supposed to be deprecated?
[4.0] Cache call() method supposed to be deprecated?
avatar franz-wohlkoenig franz-wohlkoenig - edited - 19 May 2019
avatar SharkyKZ
SharkyKZ - comment - 19 May 2019

Yes, it's marked to be removed in 4.0 and was removed in 4.0.

See #24703.

avatar franz-wohlkoenig franz-wohlkoenig - change - 19 May 2019
Status New Closed
Closed_Date 0000-00-00 00:00:00 2019-05-19 11:35:07
Closed_By franz-wohlkoenig
avatar franz-wohlkoenig franz-wohlkoenig - close - 19 May 2019
avatar C-Lodder
C-Lodder - comment - 19 May 2019

Does the deprecated notice not mean it will be deprecated in 4.0?

avatar mbabker
mbabker - comment - 20 May 2019

The generally accepted practice for a @deprecated annotation is the version will be the first version the element may be removed in. So something @deprecated 4.0 means it may be removed in 4.0. It doesn't mean "as of 4.0 this element is deprecated and will be removed at the next major version".

avatar C-Lodder
C-Lodder - comment - 20 May 2019

Ah fair enough. Thanks for clarifying

Add a Comment

Login with GitHub to post a comment