Closed (fixed)
Project:
@font-your-face
Version:
7.x-1.4
Component:
Miscellaneous
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Jul 2011 at 02:17 UTC
Updated:
31 Jul 2011 at 19:21 UTC
Hi,
these are the 2 error messages I got after upgrading to 6.x-2.7 (after update.php):
1. warning: array_merge() [function.array-merge]: Argument #2 is not an array in /home/hangar217/public_html/update.php on line 173.
2. warning: Invalid argument supplied for foreach() in /home/hangar217/public_html/update.php on line 337.
Fontyourface had this other message in the update.php results:
The following queries were executed
google_fonts_api module
Update #6200
The site seems to work ok. But anyway, are these 2 messages of any concern?
Thank you.
R.
Comments
Comment #1
sreynen commentedThat looks like something went wrong with the update code. The only thing that code does is re-import Google fonts, so if you have any trouble, just run that import again. I'll see if I can reproduce this problem. It may be a bug.
Comment #2
Québec commentedGood morning
«just run that import again» : I'm not shure to understand what you mean by this. I did another upgrade of the module and ran again update.php. Things were ok with. In the log I have found this message:
-> warning: array_pop() [function.array-pop]: The argument should be an array in /home/hangar217/public_html/update.php on line 313.
And just under the red box: «The update process was aborted prematurely while running update # in .module.»
There has been no other upgrade appart fontyourface.
Hope this help.
R.
Comment #3
Québec commentedIf this can help:
MySQL: 5.0.91
PHP: 128M
php: V. 5.2.17
Drupal 6.22 on Apache server.
Comment #4
sreynen commentedQuébec, if you're not having any problems after that error message, don't worry about it. But if you do have a problem, just go to the @font-your-face settings page (under configuration in the admin section), and click the "Import" button for Google Fonts. What happens when you click that button is exactly what the update failed to do, so that should clear up any problems with Google fonts.
Thanks for the details. I haven't had a chance to see what might be causing that error yet, but that will be very useful when I do.
Comment #5
sreynen commentedI just marked #1221148: Error while updating to 7.x.2.2 as a duplicate of this. I think between the two error message, I know what's going on. I'll try to fix this today.
Comment #6
Québec commentedHi,
I beleive that everything looks ok since all the fonts are showing like they should, where they should (I use Newswire theme).
But I did import the google fonts again and set the site back on-line.
Thanks for your time.
R.
Comment #7
sreynen commentedI just created new 7.x-1.5 and 6.x-2.8 releases that should fix this problem in both Drupal versions. The problem seemed to be a call to a function in the main .module file from the .install file when the .module file wasn't loaded, so I just added a check to force the file to load if it wasn't already.