Yep.

Requires a new database table and pulling the data out of $user->data.

See signup's signup_update_5204 for a model of how to batch something during an udpate.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

aclight’s picture

Status: Active » Needs review
FileSize
9.25 KB

Here's a patch for the D5 version of comment_notify. I don't have many users in this test site that have any comment_notify data for them in {users}.data, but for the few users I do have this update worked. I have only tested on mysql, not pgsql. I named the update function here 5201, based on my guess that #317640: Remove "notify" column from comments table will probably be committed first, but feel free to rename if necessary.

greggles’s picture

Updated version for 5.x

1) Fixes a parse error in the install
2) Fixes a notice when loading users

greggles’s picture

Let's try that again.

greggles’s picture

Here's a 6.x version and a new 5.x version that is more likely to work than the last few ;)

greggles’s picture

Status: Needs review » Fixed

I tested the 6.x on a site with a few different values in the user->data column and everything worked as expected.

I've committed the 5.x version of the patch as well, though it got no additional testing beyond my first round...here's to beta users as testers ;)

Status: Fixed » Closed (fixed)

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