Problem/Motivation

in file core/modules/views/views.theme.inc

    $variables['rows'][$id]['attributes'] = array();
    $variables['rows'][$id]['attributes'] = new Attribute($variables['rows'][$id]['attributes']);

This was caused by:
#2329917: Move views classes from preprocess to templates
#1968398: Convert Views $row_classes to $row['attributes']

Proposed resolution

remove useless line

Remaining tasks

commit

User interface changes

no

API changes

no

Data model changes

no

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost created an issue. See original summary.

andypost’s picture

Issue tags: +rc eligible

tagging

Status: Needs review » Needs work

The last submitted patch, unused-1.patch, failed testing.

andypost’s picture

Status: Needs work » Needs review
FileSize
761 bytes

proper fix

andriyun’s picture

Status: Needs review » Reviewed & tested by the community

Great!

andypost’s picture

Issue summary: View changes

added related issues that caused this

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 4: unused-4.patch, failed testing.

andypost’s picture

Status: Needs work » Reviewed & tested by the community

bot flux

catch’s picture

Title: Remove useless assining in template_preprocess_views_view_unformatted() » Remove useless assigning in template_preprocess_views_view_unformatted()

  • catch committed 28437fd on 8.1.x
    Issue #2607676 by andypost: Remove useless assigning in...

  • catch committed cb55cb5 on
    Issue #2607676 by andypost: Remove useless assigning in...
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.1.x and cherry-picked to 8.0.x, thanks!

Status: Fixed » Closed (fixed)

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