Thank you so much for taking on this module and making it much better. The 2-way integration looks superb so far and will greatly reduce the amount of "why can't I log in" queries I get... I hope.

I found an error in the export users routine at line 219. You have:

$vbuser = db_fetch_array(drupalvb_db_query("SELECT salt FROM {user} WHERE username = '%s'", $edit['name']));
It should read
$vbuser = db_fetch_array(drupalvb_db_query("SELECT salt FROM {user} WHERE username = '%s'", $user->name));
After this change I've been able to export from drupal to vb fine - and it patched up a lot of problems; for some reason with the old drupalvb module, lots of people's email and password info in the vb DB were emptied.

Cheers

Paul.

CommentFileSizeAuthor
#2 drupalvb-DRUPAL-5--2.export.patch1.37 KBsun

Comments

mistlethrush’s picture

PS... Apologies, I omitted to say that the change is in file drupalvb.inc.php.

sun’s picture

Status: Needs review » Fixed
StatusFileSize
new1.37 KB

Thanks! That's a no-brainer, so committed attached patch.

If you've encountered other issues after upgrading to this version, I'd be happy to know (please create a new report for each issue).

Anonymous’s picture

Status: Fixed » Closed (fixed)

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