CanvasOverride::nodeTypePresave() breaks config import of new Canvas enabled node types

Problem/Motivation

If I have a new node type with Canvas Override enabled:

dependencies:
  module:
    - canvas_override
    - menu_ui
third_party_settings:
  menu_ui:
    available_menus:
      - main
    parent: 'main:'
  canvas_override:
    enabled: true

Then I run drush cim on this type, it fails:

    The import failed due to the following reasons:                                                                                                                                          
    Unexpected error during import with operation create for node.type.MY_CONTENT_TYPE: Missing bundle entity, entity type node_type, entity id MY_CONTENT_TYPE.                                 
    Unexpected error during import with operation create for field.field.node.MY_CONTENT_TYPE.field_canvas_layout: Missing bundle entity, entity type node_type, entity id MY_CONTENT_TYPE.

This is because CanvasOverride::nodeTypePresave() fires unconditionally and tries to create the Canvas field on the content type before the content type exists.

Steps to reproduce

1. Create a new content type
2. Enable Canvas Override
3. ddev drush cex -y
4. Delete the content type
5. ddev drush cim -y

Proposed resolution

Remaining tasks

• ✅ File an issue
• ✅ Addition/Change/Update/Fix
• ✅ Testing to ensure no regression
• ✅ Automated unit testing coverage
• ✅ Automated functional testing coverage
• ➖ UX/UI designer responsibilities
• ➖ Readability
• ➖ Accessibility
• ➖ Performance
• ➖ Security
• ➖ Developer Documentation
• ➖ User Guide Documentation
• ➖ Reviewed by human
• ➖ Code review by maintainers
• ➖ Full testing and approval
• ✅ Credit contributors
• ➖ Review with the product owner
• ➖ Release notes snippet
• ✅ Release

User interface changes

• N/A

API changes

• N/A

Data model changes

• N/A

Release notes snippet

• N/A