Closed (fixed)
Project:
OpenX
Version:
7.x-1.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
10 Mar 2009 at 16:20 UTC
Updated:
23 May 2014 at 13:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
gordon commentedThis has been implemented in the latest updates.
Comment #3
danieltnbaker commentedThe last update did not completely fix this issue.
It does not take into account multiple terms associated with the node.
Also tokens do not work for the site - variable 'name' field.
Both I think are pretty useful features which would benefit the module?
Comment #4
gcybill commentedConsidering that OpenX is only a module to show ads from OpenX, this issue could be worked on both OpenX and OpenX Manager module.
It would be great if the OpenX Manager could export drupal data to the ads server as variable (for example: "d_taxonomy_
<taxo_name>"), while OpenX filter ads shown based on the variable.Comment #5
mropanen commentedThere is a bug in at least 7.x-1.2: in openx.inc, around lines 40-44 (I think, I fiddled with the file a bit) there's
which I think should be setting
$data['node'] = $node;instead.Comment #6
andy.s.clark commentedConfirmed and created D7 patch for fix in comment #5.
Comment #7
jm.federico commentedReaaally need this.
Tested and works!
Also, changing title to reflect what patch does.
Cheers
Comment #8
mavimo commentedPatch to D7
Comment #9
mavimo commentedComment #11
wuinfo - bill wu commentedRepeat #9
Comment #13
wuinfo - bill wu commentedCommited and pushed