Seřazení poboček Uloženky dle jména, přidán chybějící tag </select>

pavelevap / ceske-sluzby

České služby pro WordPress

56 stars 25 forks source link

Seřazení poboček Uloženky dle jména, přidán chybějící tag </select> #149

Closed vrozkovec closed 7 years ago

pavelevap commented 7 years ago

Díky moc, vypadá to dobře, podívám se na to. Myslel jsem, že se to řadí automaticky, ale možná to mají řazené pomocí nějakého ID. A koukám, že jsem tam mohl použít rovnou funkci selected()...

pavelevap commented 7 years ago

Máte pravdu, pobočky jsou rozházené. Vzpomínám si, že jsem používal nějakou jejich ukázku (tuším zde) a tam to bylo seřazené (a mám pocit, že to bylo nějakou dobu seřazené i v rámci pluginu), ale API to vrací podle id poboček (patrně tak, jak vznikaly). Ještě se podívám, zda bychom to nemohli řešit přímo na straně API...

pavelevap commented 7 years ago

Tak na straně API to asi nepůjde :-(

Pobočky Uloženky nejsou seřazené: https://api.ulozenka.cz/v3/transportservices/1/branches

Zatímco pobočky DPD seřazené jsou: https://api.ulozenka.cz/v3/transportservices/5/branches

Proč tomu tak je se mi zjistit nepodařilo, ale budeme to muset opravdu ještě řadit.

pavelevap commented 7 years ago

Tak nakonec jsem to celé ještě trochu upravil:

Řazení podle abecedy pomocí sort() mi někde nefungovalo s diakritikou a abych zamezil různým konfliktům, tak jsem tam doplnil jeden z dostupných jednoduchých skriptů, který to vcelku dobře řeší. Ale uvítám případné náměty na změnu.
Řazení jsem nakonec přesunul do samostatné metody.
Trochu jsem přeuspořádal kód funkcí pro zobrazení poboček (asi by šlo časem obě funkce Uloženka/DPD sloučit do jedné).
Doplnil jsem funkci selected() pro zpřehlednění kódu.
Doplnil jsem i chybějící element select (Uloženka/DPD).

@vrozkovec: Ještě jednou velké díky za report a návrh řešení!

vrozkovec commented 7 years ago

Bezva, díky!