User tests: Successful: Unsuccessful:
Pull Request resolves #47609.
This PR fixes storing UCM data when tags are assigned to articles.
When an article with tags is saved again, the existing #__ucm_content row is now updated instead of creating a duplicate row.
The PR also keeps #__ucm_base consistent with #__ucm_content. When tag data is removed and the related #__ucm_content row is deleted, the matching #__ucm_base row is removed too. This prevents a stale #__ucm_base row from pointing to a deleted #__ucm_content row and breaking tag assignment when the same tag is added again later.
#__ucm_content, #__ucm_base, and #__contentitem_tag_map.#__ucm_content.#__ucm_base row remains for the deleted #__ucm_content row.Saving an article with tags again creates duplicate rows in #__ucm_content.
After removing a tag, the #__ucm_content row is deleted but the related #__ucm_base row can remain. When assigning the same tag again, the article is saved but the tag assignment silently fails.
Saving an article with tags again updates the existing #__ucm_content row and does not create duplicates.
When tags are removed, the related UCM rows remain consistent. Assigning the same tag again works correctly.
Please select:
Documentation link for guide.joomla.org:
No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Libraries |
| Labels |
Added:
PR-5.4-dev
|
||
Didn't this already get fixed once
The last test was unsuccessful
Same issue with 6.1.2 (but my ucm_base is always empty)
Each Time when I save an article a new row is added to ucm_content
Same issue with 6.1.2 (but my ucm_base is always empty)
Each Time when I save an article a new row is added to ucm_content
I have tested this PR and seemed to work in every constellation.
But it uses the deprecated function storeUcmBase().
This would require a revert of the deprecation?
I have tested this PR and seemed to work in every constellation. But it uses the deprecated function storeUcmBase().
This would require a revert of the deprecation?
They should totally remove ucm tables
I have tested this PR and seemed to work in every constellation. But it uses the deprecated function storeUcmBase().
This would require a revert of the deprecation?They should totally remove ucm tables
#47664 check this PR also use the
I have tested this PR and seemed to work in every constellation. But it uses the deprecated function storeUcmBase().
This would require a revert of the deprecation?
check this PR: #47664 also uses the deprecated function.
I have tested this PR and seemed to work in every constellation. But it uses the deprecated function storeUcmBase().
This would require a revert of the deprecation?They should totally remove ucm tables
#47664 check this PR also use the
I have tested this PR and seemed to work in every constellation. But it uses the deprecated function storeUcmBase().
This would require a revert of the deprecation?check this PR: #47664 also uses the deprecated function.
Please add the same lines to the phpstan-baseline.neon so phpstan passes.
| Labels |
Added:
bug
RMDQ
|
||
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-07-30 09:49:55 |
| Closed_By | ⇒ | Reda-Muhamed |
@Reda-Muhamed Why close this?
Didn't this already get fixed once