Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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
Comment | File | Size | Author |
---|---|---|---|
#2 | cckpluginmanager_not-2763629-2.patch | 668 bytes | hussainweb |
Comments
Comment #2
hussainwebIt's a simple straight-forward patch.
Comment #3
mohit_aghera CreditAttribution: mohit_aghera as a volunteer and at Axelerant commentedThanks @hussainweb
This is simple and straight forward coding standard fix.
Moving to RTBC
Comment #4
alexpottCommitted and pushed e74b8c65b25b2a60040a3e2f5ab5981863453e1f to 8.2.x and 3cf0bf7 to 8.1.x. Thanks!