I was trying to migrate a taxonomy, and found out that the allowDuplicateTerms option, that I had set to False, was not being honored. The problem is that DrupalTermMigration instantiates the destination without any options. I attach a very simple patch to fix this.

I also checked that all destination classes that extend MigrateDestinationEntity accept options, but the Drupal*Migration classes that create them don't pass them any.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrP’s picture

Status: Active » Needs review
Dave Hill’s picture

This is a slightly improved patch that uses similar approach to source options.

mikeryan’s picture

Category: Bug report » Feature request
Status: Needs review » Needs work

Good idea, but a couple of things:

1. Please do the array_merge as sourceOptions does, which enables classes to set default options before calling their parent constructor.
2. It should be extended to all destinations that support options.

Thanks.