Problem/Motivation

Following #2842222: D7 Plain text fields incorrectly migrated to D8 as Text (formatted) and #2893061: Create a ProcessField plugin to process the field types, we need to replace the field type static_map in the d7_field migration by dedicated Field plugins.

Proposed resolution

Write a Field plugin for list fields.

Remaining tasks

Write a patch. Review.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#11 interdiff-2893075-6-11.txt1019 bytesmaxocub
#11 2893075-11.patch671 bytesmaxocub
#6 interdiff-4-6.txt639 bytesmaxocub
#6 2893075-6.patch1.08 KBmaxocub
#4 2893075-4.patch1.08 KBmaxocub
#4 interdiff-2-4.txt653 bytesmaxocub
#2 2893075-2.patch1.05 KBmaxocub

Comments

maxocub created an issue. See original summary.

maxocub’s picture

maxocub commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new 2893075-2.patch1.05 KB

Here's a first patch.

Status: Needs review » Needs work

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

maxocub’s picture

maxocub commented
Status: Needs work » Needs review
StatusFileSize
new interdiff-2-4.txt653 bytes
new 2893075-4.patch1.08 KB

Status: Needs review » Needs work

The last submitted patch, 4: 2893075-4.patch, failed testing. View results

maxocub’s picture

maxocub commented
Status: Needs work » Needs review
StatusFileSize
new 2893075-6.patch1.08 KB
new interdiff-4-6.txt639 bytes

Oups, forgot a coma.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Assigned: Unassigned » phenaproxima

Self-assigning for review.

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Assigned: phenaproxima » heddn

Assigning to myself for review this week.

heddn’s picture

heddn
Primary language English
Location Illinois
commented
Assigned: heddn » Unassigned
Status: Needs review » Postponed

Postponing on #2896507: Update FieldPluginBase with a default processFieldValues() and getFieldFormatterMap()

Version: 8.4.x-dev » 8.5.x-dev

Drupal 8.4.0-alpha1 will be released the week of July 31, 2017, which means new developments and disruptive changes should now be targeted against the 8.5.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

maxocub’s picture

maxocub commented
StatusFileSize
new 2893075-11.patch671 bytes
new interdiff-2893075-6-11.txt1019 bytes

Preparing for #2896507: Update FieldPluginBase with a default processFieldValues() and getFieldFormatterMap() getting in.

maxocub’s picture

maxocub commented
Status: Postponed » Closed (duplicate)

Since they are quite small, similar and related, let's do the for new field plugin in one patch: #2901851: Replace the static map in the d7_field migration by field plugins