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 entity reference fields.

Remaining tasks

Write a patch. Review.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#11 interdiff-2893073-5-11.txt1.04 KBmaxocub
#11 2893073-11.patch649 bytesmaxocub
#5 2893073-5.patch1.06 KBmaxocub
#5 interdiff-3-5.txt686 bytesmaxocub
#3 2893073-3.patch1.03 KBmaxocub

Comments

maxocub created an issue. See original summary.

maxocub’s picture

maxocub commented
Parent issue: #2893072: Modify the DateField Field plugin so it's used for D6 & D7 » #2842222: D7 Plain text fields incorrectly migrated to D8 as Text (formatted)
maxocub’s picture

maxocub commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new 2893073-3.patch1.03 KB

Here's a first patch.

Status: Needs review » Needs work

The last submitted patch, 3: 2893073-3.patch, failed testing. View results

maxocub’s picture

maxocub commented
Status: Needs work » Needs review
StatusFileSize
new interdiff-3-5.txt686 bytes
new 2893073-5.patch1.06 KB
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

Going to add a pre-task of updating the base class since this issue and #2893075: Create a ListField Field plugin, #2893076: Create a NumberField Field plugin, #2893077: Create a PhoneField Field plugin all depend on the exact same functionality in processFieldValues().

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

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

maxocub’s picture

maxocub commented
StatusFileSize
new 2893073-11.patch649 bytes
new interdiff-2893073-5-11.txt1.04 KB

Oups, forgot the files.

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