Date module version 2.0, which introduces date_api version 5.2, adds a field "timezone_name" to the users table. Because the user_suspend_users table does not have this field, both suspend and unsuspend operations do not work after date_api 5.2 is installed.

CommentFileSizeAuthor
#1 user_suspend-update-for-date_api-5.2.patch1.4 KBneopoet

Comments

neopoet’s picture

neopoet commented
Status: Active » Needs review
StatusFileSize
new user_suspend-update-for-date_api-5.2.patch1.4 KB

See patch. Code could be improved -- but it works in the meantime as a fix for those with the problem.

john.money’s picture

john.money commented

Thanks for patch. Will review this week and issue new release.

john.money’s picture

john.money commented

Finally had a chance to look at the reported incompatibility with Date 2.0 RC. Thanks for the patch!

I've decided to copy over only the default fields to the user_suspend tables (e.g. gracefully degrade from a condition I had never expected). I think its not so good practice to accommodate every (any?) module that chooses to alter the default core Drupal tables, particularly node and users. What this means relative to the Date module is that a user that is suspended and then unsuspended will lose the timezone_name data.

As stated, the next release will gracefully degrade and remain operational with Date 2.0 RC. It will also present a warning when the schema for the core tables differs from what is expected by User Suspend.

john.money’s picture

john.money commented
Version: 5.x-1.1 » 5.x-1.2
Status: Needs review » Closed (won't fix)

User Suspend 1.2 and Date 2.0 RC can now peacefully coexist.