Follow-up to #2839178: Fix incorrect DER field dependency calculation

Problem/Motivation

DER field doesn't add dependencies to the target bundles.

Proposed resolution

Update DERI::calculateDependencies() and DERI::onDependencyRemoval() to add dependencies to the target bundles.

Remaining tasks

  • Review.
  • Commit.

User interface changes

None

API changes

None

Data model changes

None

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jibran created an issue. See original summary.

jibran’s picture

jibran’s picture

Status: Active » Needs review
jibran’s picture

With upgrade path and tests now.

jibran’s picture

Patch for 8.x-2.x without the upgrade path.

larowlan’s picture

looks good to me

dpi’s picture

Status: Needs review » Reviewed & tested by the community

  • jibran committed ac86b98 on 8.x-2.x
    Issue #2878852 by jibran: Fix incorrect DER field config dependency...

  • jibran committed ccd4f45 on 8.x-1.x
    Issue #2878852 by jibran: Fix incorrect DER field config dependency...
jibran’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, @larowlan and @dpi for the review.

Status: Fixed » Closed (fixed)

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