[BUG] Redirect na TR

[We Are Borg]

[BUG] Redirect na TR

Heb nu wel een beschrijving hoe je de bug zelf kunt ontdekken.

1. Ga naar een topic met meerdere pagina's en ga naar de 2de pagina, bijv klik
2. Selecteer de eerste reply door op de anchor te klikken, bijv klik
3. Klik nu op TR en je krijgt de volgende url

code:

http://gathering.tweakers.net/forum/insert_topicreport/1190009/last/1190009/0/1

Die nul hoort er niet tussen waardoor de pagina parameter (1) niet goed wordt gepakt, tenminste, dat denk ik dan Immers, als ik normaal op pagina 1 een TR aanmaak zonder eerst een anchor te selecteren, kom ik op de volgende url terecht en werkt het wel

code:

http://gathering.tweakers.net/forum/insert_topicreport/1190009/last/1190009/1

[ Voor 14% gewijzigd door We Are Borg op 09-02-2007 06:29 ]

[Spider.007]

Er staat inderdaad een '/0' teveel tussen; welke {variabele} gebruik je daar crisp?

[crisp]

Spider.007 schreef op vrijdag 09 februari 2007 @ 09:38:
Er staat inderdaad een '/0' teveel tussen; welke {variabele} gebruik je daar crisp?

gewoon {topic_insertreportlink}

die wordt in de database.class gevuld met

code:

http://gathering.tweakers.net/forum/insert_topicreport/1190009/last

- en kijk nu eens naar de ranzige trim op regel 200 in topic.class en je weet waar het fout gaat

Ik zou daar een |/\d+/(\d+|last)$| gebruiken als ik jou was

oh, en die .'/0/'. hoort daar denk ik ook niet...

Maar het blijft wel vreemd; insert_topicreport accepteert maar 2 parameters: topicid en offset, dus

code:

http://gathering.tweakers.net/forum/insert_topicreport/1190009/last/1190009/0/1

en

code:

http://gathering.tweakers.net/forum/insert_topicreport/1190009/last/1190009/1

zouden equivalent moeten zijn; alles na /last/ zou immers toch niet gemapped moeten worden...

maar goed, de generated url's zijn dus sowieso al fout

[ Voor 39% gewijzigd door crisp op 09-02-2007 10:11 ]

[Spider.007]

Fixed in 1.9.5

en een extra regeltje om dubbelposts te doen