For classes that extend plugin the id is a read-only variable through Plugin::getId()

defining an id variable is both redundant and buggy

Currently DevelGenerate::id and DevelDumper::id are both incorrectly writable.

Core make this mistake alot ( 34 cases at the last count ) I am trying to fix that here.

#2917345: Parallel definition of ID is confusing

CommentFileSizeAuthor
#2 2917472-2.patch1005 bytesmartin107
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

martin107 created an issue. See original summary.

martin107’s picture

Assigned: martin107 » Unassigned
Status: Active » Needs review
FileSize
1005 bytes

Now I have an issue number here is the patch.

moshe weitzman’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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