? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
6 Oct 2020

Summary of Changes

Just a spammy looking, sofa sitting, bunch of docblock updates based on what phpStorm was complaining about

Mostly adding |null where a param has a default of null.

Testing Instructions

Review the docblocks ... that's all

And for the record - please DONT add hacktoberfext-accepted to this PR - it was not intended as a spammy T-shirt earning PR

avatar PhilETaylor PhilETaylor - open - 6 Oct 2020
avatar PhilETaylor PhilETaylor - change - 6 Oct 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Oct 2020
Category Administration com_categories
avatar PhilETaylor PhilETaylor - change - 6 Oct 2020
The description was changed
avatar PhilETaylor PhilETaylor - edited - 6 Oct 2020
avatar ceford ceford - test_item - 7 Oct 2020 - Tested successfully
avatar ceford
ceford - comment - 7 Oct 2020

I have tested this item successfully on 0adce1b

I looked at the diff, found all the changed files and checked applying the patch made the changes indicated - all good.


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

avatar wilsonge wilsonge - change - 16 Oct 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-10-16 23:03:22
Closed_By wilsonge
Labels Added: ? ?
avatar wilsonge wilsonge - close - 16 Oct 2020
avatar wilsonge wilsonge - merge - 16 Oct 2020
avatar wilsonge
wilsonge - comment - 16 Oct 2020

Added the label just because you mentioned it ;)

But thankyou!

Add a Comment

Login with GitHub to post a comment