I just found this module. I'm not sure yet if I think it's evil you forked the update_status code like this or not. ;) Either way, it would have been nice for you to participate over at #222924: Availability of modules for newer versions of Drupal somehow.

I'm also not sure what I think about the upgrade_status_moved_into_core() stuff. Seems weird to have to manually maintain a list in your module code, but I'm not sure how else that would work. I guess what you've got will work, so long as it's kept current. Not sure how you're planning to keep it current...

Anyway, you missed (at least) one. ;) http://drupal.org/project/user_status moved into core via #144859: add settings to notify users when account is active, blocked or deleted (move user_status.module into core)

Cheers,
-Derek

CommentFileSizeAuthor
#4 268578-upgrade_status-missing_modules.patch2.25 KBdww
#1 upgrade_status-DRUPAL-5.patch1.87 KBsun

Comments

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Title: upgrade_status_moved_into_core() missing case for 'user_status' » Missing core modules
Status: Active » Needs review
StatusFileSize
new upgrade_status-DRUPAL-5.patch1.87 KB

Heh... it actually has a dependency on update_status, since I forked only the bare minimum of functions. :)

Well, I suspect that other missing modules that have been moved into core will be reported equally.

By looking over the list of modules in D6's modules folder it seems that there are even more missing yet:

- actions
- openid
- user_status

Should there be a note for D5 users of user_status?

dww’s picture

dww
we/he/they
commented

Glad it's only a bare-minimum fork, at least... ;)

Re: user_status: I guess there could be a note that there's no DB upgrade path, so you'll have to re-enter your email message template subjects and bodies in the new core settings. You probably want me to write that text, don't you? ;)

Thanks,
-Derek

sun’s picture

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

oww... that would be great! :)

dww’s picture

dww
we/he/they
commented
Assigned: Unassigned » dww
Status: Needs work » Needs review
StatusFileSize
new 268578-upgrade_status-missing_modules.patch2.25 KB

How's this?

(edit: yay, #267884: Attachments failing when users preview or use "attach" button strikes again) :(

sun’s picture

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

Thanks, committed!

meh... I shouldn't create a new release containing only this change, or should I? ;)

dww’s picture

dww
we/he/they
commented

No, please don't. ;)

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

Well, if you find more (like me), please add them to #270531: Missing core modules #2 ;)

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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

isthisthingon’s picture

isthisthingon commented
Category: bug » support
Status: Closed (fixed) » Active

I downloaded the user_status 5.x-1.2 module, un-zipped it and put the user_status folder in the same modules folder that contains all the other modules I use for my site. I go into administer the module, but I don't see it listed. Any help in this regard?

Thanks for any help in getting this working.

Mike

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Status: Active » Closed (fixed)

Please do not hi-jack issues. This issue is about Upgrade Status module. You should post a new issue for User Status project, but please search the issue queue for existing issues first.