For new users after registration with number. Function user_save remove namber of the user when it login.

Please fix it:
replace
user_save($account, array('sms_user' => $data), 'mobile');
to
sms_user_save($account, array('sms_user' => $data), 'mobile');

CommentFileSizeAuthor
#4 smsframework.patch1.14 KBWiseMike
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

WiseMike’s picture

In version 6.x-2.x-dev it's too.

univate’s picture

Status: Active » Postponed (maintainer needs more info)

Not sure I understand the bug

Could you do one of the following:
1) provide a patch
2) say what line needs changing
3) describe the steps to show how to repeat the problem

oscardax’s picture

I´m experiencing a similar problem. User mobile number will get deleted from sms_user just after code validation, if number provided does not include the country code. (e.g. a number from Spain 0034650123456 or 34650123456 will be fine, but a number 650123456 will get deleted).

What´s curious is that the validation code is correctly sent to the mobile, but as the number will get deleted, that user will receive no more SMS.

Steps to reproduce the error:
Provide a mobile number in the form 650123456, submit form, introduce de validation code (you can read the code from database table sms_user, column code), submit form -> the number is deleted from the database.
I mean the sms_user row for that user is set correctly to status = 2 (number validation successful), but the number field is set to 0.

WiseMike’s picture

Status: Postponed (maintainer needs more info) » Active
FileSize
1.14 KB

attached patch

univate’s picture

Status: Active » Needs review

since there is a patch, you should set to needs review

dpi’s picture

Issue summary: View changes
Status: Needs review » Closed (outdated)
dpi’s picture