Once upon a time (a couple of months ago), a cckfield plugin for link fields was created in #2448501: Use the migrate cckfield plugin type for the link field. There was an intention to create plugins for text fields and file fields as well, but those plans got lost in the flow. It's time to rectify that.

The attached patch moves file field processing from the LoadEntity load plugin into a dedicated cckfield plugin.

Comments

phenaproxima’s picture

Issue summary: View changes
benjy’s picture

Status: Needs review » Reviewed & tested by the community

Looks good, thanks.

benjy’s picture

Status: Reviewed & tested by the community » Needs review

Just realised, I think we need to remove filefield_widget: file_generic from field_instance_widget_settings? Also, make sure it's handled in the CCK field plugin.

phenaproxima’s picture

StatusFileSize
new4.62 KB
new1.1 KB

Done.

Status: Needs review » Needs work

The last submitted patch, 4: 2528688-4.patch, failed testing.

phenaproxima’s picture

Status: Needs work » Needs review

Um, what?

phenaproxima queued 4: 2528688-4.patch for re-testing.

ultimike’s picture

Status: Needs review » Reviewed & tested by the community

Thanks for picking up my slack - I wanted to take care of these conversions but the free time to do so never materialized...

-mike

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 23fde8d and pushed to 8.0.x. Thanks!

  • alexpott committed 23fde8d on 8.0.x
    Issue #2528688 by phenaproxima: Create cckfield plugin for file fields
    

Status: Fixed » Closed (fixed)

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