Currently, the dom_id is being set in template_preprocess_views_view(), which means that functions like hook_views_pre_build() have no access to it.

Originally I thought it should be set in an implementation of hook_views_pre_view(), but that would prevent zzz_views_pre_view() from having a chance to set it first.

CommentFileSizeAuthor
#1 views-1460418-1.patch1.11 KBtim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Active » Needs review
FileSize
views-1460418-1.patch1.11 KB

See attached.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs review » Reviewed & tested by the community

The patch looks fine.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Version: 7.x-3.x-dev » 6.x-3.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed to 7.x-3.x

I think it would be worth to backport to 6.x-3.x

Chris Matthews’s picture

Chris Matthews CreditAttribution: Chris Matthews as a volunteer commented
Issue summary: View changes
Status: Patch (to be ported) » Closed (outdated)

The Drupal 6 branch is no longer supported, please check with the D6LTS project if you need further support. For more information as to why this issue was closed, please see issue #3030347: Plan to clean process issue queue