? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
10 Apr 2017

Before this PR the client view will not work at all with the following error
Call to a member function get() on null

avatar brianteeman brianteeman - open - 10 Apr 2017
avatar brianteeman brianteeman - change - 10 Apr 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Apr 2017
Category Administration Templates (admin)
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 10 Apr 2017

Without PR i don't get an Error looking at Banners-Client-List or Edit-View.

avatar brianteeman
brianteeman - comment - 10 Apr 2017

@franz-wohlkoenig with hathor?

avatar waader waader - test_item - 10 Apr 2017 - Tested successfully
avatar waader
waader - comment - 10 Apr 2017

I have tested this item successfully on a0bfb6c

With testing date installed I get several errors.


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

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 10 Apr 2017

@brianteeman yes, Hathor.

avatar joomdonation
joomdonation - comment - 10 Apr 2017

@brianteeman The issue you are fixing was fixed in this PR #15168 . Your propose fix also not a valid fix as it bypass the permission check.

So for this PR, please only keep the CS fix

avatar brianteeman
brianteeman - comment - 10 Apr 2017

and that is why it is essential to have pull requests with meaningful titles and descriptions. because you didnt you wasted my time and you wasted the time of people who are testing

avatar brianteeman brianteeman - change - 10 Apr 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-04-10 11:09:30
Closed_By brianteeman
Labels Added: ?
avatar brianteeman brianteeman - close - 10 Apr 2017
avatar joomdonation
joomdonation - comment - 10 Apr 2017

Not sure if your last message is for me but #15168 was not my PR.

Add a Comment

Login with GitHub to post a comment