? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
25 Apr 2021

Summary of Changes

Remove deprecated function calls and use short array syntax

Instead of Factory::getUser() which is deprecated use Factory::getApplication()->getIdentity()

Use short array syntax [] instead of long syntax array()

Testing Instructions

Code Review only

@Bakual :)

avatar PhilETaylor PhilETaylor - open - 25 Apr 2021
avatar PhilETaylor PhilETaylor - change - 25 Apr 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Apr 2021
Category Administration com_banners com_contact com_content com_messages com_newsfeeds com_users
avatar PhilETaylor PhilETaylor - change - 25 Apr 2021
The description was changed
avatar PhilETaylor PhilETaylor - edited - 25 Apr 2021
avatar PhilETaylor PhilETaylor - change - 25 Apr 2021
Labels Added: ?
avatar richard67 richard67 - test_item - 25 Apr 2021 - Tested successfully
avatar richard67
richard67 - comment - 25 Apr 2021

I have tested this item successfully on edabb1a

Code review.


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

avatar joomdonation joomdonation - test_item - 26 Apr 2021 - Tested successfully
avatar joomdonation
joomdonation - comment - 26 Apr 2021

I have tested this item successfully on edabb1a


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

avatar richard67 richard67 - change - 26 Apr 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 26 Apr 2021

RTC


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

avatar rdeutz rdeutz - close - 26 Apr 2021
avatar rdeutz rdeutz - merge - 26 Apr 2021
avatar rdeutz rdeutz - change - 26 Apr 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-04-26 07:17:11
Closed_By rdeutz
Labels Added: ?

Add a Comment

Login with GitHub to post a comment