FUDforum
Fast Uncompromising Discussions. FUDforum will get your users talking.

Home » FUDforum » FUDforum Suggestions » Topic notifications
Show: Today's Messages :: Unread Messages :: Show Polls :: Message Navigator
| Subscribe to topic | Bookmark topic 
Switch to threaded view of this topic Create a new topic Submit Reply
Topic notifications [message #26959] Tue, 23 August 2005 10:13 Go to next message
JamesS is currently offline  JamesS   United States
Messages: 275
Registered: July 2002
Location: Atlanta, GA
Karma: 0
Senior Member
add to buddy list
ignore all messages by this user
If I remember correctly, the software will stop sending a person notifications if he does not log in and view the messages for which he has received email notifications. I, quite frequently, just visit the link in the emails without logging in. This means I have viewed the message that I was notified of but the software doesn't realize it. I think it would be handy if the notification link included a unique user id so that the software can mark the message as read for the user even though he did not log in. It should not, however, log the user in when visiting the URL as that would be a huge security hole.
Re: Topic notifications [message #26961 is a reply to message #26959] Tue, 23 August 2005 11:11 Go to previous messageGo to next message
Ilia is currently offline  Ilia   Canada
Messages: 13241
Registered: January 2002
Karma: 0
Senior Member
Administrator
Core Developer
add to buddy list
ignore all messages by this user
Not entirely, you can disable "smart notification" at which point the forum will send e-mail notifications regardless of read status or previous notifications.

I think unique user ids and so on provide a nasty security problem.


FUDforum Core Developer
Re: Topic notifications [message #26968 is a reply to message #26959] Tue, 23 August 2005 12:52 Go to previous messageGo to next message
JamesS is currently offline  JamesS   United States
Messages: 275
Registered: July 2002
Location: Atlanta, GA
Karma: 0
Senior Member
add to buddy list
ignore all messages by this user
It would only be good for marking messages as read. If the id does absolutely nothing else then it poses no risk.
Re: Topic notifications [message #26978 is a reply to message #26968] Tue, 23 August 2005 19:19 Go to previous messageGo to next message
Ilia is currently offline  Ilia   Canada
Messages: 13241
Registered: January 2002
Karma: 0
Senior Member
Administrator
Core Developer
add to buddy list
ignore all messages by this user
There is just too much complexity involved in this, just to avoid having user to mark something read.

It'd require:
A new table with subscription type, id and unique key, entries for every e-mail sent (lots of inserts). Then we'd need to tailor email body for each user with their own, special unsubscribe link. Then there is a possible issue with what happens if the user never uses the link, the table will just keep on growing and growing. So no we need some cleanup code, another words quite a bit of a mess.


FUDforum Core Developer
Re: Topic notifications [message #26979 is a reply to message #26959] Tue, 23 August 2005 19:52 Go to previous messageGo to next message
JamesS is currently offline  JamesS   United States
Messages: 275
Registered: July 2002
Location: Atlanta, GA
Karma: 0
Senior Member
add to buddy list
ignore all messages by this user
I don't see it as that complex. I think the approach you have envisioned goes way too deep. The forum already has to pull up the thread based on the URL given to it; along with retrieving the data for the thread view it can process the "mark as read" code that already exists. Yes, the emails would have to be rendered for individual users instead of the generic notification that is sent now. I really don't see how that would be so complex either; plus it would be an opportunity to fix the notification language bug. I know that you see it as a feature in terms of speed but it is really a bug from the user's perspective.
Re: Topic notifications [message #26983 is a reply to message #26979] Tue, 23 August 2005 22:25 Go to previous messageGo to next message
Ilia is currently offline  Ilia   Canada
Messages: 13241
Registered: January 2002
Karma: 0
Senior Member
Administrator
Core Developer
add to buddy list
ignore all messages by this user
Sure, but it does not have user information unless the user is logged in. Since you want to mark it read without being logged in the extra steps are needed.

FUDforum Core Developer
Re: Topic notifications [message #26986 is a reply to message #26959] Tue, 23 August 2005 23:27 Go to previous messageGo to next message
JamesS is currently offline  JamesS   United States
Messages: 275
Registered: July 2002
Location: Atlanta, GA
Karma: 0
Senior Member
add to buddy list
ignore all messages by this user
The link that brought me to your response is:
http://fudforum.org/forum/index.php?t=rview&goto=26983#msg_26983

If the URL were something like:
http://fudforum.org/forum/index.php?t=rview&goto=26983&i=cz89t7l#ms g_26983

Where the 'i' variable identifies the person the email was sent to then I imagine it wouldn't be too difficult to adjust that user's viewed list. Would it not be one query to get the user and a second to update their list? I am assuming that this information is stored in the users table. I haven't taken the time to track down how all of this information is kept.
Re: Topic notifications [message #27006 is a reply to message #26986] Wed, 24 August 2005 09:33 Go to previous messageGo to next message
Ilia is currently offline  Ilia   Canada
Messages: 13241
Registered: January 2002
Karma: 0
Senior Member
Administrator
Core Developer
add to buddy list
ignore all messages by this user
Well, this id string needs to be kept somewhere, let's say the subscription table that has hundreds of thousands of entries. An extra field would add loads of overhead to the mix and require additional indexes.

FUDforum Core Developer
Re: Topic notifications [message #27007 is a reply to message #26959] Wed, 24 August 2005 09:38 Go to previous messageGo to next message
Anonymous   United States
Karma:
Not if you use the current index for the user. That information is already being retrieved when the emails are generated.
Re: Topic notifications [message #27008 is a reply to message #27007] Wed, 24 August 2005 09:40 Go to previous messageGo to next message
Ilia is currently offline  Ilia   Canada
Messages: 13241
Registered: January 2002
Karma: 0
Senior Member
Administrator
Core Developer
add to buddy list
ignore all messages by this user
What current index for the user, we don't have a user identifier other then the "mark-read" id.

FUDforum Core Developer
Re: Topic notifications [message #27010 is a reply to message #26959] Wed, 24 August 2005 09:55 Go to previous messageGo to next message
JamesS is currently offline  JamesS   United States
Messages: 275
Registered: July 2002
Location: Atlanta, GA
Karma: 0
Senior Member
add to buddy list
ignore all messages by this user
Looks to me like there is an "id" column in the users table and it seems to be the primary key.
Re: Topic notifications [message #27011 is a reply to message #27010] Wed, 24 August 2005 11:40 Go to previous messageGo to next message
Ilia is currently offline  Ilia   Canada
Messages: 13241
Registered: January 2002
Karma: 0
Senior Member
Administrator
Core Developer
add to buddy list
ignore all messages by this user
Sure, but if you just pass the user id, your can use the code to mark things read for ANY users. So, you need to have some validation string.

FUDforum Core Developer
Re: Topic notifications [message #27014 is a reply to message #26959] Wed, 24 August 2005 14:25 Go to previous message
JamesS is currently offline  JamesS   United States
Messages: 275
Registered: July 2002
Location: Atlanta, GA
Karma: 0
Senior Member
add to buddy list
ignore all messages by this user
That is what I wasn't thinking of. I guess there really would need to be a new column added to the user's information. Oh well, it was just a thought.
Quick Reply
Formatting Tools:   
  Switch to threaded view of this topic Create a new topic
Previous Topic: Java formatting
Next Topic: Bulk processing for User CP --> Subscriptions
Goto Forum:
  

-=] Back to Top [=-
[ Syndicate this forum (XML) ] [ RSS ]

Current Time: Sat Oct 21 08:21:42 EDT 2017

Total time taken to generate the page: 0.00819 seconds