Dashticz - Bug report

[aiolos]

I think some default styling for images instead of icons is missing then. There have been changes in those parts and I don't have images as icons, so I didn't really notice

[blacksn0w]

Screen Shot 2017-11-23 at 12.15.02.pngNow I've got this little problem. Not to serious, just need to resize them myself

Got the same, since im using the .svg resizing is not easy possible.

[aiolos]

Screen Shot 2017-11-23 at 12.15.02.png
Now I've got this little problem. Not to serious, just need to resize them myself

I created the fix for this, I expect it to be merged soon.

[robgeerts]

Merged

[EdwinK]

Thanks Rob and aiolos

you rock

[htilburgs]

In the latest beta, Dashticz won't load correct / complete in IE11.
Missing a lot of switches.

Error (in dutch):
SCRIPT5008: Ongeldige linkerkant van toewijzing (incorrect assigning on the left side)
main.js (1050,33)

line 1049 - 1051:

Code: Select all

catch (err) {
             [html, addHTML] = handleDevice(device, idx);
            }

For as far as I can find:

this is a Illegal assignment (JavaScript)
You attempted to assign a value to a read-only identifier. You cannot assign a value to a read-only identifier. For example, host defined objects and external COM objects. To correct this error --> Do not assign values to read-only identifiers.

[htilburgs]

In the latest beta, Dashticz won't load correct / complete in IE11.
Missing a lot of switches.

Error (in dutch):
SCRIPT5008: Ongeldige linkerkant van toewijzing (incorrect assigning on the left side)
main.js (1050,33)

line 1049 - 1051:

Code: Select all

catch (err) {
             [html, addHTML] = handleDevice(device, idx);
            }

For as far as I can find:

this is a Illegal assignment (JavaScript)
You attempted to assign a value to a read-only identifier. You cannot assign a value to a read-only identifier. For example, host defined objects and external COM objects. To correct this error --> Do not assign values to read-only identifiers.

Tried it also on Microsoft Edge, same problem.
In my case my dashboard is on a Microsoft Surface RT (windows 8.1), that only supports Edge and IE11

[aiolos]

In the latest beta, Dashticz won't load correct / complete in IE11.
Missing a lot of switches.

Error (in dutch):
SCRIPT5008: Ongeldige linkerkant van toewijzing (incorrect assigning on the left side)
main.js (1050,33)

line 1049 - 1051:

Code: Select all

catch (err) {
             [html, addHTML] = handleDevice(device, idx);
            }

For as far as I can find:

this is a Illegal assignment (JavaScript)
You attempted to assign a value to a read-only identifier. You cannot assign a value to a read-only identifier. For example, host defined objects and external COM objects. To correct this error --> Do not assign values to read-only identifiers.

Tried it also on Microsoft Edge, same problem.
In my case my dashboard is on a Microsoft Surface RT (windows 8.1), that only supports Edge and IE11

On IE11 I can reproduce it, on Edge it works (but that's windows 10, so that might be different.) What version of Edge do you use? (I testen on Edge 38)
It is easy to fix, but problems like this are more likely to happen if we want to use the functions of the newer Javascript versions. IE11 is a bit getting the new IE6...

[htilburgs]

I can't find the version of Edge on the Windows Surface RT. Even don't know if its the real Edge or just IE11 in another look.
For IE:

11.0.9600.18838
Update: 11.0.48 (KB4047206)

It would be disappointing if we could'nt use IE11 as browser for Dashticz.

[aiolos]

You can find the version on https://www.whatismybrowser.com/

And yes, it would be disappointing if it doesn't work in IE anymore, but on the other hand: it is an old browser and edge is now the one to use (if you are required to use a Microsoft browser). Nevertheless this needs to be fixed, but there might be a moment where it won't be, since the new javascript functions can make the code much more readable and faster. (it's a pity Microsoft still supports IE, it would be better if they drop it completely and fully focus on Edge).

[htilburgs]

But for now, can you fix it? So I can update my dashboard with te latest settings.

[aiolos]

Yes, a PR is waiting to be merged. But I'm also still wondering what Edge version also causes this problem.

[robgeerts]

Merged, check latest beta!

[MariH]

Latest Beta is not working correctly with groups/scene's
Activating this code in CONFIG.js:

Code: Select all

blocks['s1'] = {}
blocks['s1']['width'] = 4;
blocks['s1']['title'] = 'Zitkamer';
blocks['s1']['hide_data'] = true;

will not send an 'ON' or 'OFF' command to 'group 1' but to 'switch 1'
this it out the domoticz logging:

Code: Select all

2017-12-03 15:39:24.286 User: Admin initiated a switch command (1/Weer: Luchtdruk/Off)

status is reflected OK

this use to work in earlier versions

[aiolos]

Latest Beta is not working correctly with groups/scene's

...

this use to work in earlier versions

Good find! The big cleanup a few weeks ago caused this. I pushed a fix for it, which Rob will merge very soon I think.

[robgeerts]

Merged!
Check out latest beta!

[aiolos]

Merged!
Check out latest beta!

You're just waiting for my pull requests all day, aren't you?

[MariH]

Guys,
thx for the quick fix, works as a charm!

[robgeerts]

Merged!
Check out latest beta!

You're just waiting for my pull requests all day, aren't you?

Yes, I've nothing better to do
No, I got notifications about new mail on my devices, as my laptop is almost always on I merge your request immediately

[htilburgs]

Yes, a PR is waiting to be merged. But I'm also still wondering what Edge version also causes this problem.

Aiolos, first thanx for the fix. It works like a charm again.
I'm using a Microsoft Surface RT Windows 8.1. I'm not sure it's EDGE that is installed. Maybe it's just an other way to show IE11 on the Windows 8.1 way.
I've been searching, but cannot find any version information.