Notifications
Clear all

wpForo 1.x.x URLs with query parameters break pagination

9 Posts
2 Users
0 Likes
1,446 Views
Robert
Posts: 9334
Admin
(@robert)
Support Team
Joined: 6 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.

Reply
SimSync2
Posts: 7
Topic starter
(@simsync2)
Active Member
Joined: 5 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']);

 

Reply
SimSync2
Posts: 7
Topic starter
(@simsync2)
Active Member
Joined: 5 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

Reply
Robert
Posts: 9334
Admin
(@robert)
Support Team
Joined: 6 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?

Reply
Page 2 / 2
Share: