groove/Part-DB-server
1
0
Fork 0
mirror of https://github.com/Part-DB/Part-DB-server.git synced 2026-03-01 04:49:36 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

fix(generic info provider): urlencode url

Browse source
This commit is contained in:
Hannes Rüger 2026-02-11 22:24:13 +01:00
parent 41252d8bb9
commit 79513f368b
No known key found for this signature in database
GPG key ID: 3F842590882C2E0C
2 changed files with 4 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

1
src/Services/InfoProviderSystem/Providers/GenericWebProvider.php
View file

@ -310,6 +310,7 @@ class GenericWebProvider implements InfoProviderInterface
private function fixAndValidateURL(string $url): string
{
$originalUrl = $url;
$url = urldecode($url);
//Add scheme if missing
if (!preg_match('/^https?:\/\//', $url)) {

6
templates/info_providers/search/part_search.html.twig
View file

@ -110,10 +110,10 @@
{% if update_target %} {# We update an existing part #}
{% set href = path('info_providers_update_part',
{'providerKey': dto.provider_key, 'providerId': dto.provider_id, 'id': update_target.iD}) %}
{'providerKey': dto.provider_key, 'providerId': dto.provider_id|url_encode, 'id': update_target.iD}) %}
{% else %} {# Create a fresh part #}
{% set href = path('info_providers_create_part',
{'providerKey': dto.provider_key, 'providerId': dto.provider_id}) %}
{'providerKey': dto.provider_key, 'providerId': dto.provider_id|url_encode}) %}
{% endif %}
{# If we have no local part, then we can just show the create button #}
@ -133,7 +133,7 @@
target="_blank" title="{% trans %}info_providers.search.show_existing_part{% endtrans %}">
<i class="fa-solid fa-search"></i>
</a>
<a class="btn btn-primary" href="{{ path("info_providers_update_part", {'id': localPart.id, 'providerKey': dto.provider_key, 'providerId': dto.provider_id}) }}"
<a class="btn btn-primary" href="{{ path("info_providers_update_part", {'id': localPart.id, 'providerKey': dto.provider_key, 'providerId': dto.provider_id|url_encode}) }}"
target="_blank" title="{% trans %}info_providers.search.update_existing_part{% endtrans %}">
<i class="fa-solid fa-arrows-rotate"></i>
</a>