Closed (fixed)
Project:
Taxonomy Views Integrator
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 Mar 2013 at 11:36 UTC
Updated:
25 Aug 2014 at 16:20 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
duaelfrLet's see if this could work.
This patch is part of the #1day1patch initiative.
Comment #2
caspervoogt commentedNot sure if I am experiencing the same issue. I'm using metatag version 7.x-beta7 and although meta tags do show up, I can't get the term name to show up in meta tags. When I disable the tvi module, the meta title (aka page title) does include the term name by default. The meta description is fine though. I checked the default Taxonomy Term view provided by Views (though I do not have this Taxonomy Term view enabled), and its meta title settings use "[view:title]" so I used the same thing in my view and it made no difference. "[current-page:title]" also does nothing.
The metatag module does not expose term-related tokens in the views meta token browser, so I don't expect tvi would need that. But somehow the metatag module is grabbing the term name and passing it into the rendered view, and tvi is stopping that from happening, I think.
Anybody else experience this?
Comment #3
caspervoogt commentedI got it working. The trick was to use a view of Content rather than a view of a Term, and then in the Contextual Filters I used a Taxonomy Term ID (with depth) as well as a depth modifier.
Comment #4
kevinquillen commentedHas anyone tested the patch in #1?
Comment #5
caspervoogt commentedI haven't, because I'm not sure I was experiencing exactly the same problem.
Comment #6
arrubiu commentedPatch #1 works for me!
Comment #7
kevinquillen commentedComment #8
davidpetit commentedI had the same issues and with this patch I confirm it's working.
Also I updated the patch to match the last dev versions of the module.
Comment #9
davidpetit commentedComment #10
alauddin commented#9 patch works.. tested with 7.x-1.0-beta4
thanks.
Comment #11
kevinquillen commentedComment #12
Scott Robertson commentedI can also confirm that the patch in #9 solved the issue for me. Once applied, the meta tags on my taxonomy term appeared in the source code.
Comment #13
giorgoskLots of people have tested this already
so its ready to be committed
(seems like a trivial change)
applies cleanly to beta4
Comment #14
giorgoskI think it was assigned to this person by mistake
so unassigned to get the maintainer's attention
Comment #16
duaelfr:)