In /core/includes/theme.inc file couple of unused local variables.

I removed the following variables

-  $is_sticky = !empty($variables['sticky']);
-  $is_responsive = !empty($variables['responsive']);

https://www.drupal.org/node/2598500#comment-10485916
https://www.drupal.org/core/d8-allowed-changes#rc

CommentFileSizeAuthor
removing-unused-local-variables-theme.patch524 bytesrakesh.gectcr

Comments

rakesh.gectcr created an issue. See original summary.

rakesh.gectcr’s picture

rakesh.gectcr
he/him
Primary language English
Location Manchester
commented
Parent issue: » #2002650: [meta, no patch] improve maintainability by removing unused local variables
rakesh.gectcr’s picture

rakesh.gectcr
he/him
Primary language English
Location Manchester
commented
Issue tags: +rc eligible
cilefen’s picture

cilefen commented
Title: Remove unused local variable in /core/includes/theme.inc » Unused local variables in template_preprocess_table()
Status: Needs review » Reviewed & tested by the community

This is a simple fix.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed 3d816f6 and pushed to 8.0.x. Thanks!

  • alexpott committed 3d816f6 on 8.0.x 
    Issue #2601364 by rakesh.gectcr: Unused local variables in...

Status: Fixed » Closed (fixed)

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