Error on update from 5.x-1.x-dev to 5.x-1.0-beta
sreknob - March 21, 2008 - 14:46
| Project: | Account reminder |
| Version: | 5.x-1.0-beta |
| Component: | Miscellaneous |
| Category: | support request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Jump to:
Description
After running update.php for account reminder, I get the following warnings:
* warning: array_merge() [function.array-merge]: Argument #2 is not an array in /home/sreknob/public_html/cada/update.php on line 309.
* warning: Invalid argument supplied for foreach() in /home/sreknob/public_html/cada/update.php on line 542.The following queries were executed
account_reminder module
Update #1
* No queries
Update #2
* ALTER TABLE {account_reminder} ADD COLUMN temp_last_reminder INT(11)
* ALTER TABLE {account_reminder} CHANGE temp_last_reminder last_reminder INT(11)
Any thoughts on why this is happening, and if it's a problem?
Thanks!
#1
If you have access to your database can you run a command to show
the schema of the account_reminder table so I can see if the updates
completed on the database side despite the error in drupal?
Mysql> describe account_reminder;
#2
I also got this error when upgrading to 5.x-1.0-beta from 5.x-1.x-dev..
I ran the mysql above - but there was no result. If you have any more instructions on that I'd be happy to post.
I also notice that since I've been testing it with my own email, none of the tags are filling it on the emails that are sent out - for instance instead of name the email says %username, but that may be another issue.
#3
The update error has been addressed in CVS. It was a problem with update_1
which predates my time so I hadn't run across it myself.
The table update in update_2 should have run correctly. If possible can you
dump the account_reminder table schema just to verify?
As for the issue with the email reminders spitting out the %username instead
of the actual username that's most certainly due to updating rather than
installing the module as I changed the placeholders from % to ! which in
retrospect was maybe a bad idea...am looking into changing that back
to %.
#4
reverted back to the old placeholders (%) as I had changed them to (!).
Please download the -dev release when you see it updated and see
if you still have problems with the emails.
#5
Hi jaydub,
Thanks for the reply and updated module. I can't dump the table because I simply restored the database after the error on upgrade, so I don't have a copy of the modified table from the upgrade. I just updated using the CVS and there was no warning this time, so it appears to be fixed. Let me know if you need a dump of the table from the error, as it is easy enough for me to reproduce, but it looks like you know what it was anyway. Thanks for all your hard work, I'm glad you're maintaining this module!
#6
Thanks, on the placeholder issue (! changed back to %) - it's working now - I can see registered people are getting reminders with the right login URL.
Thanks much.
#7