Currently the uuidKey is being set in the constructor. As this is hardcoded for all config entities, it makes sense to move this out of the constructor. It can be overridden in extending classes without doing this in the constructor.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

Status: Needs review » Reviewed & tested by the community

This makes more sense, and follows the convention used in other classes, especially tests.

tim.plunkett’s picture

Issue tags: +VDC

tagging

xjm’s picture

FileSize
602 bytes
1023 bytes

The docblock for the property wasn't quite accurate (and also should have been one line). Fixed here.

catch’s picture

Status: Reviewed & tested by the community » Fixed

Yes that's better. Committed/pushed to 8.x.

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