Problem/Motivation

Discovered while removing the deprecations #3574717: Remove deprecated code from migrate module in EntityContentBase when we make $entity_type_bundle_info required then it's an optional parameter in $account_switcher followed by a now required parameter.

Steps to reproduce

NA

Proposed resolution

Make $account_switcher also required

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3577140

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

smustgrave created an issue. See original summary.

smustgrave’s picture

smustgrave commented
Title: Order of parameters in EntityContentBase need looking at » EntityContentBase make all parameters required
Issue summary: View changes

smustgrave opened merge request !15007

smustgrave’s picture

smustgrave commented
Status: Active » Needs review
benjifisher’s picture

benjifisher
he/him or they/them
Primary language English
Location Boston area
commented

I think we can close this issue. See my comment #3574717-9: Remove deprecated code from migrate module. I gave the detailed explanation on that issue, not this one, so that the final review of that issue will see the explanation.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Closed (duplicate)

I think it's more clear to have required params, did it in #3574717-10: Remove deprecated code from migrate module

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.