Closed (fixed)
Project:
Google AdSense integration
Version:
5.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Feb 2008 at 22:11 UTC
Updated:
26 Feb 2008 at 20:11 UTC
Jump to comment: Most recent file
Comments
Comment #1
kbahey commentedWhich selection module did you chose? adsense_basic or revenue_sharing_basic?
If you want revenue sharing you have to select the latter.
Also, there are upgrade instructions for it at http://drupal.org/project/adsense
Make sure you read it and apply it, then see if you have issues still.
Comment #2
frank.dev commentedI chose revenue_sharing_basic.
http://drupal.org/project/adsense
"Upgrading
If your site already uses revenue sharing, you need to first write down the revenue sharing settings, before installing the 5.x-2.x version of Adsense. You then need to enable the Revenue Sharing Basic module, and click on its Settings, and enter all the settings that you wrote down earlier."
I have done exactly this. ;-)
Comment #3
kbahey commentedOK, we have a bug then.
Can you join me on IRC? Free node #drupal-dev. My nick is kbahey.
Comment #4
kbahey commentedI found the problem and fixed it.
The attached file is a modified version. Download it, rename it to .module (remove .txt), and it should work fine.
Also committed to 5.x-2.x-dev, but will take some time to make it there.
Comment #5
frank.dev commentedThank you kbahey, I have installed 5.x 2.x dev with the modified version from "revenue_sharing_basic.module".
On the first sight this also functioned revenue_sharing. But... If I activate the caching mode for the Drupal-site (Administer >> Site configuration >> Performance: Caching mode: "Normal"), it is indicated again generally the Adsense client ID of a user - revenue_sharing does not work.
All the same which setting I choose - "Percentage of node views going to author: 5 - 90" - it is always the same result. Either is indicated generally the Adsense client Id of the Site owner or the author.
If I disabled the cache, the indicated ID's alternate in the source correctly.
Comment #6
kbahey commentedThat is a separate issue.
Closing this and opening a new issue for it here http://drupal.org/node/220770.
Comment #7
frank.dev commentedHi kbahey!
Ok, I understand. Many thanks for your help - and for the new feature request.
Frank
Comment #8
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.