The attached patch adds the functionality to d7 to target adverts based on taxonomy term.

If you are on a taxonomy term page (taxonomy/term/%) the patch will expose the term if applicable to google admanager
If you are on a node view page (node/%) the patch will expose all applicable terms to google admanager

This patch uses the vocabulary selects already pre-existing in the configuration form.

Please review and let me know your thoughts.

CommentFileSizeAuthor
google_admanager_taxonomy.patch2.28 KBDarren Shelley

Comments

jcisio’s picture

I haven't tested, but patch looks good.

wemmies’s picture

Tested it and it seems to work. Haven't jumped in the code as I'm still no good at Drupal coding. Bottom line is that after the patch I could target Taxonomy Vocabularies (in depth) very nice.

It took me some time to figure out how to set up DFP, as my Vocubulary name is Location, but the term to be used in DFP is v-location. So I'm guessing all vocabs shoud have v- in front of their name in DFP.

Nice work even if it is over a year old, it made my day.

jcisio’s picture

Status: Needs review » Fixed

Thanks for the review. Committed because of #1 and #2.

Status: Fixed » Closed (fixed)

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