Problem/Motivation

In #3439713: Deprecate using the simpleConfigUpdate action on config entities, and introduce a new setProperties action to replace the invalid use cases, we deprecated using simpleConfigUpdate on a config entity. That should be an exception in Drupal 12.

Issue fork drupal-3515544

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

phenaproxima created an issue. See original summary.

annmarysruthy’s picture

annmarysruthy commented
Assigned: Unassigned » annmarysruthy

annmarysruthy opened merge request !11776

annmarysruthy’s picture

annmarysruthy commented
Assigned: annmarysruthy » Unassigned
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Postponed

Will have to wait for D12

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.

smustgrave’s picture

smustgrave commented
Status: Postponed » Needs work

@annmarysruthy this is ready to be worked on again for main.

smustgrave’s picture

smustgrave commented
Related issues: +#3553503: [meta] Remove deprecated classes, methods, procedural functions and code paths outside of deprecated modules from Drupal 12

sujal kshatri made their first commit to this issue’s fork.

sujal kshatri’s picture

sujal kshatri commented

Hi everyone, I rebased the merge request on the latest main branch and resolved the conflict . Please let me know if anything else needs adjustment : ) Thanks!

sujal kshatri’s picture

sujal kshatri commented
Status: Needs work » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Rebase LGTM

godotislate closed merge request !11776

  • godotislate committed 762b7dd2 on main 
    refactor: #3515544 The simpleConfigUpdate config action should throw an...
godotislate’s picture

godotislate
he/him
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to main. Thanks!

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.

quietone’s picture

quietone commented
Parent issue: » #3553503: [meta] Remove deprecated classes, methods, procedural functions and code paths outside of deprecated modules from Drupal 12
Related issues: -#3553503: [meta] Remove deprecated classes, methods, procedural functions and code paths outside of deprecated modules from Drupal 12

Status: Fixed » Closed (fixed)

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