Currently the message_subscribe_email feature is under the general 'features' tab by default. It should be moved into the Messages project.

#2 1915364-message-subcsribe-remove-features-info-2.patch770 bytesezra-g
PASSED: [[SimpleTest]]: [MySQL] 9 pass(es). View
message_subscribe_email_package.patch501 bytesjaperry
PASSED: [[SimpleTest]]: [MySQL] 9 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


amitaibu’s picture

Status: Needs review » Needs work

Actually message_subscribe_email should not be a "feature". I prefer to manually declare all the needed hooks. Any chance you can work on it?

ezra-g’s picture

Status: Needs work » Needs review
770 bytes
PASSED: [[SimpleTest]]: [MySQL] 9 pass(es). View

Message subscribe already implements the necessary hooks. The only thing we need to do to prevent it from being a Features module is to remove the Features-specific entries from the .info file.

ezra-g’s picture

Title: Message Subscribe email should exist within other message features » Remove unnecessary Features lines from message_subscribe_info file
achton’s picture

Sorry, can't get #3 to apply against current -dev version.

patching file message_subscribe_email/
Hunk #2 FAILED at 7.
1 out of 2 hunks FAILED -- saving rejects to file message_subscribe_email/
ezra-g’s picture

It will only apply to the module pulled from Git - the dev version is packaged with added lines in the .info that cause the patch to fail.

amitaibu’s picture

Status: Needs review » Fixed


Status: Fixed » Closed (fixed)

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