Project

General

Profile

Actions

Defect #5945

closed

Forum message permalinks don't take pagination into account

Added by Felix Schäfer over 13 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Forums
Target version:
Start date:
2010-07-23
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

The forum message permalinks (i.e. what the "title" of the message (mostly "Re: Original title") points to) doesn't take into account pagination, i.e. messages not on the first page still point to /boards/:board_id/topics/:topic_id#message-:message_id, which will take you to the first page, but not to the right message.

One solution would be to make all those links like this /boards/:board_id/topics/:topic_id?r=:message_id#message-:message_id, which would take you to the right page each time, or add a parameter for the right page. The problem with the second solution is that it breaks if the pagination size is change at some point, the first solution might be a little more resource-hungry, and especially will reload the page each time you click on one of the permalinks on the page. That shouldn't happen too often though, so I'd favor the first solution more.

One thing that should also be thought out is if we the retain the AJAX pagination on the topics view, or if it should load a whole new page with the ?r=:id_of_the_first_message_on_the_next_page parameter.

An example of this behavior can be observed here: http://orga.fachschaften.org/boards/1/topics/31#message-58 doesn't take you to the correct message, http://orga.fachschaften.org/boards/1/topics/31?r=58#message-58 does.

Actions #1

Updated by Felix Schäfer over 13 years ago

  • Affected version (unused) set to devel
Actions #2

Updated by Jean-Philippe Lang over 13 years ago

  • Status changed from New to Closed
  • Target version set to 1.0.1
  • Resolution set to Fixed

Fixed in r3877.

Actions

Also available in: Atom PDF