Duplicitní lidé na hlavní straně

[janoskaz]

CO

v pravém menu na hlavní straně se zobrazují 3 náhodně vybraní lidé. Občas se stane, že je tam někdo vícekrát, protože funkce, vybírající náhodně ze seznamu lidí, nekontroluje duplicity

KDE

na hlavní straně, ale potenciálně všude, kde se používá funkce choice_n_random ze souboru _includes/js/custom.js

ŘEŠENÍ

Nahradit funkci tímto:
(zkoušel jsem udělat pull request, ale nějak to nedopadlo a popravdě nepracuji s jekyllem, tak nevím, jestli je třeba i kompilace apod.)

var choice_n_random = function(data, n) { n = (typeof n !== 'undefined') ? n : 3; var choice = []; var selected = []; while(true) { var nr = Math.floor(Math.random()*data.length); if (selected.indexOf(nr) === -1 || data.length < n) { var value = data[nr]; if(value.description) { choice.push(value); selected.push(nr) if(choice.length == n) { break; } } } } return choice; };

[janoskaz]

Stále trvá, tady je screenshot::

[Aktron]

Tuším, že to je proto, že František Navrkal má uvedeny kategorie "poslanec" a "ústecký kraj" Pak ho to omylem vybere dvakrát. Asi by se tam měl dát nějaký IF. Každopádně jsem aspoň k F. Navrkalovi dodal, že je také poslanec aby se tam neobjevoval ten předseda kraje sám.

[janoskaz]

V prvním příspěvku jsem postnul upravenou funkci choice_n_random, kde jsou ty duplicity ošetřené, ale protože nepracuju s jekyllem a nevím, jestli to stačí takto opravit na jednom místě, nebo jestli se kompiluje apod, tak jsem to nemohl vyzkoušet. Myslím ale, že by mělo stačit nahradit tu jednu funkci a bude vyřešeno

[Aktron]

Na mně je to bohužel technicky také příliš složité to nahradit :-( Mrzí mně to, ale musel by nám pomoci někdo jiný kdo s tím umí víc. Ach jo.

[jitka]

Ptala jsem se Honzy Vorbela coby hlavního správce obsahu webu, jak si představuje banner. Jestli máme opravovat tohle generováňí nebo udělat konkrétní list či generovat z menši množiny. Odpovědí je, že nemá smysl to teď řešit.

čau, prosím tě, my připravujeme pořádnou koncepci, zatím ty "lidé" klidně skryj, budeme realizovat v horizontu pár měsíců komplexní update a nemá smysl teď řešit detaily.

[Fafrin]

Ok, tak můžem aspoň zabít nepříliš aktuální dole rotující baner na stránce lide?

[jitka]

Ok.