Problem/Motivation

#3331366: Remove epp default dependency in all fields config sync files corrects the creation of 'empty' EPP config data and unneeded dependency. Now we need to clean up existing 'empty' data, eg:

dependencies:
  ...
  module:
    - epp
  ...
third_party_settings:
  epp:
    value: ''
    on_update: 0

Proposed resolution

Implement an update hook to remove EPP third_party_settings and dependency where both value and on_update are empty/false in field config items.

Issue fork epp-3348750

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:

Comments

justcaldwell created an issue. See original summary.

geek-merlin’s picture

Title: Remove empty config and unneeded dependency » Add migration hook to remove empty config and unneeded dependency

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

vitaliyb98’s picture

Status: Active » Needs review

Added hook update, which will remove unneeded dependencies if third_party_settings.epp.value = ''

geek-merlin’s picture

Code lgtm. Automated test not necessary, if you did a manual test.

  • vitaliyb98 committed 5650fe8f on 8.x-1.x
    Issue #3348750 by vitaliyb98: Add migration hook to remove empty config...
vitaliyb98’s picture

Status: Needs review » Fixed

Tested again and merged into 8.x-1.x, thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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