Remove Unused local variables from /core/modules/views/views.theme.inc [#2064289]

Part of meta-issue #2002650: [meta, no patch] improve maintainability by removing unused local variables

File /core/modules/views/views.theme.inc

Line 23: Unused local variable $base_path
Line 309: Unused local variable $id
Line 350: Unused local variable $view
Line 351: Unused local variable $field
Line 352: Unused local variable $row
Line 517: Unused local variable $default_row_class
Line 667: Unused local variable $num
Line 671: Unused local variable $num
Line 958: Unused local variable $base_url
Line 1055: Unused local variable $id

Comment	File	Size	Author
#1	drupal8.other_.2064289-1.patch	2.7 KB	sergeypavlenko
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

sergeypavlenko CreditAttribution: sergeypavlenko commented 13 August 2013 at 16:09

Assigned:	sergeypavlenko	» Unassigned
Status:	Active	» Needs review

File	Size
drupal8.other_.2064289-1.patch	2.7 KB

Comment #2

alphawebgroup

he/him

English

Kiev / Toronto

CreditAttribution: alphawebgroup commented 13 August 2013 at 18:56

Status:

Needs review

» Reviewed & tested by the community

just applied the patch and reviewed the changes
tried the main views functionality... works good for me
so, don't see any reason why do not change the status to rtbc

Comment #3

alexpott

he/they

English

🇪🇺🌍

CreditAttribution: alexpott commented 14 August 2013 at 12:21

Title:	Remove Unused local variable $base_path from /core/modules/views/views.theme.inc	» Remove Unused local variables from /core/modules/views/views.theme.inc
Status:	Reviewed & tested by the community	» Fixed

Committed 0bbd2b0 and pushed to 8.x. Thanks!