Closed (won't fix)
Project:
Drupal core
Version:
8.0.x-dev
Component:
tracker.module
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
25 May 2003 at 18:16 UTC
Updated:
29 Jul 2014 at 17:23 UTC
Jump to comment: Most recent
Comments
Comment #1
moshe weitzman commenteda patch for this is in my sandbox and awaits review. does not implement forum.module piece.
Comment #2
moshe weitzman commentedkjartan proposed a better one, though that has notm hit core yet.
Comment #3
alexandreracine commentedI think this still stand. Follow up?
Comment #4
Wesley Tanaka commentedsee also http://drupal.org/node/24883
Comment #5
catchdon't see this getting into 5.x so changing to 6.x
see also:
http://drupal.org/node/63416
Comment #6
moshe weitzman commentedKjartan'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
Comment #7
bradlis7 commentedComment #8
bjaspan commentedI 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).
Comment #9
jody lynnThis is still a valid feature request for tracker.
Comment #10
marvil07 commentedLink mentioned on comment 6 is now broken.
I would like to say that to get kind of the same functionality on a D7 site I ended up creating a module(I started at http://drupal.org/node/515418) that adds a new action for marking as read one node(as mentioned on comment 8, it uses
node_tag_new()). Then I used views bulk operations to actually perform the multiple operation.In the other side, as mentioned on point 1 at comment 8, this would only part of the solution, so I would say to just only provide an action that calls
node_tag_new()in core, and leave the rest to contrib.Here a quick paste of the code required to implement such an action. Please let me now if it's a good idea and I can convert it into a patch against current mainline(fixing strings and moving to the right place).
Maybe change this issue title to "Add an action to mark as read" if the idea sounds good.
Comment #11
kattekrab commented11 years later, doesn't look like this is going to happen.