Hello,

I enabled breadcrumbs for my default taxonomy term view, and now I am seeing notices on all taxonomy pages:
Warning: Illegal offset type in views_taxonomy_set_breadcrumb() (line 505 of /sites/all/modules/views/modules/taxonomy.views.inc).

CommentFileSizeAuthor
#1 1504602.patch466 bytesdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Active » Needs review
FileSize
1504602.patch466 bytes

This patch should fix your issue, though it would be cool if you could try it out.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Version: 7.x-3.3 » 7.x-3.x-dev

Changing version for the testbot.

Status: Needs review » Needs work

The last submitted patch, 1504602.patch, failed testing.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Needs work » Needs review

#1: 1504602.patch queued for re-testing.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
CreditAttribution: tim.plunkett commented
Status: Needs review » Reviewed & tested by the community

Looking at http://api.drupal.org/api/search/7/taxonomy_term_uri, this is correct.

dawehner’s picture

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

Thanks for the review! Committed to 7.x-3.x

dimitriseng’s picture

dimitriseng CreditAttribution: dimitriseng commented

I can confirm that #1 removes these warnings, thanks.

Status: Fixed » Closed (fixed)

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

godinvestor’s picture

godinvestor CreditAttribution: godinvestor commented

I had the same warning. But after implementation the above mentioned patch it removes. Thanks for the hint.

damic’s picture

damic CreditAttribution: damic commented

Didn't work for me:
modules/taxonomy.views.inc has type 100755, expected 100644