I forgot about the anon_mail property in registration_get_properties method while working in #1664218: Deprecate autoloaded entities for Entity Metadata Wrappers. It isn't actually returning anything in rules anymore. Whoops!

Sorry about that.

CommentFileSizeAuthor
0001-Issue-1664218-by-mradcliffe-Forgot-to-return-anythin.patch947 bytesmradcliffe

Comments

babbage’s picture

babbage commented
Priority: Normal » Critical

So I just noticed when I upgraded. Hooray for a patch to test.

Upgrading to critical as this prevents a confirmation email being sent to anonymous registrants, and results in an error message displayed to them on the site, if you're using Rules to send the email.

babbage’s picture

babbage commented
Status: Needs review » Reviewed & tested by the community

Problem solved. This fix does exactly what it needs to. Marking RTBC.

I manually applied this patch, as for the particular site in question it was easier. So I haven't tested the well-formedness of the patch file per se, but the fix is bang on.

babbage’s picture

babbage commented

Problem solved. This fix does exactly what it needs to. Marking RTBC.

I manually applied this patch, as for the particular site in question it was easier. So I haven't tested the well-formedness of the patch file per se, but the fix is bang on.

bryanhidalgo’s picture

bryanhidalgo commented

Patch works for me, thanks!!!

levelos’s picture

levelos commented
Status: Reviewed & tested by the community » Fixed

Thanks @mradcliffe, got it.

Status: Fixed » Closed (fixed)

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

  • levelos committed a5588d1 on 7.x-1.x, panels, any-entity, slots, integrations, hold_state authored by mradcliffe 
    #1790810 by mradcliffe: Add anon_mail back to...