CoreCommunity ExtensionsIncubatorDistributionsTYPO3 4.5 ProjectsTYPO3 4.7 ProjectsTYPO3 6.0 ProjectsTYPO3 6.1 ProjectsTYPO3 6.2 Projects (+)

Bug #4778

Infinite redirect loop

Added by Peter Schuster over 3 years ago. Updated almost 2 years ago.

Status:New Start date:2009-09-27
Priority:Should have Due date:
Assignee:Peter Schuster % Done:

0%
Category:Forum
Target version:1.10.0
Votes: 1 (View)

Description

When after deleting a post mm_forum gets into an infinite redirect loop.
I think this is caused by the parameter "page=1" also there is only one page.

infinite_redirect.diff (517 Bytes) Peter Schuster, 2009-09-27 18:15

Associated revisions

Revision 4662
Added by Dmitry Dulepov over 6 years ago

Fixed bug 4812: RTE field does not respect system RTE configuration
Fixed bug 4778: Filelinks in FCEs are not registered as "references"

History

#1
Updated by Peter Schuster over 3 years ago

I think I found the bug: in get_pid_link() the page number is set to 1, instead of 0.
I attached a patch.

#2
Updated by Peter Schuster over 3 years ago

  • Category set to Forum
  • Assignee set to Peter Schuster
  • Target version set to 1.9.0

I think we either should drop the old, built-in pagebrowser or change the old pagebrowser to start with page '0' as first page.
That's the way tx_pagebrowse is doing it.

Currently there are lots of checks for 'doNotUsePageBrowseExtension' and it is not clear when to decrease or increase a page number to get the "right" one (as in this bug report).

What do you think about it?

#3
Updated by Martin Helmich about 3 years ago

  • Target version changed from 1.9.0 to 1.10.0

#4
Updated by Dmitry Dulepov over 2 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100

Applied in changeset r4662.

#5
Updated by Hauke Hain about 2 years ago

I support the idea to drop the support for the old style mm_forum pagebrowser.

#6
Updated by Ines Willenbrock almost 2 years ago

I too support this

regards - Ines

Also available in: Atom PDF