tas2580 / seourls

phpBB Extension: URL rewriting for phpBB forums
https://tas2580.net/downloads/phpbb-seo-url/
GNU General Public License v2.0
44 stars 25 forks source link

Nice to have: rendered redirect URL #58

Open RonaldKools opened 6 years ago

RonaldKools commented 6 years ago

Some templatesets (like Metro) use a popup to login. This means the redirect URL in the template variable S_LOGIN_REDIRECT will be filled with the current URL when you are reading a thread in viewtopic.php. This works perfectly without SEOUrls enabled, but it breaks when you use this extension (probably because phpBB doesn't "see" the topic ID).

A simple fix is to add this to listener.php:

'S_LOGIN_REDIRECT' => append_sid($this->base->generate_topic_link($event['forum_id'] , $data['forum_name'], $event['topic_id'], $data['topic_title'], $event['start'])),

Maybe this can be added in a future release?