Get rid of the repo query param for single packages
We keep package names unique, and if for some reason they wouldn't be we show both packages on that page then. Let's not add a query param to every URL for such an edge case.
This commit is contained in:
parent
106f1e42ec
commit
3c7444c0a3
@ -96,14 +96,12 @@ def package_url(request: Request, package: Package, name: str | None = None) ->
|
|||||||
res: str = ""
|
res: str = ""
|
||||||
if name is None:
|
if name is None:
|
||||||
res = str(request.url_for("package", package_name=name or package.name))
|
res = str(request.url_for("package", package_name=name or package.name))
|
||||||
res += "?repo=" + package.repo
|
|
||||||
if package.repo_variant:
|
if package.repo_variant:
|
||||||
res += "&variant=" + package.repo_variant
|
res += "?variant=" + package.repo_variant
|
||||||
else:
|
else:
|
||||||
res = str(request.url_for("package", package_name=re.split("[<>=]+", name)[0]))
|
res = str(request.url_for("package", package_name=re.split("[<>=]+", name)[0]))
|
||||||
if package.repo_variant:
|
if package.repo_variant:
|
||||||
res += "?repo=" + package.repo
|
res += "?variant=" + package.repo_variant
|
||||||
res += "&variant=" + package.repo_variant
|
|
||||||
return res
|
return res
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user