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

CommentFileSizeAuthor
#2 cckpluginmanager_not-2763629-2.patch668 byteshussainweb

Comments

hussainweb created an issue. See original summary.

hussainweb’s picture

hussainweb
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new cckpluginmanager_not-2763629-2.patch668 bytes

It's a simple straight-forward patch.

mohit_aghera’s picture

mohit_aghera commented
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

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
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.