Notifications
Clear all

Limited Support

Our team is currently on holiday, so support will be limited during this period. Response times may be slower than usual, and some inquiries may be delayed.
We appreciate your patience and understanding, and we’ll resume our usual support by the end of August.

 

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

9 Posts
2 Users
0 Reactions
2,532 Views
Robert
Posts: 10616
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: 10616
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