Now that #2082117: Install default config only when the owner and provider modules are both enabled has landed, I think we can just remove our loadMultiple() method in the ViewStorageController, as now default config will only get installed if both the owner and provider are enabled. So we can (finally) remove the module exists check...

There was an issue for this a very long time ago, but I just couldn't find it, so opened a new one :/

CommentFileSizeAuthor
vdc.remove-ViewStorageController-loadMultiple.patch973 bytesdamiankloip
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Needs review » Postponed

Postponed on #2080823: Create API to discover config entities' soft dependencies and use this to present a confirm form on module uninstall :(

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost commented
Issue summary: View changes
Status: Postponed » Needs review
damiankloip’s picture

damiankloip CreditAttribution: damiankloip commented
Status: Needs review » Fixed

Views just uses ConfigEntityStorage now.

Status: Fixed » Closed (fixed)

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