Actually, I’ve just look at that Woo Framework code again, and it is worse than I thought, and I can see no workaround.
What the woo_google_webfonts() function does is this:
1. It loops through all the theme options and picks out any option with a “face” element. These will be the fonts selected in the theme, and will be selected in no particular order.
2. For each “face” found, the function will loop over every single Google font that it knows about. These will be the fonts supplied by this plugin.
3. If the “face” name matches a Google font in the loop AND the font name does not already appear somewhere in the header font link, then it will be added to the font list in the header.
Note that even if the font does match, and has not already been used, the loop will continue over all remaining Google fonts, checking each in turn (even though we know none will match).
So the order in which the fonts are added to the page head will depend on the order in which they happen to appear in the theme options. It makes no difference what order the Google fonts are listed or supplied.
The worst bit about this is the long loops within loops; it is inefficient and a waste of time. It should collect the list of fonts used in the theme together first – that is one loop, Duplicates can be discarded at this stage.
Then this list of fonts can be searched for in the Google Font list. That is a second and final loop. The fonts, as they are found and added to the head, would be ticked off the list so that none are missed out.
So in conclusion: the original issue cannot be fixed in the plugin. It needs to be fixed in the Woo Framework, specifically the woo_google_webfonts function. That function *could* be overwridden in the plugin, but I would like to avoid going down that route.
If you are interested, this fixes the framework:-
Replace this in woo_google_webfonts()
foreach ( $woo_options as $option ) {
// Check if option has "face" in array
if ( is_array($option) && isset($option['face']) ) {
// Go through the google font array
foreach ($google_fonts as $font) {
// Check if the google font name exists in the current "face" option
if ( $option['face'] == $font['name'] AND !strstr($fonts, $font['name']) ) {
// Add google font to output
$fonts .= $font['name'].$font['variant']."|";
} // End If Statement
} // End Foreach Loop
} // End If Statement
} // End Foreach Loop
with this new code:
$fonts_used = array();
foreach ( $woo_options as $option ) {
if ( is_array($option) && isset($option['face']) ) $fonts_used[$option['face']] = $option['face'];
}
foreach ($google_fonts as $font) {
if (isset($fonts_used[$font['name']])) $fonts .= $font['name'].$font['variant']."|";
}
That gives you Cabin *and* Cabin Condensed.
