Problem/Motivation

Similar to #2232275: System menu blocks need to be able to depend on their menu entities, views blocks don't depend on the view they are displaying. That makes it impossible to sync or provide a default views block.

Proposed resolution

Ensure the dependency is set.

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#1 2236851.1.patch5.63 KBalexpott
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Priority: Major » Critical
Issue summary: View changes
Status: Active » Needs review
Issue tags: +Configurables, +Configuration system, +beta blocker, +missing config dependencies
FileSize
2236851.1.patch5.63 KB

Using the same definition key as the system menu block deriver...

Beta blocker due to the data model change of the views blocks and because it would require a hook_update_N().

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

Nothing really to complain about.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

  • Commit f97d2fa on 8.x by webchick: 
    Issue #2236851 by alexpott | Berdir: Views blocks are missing dependency...

Status: Fixed » Closed (fixed)

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

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented

Here is another issue related with blocks and dependencies and views :) #2309247: Views do not depend on modules providing their displays.