I was investigating how easy it would be to have the ability to add "foreign" nodes to my personal tracker, and it seems to be possible.

This patch should do that and also allow you to remove nodes from your tracker (except for nodes thatyou created which will always stay on).

This is just a starting point and I assume I have done many things wrong and every string added is IMO poor.

Note - this does NOT add email subscriptions - just lists them in your personal tracker.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch failed testing.

NaheemSays’s picture

Status: Needs work » Needs review
FileSize
5.25 KB

and now with a few fixes.

catch’s picture

Version: 7.x-dev » 8.x-dev
Status: Needs review » Needs work

This is a completely new feature and can't be considered during code slush, it also appears to remove the auto-subscription-by-commenting feature of tracker module.

NaheemSays’s picture

AFAIK it shouldn't remove the auto subscription (but I will look into it further).

My motivation for this was once drupal.org upgrades to drupal 7, this feature would be enough for many people - but that does not make it necessary for core.

(it also does not change any api, so is a lower risk feature, but at the same time I can see hundreds of patches that are either critical or need review for d7, postponing is probably expected.)

NaheemSays’s picture

Title: Tracker Subscriptions? » Refactor Tracker to allow for subscriptions
Version: 8.x-dev » 7.x-dev
Status: Needs work » Needs review
FileSize
3.9 KB

Changing tack - but it still may be too late for drupal 7.

This does not change the API, just refactors the code to move the tracker removal checks elsewhere, and thus will allow for a contrib module to provide the subscription UI without the need to fork _tracker_remove().

This patch should not cause any change in out of the box functionality, but allow the tracker module to be used to track "subscriptions" more freely.

NaheemSays’s picture

FileSize
3.97 KB

and fixing the logic.

Status: Needs review » Needs work

The last submitted patch failed testing.