?
avatar ReLater
ReLater
16 Aug 2020

Steps to reproduce the issue

@deprecated 5.0 Use the class \Joomla\Component\Content\Site\Service\HTML\Icon instead

Expected result

????

  • @deprecated 5.0 Use the class \Joomla\Component\Content\Administrator\Service\HTML\Icon instead

????

avatar ReLater ReLater - open - 16 Aug 2020
avatar joomla-cms-bot joomla-cms-bot - change - 16 Aug 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 16 Aug 2020
avatar SharkyKZ
SharkyKZ - comment - 16 Aug 2020

Not correct.

avatar richard67
richard67 - comment - 16 Aug 2020

@SharkyKZ Sorry, what's not correct? The deprecation string? Or the solution proposed above? I assume the first, but it's not 100 clear to me. If the first: Is the proposed correction here correct? Or maybe in shorter words: Do you want to make a PR? ;-)

avatar SharkyKZ
SharkyKZ - comment - 16 Aug 2020

The first.

avatar richard67
richard67 - comment - 16 Aug 2020

@ReLater Wanna make a PR?

avatar ReLater
ReLater - comment - 16 Aug 2020

Will do

avatar ReLater
ReLater - comment - 16 Aug 2020

see pr #30389

avatar ReLater ReLater - close - 16 Aug 2020
avatar ReLater ReLater - change - 16 Aug 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-08-16 18:12:24
Closed_By ReLater

Add a Comment

Login with GitHub to post a comment