? ? Pending

User tests: Successful: Unsuccessful:

avatar wilsonge
wilsonge
14 Jan 2017

Pull Request for Issue #8770 . Alternative fix to #13584

Summary of Changes

Deleting an article doesn't remove the article from the ucm_content table because an array is sent (containing array('primary_key_name' => 'contentItemId')) while an integer is expected. This pull request fixes this.

Testing Instructions

  1. Create an article and make sure to assign a tag to the article
  2. Check the ucm_content table to see that the article is there as well
  3. Trash the article
  4. Delete the article from trash
  5. Check the ucm_content table and see that the article is still there
  6. Apply patch
  7. Create an article and make sure to assign a tag to the article
  8. Check the ucm_content table to see that the article is there as well
  9. Trash the article
  10. Delete the article from trash
  11. Check the ucm_content table and see that the article has now been removed

Documentation Changes Required

None

avatar wilsonge wilsonge - open - 14 Jan 2017
avatar wilsonge wilsonge - change - 14 Jan 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Jan 2017
Category Libraries
avatar wilsonge wilsonge - change - 14 Jan 2017
Labels Added: ?
avatar ghazal
ghazal - comment - 15 Jan 2017

I have tested this item successfully on d1d63a6


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

avatar ghazal ghazal - test_item - 15 Jan 2017 - Tested successfully
avatar sanderpotjer
sanderpotjer - comment - 15 Jan 2017

I have tested this item successfully on d1d63a6


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

avatar sanderpotjer sanderpotjer - test_item - 15 Jan 2017 - Tested successfully
avatar wilsonge wilsonge - change - 15 Jan 2017
Status Pending Ready to Commit
avatar wilsonge
wilsonge - comment - 15 Jan 2017

RTC. Thanks guys!


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

avatar rdeutz rdeutz - change - 15 Jan 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-01-15 21:10:10
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 15 Jan 2017
avatar rdeutz rdeutz - merge - 15 Jan 2017

Add a Comment

Login with GitHub to post a comment