Problem/Motivation

Opacity value not save on migration.

Steps to reproduce

create a color_field type in Drupal 7 and migrate in Drupal 8/Drupal 9.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#3 opacity-value-not-migrated-from-D7-to-D8-D9-3224459-3.patch412 bytesamarlata
#2 opacity-value-not-migrated-from-D7-to-D8:D9-3224459-2.patch412 bytesamarlata

Comments

amarlata created an issue. See original summary.

amarlata’s picture

amarlata
Amarlata
Primary language English
commented
StatusFileSize
new opacity-value-not-migrated-from-D7-to-D8:D9-3224459-2.patch412 bytes
amarlata’s picture

amarlata
Amarlata
Primary language English
commented
StatusFileSize
new opacity-value-not-migrated-from-D7-to-D8-D9-3224459-3.patch412 bytes
imalabya’s picture

imalabya
Primary language English
Location Bangalore
commented
Status: Active » Needs review
imalabya’s picture

imalabya
Primary language English
Location Bangalore
commented
Status: Needs review » Reviewed & tested by the community

Looks good to me. Moving to RTBC

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Title: Opacity value not migrated from D7 to D8/D9 » Follow-up for #3054676: Opacity value not migrated from D7 to D8/D9
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented 
+++ b/src/Plugin/migrate/field/ColorField.php
@@ -29,6 +29,7 @@ class ColorField extends FieldPluginBase {
+        'opacity' => 'opacity',

Looks good based on https://git.drupalcode.org/project/color_field/-/blob/7.x-2.x/color_fiel... and https://git.drupalcode.org/project/color_field/-/blob/8.x-2.x/src/Plugin... :)

nickdickinsonwilde’s picture

nickdickinsonwilde
He/Him/His
Primary language English
Location Victoria, BC (T'So-uke lands)
commented
Assigned: amarlata » Unassigned
Status: Reviewed & tested by the community » Fixed

committed, thanks

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

@NickDickinsonWilde Any chance you're tagging a release soon? 🤞🤓

nickdickinsonwilde’s picture

nickdickinsonwilde
He/Him/His
Primary language English
Location Victoria, BC (T'So-uke lands)
commented

Well... got half a dozen bug fixes commited in the last couple days. I'm in favour of not letting stuff sit on dev too long. Either Wednesday or next weekend for the release. Want to properly review/resolve if possible in a short timeframe most of the other tickets too.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

Lovely, thank you! :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.