Problem/Motivation

The restriction type plugins define set their weight, but that value is never used. It does not even change how user restrictions are applied.

Proposed resolution

Remove the weight value set inside #[UserRestrictionType()] (3.0.x), or inside @UserRestrictionType() (2.0.x).

CommentFileSizeAuthor
#4 98c4cb9d6fcbb1e18c444f18a7e7c49841acb85f.png5.87 KBavpaderno

Issue fork user_restrictions-3553636

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

avpaderno created an issue. See original summary.

avpaderno opened merge request !87

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Active » Needs review
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
StatusFileSize
new 98c4cb9d6fcbb1e18c444f18a7e7c49841acb85f.png5.87 KB

The only change is the order used to show the user restriction types in the edit/create form.

screenshot

  • avpaderno committed 5011e89d on 3.0.x 
    Issue #3553636: Remove the weight value from the restriction type...
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 3.0.x-dev » 2.1.x-dev

avpaderno opened merge request !89

  • avpaderno committed 5ace7731 on 2.1.x 
    Issue #3553636: Remove the weight value from the restriction type...
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Fixed

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

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

Maintainers, please credit people who helped resolve this issue.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Priority: Normal » Minor
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented

The Restriction type options are now ordered by plugin ID.

Status: Fixed » Closed (fixed)

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