Follow-up to #2683435: CCK does not exist in Drupal 7, rename Migrate's cckfield plugins to field plugins

Problem/Motivation

While working on https://www.drupal.org/node/2879193, I noticed we are still using deprecated CCK inside of our yaml files. Let's fix that.

Proposed resolution

Reference FieldMigration instead.

Remaining tasks

  • Code it
  • Review it

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 2896538-4.patch5.34 KBheddn
#4 interdiff_2-4.txt4.67 KBheddn
#2 2896538.patch4.91 KBheddn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heddn created an issue. See original summary.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented
Issue summary: View changes
Status: Active » Needs review
FileSize
2896538.patch4.91 KB

Status: Needs review » Needs work

The last submitted patch, 2: 2896538.patch, failed testing. View results

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented
Status: Needs work » Needs review
FileSize
interdiff_2-4.txt4.67 KB
2896538-4.patch5.34 KB
quietone’s picture

quietone CreditAttribution: quietone commented
Status: Needs review » Closed (duplicate)

Yes, they are still using CckMigration but the work to change that is at #2833206: Convert Migrate's cckfield plugins to use the new field plugins.

Closing as a duplicate.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Acro Commerce commented

That makes sense. I couldn't image us not doing that. In fact, I *knew* about the duplicate issue. Heading over there now for a review.