Bug URLField?

[MirekZv]

Narazil jste někdo na toto chování?

Pastnuté URL je v pořádku, ale při sledování odkazu přibyde v místě kurzoru "=" navíc (jak je vidět dole).

Debian 10 Testing KDE, Py 3.9.1, Dj 3.1.6, Chrome i Firefox

[starenka .]

Jen pro sichr, do zdrojaku toho html ses dival, ze to tam fakt je (tj, ze browserum nehrabe...)?

---
In Perl you shoot yourself in the foot, but nobody can understand how you did it. Six months later, neither can you. | print 'aknerats'[::-1]

--
--
E-mailová skupina djan...@googlegroups.com
Správa: http://groups.google.cz/group/django-cs
---
Tuto zprávu jste obdrželi, protože jste přihlášeni k odběru skupiny „django-cs“ ve Skupinách Google.
Chcete-li zrušit odběr skupiny a přestat dostávat e‑maily ze skupiny, zašlete e-mail na adresu django-cs+...@googlegroups.com.
Chcete-li tuto diskusi zobrazit na webu, navštivte https://groups.google.com/d/msgid/django-cs/0a4427a7-46c8-443b-a8f2-339034a178f3n%40googlegroups.com.

[Vladimir Linhart]

navic to asi nicemu nevadi? to = navic

Chcete-li tuto diskusi zobrazit na webu, navštivte https://groups.google.com/d/msgid/django-cs/CA%2B7MNVo0WRfmiknrf%3DdhOJuXiwwof0yQ9%3Dp9OC%3D3tWpD6G_i6w%40mail.gmail.com.

[MirekZv]

@Starenka: No jo. <p class="url">Aktuálně: <a href="https://mapy.cz/zakladni?planovani-trasy=&amp;x=14.3997275&amp;y=49.8658414&amp;z=14&amp;rc=9gzb-xWeX3h.ihoDdqEirpfDOiZIfdm5x-iZLdrC&amp;rs=coor&amp;rs=stre&amp;rs=stre&amp;rs=stre&amp;rs=coor&amp;rs=coor&amp;ri=&amp;ri=5184552&amp;ri=5184960&amp;ri=5185096&amp;ri=&amp;ri=&amp;mrp=%7B%22c%22%3A131%7D&amp;xc=%5B%5D">https://mapy.cz/zakladni?planovani-trasy&amp;x=14.3997275&amp;y=49.8658414&amp;z=14&amp;rc=9gzb-xWeX3h.ihoDdqEirpfDOiZIfdm5x-iZLdrC&amp;rs=coor&amp;rs=stre&amp;rs=stre&amp;rs=stre&amp;rs=coor&amp;rs=coor&amp;ri=&amp;ri=5184552&amp;ri=5184960&amp;ri=5185096&amp;ri=&amp;ri=&amp;mrp=%7B%22c%22%3A131%7D&amp;xc=%5B%5D</a>

@Ing.Vladimír: Jak nevadí ???? Proč si asi ukládáš linky? Aby někam vedly, né ???

Dne středa 17. února 2021 v 14:39:14 UTC+1 uživatel Ing. Vladimir napsal:

[starenka .]

tak to = tam fakt nic nedela, jak rika Vladimir...

---
In Perl you shoot yourself in the foot, but nobody can understand how you did it. Six months later, neither can you. | print 'aknerats'[::-1]

Chcete-li tuto diskusi zobrazit na webu, navštivte https://groups.google.com/d/msgid/django-cs/d34d76ed-e2aa-4dbe-80c8-164879c25d57n%40googlegroups.com.

[MirekZv]

PS @Ing.Vladimír: Ten odkaz tady v diskuzi jde prokliknout. Tak si to zkus a pak zkus odstranit to = za tím planovani-trasy. Trochu rozdíl, ne? A to je ještě úplná náhoda, že to není rozbité úplně.

Dne středa 17. února 2021 v 16:44:20 UTC+1 uživatel MirekZv napsal:

[MirekZv]

No, jestli myslíš "nic nedělá" ve smyslu že "nezobrazí tu trasu", tak to máš pravdu.

Mimochodem můžete přijít na výlet, zítra Čt 9:05 ze Smíchovského nádraží 338 do Štěchovic.

Dne středa 17. února 2021 v 16:46:31 UTC+1 uživatel starenka napsal:

[Radim Novotny]

Já bych tomu rozuměl tak, že to za otazníkem jsou parametry (query string). Parametr má název a hodnotu. Hodnota je nepovinná, takže ?param1=&param2=9 je ok. Pokud je to ve formátu ?param1&param2=9 tak se to zřejmě někde  normalizuje do toho ?param1=&param2=9 a param1 sice existuje, ale nemá hodnotu. Proto ta poznámka, že = nic nedělá. 

Přijde mi, že mapy.cz to používají nějak nestandardně.

URLField s tím evidentně nedělá žádný vopičárny. Udělá něco jako urlunsplit(urlsplit(value)) aby to bylo validní. Podíval bych se co je opravdu v databázi a jestli se to nepomate někde jinde.

Radim

Chcete-li tuto diskusi zobrazit na webu, navštivte https://groups.google.com/d/msgid/django-cs/5f488e00-9fef-41c4-bc8c-76c2c95b07e0n%40googlegroups.com.

[MirekZv]

@novotn

Máš pravdu, že to rovnítko se objevuje až za otazníkem a že je to teda nestandardní a dost možná to odporuje validnímu url.

To jsem si neuvědomil.

Ale i tak mi přijde dost divoké, aby si Django samo to url upravovalo.

I když asi z nevalidního na validní.

Dne středa 17. února 2021 v 17:32:12 UTC+1 uživatel novotn...@gmail.com napsal:

[MirekZv]

@novotn

PS: Ještě bych pochopil (i když potěšilo by mě to ještě míň), že by Django neumožnilo pomocí validace toho URLFieldu tu chybnou hodnotu uložit. Ale ono ji uloží a v databázi je ta správná (i když tedy zřejmě nevalidní). Pokroutí ji až při renderování href=..

Dne středa 17. února 2021 v 17:32:12 UTC+1 uživatel novotn...@gmail.com napsal:

Já bych tomu rozuměl tak, že to za otazníkem jsou parametry (query string). Parametr má název a hodnotu. Hodnota je nepovinná, takže ?param1=&param2=9 je ok. Pokud je to ve formátu ?param1&param2=9 tak se to zřejmě někde  normalizuje do toho ?param1=&param2=9 a param1 sice existuje, ale nemá hodnotu. Proto ta poznámka, že = nic nedělá. 

[Radim Novotny]

URLField validuje (upravuje) strukturu odkazu (scheme, host, port, path, fragment, query,) ale nevaliduje obsah  jednotlivých částí, takže URL je z tohoto pohledu validní

>>> urlsplit('scheme://blah:34787654/?query#frgmnt')
SplitResult(scheme='scheme', netloc='blah:34787654', path='/', query='query', fragment='frgmnt')

Ten problém je tedy někde v renderování. Zkusil jsem si to, ale nepodařilo se mi to zopakovat. 

Jak to renderuješ? Nemůže být problém někde tam? Tohle se ve standardním Django templates vyrenderuje stejně, jak je to v DB. 

<a href="{{ obj.url }}">{{ obj.url }}</a>

Radim

Chcete-li tuto diskusi zobrazit na webu, navštivte https://groups.google.com/d/msgid/django-cs/c0bd13df-649e-4264-85fc-f53c42015496n%40googlegroups.com.

[MirekZv]

Díky, Radime.

Já se renderováním (zatím) nijak nezabývám. Je to standardní widget pro URLField ve standardním adminu.

Dne čtvrtek 18. února 2021 v 7:14:09 UTC+1 uživatel novotn...@gmail.com napsal:

[Radim Novotny]

Pravda, já hlupák jsem nezkusil Edit v Django adminu, tam bych to viděl. 

Pokazí se to až v Adminu, kde AdminURLFieldWidget upraví href takto:
context['widget']['href'] = smart_urlquote(context['widget']['value']) if value else ''

No, a to smart_urlquote s tím provede to co s tím provede. Viz 
https://github.com/django/django/blob/3.1.6/django/utils/html.py#L221

(Pdb) value

'https://mapy.cz/zakladni?planovani-trasy&x=14.3997275&y=49.8658414&z=14&rc=9gzb-xWeX3h.ihoDdqEirpfDOiZIfdm5x-iZLdrC&rs=coor&rs=stre&rs=stre&rs=stre&rs=coor&rs=coor&ri=&ri=5184552&ri=5184960&ri=5185096'
(Pdb) smart_urlquote(value)
'https://mapy.cz/zakladni?planovani-trasy=&x=14.3997275&y=49.8658414&z=14&rc=9gzb-xWeX3h.ihoDdqEirpfDOiZIfdm5x-iZLdrC&rs=coor&rs=stre&rs=stre&rs=stre&rs=coor&rs=coor&ri=&ri=5184552&ri=5184960&ri=5185096'

Řešením by mohlo být použít ModelAdmin.formfield_overrides a pro URLField si udělat subclass AdminURLFieldWidget, který by to href neupravoval

a nebo se zahloubat do specifikace a vymyslet, jestli je chyba v Django smart_urlquote nebo urllib.parse.urlencode nebo jestli mají mapy.cz používají URL v rozporu se standardy :)

Radim

Chcete-li tuto diskusi zobrazit na webu, navštivte https://groups.google.com/d/msgid/django-cs/366ece3f-3cd0-4089-8e3d-1efeab465832n%40googlegroups.com.

[MirekZv]

Super. Děkuju za perfektní rozbor. Jistě za tím bylo plno práce.

Dne pátek 19. února 2021 v 11:23:42 UTC+1 uživatel novotn...@gmail.com napsal: