Notifications
Clear all

wpForo 1.x.x [Closed] URLs with query parameters break pagination

9 Posts
2 Users
0 Reactions
2,244 Views
Robert
Posts: 10591
Admin
(@robert)
Support Team
Joined: 9 years ago

I'm sorry but we don't support ?v=x cases yet. In regular cases this should not be in URL. We'll take this under consideration in future versions. Please let us know, If changing of SERVER variable helps. We may add an option for this.

SimSync2
Posts: 7
Topic starter
(@simsync2)
Active Member
Joined: 8 years ago

To make it work on my site, i just change line 242 in file "functions.php" after every update of wpforo:

old:
return $protocol . "://" . $_SERVER['HTTP_HOST'] . ($with_port ? $port : '') . $_SERVER['REQUEST_URI'];

new:
return $protocol . "://" . $_SERVER['HTTP_HOST'] . ($with_port ? $port : '') . (strpos($_SERVER['REQUEST_URI'], "?") ? substr($_SERVER['REQUEST_URI'], 0, strpos($_SERVER['REQUEST_URI'], "?")) : $_SERVER['REQUEST_URI']);

 

SimSync2
Posts: 7
Topic starter
(@simsync2)
Active Member
Joined: 8 years ago

Issue still active in version 1.4.5

My solution still works (at least for me), just the line hast changed: Now it's in line 237

Robert
Posts: 10591
Admin
(@robert)
Support Team
Joined: 9 years ago

As far as I remember your website is very specific, it has ?a=b Get variable in each page URL. So will your solution also work on regular websites? Have you tested for standard permalinks?

Page 2 / 2