-
Notifications
You must be signed in to change notification settings - Fork 640
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[5.x]: Merged fields are not properly merged or deleted #16198
Comments
Oh! One other thing worth mentioning: I can't delete fields with no usages. When I click the delete button, then attempt to delete the field, absolutely nothing happens, even though the site says it's deleted the field. |
Hi, thanks for reaching out! Could you send your |
Thanks for getting back to me! I did not make database exports before the |
I am having this problem too. I am creating migrations, deploying them, and when they run they do not delete the merged fields even though it says it ran the migration, and all the guids match in the environment. This has cost me many hours and I'm having to merge fields in production. I am using |
The field deletion is handled by the project config being applied. @LinneaHarts, when you deploy your |
I have managed to re-reproduce this. I am going to send the SQL files and the Composer files over as requested, but wanted to quickly add a few more notes in case somebody else in the community is experiencing this:
Unfortunately, this makes the Craft update impossible for me right now, even though both the client and I would love to do it. |
I'm using When I deploy it says the migrationos have run, but the fields are neither merged nor deleted. |
I got a lot of help from Tommy at support, who got a few database dumps from me and did some experimentation on his end. A clean install solved it for me. Still kind of a mystery what caused it, though. |
@nathansnelgrove does that mean you deleted your vendor directory and reinstalled? or something else? And was this happening for you on deploy? My merges are working on one server, but not if I copy the migration to another server, even if the database is identical. |
@LinneaHarts it means I set up a whole new Docker container and did a clean install of Craft 5 without an existing database. Then imported the DB and migrated it from there to Craft 5. I have not migrated to staging or production yet; I'm still replacing all the old Supertable field groups I had before with generic field types. |
Thanks @nathansnelgrove ! Yeah, sounds like a somewhat different problem than I'm having. @i-just should I open a new issue? |
@LinneaHarts, it would be best if you could send your |
What happened?
Description
Just upgraded to Craft 5 on a client project. Huge multisite project, hundreds of fields. I had a lot of fields I could quickly merge that existed in matrixes and SuperTables before. I followed the instructions as per the upgrade docs, and I did this a couple different times with different DDEV and remote staging environments to make sure I wasn't missing anything.
Once I merged the fields, the merged fields do not delete. Here's an example below, where many redundant fields are still listed:
Furthermore, the fields are not actually merged properly in the content. Below is a screenshot of a matrix. Each block in the matrix is an Internal Link, but instead of referring to the now-merged
internalLink
, this content is stillinternalLink2
. To fix this, I have to manually replace each of these fields with the mergedinternalLink
field and re-add my content.Steps to reproduce
craft fields/auto-merge
command.Expected behavior
I expect fields that are merged to be completely removed from the Fields screen, and their instances within content to be updated.
Actual behavior
Fields that are merged say they are used 0 times, remain in the Fields screen, and their instances do not update.
Craft CMS version
5.5.2
PHP version
8.3.14
Operating system and version
No response
Database type and version
MySQL 8.0
Image driver and version
No response
Installed plugins and versions
The text was updated successfully, but these errors were encountered: