Closed (outdated)
Project:
Remindme
Version:
master
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
9 Jan 2006 at 20:47 UTC
Updated:
3 Aug 2020 at 15:26 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
killes@www.drop.org commentedthis module isn't yet updated for 4.7. Patches welcome.
Comment #2
Sverre commentedI'm afraid I don't have time to learn about creating and submitting patches, but this worked fine for my set up with 4.7.
I simply used the form updater at http://www.lullabot.com/formupdater to replace the old form declarations by pasting the new forms api over the top of the old.
I also changed a couple of $outputs to print or something like that!
Comment #3
Sverre commentedGiven the attachment above I've changed the status to 'code needs review'. That should probably read 'code needs accepting and testing by somebody a bit more in the know than me!' ;-)
Comment #4
killes@www.drop.org commentedThanks, Sverre, I've worked a bit more on it and committed the changes. I will not yet branch for 4.7 as some work needs to be done, mainly a .install file and some more form changes. Mainly testing of course.
Comment #5
Sverre commentedNo problems Killes, I'll look forward to the 4.7 version, and will help test where possible. This module is an excellent add on feature for any online calendar! Thank you and well done.
Just a note for anybody that is interested, I also discovered with a bit more testing that the sql query for getting and sending reminders also needed updating. I added a db_rewrite_sql() around the query and all now seems okay.
Also for any flexinode users; if you use flexinode for events like me, look for the following code at about line 35:
and change it to:
Substitute the 1 in 'flexinode-1' as appropriate.
Comment #6
killes@www.drop.org commentedI've committed a version which includes these changes. Please try it.
Comment #7
pcwick commentedremindme.module,v 1.28 2006/05/28 01:18:31 killes Exp
Good News is that there is now a RemindMe! link in the event nodes. When clicked the link leads to the RemindMe setup page where times prior to the event can be selected for reminders to be sent. This page appears to work correctly as no errors are triggered when the reminder is saved.
Unfortunately, the RemindMe! menu item triggeres this error:
Fatal error: [] operator not supported for strings in /drupal/modules/remindme/remindme.module on line 184
I am still testing to see if the reminders are actually sent by the system and will report back.
Comment #8
Timotheos commentedpcwick, a patch for that is here http://drupal.org/node/65923
Comment #9
pcwick commentedThank you Timotheos! I missed that one. The menu item now works. No error.
Comment #10
avpadernoI am closing this issue, since it's for a Drupal version that isn't supported.