KIZI / EasyMiner-MiningUI

Web UI for mining in new version EasyMiner Project
Apache License 2.0
3 stars 1 forks source link

Odebrání atributu ze vzoru pravidla někdy vyhazuje výjimku #46

Open vojir opened 9 years ago

vojir commented 9 years ago

v některých případech je při odebrání atributu z antecedentu či consequentu vyhocena výjimka u element.appendChild s chybou element is null

pokud šlo o poslední atribut v dané části pravidla a zároveň pouze jen někdy...

PreVaDu commented 9 years ago

zatím se mi podařilo zjistit, že je ten appendChild volán při grabu. Jelikož má grab výchozí hodnotu 'bottom', je asi tisíc možností, kde to může být (ta chyba)... Inject jsem postupným testováním vyloučil. Je třeba se asi zaměřit na graby cedentu/pravidla a seznamu attributů.

PreVaDu commented 9 years ago

Právě se mi stalo, že se chyba zobrazila při přidání atributu, druhého. Byl jsem hodně rychlý. Mám obecně podezření, že jde o chybu z přílišné rychlosti požadavků na UI a ono nestíhá těžkopádné přebarvování drop zón... Je na zvážení, zda neodlehčit ty animace o(d)barvování drop zón a nedat to jen jako přidání třídy, která dává danou barvu, dost by to ulehčilo systému...

vojir commented 9 years ago

Ohledně javascriptových animací jsem rozhodně pro jejich odebrání... (případně tam můžeme zkus dát animaci při změně background v css3)

PreVaDu commented 9 years ago

Zkusím se na to někdy v brzké době podívat.

PreVaDu commented 9 years ago

Animace jsem zrušil a nahradil je přímou změnou stylu - osobně v tom nevidím prakticky rozdíl a přitom to dělalo neplechu. Zatím se mi nepodařilo při testech na dříve avizovanou chybu narazit, tak snad to pomohlo.

vojir commented 9 years ago

problém stále přetrvává, chyba se objevuje nepravidelně, při odebírání položek z antecedentu klikáním na odkaz pro odebrání (vždy až u několikátého atributu v pořadí)

screen

PreVaDu commented 9 years ago

Situace je stále stejná a to i při přidání atributu. Souvislost s animacemi se zdá být mylná, zkusím se zaměřit na dragdrop funkcionalitu, zda není v nějaké své části voláno něco, co občas ztratí "element". V každém případě je na zvážení, do jaké míry je to tak zásadní bug, funkcionalitu neomezuje, UI nezamrzne.

vojir commented 9 years ago

Souhlasím s tím, že to nezpůsobuje zamrznutí UI, přesto nepůsobí nejlíp, pokud aplikace vyhazuje chyby. Napadá mne, jestli to nesouvisí s překreslováním vzoru pravidla po drag&drop operaci.

Případně zkuste doplnit k voláním funkcí pro přidání atributu do vzoru pravidla kontrolu, jestli je ho kam přidat.

PreVaDu commented 9 years ago

Kontrolou myslíte co? Funkce pro přidání atributu by necházely chybu při odebrání. Osobně to tipuji na něco u D&D, ale možností je mnoho. Budu postupně zkoušet. Nejhorší je ta nepravidelnost, takže se nelze spolehnout, že je úspěšné testování skutečně úspěšné...

vojir commented 9 years ago

chyba se objevuje náhodně, bez vlivu na běh aplikace (pravděpodobně jde o knihu při překreslování prvků)

odloženo na později...