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.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | tracker_refactor.patch | 3.97 KB | naheemsays |
| #5 | tracker_refactor.patch | 3.9 KB | naheemsays |
| #2 | tracker_subscriptions.patch | 5.25 KB | naheemsays |
| tracker_subscriptions.patch | 4.63 KB | naheemsays |
Comments
Comment #2
naheemsays commentedand now with a few fixes.
Comment #3
catchThis 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.
Comment #4
naheemsays commentedAFAIK 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.)
Comment #5
naheemsays commentedChanging 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.
Comment #6
naheemsays commentedand fixing the logic.