Posted by jumpfightgo@gro... on November 21, 2008 at 10:11pm
Jump to:
| Project: | Google Ad Manager |
| Version: | 6.x-1.x-dev |
| Component: | Documentation |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
In the comments in google_admanager.module on line 110, the module saysThis function should be called from pianofiles_preprocess_page(&$vars) as followed
But the README.txt that comes with the module says to use...function _phptemplate_variables($hook, $vars)
The module should say "phptemplate_preprocess" instead of "pianofiles_preprocess". Suggested revision for README.txt follows:
4. Add the following code into your theme's template.php file. (If phptemplate_preprocess_page(&$vars) already exists, copy the four lines starting with "// Insert Google..." into the function.)
function phptemplate_preprocess_page(&$vars) {
// Insert Google Ad Manager scripts into header
if (module_exists('google_admanager')) {
$vars['scripts'] .= google_admanager_add_js();
}
}
Comments
#1
Thx!!
#2
Automatically closed -- issue fixed for two weeks with no activity.