Closed Bug 410721 Opened 17 years ago Closed 17 years ago

Forum topics are marked as not visited after a new topic is posted

Categories

(support.mozilla.org :: Forum, task)

task
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: jason.barnabe, Assigned: jason.barnabe)

Details

(Whiteboard: tiki_test)

Attachments

(1 file)

Topic links include topics_offset parameter, which causes every post to have a new URL after a new topic is posted. This makes Firefox think of the topic as a new page, and they are marked as not having been visited. Can we remove this parameter? What does it do?
It looks like this may have to do with the next topic/previous topic feature. There should be a way to have that feature without passing a parameter in all the time - just go to the next or previous thread in that forum based on the time of the last post.
http://support.mozilla.com/tiki-view_forum_thread.php?comments_parentId=3253&topics_offset=3&topics_sort_mode=lastPost_desc&forumId=3 After some discussion with forum contributors, I think we can live without the next/previous topic feature.
Assignee: nobody → jason_barnabe
Status: NEW → ASSIGNED
Attachment #298397 - Flags: review?(nelson)
It seems like the Tiki templates could use a URL generation function, something that takes a base and a hash of arguments and gives you a URL. I had to shift around a bunch of parameters because otherwise I would've ended up with a question mark then an ampersand in the URL after removing topics_offset.
The "feature" for "prev/next topic" is actually implemented using a bunch of 4 params - topics_threshold, topics_offset, topics_find, and topics_sortmode. I have removed all 4. Other than the change in tiki-view_forum.php, which is removing superflouus default vars anyway, I have not made any changes to php files. The templates have been modified to make this feature hidden.
Attachment #298397 - Flags: review?(nelson) → review+
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Much shorter URLs are good :)
Now on live server.
Status: RESOLVED → VERIFIED
Whiteboard: tiki_triage
Whiteboard: tiki_triage → tiki_test
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: