Even there is another cleanup task for the base.js this one is about throwing out code which is not used at all.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

Status: Active » Needs review
Issue tags: +Needs backport to D7
FileSize
1.34 KB

Funny fact, this can be backported to Drupal 7 :)

damiankloip’s picture

Status: Needs review » Reviewed & tested by the community

Yeah, looks like this is redundant.

nod_’s picture

nice, thanks :)

nod_’s picture

Issue tags: +JavaScript

and tag.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

I like these kinds of patches. ;-) Committed to 8.x. Thanks!

Dries’s picture

Version: 8.x-dev » 7.x-dev
Status: Fixed » Reviewed & tested by the community

Actually, let me move this to 7.x. It should be considered.

dawehner’s picture

Project: Drupal core » Views (for Drupal 7)
Version: 7.x-dev » 7.x-3.x-dev
Component: views.module » Code
Status: Reviewed & tested by the community » Patch (to be ported)
Valentine94’s picture

Issue summary: View changes
Status: Patch (to be ported) » Needs review
Issue tags: -Needs backport to D7
FileSize
1.19 KB

Backport for Drupal 7

Chris Matthews’s picture

Status: Needs review » Needs work
Issue tags: +Needs reroll

The 4 year old patch in #8 to base.js does not apply to the latest views 7.x-3.x-dev and if still relevant needs to be rerolled.

Andrew Answer’s picture

Status: Needs work » Needs review
Issue tags: -Needs reroll
FileSize
1.28 KB

Patch rerolled.