? Success

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
23 Jul 2015

****Same issue as #7535

Contact list displays trashed items - this PR fixes it

Before

Create multiple newsfeeds
Create menu items for "News Feeds ยป List News Feeds in a Category"
Trash one feed and unpublish one feed
Check the menu without being logged in - the trashed and unpublished items do not appear
Login to the frontend as admin (or anyone with Special access)
The trashed and unpublished newsfeed is present

lwma

After

Apply the patch
Check the menu without being logged in - the trashed and unpublished items do not appear
Login to the frontend as admin (or anyone with Special access)
The unpublished newsfeed is present and the trashed one is not

lwma 1

avatar brianteeman brianteeman - open - 23 Jul 2015
avatar brianteeman brianteeman - change - 23 Jul 2015
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 23 Jul 2015
Labels Added: ?
avatar Fedik
Fedik - comment - 23 Jul 2015

test good

avatar Fedik Fedik - test_item - 23 Jul 2015 - Tested successfully
avatar brianteeman brianteeman - change - 23 Jul 2015
Category ACL
avatar jwaisner
jwaisner - comment - 24 Jul 2015

@test

PR works as intended.


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

avatar jwaisner jwaisner - test_item - 24 Jul 2015 - Tested successfully
avatar brianteeman brianteeman - change - 24 Jul 2015
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 24 Jul 2015

Setting RTC - thanks for testing


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

avatar joomla-cms-bot joomla-cms-bot - change - 24 Jul 2015
Labels Added: ?
avatar brianteeman
brianteeman - comment - 24 Jul 2015

Setting RTC - thanks for testing


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

avatar Bakual Bakual - change - 24 Jul 2015
Status Ready to Commit Pending
avatar Bakual Bakual - change - 24 Jul 2015
Labels Removed: ?
avatar Bakual
Bakual - comment - 24 Jul 2015

Same here as in the other one. The setting of the state doesn't do anything and is unneeded. Please remove those lines, then it will be fine for merging.

avatar brianteeman
brianteeman - comment - 24 Jul 2015

Done the update

avatar brianteeman brianteeman - change - 24 Jul 2015
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2015-07-24 07:59:50
Closed_By brianteeman
avatar brianteeman brianteeman - close - 24 Jul 2015
avatar brianteeman brianteeman - close - 24 Jul 2015
avatar brianteeman brianteeman - change - 24 Jul 2015
The description was changed
Status Closed Ready to Commit
Closed_Date 2015-07-24 07:59:50
Closed_By brianteeman
avatar brianteeman
brianteeman - comment - 24 Jul 2015

Changes made - travis passes - set back to RTC


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

avatar brianteeman brianteeman - reopen - 24 Jul 2015
avatar brianteeman brianteeman - reopen - 24 Jul 2015
avatar joomla-cms-bot joomla-cms-bot - change - 24 Jul 2015
Labels Added: ?
avatar brianteeman
brianteeman - comment - 24 Jul 2015

OOps I changed the status at the same time that @Bakual committed it

avatar brianteeman brianteeman - change - 24 Jul 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-07-24 08:03:21
Closed_By brianteeman
avatar brianteeman brianteeman - close - 24 Jul 2015
avatar joomla-cms-bot joomla-cms-bot - close - 24 Jul 2015
avatar brianteeman brianteeman - close - 24 Jul 2015
avatar joomla-cms-bot joomla-cms-bot - change - 24 Jul 2015
Labels Removed: ?
avatar Bakual
Bakual - comment - 24 Jul 2015

:smile:

avatar Bakual Bakual - change - 24 Jul 2015
Milestone Added:
avatar brianteeman brianteeman - head_ref_deleted - 24 Jul 2015

Add a Comment

Login with GitHub to post a comment