Drobnosti k JS, 31.08.2015 16:52, Tori(m)

Týká se posílání nového příspěvku do galerie:

1. prvek s ID "showMoreDiv1" by měl být spíš div, když se do něj dávají blokové prvky.

2. Bug: Vyberu obrázek, objeví se mi select na přiřazení k atlasu. Změním obrázek na jiný, objeví se druhý select. Změním obrázek znova, objeví se třetí select. Tohle by se nemělo stát, bo všechny selecty mají stejné name a prázdné hodnota z nových selectů přepíše vybraný druh z prvního selectu. Stačilo by zjistit, jestli cílový prvek už něco obsahuje nebo ne, např. přidat do common. js před ř. 56 následující:

if (document. getElementById(target). childNodes. length != 0) { exit; }

(jen se musí zajistit, aby prvek #showMoreDivX neobsahoval ani bílé znaky)

3. Místo používání eval() na zpracování návratové hodnoty XHR (např. ve funkcích getAtlasList, validateUser, validateNazev) by se dalo zpracovat odpověď serveru jako JSON. Všechny současné prohlížeče to už pár let umí parsovat nativně, pro starší mám dobrou zkušenost s parserem json-sans-eval. Ze serveru už zřejmě jde JSON, zdá se mi.

• odpovědět
• 0
• 0

Re: Drobnosti k JS, 31.08.2015 17:06, alesjirsa(m)

Já myslím, že jsem dokonce jednou přiřadil jeden obrázek přes přidávání příspěvků do galerie do atlasu, ale vůbec ho to tam nepřidalo. Sice nevím, jestli jsem třeba neudělal něco špatně, a přece jen už to je dlouho, ale myslím, že by to stálo za ověření.

• odpovědět
• 0
• 0

Re: Drobnosti k JS, 06.09.2015 22:11, admin(m)

Teda, aby mi tu někdo takhle podrobně analyzoval javascript, to jsem ještě nezažil, krom basseta a jeho cenných rad s sql, díky Tori

ad 1. opraveno
ad 2. opraveno
ad 3. máš pravdu, ze serveru jde JSON, nemám ale sílu to teď přepisovat, na funkčnost to asi nemá vliv

• odpovědět
• 0
• 0