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

Files: 
CommentFileSizeAuthor
#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 ]

Comments

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
StatusFileSize
new770 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/message_subscribe_email.info
Hunk #2 FAILED at 7.
1 out of 2 hunks FAILED -- saving rejects to file message_subscribe_email/message_subscribe_email.info.rej
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

Committed.

Status:Fixed» Closed (fixed)

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