Problem/Motivation

In order to leverage the MigrateFileUriAsIs class, the DrupalFileMigration class allows migrations to inject their own value to the file_class argument. However, DrupalFile7Migration::prepareRow() has an else condition that will apply $this->destination->setFileClass('MigrateFileUri'); to any file row that isn't a Youtube or oEmbed asset.

Proposed resolution

Remove that else condition to allow the migration argument setting to be the default behavior.

Remaining tasks

All the things.

User interface changes

None.

API changes

Restores function of the file_class argument.

Data model changes

None.

CommentFileSizeAuthor
#2 2903724-2_use_file_class_default.patch498 bytesGrayside
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Grayside created an issue. See original summary.

Grayside’s picture

Status: Active » Needs review
FileSize
498 bytes