Notice: Undefined property: stdClass::$data in recurly_process_push_notification() (line 482 of /dev/sites/all/modules/recurly/recurly.module).

Getting this on every new push notification. There doesnt seem to be any '$local_account->data' when doing the recurly_account_load.

CommentFileSizeAuthor
#4 undefined_property-1899350.patch1.26 KB7wonders
#1 patch_commit_d2a1d78f6fe4.patch1.45 KBstiknoltz

Comments

stiknoltz’s picture

stiknoltz commented
StatusFileSize
new patch_commit_d2a1d78f6fe4.patch1.45 KB

Oddly enough, it doesn't even look like the function itself uses the $data variable. I was able to solve by just removing it from the function call and from the function declaration itself.

It doesn't appear as though any other instance of the function call uses this variable so it appears to be a legacy thing.

[Adding a patch which may fail since I don't know how to roll them and this my first attempt]

stiknoltz’s picture

stiknoltz commented
Status: Active » Needs review
7wonders’s picture

7wonders commented

^ That was exactly what I did to kill the error too and it doesnt seem to have any other effects so it gets the thumbs up from me.

7wonders’s picture

7wonders commented
StatusFileSize
new undefined_property-1899350.patch1.26 KB

You should check out - http://drupal.org/project/recurly/git-instructions

Here is a patch that should apply cleanly.

quicksketch’s picture

quicksketch
he/him
commented
Status: Needs review » Fixed

Thanks guys, yeah the "data" attribute used to be in the recurly_account table but it was removed. I think the patch suggested here makes sense. Committed to 7.x-1.x branch. Thanks guys!

Status: Fixed » Closed (fixed)

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