This thread is intended to track the development of the Drupal 7 port of Subscriptions.

Please test the versions as they become available and provide feedback, both good and bad. Both types of feedback are equally important, but bug reports need to go to separate issues to keep this thread from falling apart.

Comments

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 » Subscriptions for D7 — ALPHA1
Version: 7.x-1.x-dev » 7.x-1.0-alpha1

Here's ALPHA1 of Subscriptions.

It has had very little testing so far and is certainly not ready for production yet.

There is no D7 version of Mail Editor yet, which means the email templates cannot be customized yet.
Please test this version and provide feedback, both good and bad:

IF it works for you, let us know by adding a note to THIS issue below.

IF YOU FIND A BUG, please check the issues queue, and if it hasn't been reported yet, then OPEN A NEW ISSUE!
Do NOT add bug reports to this issue here.

Zarevac’s picture

Zarevac commented

Ok, I am running into trouble with Cron: Fatal error: Call to undefined function kpr() in /sites/all/modules/subscriptions/subscriptions_mail.cron.inc on line 150

salvis’s picture

salvis
he/his
commented

Thanks, but — did you read the bold text?

Here's ALPHA2 of Subscriptions.

It has had very little testing so far and is certainly not ready for production yet.

Please test this version and provide feedback, both good and bad:

IF it works for you, let us know by adding a note to THIS issue below.

IF YOU FIND A BUG, please check the issues queue, and if it hasn't been reported yet, then OPEN A NEW ISSUE!
Do NOT add bug reports to this issue here.

EDIT: As a reminder: we're currently using a copy of mail.inc from D6 because drupal_html_to_text() is completely broken in D7. The core fix has been committed and will appear in Drupal 7.5, see #1130198-58: Regression: line-breaks are mangled by \Drupal\Core\Utility\Mail::htmlToText().

Zarevac’s picture

Zarevac commented
Title: Subscriptions for D7 — ALPHA1 » Subscriptions for D7 — ALPHA2
Version: 7.x-1.0-alpha1 » 7.x-1.0-alpha2

Oops my bad. Dev works fine :-))

salvis’s picture

salvis
he/his
commented

So should ALPHA2.

rogueturnip’s picture

rogueturnip commented

Just wondering if there is any update on this module moving to Alpha 3 or even a beta.

Thx!

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 — ALPHA2 » Subscriptions for D7 — ALPHA3
Version: 7.x-1.0-alpha2 » 7.x-1.0-alpha3

Here's ALPHA3 of Subscriptions.

It has had very little testing so far and is certainly not ready for production yet.

ALPHA3 has a number of important bug fixes and it uses the D7 mail system paradigm. This means you should finally be able to send HTML mails. Please let us know in a new issue if you want to try this out.

Please test this version and provide feedback, both good and bad:

IF it works for you, let us know by adding a comment to THIS issue below.

IF YOU FIND A BUG, please check the issues queue and the change log, and if it hasn't been reported/fixed yet, then OPEN A NEW ISSUE!
Do NOT add bug reports to this issue here.

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 — ALPHA3 » Subscriptions for D7 — ALPHA4
Version: 7.x-1.0-alpha3 » 7.x-1.0-alpha4

Here's ALPHA4 of Subscriptions. It requires Drupal 7.5 or later!

It has had little testing so far and is certainly not ready for production yet.

ALPHA4 fixes a sizable number of bugs and removes the SubscriptionsMailSystem, which was a stopgap measure to work around a bug in Drupal <7.5. See Creating HTML formatted emails in Drupal 7 for hints on a long-standing request. Please let us know in a new issue if you want to try this out.

Please test this version and provide feedback, both good and bad:

IF it works for you, let us know by adding a comment to THIS issue below.

IF YOU FIND A BUG, please check the issues queue and the change log, and if it hasn't been reported/fixed yet, then OPEN A NEW ISSUE!
Do NOT add bug reports to this issue here.

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 — ALPHA4 » Subscriptions for D7 — ALPHA5
Version: 7.x-1.0-alpha4 » 7.x-1.0-alpha5

Here's ALPHA5 of Subscriptions.

ALPHA5 has the following requirements:

  1. Drupal 7.5 or later,
  2. Mail Editor 7.x-1.0-alpha2 or later,
  3. Token 7.x-1.0-beta6, plus
  4. the Token patch in #1195874-20: Need to figure out how to create nested tokens from the array token type (not any later patch!)
  5. Be sure to run update.php after installing it!

The Token patch will not change existing functionality in Token, but if you're not able or not willing to patch Token, then ALPHA5 will not work for you! The patch adds the ability to handle arrays of objects (rather than just strings), which is needed for tokens like [subs:comments] and [subs:files:field_FILES]. We hope to get the patch accepted into Token, but if we can't, we'll provide the necessary functionality outside of Token. If these tokens work for you, then we'd appreciate a favorable comment in the Token issue.

Much of ALPHA5 is new code that has had little testing so far and is certainly not ready for production yet.

ALPHA5 fully embraces Drupal core's Token system. Together with Mail Editor, which is now required, ALPHA5 offers an unprecedented level of customizability for your outgoing notifications. With the help from HTML Mail, you should now be able to send HTML notifications. Please let us know in a new issue if you try this out.

ALPHA5 will attempt to upgrade your D6 templates if you're upgrading a D6 site. This cannot work perfectly, because the syntax has changed, but it's a best effort. If you try this, PLEASE let us know how it worked, in a new issue!

Digest support is not working yet.

Please test this version and provide feedback, both good and bad:

IF it works for you, let us know by adding a comment to THIS issue below.

IF YOU FIND A BUG, have a question or a feature request, please check the issues queue and the change log, and if it hasn't been reported/fixed yet, then OPEN A NEW ISSUE!

Please do NOT add bug reports / questions / feature requests to this issue here.

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 — ALPHA5 » Subscriptions for D7 — ALPHA6
Version: 7.x-1.0-alpha5 » 7.x-1.0-alpha6

Here's ALPHA6 of Subscriptions.

ALPHA6 requires a fix in the new Mail Editor 7.x-1.0-beta1 or later, but it does not need the Token patch anymore. If you've applied that patch, then remove it or simply update Token. I would have liked to contribute the typed array functionality for the benefit of all Token users, but there isn't sufficient interest and support for getting this committed in a reasonable time frame.

ALPHA6 has a number of important bug fixes, internal reorganizations, and new features. The default templates have been expanded and some tokens have changed. If you've customized your templates, then make sure that they still work as expected, and take a look at the new features of the new templates (category, forum, terms fields).

Digest support is not working yet.

Please test this version and provide feedback, both good and bad:

IF it works for you, let us know by adding a comment to THIS issue below.

IF YOU FIND A BUG, have a question or a feature request, please check the issues queue and the change log, and if it hasn't been reported/fixed yet, then OPEN A NEW ISSUE!

Please do NOT add bug reports / questions / feature requests to this issue here.

Yuri’s picture

Yuri commented

I installed the alpha6 version of Subscriptions, and Subscription emails now show their title and contents correctly.
I actually used the latest dev version which does not work (for me), showing empty emails. after reverting back to alpha6 it worked.

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 — ALPHA6 » Subscriptions for D7 — BETA1
Version: 7.x-1.0-alpha6 » 7.x-1.0-beta1

Here's BETA1 of Subscriptions.

BETA1 again requires updating Mail Editor to 7.x-1.0-beta2 or later, and it finally delivers the long-awaited Digest Mode.

Note that if you have customized your templates, you must adjust them for use with digest mode! Please read the release notes!

Besides digest mode, BETA1 has a number of important bug fixes, and we recommend updating to it.

Please test this version and provide feedback, both good and bad:

IF it works for you, let us know by adding a comment to THIS issue below.

IF YOU FIND A BUG, have a question or a feature request, please check the issues queue and the change log, and if it hasn't been reported/fixed yet, then OPEN A NEW ISSUE!

Please do NOT add bug reports / questions / feature requests to this issue here.

vegantriathlete’s picture

vegantriathlete
he/him/his
Primary language English
Location Lakewood, CO
commented

Yay for moving the admin to admin > config > system > subscriptions! Just remember that you will have to menu_rebuild() in order for it to show up if you have previously installed the module.

I have installed beta1 and will wait to see what happens when I actually update the site (I don't want to drive my members crazy with any more pseudo updates). Currently I have all members, except for one, set for single mode. One member is set for digest mode. All members are set for Daily delivery. I'd like to keep the Daily delivery option, but could set all, some, or none for Digest mode to help with testing. What scenario would you like me to set up?

salvis’s picture

salvis
he/his
commented

Please use a separate thread for discussing specific features / issues.

This here is the meta-thread that will be "fixed" when 1.0 is released.

vegantriathlete’s picture

vegantriathlete
he/him/his
Primary language English
Location Lakewood, CO
commented

Success: There was an item in the queue (for a user who is set to use Digest Mode with a Daily interval) that appears to have been sent correctly.

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 — BETA1 » Subscriptions for D7 — BETA2
Version: 7.x-1.0-beta1 » 7.x-1.0-beta2

Here's BETA2 of Subscriptions. It fixes all active known issues.

Please see the BETA1 release notes if you are upgrading from an ALPHA version.

Please test this version and provide feedback, both good and bad:

IF it works for you, let us know by adding a comment to THIS issue below.

IF YOU FIND A BUG, have a question or a feature request, please check the issues queue and the change log, and if it hasn't been reported/fixed yet, then OPEN A NEW ISSUE!

Please do NOT add bug reports / questions / feature requests to this issue here.

Anonymous’s picture

Anonymous (not verified) commented

I just downloaded, installed and started testing BETA2 over the weekend and it is working fairly well. THANK YOU!!! This is my first time using Subscriptions so I am learning how to use it.

One comment I have is that I was hoping for more control over what Content Types a User can choose from. For example, not all users have access to the site's forum but all users can choose to subscribe to forum posts. There is other content types taht no all users can access but those also are listed as choices to subscribe to. Looks like Subscriptions is not using the site's permissions when displaying subscription choices to the user.

salvis’s picture

salvis
he/his
commented

Thank you for the encouraging feedback. Please create a new issue for the issue so we can track and fix it.

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 — BETA2 » Subscriptions for D7 — BETA3
Version: 7.x-1.0-beta2 » 7.x-1.0-beta3

Here's BETA3 of Subscriptions. It fixes all active known issues.

Please see the BETA1 release notes if you are upgrading from an ALPHA version.

Please test this version and provide feedback, both good and bad:

IF it works for you, let us know by adding a comment to THIS issue below.

IF YOU FIND A BUG, have a question or a feature request, please check the issues queue and the change log, and if it hasn't been reported/fixed yet, then OPEN A NEW ISSUE!

Please do NOT add bug reports / questions / feature requests to this issue here.

LaFabrik’s picture

LaFabrik commented

Seems to work as espected :)

salvis’s picture

salvis
he/his
commented

Thanks, LaFabrik.

@all: If you'd like the [node:summary] token to work, then please review the #1300920: The [node:summary] token does not output anything for body fields without a manual summary core issue. This needs to be committed to D8 first, then it can be back-ported to D7...

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 — BETA3 » Subscriptions for D7 — BETA4
Version: 7.x-1.0-beta3 » 7.x-1.0-beta4

Here's BETA4 of Subscriptions. It fixes all active known issues. We expect this to be the last beta version before the release candidate.

If you are upgrading from an ALPHA version then see the BETA1 release notes for important information.

Please test this version and provide feedback, both good and bad:

IF it works for you, let us know by adding a comment to THIS issue below.

IF YOU FIND A BUG, have a question or a feature request, please check the issues queue and the change log, and if it hasn't been reported/fixed yet, then OPEN A NEW ISSUE!

Please do NOT add bug reports / questions / feature requests to this issue here.

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 — BETA4 » Subscriptions for D7 — RC1
Version: 7.x-1.0-beta4 » 7.x-1.0-rc1
Status: Active » Needs review

Here's RC1 of Subscriptions. It fixes a few remaining minor issues, mostly cosmetic. We're confident that it's ready for release.

If you are upgrading from an ALPHA version then see the BETA1 release notes for important information.

Please give this version one last work-out and provide feedback (both good and bad!) in the issues queue.

IF it works for you, let us know by adding a comment to THIS issue below.

IF YOU FIND A BUG, have a question or a feature request, please check the issues queue and the change log, and if it hasn't been reported/fixed yet, then OPEN A NEW ISSUE!

Please do NOT add bug reports / questions / feature requests to this issue here.

salvis’s picture

salvis
he/his
commented
Title: Subscriptions for D7 — RC1 » Subscriptions for D7
Version: 7.x-1.0-rc1 » 7.x-1.0
Assigned: salvis » Unassigned

We have Subscriptions 1.0.

Thanks to all who have helped to pull this through!

lhubbert’s picture

lhubbert commented

Awesome. Flawless upgrade from BETA, and just as I needed it most. Thank you!

dark_underground’s picture

dark_underground commented
Version: 7.x-1.0 » 7.x-1.0-beta3
Component: Code » Miscellaneous

The use of HTML in the mail template works perfectly although the compliance to the mail clients like thunderbird or outlook as well as webmailer like googlemail, gmx, yahoo has to be watched because they interpret it differently and don't accept some css-instructions.

salvis’s picture

salvis
he/his
commented
Version: 7.x-1.0-beta3 » 7.x-1.0
Status: Needs review » Fixed

Oh, great, that's very encouraging — thanks!

(Should have marked this fixed in #24)

Status: Fixed » Closed (fixed)

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