Problem/Motivation

In Drupal, the namespace for entity classes is different from the namespace for the interface they implement. For example:

  • Drupal\node\Entity\Node implements Drupal\node\NodeInterface
  • Drupal\user\Entity\User implements Drupal\user\UserInterface

Proposed resolution

Rename the Drupal\user_restrictions\Entity\UserRestrictionInterface interface Drupal\user_restrictions\UserRestrictionInterface.

Issue fork user_restrictions-3550727

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.

tanushree gupta’s picture

tanushree gupta commented
Assigned: Unassigned » tanushree gupta

Working on it!!

tanushree gupta opened merge request !37

tanushree gupta’s picture

tanushree gupta commented
Assigned: tanushree gupta » Unassigned
Status: Active » Needs review
avpaderno’s picture

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

Classloader with support for moving/deprecating classes explains what to do when a class/interface/trait file is moved.

avpaderno’s picture

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

I need to verify it works also for interfaces.

avpaderno’s picture

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

avpaderno changed the visibility of the branch 3550727-rename-the-drupaluserrestrictionsentityuserrestrictioninterface to hidden.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

  • avpaderno committed 5b3081c4 on 3.0.x 
    Issue #3550727: Rename the Drupal\user_restrictions\Entity\...
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Status: Reviewed & tested by the community » 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 closed merge request !37

Status: Fixed » Closed (fixed)

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