User tests: Successful: Unsuccessful:
Pull Request resolves #47609.
The PR #44910 deprecated the UCM code, but broke storing the data in the #__ucm_base table. This PR reverts this change and fixes the problem.
Create an article and assign a tag to it. Click on save, wait for the page to reload and click on save again.
In the #__ucm_content table, for each click on save, a new entry is created in the table. No entry for the item is generated in #__ucm_base.
An entry is created in #__ucm_base for the item and the corresponding entry in #__ucm_content is only updated, no duplicates are created.
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:
bug
PR-5.4-dev
|
||
I have tested this item ✅ successfully on 8c07a71
I have tested this item ✅ successfully on 8c07a71
I have tested this item ✅ successfully on 8c07a71.
I have screenshots I will provide in a different comment because I cannot upload them on the tracker as I have 2 issues when I try to upload them:
1- the opacity is set to 0, I could solve that.
2- when clicking Start upload I get this error: Error Empty file upload result. Unfortunately, I couldn't solve that.
I have tested this item ✅ successfully on 8c07a71.
I have screenshots I will provide in a different comment because I cannot upload them on the tracker as I have 2 issues when I try to upload them:
1- the opacity is set to 0, I could solve that.
2- when clicking Start upload I get this error: Error Empty file upload result. Unfortunately, I couldn't solve that.
I have tested this item ✅ successfully on 8c07a71
I have tested this item ✅ successfully on 8c07a71
Tested it successfully: no duplicates are created in ucm_content table after saving an article again, corresponding row is updated.
@sertlan Please mark your test as successfully at https://issues.joomla.org/tracker/joomla-cms/47664
@sertlan Please mark your test as successfully at https://issues.joomla.org/tracker/joomla-cms/47664
Well this PR already has 2 Successful human Tests using issue tracker which is enough to get it RTCed (some maintainer will add the RTC tag as soon as they look into this PR)
Well more testing is always good - but as @sertlan already reported it here about their Successful test - after 2 human Successful test via joomla issue tracker It's not really needed for them to mark it using Issue tracker.
Thanks.
I have tested this item ✅ successfully on 8c07a71
I have tested this item ✅ successfully on 8c07a71
It's not really needed for them to mark it using Issue tracker
@krishnagandhicode please think also about less experienced users.
| Status | Pending | ⇒ | Ready to Commit |
RTC
RTC
Is it safe to delete duplicates in ucm_content table with this SQL query?
DELETE FROM `[prefix]_ucm_content`
WHERE core_type_alias = 'com_content.article'
AND core_content_id NOT IN (
SELECT max_id FROM (
SELECT MAX(core_content_id) as max_id
FROM `[prefix]_ucm_content`
WHERE core_type_alias = 'com_content.article'
GROUP BY core_content_item_id
) as tmp
);
Is it safe to delete duplicates in
ucm_contenttable with this SQL query?DELETE FROM `[prefix]_ucm_content` WHERE core_type_alias = 'com_content.article' AND core_content_id NOT IN ( SELECT max_id FROM ( SELECT MAX(core_content_id) as max_id FROM `[prefix]_ucm_content` WHERE core_type_alias = 'com_content.article' GROUP BY core_content_item_id ) as tmp );
No. The query is wrong. On MySQL 8 in strict mode you will get an SQL error because you are using a group by but the column by which you are grouping is not part of the select statement.
No. The query is wrong. On MySQL 8 in strict mode you will get an SQL error
For me (MariaDB 10.6.17) this SQL query works.
But is it safe to delete duplicates in the ucm_content table?
No. The query is wrong. On MySQL 8 in strict mode you will get an SQL error
For me (MariaDB 10.6.17) this SQL query works.
But is it safe to delete duplicates in ucm_content table?
No. The query is wrong. On MySQL 8 in strict mode you will get an SQL error
For me (MariaDB 10.6.17) this SQL query works.
As said, try it on MySQL 8 or PostgreSQL and you will get an SQL error.
| Labels |
Added:
RTC
|
||
I have tested this item 🔴 unsuccessfully on 779a9ca
Starting final test before merge using JBT
ucm_content entry is createducm_content entry is createducm_base and one ucm_content entry is created for assigning a tagucm_content entry is also deleted; the ucm_base entry remainsucm_base entry, if this is deleted manually, the article tag assignment is working againI have tested this item 🔴 unsuccessfully on 779a9ca
Starting final test before merge using JBT
ucm_content entry is createducm_content entry is createducm_base and one ucm_content entry is created for assigning a tagucm_content entry is also deleted; the ucm_base entry remainsucm_base entry, if this is deleted manually, the article tag assignment is working again| Status | Ready to Commit | ⇒ | Pending |
back to pending
back to pending
When the tag assignment is deleted, the
ucm_contententry is also deleted; theucm_baseentry remains
Trying to assign the tag a second time silently fails, article is saved, but the tag is missing
This may be related to theucm_baseentry, if this is deleted manually, the article tag assignment is working again
Tested it. Unfortunately, it is.
I have tested this item ✅ successfully on 8c07a71
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/47664.