Closed (fixed)
Project:
Google AdSense integration
Version:
6.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
11 Nov 2008 at 13:13 UTC
Updated:
9 Feb 2009 at 20:50 UTC
Jump to comment: Most recent
Comments
Comment #1
kbahey commentedThe module assumes that clean urls are enabled.
Can you enable it on your site and test again?
Comment #2
le_top commentedThanks for looking into this ;-)
I can not enable it on my site - htaccess does not allow it (= the provider) and other methods have proven not to be effective.
I supposed that your module supposes that Clean URLs are available, but IMHO it should work regardless.
Comment #3
DonWMason commentedAny help would be appreciated on this issue for those of us running IIS. We cannot use Clean URLs.
Comment #4
robbes commentedI made a fix for this:
Edit modules/adsense/cse/adsense_cse.module
replace: define('ADSENSE_CSE_RESULTS_PATH', 'adsense/cse');
With: define('ADSENSE_CSE_RESULTS_PATH', url('adsense/cse'));
Search for:
Add underneath:
You can see it at work at my drupal: www.excelhet.nl
Comment #5
jcnventuraJust committed some code that fixes this.
Comment #6
robbes commentedHmm, looks like i left something out:
Search for:
Add underneath:
Comment #7
jcnventuraYou're still leaving something out...
You have to insert the code between
<code></ code>tags, or else it will be filtered out by Drupal.org's input filters.Anyway, I have already committed some code which I tested, so I would ask you to try the dev that will get built in 3-4 hours from now.
João
Comment #9
ElSanto commentedI could help I tried everything but we still do not work for me, on this subject(#4) say you must change the module define('ADSENSE_CSE_RESULTS_PATH', url('adsense/cse')); , but still did not work for me, could you tell me step by step, as set up in adsense
still continues to include adsense cse will be part of the keywords
http://www.mywebsite.com/?q=adsense%2Fcse&cx=partner-pub..............
Comment #10
jcnventuraThe suggestion in #4 wasn't the only thing used in the patch that fixed this (http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/adsense/cse...).
I would recommend that you update your site to the latest dev version and see if it starts working..
João
Comment #11
le_top commentedHI
I just installed the 6.x-1.0 release and the problem is still there.
The search box is added to http://bt747.free.fr/content/?q=contact again.
Comment #12
ElSanto commentedI installed the 6.x-1.0 release and have the same problem, still continues to include adsense cse will be part of the keywords
Comment #13
jcnventuraHi,
There's no way around the fact that Google will include the path to the search results page when clean URLs are disabled.
The only solution is to use a path which can't be mistaken for a keyword. I have committed some code to CVS which uses the '.' path instead of 'adsense/cse'. It's simple enough that it can't be mistaken for a keyword and shouldn't affect search results, and it will make the AdSense for search start working in your case.
Of course the real solution would be one of the following:
1. That you use clean URLs (which I am sure you would, if you could), or
2. That Drupal changes the internal PHP query to the index.php file from 'q' to something else other than 'as_q', of course (the words snowball and hell come to mind).
João