See the change record: New method \Drupal\Core\Config\Config::hasOverrides($key) determines if a configuration object or any of its keys are overridden.
This will probably be very useful for replacing/improving our own override detection in entity CRUD hook methods.

Postponed until we depend on Drupal 8.5, of course, but we could already create the patch before that.

CommentFileSizeAuthor
#2 2936630-2--use_config_hasOverrides_method.patch4.71 KBdrunken monkey

Comments

drunken monkey created an issue. See original summary.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Postponed » Needs review
StatusFileSize
new 2936630-2--use_config_hasOverrides_method.patch4.71 KB

Not as much a simplification as I'd hoped, since we still can't get the actual override values (for whatever reason the Powers That Be thought that wasn't a good idea), but at least it looks a bit cleaner now, and should also be more reliable/stable (since we don't manually muck about with tagged services, etc.).

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
commented
Status: Needs review » Reviewed & tested by the community

Still an improvement though.

  • drunken monkey committed cb75093 on 8.x-1.x 
    Issue #2936630 by drunken monkey, borisson_: Adapted override detection...
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Reviewed & tested by the community » Fixed

Thanks for reviewing!
Committed.

Status: Fixed » Closed (fixed)

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