Create a module to capture the migration preferences of current d.o CVS account holders

Are the anonymized email addresses based on values from {versioncontrol_accounts}? For example would the rough idea for the initial data population be:

INSERT INTO {versioncontrol_migration} (`uid`, `vcs_username`, `mail`)
SELECT `uid`, `vcs_username`, CONCAT(`vcs_username`, '@', `uid`, '.no-reply.drupal.org') AS mail
FROM {versioncontrol_accounts}

Would this be a D6 module?

Drupal.org runs D6, so yes.

Assuming I'm on the right track with the query above, I should have some code ready for testing by tomorrow afternoon.

{versioncontrol_accounts} is probably going to disappear(see #983926: Remove account class), but probably that data also is going to live at versioncontrol_account_status.

Anyway, the data should be written on an independent module and naturally you can not assume that it lives there(versioncontrol is not on d.o now).

The data about accounts is now at {cvs_accounts} table (see cvslog project). And the PK in that table is the "cvs_user" field, so, @sdboyer it seems like it is tracking the many accounts per user :-)

Thank you for the info, Marco!

I've posted a preliminary version at: https://github.com/webbj74/cvsmigration

[removed verbose status info --JW]

So far it appears to be operating as desired. I should have the Multiple Email Address functionality in time for the stand-up tomorrow afternoon.

This module is ready for review: https://github.com/webbj74/cvsmigration

What it does presently:

• When enabled it goes through {cvs_accounts} and associates anonymized cvs_user@uid.no-reply.drupal.org email addresses with all of the cvs_user entries that don't already have a repository mail associated with them.
• Adds "Repository Email" select item to the cvs_user_edit_form; options include:
• User's d.o email address
• List of all repository emails for the user (i.e. allows the 'agentken' to 'agentrickard' situation).
• List of all emails for the user from Multiple Emails Module (if enabled).
• Anonymized cvs_user@uid.no-reply.drupal.org email (if it isn't already listed)
• Drush command to download a flat-file
• Command: drush cvsmigration-export [filepath]
• Example CSV Output:
  

  "cvs_username1","example1@999.no-reply.drupal.org"
  "cvs_username2","example2@999.no-reply.drupal.org"
  

Tried to upload an example of the csv file output, but d.o is not allowing it.

Name the file blah.txt and it should allow it. Shouldn't the output include the uid?

@josh the geek technically it does in the fake email address. I don't see how discreetly having it would help me.

@neclimdul OK.
The code looks good, but I don't have a CVS repo to test it with.

Thanks for the feedback. After I pull in your changes, I'll update the .install to also perform the change based on Drupal user name (since that is where the emails are created en masse).

I may modify the set of invalid email username characters to be a little more conservative "just in case" (right now invalid chars are converted to underscores). I'm just a little concerned about the liberalness of the characters allowed in Drupal usernames (such as apostrophes) and whether they translate into actually valid email addresses. I currently use the same regular expression as Drupal's valid_email_address, but this expression isn't a perfect match to RFC 2822.

Also I think it makes the most sense to add $user['full-name'] in the Drush stage. So I will make that change as well (and I will include the uid to the output for good measure).

Updated: https://github.com/webbj74/cvsmigration to include sdboyer's changes, and some minor mods.

The drush output has 4 fields: "d.o uid","d.o Full Name","CVS Username","Email" Example (q.v. attached zip):

"1","Jonathan W. Webb","jonathan","developer2011+cvsmigration@jwwebb.info"
"21","Test\", \"CSV Injection","testuser","trasiwitucl@21.no-reply.drupal.org"
"24","shuswucl","shuswucl","shuswucl@24.no-reply.drupal.org"

In the event that the full name field is not populated, drush will fall back to $user->name. I use addslashes() to clean the "Full Name" field.

I also reduced the set of valid email address characters to [a-zA-Z0-9_\-\+]+. If $user->name has chars outside of this set, they are converted to underscores.

You should probably reduce the full name to a-zA-Z0-9_-+ too. The slashes really mess up quicklook on the mac. I'll fork your repo and fix this. My GitHub username is JoshTheGeek.

See my pull request at https://github.com/webbj74/cvsmigration/pull/2 . It also allows spaces.

I merged this change to the drush output, but modified the regular expression to strip just control chars and quotes (rather than transform them to underscores). Thanks!

Tagging for Git Sprint 7 with the intention of having this poised for launch right at the beginning of Sprint 8.

I will create a project on d.o and upload it to CVS.

Project CVS Migration Prefs has been added. I imported the project history from github using git-cvsexportcommit (not perfect, but was able to maintain credits for the contribs by sdboyer & joshthegeek). Dev snapshot is pending. Thanks for the assistance everyone!

dev snapshot avalaible :-) http://ftp.drupal.org/files/projects/cvsmigration-6.x-1.x-dev.tar.gz

Shouldn't we set this to fixed after it's launched on d.o?

The module is created and hosted on d.o. which satisfies the issue. When and how it is deployed will be accounted for in a different issue as part of the specific deployment planning process.