I tested fontyourface with font-squirrel fonts.

When I tried to activate Gentium Basic, I got the following error (this is also true for other fonts):

   * Warning: zip_read() expects parameter 1 to be resource, integer given in fontsquirrel_fontyourface_enable() (line 163 of /home/xxxx/public_html/sites/all/modules/fontyourface/modules/fontsquirrel/fontsquirrel.module).
    * Warning: zip_close() expects parameter 1 to be resource, integer given in fontsquirrel_fontyourface_enable() (line 177 of /home/xxxx/public_html/sites/all/modules/fontyourface/modules/fontsquirrel/fontsquirrel.module).

This stops font implementation right there.

CommentFileSizeAuthor
#5 zip.patch1.52 KBsreynen
#1 zip.patch1.03 KBsreynen

Comments

sreynen’s picture

Status: Active » Needs review
StatusFileSize
new1.03 KB

@LEternity can you try applying this patch (instructions)? I think this should fix it, but I don't have a server available where PHP is compiled with ZIP, so I can't actually test it.

Status: Needs review » Needs work

The last submitted patch, zip.patch, failed testing.

sreynen’s picture

Looks like tests are broken in D7. I created a separate issue for that, but please still test the patch.

LEternity’s picture

The patch gets rid of the error, but font squirrel fonts don't work for some reason. The fonts don't get downloaded to the server directory as in the Drupal 6 version.

sreynen’s picture

Version: 7.x-1.0-beta1 » 7.x-1.x-dev
Status: Needs work » Needs review
StatusFileSize
new1.52 KB

Okay, I was able to try this on a server with zip and track down the problem. This patch should fix it. Please test it.

sreynen’s picture

Status: Needs review » Fixed

I'm going ahead and marking this as fixed. It should be in the beta2 release in a few minutes. Please reopen this issue if that release still doesn't work for you.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.