? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
17 May 2017

Pull Request for Issue #16067 .

Summary of Changes

Dont run code block when there are no items linked to a tag

Testing Instructions

Install Joomla,
Create a tag
go to front end and load that tags page with url like
http://joomla-cms/index.php?option=com_tags&view=tag&id=3

Expected result

No PHP notices

Actual result

PHP notice

Notice: Trying to get property of non-object in .../components/com_tags/views/tag/view.html.php on line 120

Documentation Changes Required

None

avatar PhilETaylor PhilETaylor - open - 17 May 2017
avatar PhilETaylor PhilETaylor - change - 17 May 2017
Status New Pending
avatar PhilETaylor PhilETaylor - change - 17 May 2017
The description was changed
avatar PhilETaylor PhilETaylor - edited - 17 May 2017
avatar joomla-cms-bot joomla-cms-bot - change - 17 May 2017
Category com_tags Front End
avatar PhilETaylor PhilETaylor - change - 17 May 2017
Labels Added: ?
avatar Quy
Quy - comment - 17 May 2017

I have tested this item successfully on 6afe198


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

avatar Quy Quy - test_item - 17 May 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 18 May 2017

I have tested this item successfully on 6afe198

With applied PR got no more Notice in php_error.log


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 18 May 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 18 May 2017

@PhilETaylor can you fix failed Build to set RTC?

avatar PhilETaylor
PhilETaylor - comment - 18 May 2017

The failed build is nothing to do with the PR.

It is already fixed by fd6d3b9

I cannot restart the build as I have no permissions to do that.

avatar rdeutz
rdeutz - comment - 18 May 2017

can't be fixed within this PR, can be set to RTC

avatar rdeutz rdeutz - change - 18 May 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 18 May 2017

RTC after two successful tests.

avatar wilsonge wilsonge - change - 21 May 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-05-21 12:56:33
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 21 May 2017
avatar wilsonge wilsonge - merge - 21 May 2017
avatar wilsonge
wilsonge - comment - 21 May 2017

Thanks Phil!

Add a Comment

Login with GitHub to post a comment