howardjones / network-weathermap

Network Weathermap draws diagrams from data
http://www.network-weathermap.com/
MIT License
426 stars 94 forks source link

New bug intoduced in editor in latest build (artifacts on top of editor see pic) #96

Closed ikorzha closed 7 years ago

ikorzha commented 7 years ago

artifact

Plus moving a parent node with child locked to it corrupts the parent node and results in PHP errors below:

Notice: Undefined index: node_iconfilename in /var/www/html/plugins/weathermap/editor.php on line 380 Notice: Undefined index: node_iconfilename in /var/www/html/plugins/weathermap/editor.php on line 384 Notice: Undefined index: node_iconfilename in /var/www/html/plugins/weathermap/editor.php on line 385

PS: I will get back to you on WARN26 issue. I'm will create a simple case scenario and get back to you...

ikorzha commented 7 years ago

Tested the patch, Howie that you did 40min ago for editor.php original problem reported still there. Please validate

Notice: Undefined index: node_iconfilename in /var/www/html/plugins/weathermap/editor.php on line 380 Notice: Undefined index: node_iconfilename in /var/www/html/plugins/weathermap/editor.php on line 384 Notice: Undefined index: node_iconfilename in /var/www/html/plugins/weathermap/editor.php on line 385 --NO ICON----ARTIFICIAL ICON--

howardjones commented 7 years ago

It works for me... both issues are part of the same problem (outputting part of the page in the wrong order).

ikorzha commented 7 years ago

I'm sorry must have been cache issue works now thanks :)