We are going to use this module in a commerce site.

Test it with place order and checkout, results look great!

However, the email field for this user is empty; when we edit user profile, and input email, save it. there will be error message:

Your current password is missing or incorrect; it's required to change the E-mail address.

So after a qq login user place an order and checkout successfully, no notify email will be sent to this user. On the other hand, almost all the b2b sites are supposed to send an email to the user after he/she has successfully place an order.

I think this is reasonable feature request and thanks for your attention.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

alfababy’s picture

Hi zterry95,

Thanks for your feature request. Looks I can't implement your request now. Because:
1) I can't get user's email address.
2) I can't get user's password.
So I only can give a random password, and no email.

But I can give you a suggestion.
1) You can create a custom form to change email address.
2) Validate the user whether login from third party and email is empty.
3) Use user_save() to save it.

I will think how to set email when user are loginning. And required the email.

miloyz’s picture

Hi Guys,

I don't think implement our own firm is the right way to do. Is it possible to take out the password input field in profile edit form by detecting the user type (QQ user for example)?
I suggest we can look into how other advanced module handle this feature, like facebook or twitter integration module, do they allow use access profile editing page?
Please try not to reinvent the wheel. -_^

DYdave’s picture

Thanks a lot guys for following up on this.
This is a feature that we need badly, since currently the module is not really exploitable with E-Commerce websites for which the email address is particularly important.

Could you perhaps try investigating further how other modules do it?
A few posts I could find:

This doesn't isn't necessarily very easy or straight forward, so I guess there should probably be more discussions and investigations.

@alfababy:
Concretelly speaking, I think we should try to really test all these other third party authentication modules and see if/how they could achieve this feature.

Any further comments, feedback, questions, issues, objections, recommendations would surely be highly appreciated.
Thanks very much to all in advance.

DYdave’s picture

Priority: Normal » Major

Any update on this issue?

It would certainly be great to be able to capture more information from users, especially the email address which is really a critical blocker if we would like to use this module on E-Com production sites......

Bumping this to Major.

Any further comments, feedback, questions, issues, objections, recommendations would surely be highly appreciated.
Thanks very much to all in advance.

alfababy’s picture

Looks this is a major problem now.

I will give a good solution on this soon.
Let me research some similar modules first.

Thanks.

skyredwang’s picture

Issue summary: View changes

A lot of OAuth providers don't provide User email data, for example Twitter and Weibo.

This is an example response from Weibo User request:

{
	weibo=null, 
	id=1803876591, 
	name='loopa', 
	screenName='loopa', 
	location='北京 海淀区', 
	description='2fBAcRG0]8OGRvp', 
	profileImageUrl='http://tp4.sinaimg.cn/1803876591/50/0', 
	url='', 
	isProtected=false, 
	followersCount=23, 
	statusCreatedAt=Mon Sep 27 13:50:14 CST 2010, 
	statusId=2847344825, 
	statusText='uhYFQJ[', 
	statusSource='<a href="" rel="nofollow">微博开放平台接口</a>', 
	statusTruncated=false, 
	statusInReplyToStatusId=0, 
	statusInReplyToUserId=0, 
	statusFavorited=false, 
	statusInReplyToScreenName='', 
	profileBackgroundColor='', 
	profileTextColor='', 
	profileLinkColor='', 
	profileSidebarFillColor='', 
	profileSidebarBorderColor='', 
	friendsCount=1, 
	createdAt=Fri Aug 27 00:00:00 CST 2010, 
	favouritesCount=0, 
	utcOffset=-1, 
	timeZone='', 
	profileBackgroundImageUrl='', 
	profileBackgroundTile='', 
	following=false, 
	notificationEnabled=false, 
	statusesCount=8509, 
	geoEnabled=false, 
	verified=false
}

For this module, after a successful user login, the callback URL should be a new page that requires user entering email or additional fields required.

edxxu’s picture

As @skyredwang mentioned, not all OAuth providers provide email data, so i write this patch to allow user set email after login via CSNA link.

Please review this patch, thx.

edxxu’s picture

FileSize
4.26 KB

Sorry, the patch in #7 is wrong, here is the patch.

skyredwang’s picture

Status: Active » Needs work
+++ b/csna.module
@@ -275,3 +288,12 @@ function theme_csna_providers($vars) {
\ No newline at end of file

Need a newline here

edxxu’s picture

FileSize
4.23 KB

Here is the new patch, please review.

skyredwang’s picture

Status: Needs work » Reviewed & tested by the community
rogical’s picture

committed, thanks for all.

  • edxxu authored 654ca26 on 7.x-1.x
    Issue #1948808 by edxxu: allow user to set email address
    
rogical’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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