Block visibility problem with RTL theme

I think if you do not fill in at least one keyword, the rule will be deleted.

You need the ability to just test for google and not check for google keywords I think. I didn't really think of that. Perhaps when I get time I can redesign.

In the meantime I think you can set the keyword to something like "[not contains] supercalifragilisticexpialidocious" to accept any other keyword. Don't use this example if your site is about Mary Poppins.

Also have you done the thing in template.php that it says in the README file ?

OK I'm not sure what's happening with the blocks it seems fine on mine, but I did notice some bad code there so I fixed it up.

I have fixed the prepending/appending on the variables title, content, and head title - which turned out to only do replacements.

I have also added permissions so you have to set those up properly to effect the correct user roles, usually anonymous/unregistered (it was annoying to the webmasters etc...)

I checked the box saying "do not display blocks" and it displays them fine - either the boxes are crossed, or it's a weird themeing issue (I have a Hebrew, RTL site).
However, I couldn't make the display work. It always disregards the tag.

I don't have a problem with blocks on mine. One of my biggest worries when making this module was compatibility with themes. Unfortunately I need some more information to solve this, possibly a screenshot of your google keyhaviour settings, and maybe I could try your theme?

Also - is it possible to use php in the text I'm prepending to the node content? Would be useful for themeing the message.

No it's not atm, perhaps in the future.

My plan to find a solution - test this module with two different themes, like Garland and GarlandRTL and compare the difference.

OK I believe this problem has nothing to do with RTL vs other themes. What it has to do with is how the original function in template.php looked where you made the change.
In many templates the 'return' statement is within another if statement, that prevents it from being called when needed.

For example in GarlandRTL

function _phptemplate_variables($hook, $vars) {
  if ($hook == 'page') {

    if ($secondary = menu_secondary_local_tasks()) {
      $output = '<span class="clear"></span>';
      $output .= "<ul class=\"tabs secondary\">\n". $secondary ."</ul>\n";
      $vars['tabs2'] = $output;
    }

    // Hook into color.module
    if (module_exists('color')) {
      _color_page_alter($vars);
    }
    return google_keyhaviour($hook, $vars); 
  }
  return array();
}

That won't work.

it should probably be

function _phptemplate_variables($hook, $vars) {
  if ($hook == 'page') {

    if ($secondary = menu_secondary_local_tasks()) {
      $output = '<span class="clear"></span>';
      $output .= "<ul class=\"tabs secondary\">\n". $secondary ."</ul>\n";
      $vars['tabs2'] = $output;
    }

    // Hook into color.module
    if (module_exists('color')) {
      _color_page_alter($vars);
    }

  }
  return google_keyhaviour($hook, $vars);
}

Going to mark as fixed, if you have similar problems please start a new issue and I'll help you work through it.