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.
| Comment | File | Size | Author |
|---|---|---|---|
| #12 | 1454152-attribute-incorrect-breadcrumbs.patch | 1.36 KB | longwave |
| #7 | 1454152-attribute-incorrect-breadcrumbs.patch | 1.36 KB | longwave |
| #6 | with display price.png | 153.17 KB | alexp999 |
| #6 | without display price.png | 152.47 KB | alexp999 |
Comments
Comment #1
alexp999 commentedJust to say this only happens with the "Display Price" field as part of the view
Comment #2
longwaveAlready fixed in -dev by #1419128: Breadcrumbs littered by products in list views
Comment #3
alexp999 commentedTried that patch and its still doing it but only on some pages.
It seems to be related to having options/attributes.
Comment #4
alexp999 commentedSorry 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.
Comment #5
longwaveWhat do you mean by "only on some pages" and "it seems to be related to having options/attributes" - can you give screenshots or examples?
Comment #6
alexp999 commentedHere is a screenshot with "Product: Display Price" loaded into the views, but excluded from display:
And here is a screenshot with it removed from the View. I am using "Product: Sell Price" in its place which works fine:
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.
Comment #7
longwaveCan you test the attached patch to see if that fixes this?
Comment #8
tr commentedMoving to -dev for the testbot.
Comment #9
longwaveComment #10
longwaveComment #11
tr commentedWe'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.
Comment #12
longwaveComment #13
alexp999 commentedI applied the patch and can confirm that this seems to have fixed the issue!
Thanks.
Comment #14
longwaveCommitted #12.