aboutsummaryrefslogtreecommitdiffstats
path: root/Zotlabs/Widget
diff options
context:
space:
mode:
authorHarald Eilertsen <haraldei@anduin.net>2022-11-26 15:45:37 +0100
committerHarald Eilertsen <haraldei@anduin.net>2022-11-26 15:45:37 +0100
commit2de2c8e207aae66280a7be6bccfe64558534a54e (patch)
tree92803e770316fa0154d3c7a725b651d1015178a0 /Zotlabs/Widget
parent41376ec2cda617549b717769f942019ebc2ba797 (diff)
downloadvolse-hubzilla-2de2c8e207aae66280a7be6bccfe64558534a54e.tar.gz
volse-hubzilla-2de2c8e207aae66280a7be6bccfe64558534a54e.tar.bz2
volse-hubzilla-2de2c8e207aae66280a7be6bccfe64558534a54e.zip
Fix Category widget URLs
The category widget template expects a URL without any query params, as it appends `/?cat=<selected category>` to the base URL passed to it. The Widget code tried to preserve any query param passed to it except for the `cat` query param. When passed to the template, this caused the invalid URLs to be generated. Example input url: https://example.com/channel/user?f=&tag=pasta The URL generated for the "Dinner" category would then be: https://example.com/channel/user&tag=pasta&tag=pasta/?cat=Dinner Which is troublesome in more than one way, and cause at least some search bots to go wild by sending requests with increasingly long URLs. This patch will simply discard the existing query params in the URL, so with the same input url, the generated URL for the "Dinner" category will now be: https://example.com/channel/user/?cat=Dinner This is comparable to what the Category Cloud and Tag Cloud widgets already do.
Diffstat (limited to 'Zotlabs/Widget')
-rw-r--r--Zotlabs/Widget/Categories.php11
1 files changed, 6 insertions, 5 deletions
diff --git a/Zotlabs/Widget/Categories.php b/Zotlabs/Widget/Categories.php
index b31856e48..b0eda253b 100644
--- a/Zotlabs/Widget/Categories.php
+++ b/Zotlabs/Widget/Categories.php
@@ -24,15 +24,16 @@ class Categories {
}
$cat = ((x($_REQUEST, 'cat')) ? htmlspecialchars($_REQUEST['cat'], ENT_COMPAT, 'UTF-8') : '');
- $srchurl = App::$query_string;
- $srchurl = rtrim(preg_replace('/cat\=[^\&].*?(\&|$)/is', '', $srchurl), '&');
- $srchurl = str_replace(['?f=','&f=', '/?'], ['', '', ''], $srchurl);
+
+ // Discard queries from the current URL, as the template expects a base
+ // URL without any queries.
+ $base = substr(App::$query_string, 0, strcspn(App::$query_string, '?'));
if($files) {
- return filecategories_widget($srchurl, $cat);
+ return filecategories_widget($base, $cat);
}
- return categories_widget($srchurl, $cat);
+ return categories_widget($base, $cat);
}
}