I'm not sure if this is due to a recent Views update or Ubercart update but I have just noticed that if I go to a Catalog "Browse" page that has products in more than one product category, those terms are added to the breadcrumb trail.

I can stop this happening by unticking the catalog breadcrumbs option in the Store -> Catalog settings, but then I also loose the breadcrumb trail completely on the product pages.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alexp999’s picture

Just to say this only happens with the "Display Price" field as part of the view

longwave’s picture

Status: Active » Closed (duplicate)
alexp999’s picture

Tried that patch and its still doing it but only on some pages.

It seems to be related to having options/attributes.

alexp999’s picture

Status: Closed (duplicate) » Active

Sorry to change the status, but I dont want it to get missed that I'm still having a problem on some pages with the linked patch.

longwave’s picture

Status: Active » Postponed (maintainer needs more info)

What do you mean by "only on some pages" and "it seems to be related to having options/attributes" - can you give screenshots or examples?

alexp999’s picture

Here is a screenshot with "Product: Display Price" loaded into the views, but excluded from display:

with display price

And here is a screenshot with it removed from the View. I am using "Product: Sell Price" in its place which works fine:

without display price

This is one of very few pages it happens on, and the only difference I can see between this page and other pages with products from multiple categories, is that this category contains the product shown outlined in red, which has product attributes/options AND is in multiple categories.

longwave’s picture

Status: Postponed (maintainer needs more info) » Needs review
FileSize
1.36 KB

Can you test the attached patch to see if that fixes this?

TR’s picture

Version: 7.x-3.0 » 7.x-3.x-dev

Moving to -dev for the testbot.

longwave’s picture

Status: Needs review » Needs work
longwave’s picture

Status: Needs work » Needs review
TR’s picture

We're in a Catch-22 here - the issue was originally set to 7.x-3.0 so the testbot put the test in #7 into "Postponed" until the Ubercart 7.x-3.0 branch tests have run. But we don't test fixed-point branches, so that will never happen (and in fact I tried turning on the bot for 7.x-3.0, and it failed miserably, so that's not the solution either). And there's no way to re-test #7 or kill the pending test, unless you're a d.o webmaster. I think it might work if you upload the patch in a new comment, now that the issue is set to 7.x-3.x.

longwave’s picture

alexp999’s picture

I applied the patch and can confirm that this seems to have fixed the issue!

Thanks.

longwave’s picture

Status: Needs review » Fixed

Committed #12.

Status: Fixed » Closed (fixed)

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