Beta phase evaluation

Reference: https://www.drupal.org/core/beta-changes
Issue category Bug, as its broken functionality
Issue priority Major, because it will hit pretty much everyone working in the views UI
Disruption Not disruptive, its just fixing internal details

Problem/Motivation

  • Add a new view of nodes
  • Click on "Content"
  • Problem: You don't see content anymore

This is caused by #2429447: Use data table as views base table, if available.

Proposed resolution

Use the data table as base table, if available, and write tests to ensure that this never happens.

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#3 2447313-3.patch1.82 KBdawehner
#3 2447313-fail.patch1 KBdawehner
#1 2447313-1.patch835 bytesdawehner

Comments

dawehner’s picture

dawehner
Primary language German
commented
Status: Active » Needs review
StatusFileSize
new 2447313-1.patch835 bytes

.

aspilicious’s picture

aspilicious commented

Probably needs a test?

dawehner’s picture

dawehner
Primary language German
commented
Issue summary: View changes
Issue tags: -Needs tests
StatusFileSize
new 2447313-fail.patch1 KB
new 2447313-3.patch1.82 KB

@aspilicious
Right, tags say so :P Let's remove the tag.

The last submitted patch, 3: 2447313-fail.patch, failed testing.

amateescu’s picture

amateescu commented
Status: Needs review » Reviewed & tested by the community

Nice and simple fix, also very good to have this tested.

aspilicious’s picture

aspilicious commented

Verified the fix :)

webchick’s picture

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

Committed and pushed to 8.0.x. Thanks!

  • webchick committed d9ed026 on 8.0.x 
    Issue #2447313 by dawehner: Views UI misses entity row plugins

Status: Fixed » Closed (fixed)

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