Fonts with a space in the name don't load properly

Shouldn't _google_fonts_family_array_key_dencode() be changed as well? Otherwise this is RTBC.

Bumping to critical as this makes half of the fonts unusable.

Thanks, I'll look into it. However, you might want to switch to @font-your-face. As stated on the module page, I've decided to put my efforts in the FYF module instead of the Google Fonts module. The FYF module also supports Google Fonts, as well as many others.

Tom,

I checked your patch but this does not seem to be an issue. The module already replaces spaces with a + sign, in <code>_google_fonts_family_pathname(). If you check the call to the CSS file, you'l see that the request points to (example) http://fonts.googleapis.com/css?family=Covered+By+Your+Grace&subset=latin

Does the current code fails in your configuration? Could you please clarify what the URL is of the CSS file that is requested?

Change in google_fonts.module

line 153:
function _google_fonts_family_array_key_encode($string) {
return str_replace(' ', '_', $string);
}

to

function _google_fonts_family_array_key_encode($string) {
return str_replace(' ', '+', $string);
}

I got it to work by replacing line 85

 drupal_add_css($protocol . '://fonts.googleapis.com/css?family=' . implode('|', $chunk), array('type' => 'external', 'weight' => CSS_SYSTEM));

with

  drupal_add_css($protocol . '://fonts.googleapis.com/css?family=' . str_replace("_","+",implode('|', $chunk)), array('type' => 'external', 'weight' => CSS_SYSTEM));