Pre-populate name/email fields on Add new subscription form; See: websharks/comment-mail#204

[kristineds]

Pre-populate name/email fields on Add new subscription form;

See: wpsharks/comment-mail#204

[kristineds]

@jaswsinc @raamdev I'm submitting the PR for this with the code you provided but it's not pre-populating the fields for the last name and email fields?

[jaswrks]

@kristineds Were you logged into the site whenever you tested this?

[kristineds]

Were you logged into the site whenever you tested this?

@jaswsinc When I was logged-in as a user, only the "First name" is pre-populated.

This works perfectly, i.e. email and first name fields are pre-populated, when a cookie exists from previously leaving a comment.

[jaswrks]

When I was logged-in as a user, only the "First name" is pre-populated

And I assume that your account in WordPress had both the First and Last name filled? In other words, do you feel there is a bug in that specific scenario? Or was the Last name missing from the form because it was missing in your account profile in WordPress?

[jaswrks]

This works perfectly, i.e. email and first name fields are pre-populated, when a cookie exists from previously leaving a comment.

Cool!

[raamdev]

@kristineds Can we get an update here?

[kristineds]

@jaswsinc @raamdev I tested this again. See below for my observations. What are the possible causes for this?

• Signed up and registered for a new user account: http://kristine.websharks-inc.net/wp-login.php (Site has no other plugins enabled except Comment Mail Pro)
• Logged in with credentials and edited profile, added first name and last name

• Went back to a post and selected "you can subscribe without commenting"

• Got redirected to the Add New Subscription Form. First name is pre-populated but not that Email and Last name fields.

[jaswrks]

@kristineds My last commit/tweak should correct this behavior. Working good on my side. If you can test it and confirm I'd appreciate it. Also, if it works for you, can you apply the same to the lite PR?

[kristineds]

@jaswsinc Got the last name to show up but not the email. That should be filled since email is a required field, isn't it?

I'll be submitting the one for the Lite PR, so it's easier to track the changes for both

[jaswrks]

@kristineds Try changing $current->email to $current->user_email.

[kristineds]

@jaswsinc That works. Thanks! 👍 This is ready for review. @raamdev

[raamdev]

@kristineds Reviewed, tested, and merged. Nice work! Thank you.