Problem/Motivation

This is a straight-forward fix. The class \Drupal\migrate_drupal\Plugin\migrate\CckMigration does not explicitly declare cckPluginManager in properties which results in some warnings.

Proposed resolution

Add the property.

Remaining tasks

Patch, review, and commit.

User interface changes

None

API changes

None, as the field is already used, just not declared.

Data model changes

None

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hussainweb created an issue. See original summary.

hussainweb’s picture

Status: Active » Needs review
FileSize
668 bytes

It's a simple straight-forward patch.

mohit_aghera’s picture

Status: Needs review » Reviewed & tested by the community

Thanks @hussainweb
This is simple and straight forward coding standard fix.
Moving to RTBC

alexpott’s picture

Priority: Normal » Minor
Status: Reviewed & tested by the community » Fixed

Committed and pushed e74b8c65b25b2a60040a3e2f5ab5981863453e1f to 8.2.x and 3cf0bf7 to 8.1.x. Thanks!

  • alexpott committed e74b8c6 on 8.2.x
    Issue #2763629 by hussainweb: cckPluginManager not explicitly declared...

  • alexpott committed 3cf0bf7 on 8.1.x
    Issue #2763629 by hussainweb: cckPluginManager not explicitly declared...

Status: Fixed » Closed (fixed)

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