Closed (fixed)
Project:
@font-your-face
Version:
7.x-2.x-dev
Component:
Fonts.com (provider)
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Jan 2012 at 11:00 UTC
Updated:
24 Dec 2012 at 19:15 UTC
If an Bug on Fonts.com module of font your Face.
Fatal error: Unsupported operand types in ..../sites/all/modules/fontyourface/modules/fonts_com/fonts_com.module on line 1372
This error comes to select ..../admin/appearance/fontyourface
Greets Lennard
Comments
Comment #1
Lennard commentedComment #2
sreynen commentedThis looks like it's probably caused by a failure to connect to the Fonts.com API for some reason, so it's probably not happening reliably. I just made a change to the 7.x-2.x-dev version to prevent this error even when the Fonts.com API is down, so if you are still seeing it, please try the dev version and see if that fixes it for you.
Also, please read and use the Drupal.org Priority guidelines. This is the second issue you've marked as critical, apparently because it's critical for you personally. The priority field is for priority for the module, not for individual people. This falls under the definition of normal priority: "Bugs that affect one piece of functionality are normal priority."
Comment #3
Lennard commentedSorry for used that as critical. Thank for your Informations.
Comment #4
johnpitcairn commentedI am also getting this error when attempting to access /admin/appearance/fontyourface/browse/fonts_com. I'm using 7.x-2.0. There is also an error message:
"There was an error getting filter choices from Fonts.com."
The fonts.com API is not down, I can include their stylesheets in template.php.
Comment #5
sreynen commentedJohn Pitcairn, please try the 2.x-dev version and see if that makes the error go away.
Comment #6
johnpitcairn commentedActually I think I'll just uninstall and add fonts.com stylesheets in template.php. Apologies for the noise.
Comment #7
sreynen commentedAlright, I don't think anyone is going to review this, so I'm marking it as fixed.
Comment #9
robmc commentedHi Scott,
I did test the dev version. This issue still remains or at least one that is identical. The steps I did to cause the issue were: fresh install of latest drupal (7.18 in this instance) installed and enabled Omega, Omega Tools, context, delta, views, fontyourface and fonts_com. Then browsed to /admin/appearance/fontyourface/browse/fonts_com and got a WSOD. In the apache/php error logs I got the reported issue.
I used a debugger to track down the issue to:
I was never prompted or found a way to set fonts_com_token through the UI or during install. This results in the Header element being undefined during the drupal_http_request call. The specific issue arises in drupal_http_request while attempting to += more header options.
I'm hoping this is a quick fix for you ;-) if it's not let me know and I'll dig in and put together a patch.
Cheer,
Rob McCrea
Comment #10
robmc commentedSetting back to active.
Comment #11
Drave Robber commented@robmc: the fix is out there and needs review - please see the patch in #5 of #1857404: Point fonts.com error messages to the configuration page.
Comment #12
robmc commentedThanks! I hadn't spotted that one. I'll move to that thread.
Comment #12.0
robmc commentedGiv more Information to fix