I updated to 7.x-1.4 yesterday and now I'm getting a number of errors during cron. I updated to 7.x-1.x-dev to see if it resolved the issue, but it did not. This is what I am seeing:
Notice: Trying to get property of non-object in registration_cron() (line 1156 of /site_root/sites/all/modules/registration/registration.module).
Notice: Trying to get property of non-object in registration_cron() (line 1154 of /site_root/sites/all/modules/registration/registration.module).
Notice: Undefined variable: result in registration_cron() (line 1154 of /site_root/sites/all/modules/registration/registration.module).
Notice: Trying to get property of non-object in registration_cron() (line 1154 of /site_root/sites/all/modules/registration/registration.module).
These errors repeat for every registration that is in the held state. I am using registration types that were created before the update. Is there a problem with existing registrations and the new held state?
Comment | File | Size | Author |
---|---|---|---|
#5 | registration-expiry-2476647-5.patch | 1.08 KB | LonitaD |
Comments
Comment #1
caxy4 CreditAttribution: caxy4 commentedDid you run update.php after updating the Registration module to 1.4?
Comment #2
LonitaD CreditAttribution: LonitaD commentedYes, I ran update.php. I didn't see any problems during the update.
Comment #3
thatoneguy CreditAttribution: thatoneguy as a volunteer commentedI ran into the same issue when I tested this myself. Here is a patch that fixes the problem.
Comment #4
thatoneguy CreditAttribution: thatoneguy as a volunteer commentedScratch that last patch. I missed a line.
Comment #5
LonitaD CreditAttribution: LonitaD commentedThe patch in #4 fixes the cron errors. However, it cancels registrations that have the held state if the hold expiration is set to 0, which the description says disables the expiration behavior. I created a new patch which first checks if the expiration is set to 0 before changing the state to canceled.
Comment #6
AaronBaumanPatch looks good to me.
1 vote for RTBC
Comment #8
caxy4 CreditAttribution: caxy4 commentedCommitted - 5c2a03e