I have used variable replacements so that VB users are directed to Drupal login, registration, password reminder. Problem is that if they haven't logged in to Drupal yet then Drupal is not recognizing their username or email to send them a new password. Some users try to register a new username in Drupal they are told their email already exists. If replacement variable is deleted then some users may still try to request a new password from Drupal.

Any simple way around this I am missing?

Import all VB users with "VBulletin to Drupal" module then turn Drupal forum off?

[Daniel - Tausend Dank!]

Comments

fox’s picture

fox commented

Anyone have a solution for this? The only thing I can figure is to manually set their password in VB admin when they email me directly asking for help. Problem is that I know there are lots of people who give up and don't email me for help.

Docc’s picture

Docc commented
Component: Miscellaneous » Code
Category: support » bug
Status: Active » Needs review

Because its buggy :)

k here we go. Dont have time for proper patch

In drupalvb.module replace line 185 

$form['#validate'] = array_merge(array('drupalvb_user_pass_validate' => array()), $form['#validate']);

with 

$form['#validate'] = array_merge(array('drupalvb_user_pass_validate'), $form['#validate']);

And line 268

$name = $form_values['name'];

with 

$name = $form_values['values']['name'];

That should do it.

Docc’s picture

Docc commented
Title: How to handle VB users requesting password reminder » Password reminder validation does not function properly
fox’s picture

fox commented

Thanks Doc - what will this do? Pull password from VB?

Docc’s picture

Docc commented

From the function documentation

* Validation handler for the password recovery form.
*
* Create a non-existent Drupal user if one with the requested username or mail
* exists in vB. Note that the user_pass form asks the user to enter a username
* OR email address in one form field internally called 'name'.

Bssicly it creates/sync the user in drupal so the password reminder can do his thing.

fox’s picture

fox commented

Excellent - worked for me - thanks!

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Needs review » Fixed

Thanks for reporting, reviewing, and testing! Committed the changes in #2 to D6.

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

Status: Fixed » Closed (fixed)

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