? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
14 Apr 2017

Summary of Changes

Use @link over @see for URL's and use correct return types in the doc blocks

Testing Instructions

Code review as there a no production code changed

Documentation Changes Required

None

Additional Informations

I have excluded the externeal components such as fof (which has a lot of errors too) and everything in the vendor folder.

avatar zero-24 zero-24 - open - 14 Apr 2017
avatar zero-24 zero-24 - change - 14 Apr 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Apr 2017
Category Administration com_admin com_associations com_categories com_fields com_languages com_login com_redirect com_templates Repository Front End com_content Libraries
avatar zero-24 zero-24 - change - 14 Apr 2017
Labels Added: ?
avatar zero-24
zero-24 - comment - 14 Apr 2017

Thanks @mbabker i have fixed the inline comments.

e4e2b13 14 Apr 2017 avatar zero-24 drone
avatar alikon alikon - test_item - 16 Apr 2017 - Tested successfully
avatar alikon
alikon - comment - 16 Apr 2017

I have tested this item successfully on 853e77d

review


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

avatar rdeutz rdeutz - change - 23 May 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-05-23 20:12:25
Closed_By rdeutz
avatar rdeutz rdeutz - close - 23 May 2017
avatar rdeutz rdeutz - merge - 23 May 2017
avatar zero-24
zero-24 - comment - 23 May 2017

Thanks 👍

Add a Comment

Login with GitHub to post a comment