Při skrytí atributu v paletě "Attributes" není zobrazen odkaz pro jeho opětovné zobrazení

KIZI / EasyMiner-MiningUI

Web UI for mining in new version EasyMiner Project

Apache License 2.0

3 stars 1 forks source link

Při skrytí atributu v paletě "Attributes" není zobrazen odkaz pro jeho opětovné zobrazení #59

Closed vojir closed 9 years ago

PreVaDu commented 9 years ago

[x] Upravit title "remove" vedle atributu v paletě atributů na "hide".
[x] Znovu zobrazovat odkaz pro zobrazení skrytých atributů.

PreVaDu commented 9 years ago

Snažil jsem se dohledat čím se to pokazilo, když to bylo dřív ok, neúspěšně. Přidal jsem tedy zobrazení a skrytí na patřičná místa a zrušil kontrolu, která to špatně vyhodnocovala. Je možné, že je problém ve stylu vykreslování, že už nemáme tolik reloadů nebo něco takového, skutečně nevím. Pokud to ale bude fungovat, tak bych to asi neřešil.

vojir commented 9 years ago

Potvrzuji, že funguje. Nešlo by to ale udělat bez reloadu celého UI?

PreVaDu commented 9 years ago

Ten tam byl a nevím vůbec proč. :( ARBuilder.js ř. 346 volá reloadData, mohu dát reloadAttributes (ř. 300), ale to má u sebe nějaké TODO a dělá to to samé - reloaduje Data.

Divné je, že DataDescription.js vrací u funkce hasHiddenAttributes (ř. 124) vždy 0. Přitom když si vypíšu keys toho allHiddenAttributes, mám tam vždy 22, this.$id je také vždy 22. Díky tomu nefunguje pořádně kontrola.

Celé skrývání je postavené na ukládání prvků do storage. Nechápu ten princip, proč to nemůže být obyčejné skrytí prvku? Bylo by to snažší, přirozenější a méně náchylné na chybu. Ale co hlavně nechápu je jak se to vlastně stalo, že to přestalo fungovat, kdy, v jaké verzi. Procházel jsem historii GIT a nic tomu nenasvědčuje.

vojir commented 9 years ago

Aha... Tak to budeme muset ještě nějak poladit - správně se to mělo ukládat na server (funkcí, která v praxi vlastně nic neukládala - tj. byla před několika měsíci zrušena).

Z hlediska zobrazení skrytých atributů to zatím nechte s tím reloadem. Připravím v rámci EasyMinerCenter funkci pro ukládání nastavení ke konkrétnímu mineru.

vojir commented 9 years ago

Blocked by KIZI/EasyMiner-EasyMinerCenter#79

PreVaDu commented 9 years ago

Možná přestalo zobrazování odkazu fungovat tím zrušením ukládání. Budeme to muset asi celé předělat. ;)

vojir commented 9 years ago

Na straně EasyMinerCenter připravena podpora pro ukládání nastavení mineru - adresy jsou v configu. Pro tento účel navrhuji nastavení cca: hiddenAttributes: názvy oddělené čárkami

Uložená nastavení jsou vracena nejen na požádání pomocí URL z configu, ale jsou zahrnuta také do výchozích informací o mineru v rámci getData requestu

PreVaDu commented 9 years ago

Funkce přepsána do nové podoby

používá uložení na serveru
JS ukládá v proměnné bez localStorage
skrytí pouze pomocí přidání třídy hidden

Zároveň byl upraven Config (config.latte na serverové části), byl o tom zaslán mail.

vojir commented 9 years ago

Bylo nutné opravit DataDescription.js (je nutné počítat s minerem, který nemá na serveru uloženou žádnou informaci o skrytých atributech). Zbytek ještě dootestuji.

PreVaDu commented 9 years ago

To jsem si neuvědomil, děkuji.