The variable typing in MigrateMetatagsquickFieldHandler::prepare() is not the same as MigrateFieldHandler::prepare().
MigrateFieldHandler::prepare()
public function prepare(stdClass $entity, array $field_info, array $instance, array $values);
MigrateMetatagsquickFieldHandler::prepare()
public function prepare($entity, array $field_info, array $instance, array $values)
Notice the missing stdClass variable type for $entity.
I haven't had this be an issue on all sites but on some it's caused a fatal error when running update.php and other admin functions. So it may be a php setting that causes more strictness on the ones where it throws an error. Either way it's safer to do it correctly then not.
Patch to come.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | metatags_quick-prepare_definition-1611770-1.patch | 794 bytes | arcaneadam |
Comments
Comment #1
arcaneadam commentedPatch Attached
Comment #2
arcaneadam commentedComment #3
valthebaldThanks for pointing this out!
Committed to 7.x-2.x-dev