? Pending

User tests: Successful: Unsuccessful:

avatar izharaazmi
izharaazmi
31 Aug 2017

Summary of Changes

Removed all occurrences of use statements that were either not used or not necessary being in the same namespace as the file.

Testing Instructions

Code review should be good enough. Full file review would be needed though and not just the changed lines so that we can find if the removed imports are indeed unnecessary.

Documentation Changes Required

None

avatar joomla-cms-bot joomla-cms-bot - change - 31 Aug 2017
Category Administration com_languages Templates (admin) Repository Front End com_contact com_newsfeeds Libraries External Library Composer Change
avatar izharaazmi izharaazmi - open - 31 Aug 2017
avatar izharaazmi izharaazmi - change - 31 Aug 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 31 Aug 2017
Category Administration com_languages Templates (admin) Repository Front End com_contact com_newsfeeds Libraries External Library Composer Change Administration com_languages Templates (admin) Repository Front End com_contact com_newsfeeds Libraries
avatar izharaazmi izharaazmi - change - 31 Aug 2017
Labels Added: ? ?
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Nov 2017

Code review should be good enough.

@Quy something for you?

avatar brianteeman
brianteeman - comment - 22 Jul 2018

It has been a very long time since this PR was submitted and as there has been no response to the comments and no tests I am closing this as an abandoned PR. It can always be reopened if updated

avatar brianteeman brianteeman - change - 22 Jul 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-07-22 09:06:02
Closed_By brianteeman
Labels Removed: ?
avatar brianteeman brianteeman - close - 22 Jul 2018

Add a Comment

Login with GitHub to post a comment