I've hit the following error trying to get font-your-face working nicely with the Google Webfont Loader API:

Fatal error: Call to undefined function _google_webfont_loader_api_load_font() in {...}/sites/all/modules/fontyourface/modules/google_fonts_api/google_fonts_api.module on line 105

Searching here, #1403100: Google fonts do not load makes reference that the dev version of the Webfont Loader API needs to be installed, but this doesn't appear to make a difference. _google_webfont_loader_api_load_font() doesn't appear to exist as a function in the 6.x-1.x-dev version of the webfont loader project either.

Am I missing something here? Is it possible to get the loader working for the 6.x branch?

Comments

btmash’s picture

Umm...I'm not sure why the 6.x branch of font-your-face has integration code with webfont loader since @sreynen and I had only talked about there being integration in the 7.x branch (from what I recall, drupal 6.x does not have the necessary hooks we need to be able to get a proper integration). I'll be able to take a look into this later this week though and give a proper confirmation.

trevorbradley’s picture

As much as I'd like to see it working in 6.x, it sounds like nixing the Webfont Loader support code in the 6.x branch is the way to go here. Thanks for the reply!

sreynen’s picture

Title: Can't get Google Webfont Loader API working » Google Webfont Loader API doesn't work with 6.x version, should not be called
Category: support » bug

Looks like BarisW included the webfont loader code when he backported a much larger change from 7.x to 6.x. He probably didn't even notice that was there. We should just remove that code from the 6.x version.

BarisW’s picture

Ah, my bad. I saw the piece of code and I recall checking if there was a D6 version of that module, but I didn't test it with that module enabled.

Drave Robber’s picture

Version: 6.x-2.10 » 6.x-2.x-dev
Component: Code (general) » Google Fonts (provider)
Status: Active » Needs review
StatusFileSize
new1.15 KB

We should just remove that code from the 6.x version.

Here's a patch to do just that.

neslee canil pinto’s picture

Issue summary: View changes
Status: Needs review » Closed (outdated)