Problem/Motivation

The website status report page shows the "Entity/field definitions" error for the Site Setting entity.

This problem was fixed in the related issue #3463953 for those, who had not installed the module or run any module updates. However, those who have already run the site_settings_post_update_1_make_revisionable post update, the following error is shown:

Mismatched entity and/or field definitions

The following changes were detected in the entity type and field definitions.
Site Setting
The Revision log message field needs to be installed.
The Revision log message field needs to be uninstalled.
The Publishing status field needs to be updated.

Steps to reproduce

Go to the /admin/reports/status page and check for the problems with the field definitions.

Proposed resolution

Add a separate post_update with the changes to affect only those users, who have already run the
site_settings_post_update_1_make_revisionable previously.

User interface changes

The definition mismatch problem is gone for all websites.

CommentFileSizeAuthor
#10 3501378-field-definitions-mismatch-10.patch3.33 KBevilargest
#9 3501378-field-definitions-mismatch-9.patch13.28 KBevilargest
#5 3501378-field-definitions-mismatch-5.patch4.08 KBevilargest

Issue fork site_settings-3501378

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

evilargest created an issue. See original summary.

evilargest changed the visibility of the branch 3501378-field-definitions-mismatch to hidden.

evilargest changed the visibility of the branch 3501378-field-definitions-mismatch to active.

evilargest opened merge request !48

evilargest’s picture

evilargest commented
Status: Active » Needs review
StatusFileSize
new 3501378-field-definitions-mismatch-5.patch4.08 KB
scott_euser’s picture

scott_euser commented
Status: Needs review » Needs work

Thanks for the contribution (and apologies you had to deal with it in the first place!). Happy to merge this, just needs the minor phpcs/cspell nitpicks resolved.

evilargest’s picture

evilargest commented
Status: Needs work » Needs review

The pipelines are green now

taran2l made their first commit to this issue’s fork.

evilargest’s picture

evilargest commented
StatusFileSize
new 3501378-field-definitions-mismatch-9.patch13.28 KB

Updating the patch

evilargest’s picture

evilargest commented
StatusFileSize
new 3501378-field-definitions-mismatch-10.patch3.33 KB

whoops, that was not a plain diff

scott_euser’s picture

scott_euser commented
Assigned: evilargest » Unassigned
Status: Needs review » Fixed

Thanks both!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.