When I have 'name' disabled for the signup (only collecting the email) and the user tries to sign up, they get a warning that the index 'Name' does not exist. The patch in the upcoming comment rectifies that by checking if Name is not empty and if it is not, then use that; otherwise, use the email address.

CommentFileSizeAuthor
#2 signup-1212392-2.patch1.27 KBsgabe
#1 signup_anonymous_form_name_fix_1212392_btmash_1.patch650 bytesbtmash

Comments

btmash’s picture

btmash commented
Status: Active » Needs review
StatusFileSize
new signup_anonymous_form_name_fix_1212392_btmash_1.patch650 bytes

Here is the patch.

sgabe’s picture

sgabe commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new signup-1212392-2.patch1.27 KB

We should remove the unnecessary comments, besides that is fine. I am attaching a revised patch.

sgabe’s picture

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

Committed, thanks!

Status: Fixed » Closed (fixed)

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