Improve the subscription interface for forums

My module is currently only for D5.

I have built it to be basically the same as Subscriptions UI. The assumption is that someone will use either Subscriptions UI (SUI) or Subscriptions Simple UI (SSUI) (my module). I have tried to make my module distinct enough that theoretically it could be used with another UI module, but I have not written that in specifically.

Because I wanted to allow for someone to use EITHER SUI or SSUI, I made a small change to the Subscriptions module so that if you enabled the SSUI module you do not get the warning about the dependencies.

The changes I made are:
In subscriptions.admin.inc -> subscriptions_settings_form() I made the following change...

// From this...
if (variable_get('subscriptions_show_install_info', 1) && (
        !module_exists('subscriptions_ui') ||
        !module_exists('subscriptions_mail') ||
        !module_exists('subscriptions_content') ||
        !module_exists('subscriptions_taxonomy') ||
        !module_exists('html_to_text')))

// To this...
if (variable_get('subscriptions_show_install_info', 1) && (
	(!module_exists('subscriptions_ui') && !module_exists('subscriptions_simple_ui')) ||
        !module_exists('subscriptions_mail') ||
        !module_exists('subscriptions_content') ||
        !module_exists('subscriptions_taxonomy') ||
        !module_exists('html_to_text')))

To reflect this change in the print out, I also made the following change:
In subscriptions.install.inc -> _subscriptions_install_information() I made the following change...

// From this...
drupal_set_message($t('Note: for standard Subscriptions functionality you need to enable the following modules:')
                     .'<ul><li>'. $tr('Subscriptions UI')
                     .'</li><li>'. $tr('Subscriptions Mail ')
                       .'<ul><li>'. $t('!mail_edit (required)', array('!mail_edit' => l($tr('Mail Editor'), 'http://drupal.org/project/mail_edit', array('target' => '_blank')))) .'</li>'
                       .'<li>'. $t('!HTML_to_text (strongly recommended)', array('!HTML_to_text' => l($tr('HTML to text'), 'http://drupal.org/project/html_to_text', array('target' => '_blank')))) .'</li></ul>'
                     .'</li><li>'. $tr('Content Subscriptions')
                     .'</li><li>'. $tr('Taxonomy Subscriptions')
                     .'</li></ul>');

// To this...
drupal_set_message($t('Note: for standard Subscriptions functionality you need to enable the following modules:')
                     .'<ul><li>'. $tr('Subscriptions UI'). " or ". $tr('Subscriptions Simple UI')
                     .'</li><li>'. $tr('Subscriptions Mail ')
                       .'<ul><li>'. $t('!mail_edit (required)', array('!mail_edit' => l($tr('Mail Editor'), 'http://drupal.org/project/mail_edit', array('target' => '_blank')))) .'</li>'
                       .'<li>'. $t('!HTML_to_text (strongly recommended)', array('!HTML_to_text' => l($tr('HTML to text'), 'http://drupal.org/project/html_to_text', array('target' => '_blank')))) .'</li></ul>'
                     .'</li><li>'. $tr('Content Subscriptions')
                     .'</li><li>'. $tr('Taxonomy Subscriptions')
                     .'</li></ul>');

At this point you can enable either SUI or SSUI and you are good to go without getting any user warnings...

There is one more minor issue in the Advanced Forum (AF) module that needs to be addressed in order for SSUI to work with AF. I have laid out in that thread how to solve that problem...

I think that is it. I am attaching my current version of the module. If you have comments or suggestions for improving the module please let me know...

Cheers...

Don't use the module or method described in #7... I am changing some things and improving the modules integration with Subscriptions so that you can choose which UI you want to use depending on Content Type.

More too come soon...

I have written a new version of my D5 Subscriptions Simple UI module.

My module no longer requires any changes to the Subscriptions module. It currently depends on a minor issue in Advanced Forum if you are using that module, but in that issue I have described how to solve the problem.

From the previous version I have added the following:
- Now Subscriptions UI (SUI) and Subscriptions Simple UI (SSUI) will work side by side. However, salvis has not exposed an override hook yet, so both subscription methods will show up for all of the Subscriptions Simple UI configured content types unless Subscriptions UI is disabled.
- SSUI will work fine without SUI enabled (turn off the warning at the top of 'admin/settings/subscriptions').
- SSUI now requires you to select the content types that you would like to use the SSUI interface with. This is selected in the 'Subscriptions Simple UI - Display Settings' at 'admin/settings/subscriptions'.
- With SSUI, it is now impossible to subscribe to pieces of content that comments are not enabled for.

I think that is it for now. Let me know what you guys think...

Cheers...