PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
7 Jul 2025

Pull Request for Issue #45584 .

Summary of Changes

added the delete method

Testing Instructions

Tags

  • Create one tag
  • Check database table __history and see item_id entry com_tags.tag.id
  • Trash and delete the tag
  • The database table __history entry for com_tags.tag.id is still existing

Content Categories

  • Create one content category
  • Check database table __history and see item_id entry com_content.category.id
  • Trash and delete the category
  • The database table __history entry for com_content.category.id is still existing

Actual result BEFORE applying this Pull Request

Related contenthistory entries are are still existing after deleting a tag or a content category.

Expected result AFTER applying this Pull Request

Related contenthistory entries are deleted together with the tag and the content category.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar alikon alikon - open - 7 Jul 2025
avatar alikon alikon - change - 7 Jul 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 Jul 2025
Category Administration com_categories
avatar alikon alikon - change - 7 Jul 2025
Labels Added: PR-5.3-dev
avatar joomla-cms-bot joomla-cms-bot - change - 7 Jul 2025
Category Administration com_categories Administration com_categories com_tags
a7e2a5b 7 Jul 2025 avatar alikon cs
avatar alikon alikon - change - 7 Jul 2025
The description was changed
avatar alikon alikon - edited - 7 Jul 2025
c24c140 7 Jul 2025 avatar alikon cs
avatar alikon alikon - change - 6 Aug 2025
The description was changed
avatar alikon alikon - edited - 6 Aug 2025
avatar alikon alikon - change - 6 Aug 2025
Title
[wip] delete category history
[5.3] contenthistory entries are now deleted with tag and content category
avatar alikon alikon - edited - 6 Aug 2025
avatar muhme muhme - test_item - 7 Aug 2025 - Tested successfully
avatar muhme
muhme - comment - 7 Aug 2025

I have tested this item ✅ successfully on b93c809

Tested with JBT, first current behavier with 5.3-dev and checking the _history table:

  • Created, trashed and deleted one tag -> com_tags.tag.2 entry remains
  • Created, trashed and deleted one content category -> com_content.category.8 entry remains

Grafted Joomla_5.3.3-dev+pr.45701-Development-Full_Package.zip

  • Enabled 'Log Almost Everything'
  • Created, trashed and deleted one tag -> com_tags.tag.2 entry is deleted together with the tag
  • Created, trashed and deleted one content category -> com_content.category.8 entry is deleted together with the content category
  • Created three tags, edited each two times -> each with three entries -> 9 entries com_tags.tag.*
    • Deleted one version entry -> 8
    • Deleted this tag -> 6
    • Deleted remaining two tags -> 0
  • Created three content catgories, editied each two times -> each with three entries -> 9 entries com_content.category.*
    • Deleted one version entry -> 8
    • Deleted this content catgory -> 6
    • Deleted the two new created content categories and 'Uncategorised' -> 0
  • Checked no entries in administrator/logs/*

Well done, thanks @alikon for fixing the issue!


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

avatar exlemor exlemor - test_item - 7 Aug 2025 - Tested successfully
avatar exlemor
exlemor - comment - 7 Aug 2025

I have tested this item ✅ successfully on b93c809

I have tested this successfully. Nice find/fix. Thanks @alikon


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

avatar alikon alikon - change - 7 Aug 2025
Status Pending Ready to Commit
avatar alikon
alikon - comment - 7 Aug 2025

RTC


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

avatar rdeutz
rdeutz - comment - 8 Aug 2025

I don't think it is a good way how this is implemented, I would prefer a more generic approach

Add a Comment

Login with GitHub to post a comment