tracker: mark all as read
moshe weitzman - May 25, 2003 - 18:16
| Project: | Drupal |
| Version: | 7.x-dev |
| Component: | tracker.module |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
Jump to:
Description
I'd like to be able to 'catch up' all nodes in the tracker without having to actually visit all of them ... separately, this would be useful for forum.module as well.
#1
a patch for this is in my sandbox and awaits review. does not implement forum.module piece.
#2
kjartan proposed a better one, though that has notm hit core yet.
#3
I think this still stand. Follow up?
#4
see also http://drupal.org/node/24883
#5
don't see this getting into 5.x so changing to 6.x
see also:
http://drupal.org/node/63416
#6
Kjartan's tiny old patch is still a good way to do this. Needs refreshing. See http://lists.drupal.org/archives/drupal-user/2003-11/msg00008.html
#7
#8
I don't like this patch for two reasons.
1. It provides the ability to mark all nodes on the entire site as read. There are few sites for which that is what people really want to do. Separate groups, separate forums, separate node types, etc., are often viewed in different containers and people typically want to mark only those posts in a specific container as read. So, functionally, this patch is not a complete solution.
2. The implementation of adding a node 0 entry to the history table to represent "all nodes" is a hack. If we ever want to support referential integrity (which, granted, I do not think is possible anyway), this hack would be incompatible with it. The right way to mark a set of nodes as read, including all of them, is to add an entry to history for each node saying it is read; this, after all, is what would happen if all the nodes actually were read.
We should implement a function that takes an array of nids and marks them all as read.
Oh, and could someone explain to me why node_tag_new() is the name of the function that marks a node as no longer being "new" for a user? Why not node_tag_read($nid, $time = NULL) (defaults to now).