Make install instructions consistent

jumpfightgo@gro... - November 21, 2008 - 22:11
Project:Google Ad Manager
Version:6.x-1.x-dev
Component:Documentation
Category:task
Priority:normal
Assigned:Unassigned
Status:closed
Description

In the comments in google_admanager.module on line 110, the module says
This 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();
  }

}

#1

toemaz - November 21, 2008 - 23:36
Status:active» fixed

Thx!!

#2

System Message - December 6, 2008 - 00:15
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.