Configuration pencil icon missing from third party node. How to debug?

fahrvergnuugen · 25 June 2018 11:31

I want to use a third party node which connects to a websocket of an ISY994i. This node requires configuration, however the pencil icon used to open the config panel is missing and I can't figure out why.

I submitted this issue to the developer, but haven't gotten a response yet.

Screenshot (I can't provide a screenshot because new users are only allowed one image per post??)

There is also an error in the browser console: Screenshot2

I'm hoping someone here might be able to shed some light on this or at least confirm that my issue exists for them as well.

I can read/write javascript just fine but I'm having problems figuring out how to debug this particular problem.

fahrvergnuugen · 25 June 2018 11:32

Here's the screenshot of the missing config pencil icon

knolleary · 25 June 2018 12:06

Can you share your node’s HTML - in particular the object you pass to the RED.nodes.registerType function?

fahrvergnuugen · 25 June 2018 12:39

@knolleary

EDIT: Here’s the Node’s HTML file:

github.com

fahrer16/node-red-contrib-isy/blob/master/nodes/isy-nodes-websocket.html

<!--START Websocket Node-->
<script type="text/x-red" data-template-name="ISY Websocket">
    <div class="form-row">
        <label for="node-input-controller"><i class="fa fa-server"></i> Controller</label>
        <input type="text" id="node-input-controller">
    </div>
    <div class="form-row">
        <label for="node-input-name"><i class="fa fa-header"></i> Name</label>
        <input type="text" id="node-input-name" />
    </div>
</script>

<script type="text/x-red" data-help-name="ISY Websocket">
    <p>Connects to an ISY 994i controller and subscribes to all websocket messages.</p>
    <h3>Outputs</h3>
    <dl class="message-properties">
        <dt>payload <span class="property-type">xml document</span></dt>
        <dd>The raw XML included in the ISY Websocket event.</dd>
        <dt>info.event_type <span class="property-type">string</span></dt>
        <dd>Description of the event type (from XML control element).</dd>

This file has been truncated. show original

knolleary · 25 June 2018 12:45

Are there any errors in the browser’s javascript console when you open your edit dialog?

fahrvergnuugen · 25 June 2018 12:57

Yes, I posted a screenshot of the error in my first post: https://imgur.com/uLCzqS1.png

zenofmud · 25 June 2018 13:58

Works fine for me.

what versions of NR and node.js ?
What platform and OS?
any error messages on the NR log at startup?

fahrvergnuugen · 25 June 2018 14:35

@zenofmud:

NodeJS v 5.2.0
Node-Red v 0.18.7
Darwin 10.8.0 x64 LE (Mac OSX)
No errors of consequence in the startup log. Log is attached just in case…
25 Jun 13:29:19 - [info] Node-RED version: v0.18.7
25 Jun 13:29:19 - [info] Node.js version: v5.2.0
25 Jun 13:29:19 - [info] Darwin 10.8.0 x64 LE
25 Jun 13:29:22 - [info] Loading palette nodes
25 Jun 13:29:43 - [info] Dashboard version 2.8.2 started at /ui
25 Jun 13:29:43 - [warn] ------------------------------------------------------
25 Jun 13:29:43 - [warn] [node-red/rpi-gpio] Info : Ignoring Raspberry Pi specific node
25 Jun 13:29:43 - [warn] [node-red-contrib-isy/isy-controller] SyntaxError: Block-scoped declarations (let, const, function, class) not yet supported outside strict mode (line:1)
25 Jun 13:29:43 - [warn] ------------------------------------------------------
25 Jun 13:29:43 - [info] Settings file : /Users/paul/.node-red/settings.js
25 Jun 13:29:43 - [info] User directory : /Users/paul/.node-red
25 Jun 13:29:43 - [warn] Projects disabled : set editorTheme.projects.enabled=true to enable
25 Jun 13:29:43 - [info] Flows file : /Users/paul/.node-red/flows_mini.local.json
25 Jun 13:29:44 - [info] Server now running at http://127.0.0.1:1880/
25 Jun 13:29:44 - [warn]

Your flow credentials file is encrypted using a system-generated key.

If the system-generated key is lost for any reason, your credentials
file will not be recoverable, you will have to delete it and re-enter
your credentials.

You should set your own key using the ‘credentialSecret’ option in
your settings file. Node-RED will then re-encrypt your credentials
file using your chosen key the next time you deploy a change.

25 Jun 13:29:44 - [info] Starting flows
25 Jun 13:29:45 - [info] Started flows
25 Jun 13:29:45 - [error] [mysql:WeathercatDB] Database not connected
25 Jun 13:29:45 - [error] [mysql:WeathercatDB] Database not connected
25 Jun 13:29:45 - [info] [mqtt-broker:83cab2d2.393bd] Connected to broker: mqtt://127.0.0.1:1883
25 Jun 13:29:45 - [info] [mqtt-broker:46636c6c.207d1c] Connected to broker: mqtt://127.0.0.1:1883

ukmoose · 25 June 2018 14:51

nodejs v5 has been end of life for 2 years, it’s time to upgrade

The recommended nodejs version is the current LTS version

fahrvergnuugen · 25 June 2018 14:52

Yeah, that’s going to be a problem due to my hardware unfortunately. I’m stuck because everything requires a newer OS in order to compile and I can’t upgrade the OS ~~without completely killing the performance of the mac mini~~ because 10.6 is the last OS that supports the Core 2 Duo.

I will do some testing on my laptop and see if I can tie this issue to the nodejs version.

knolleary · 25 June 2018 14:56

@fahrvergnuugen I’ve just installed the ISY node and I get the same error as you when I add the node to the workspace.

So yes, something is broken with the node, but Node-RED ought to be handling it better whatever it is.

fahrvergnuugen · 25 June 2018 14:59

@knolleary Well that’s a relief - I’m really hoping this error isn’t due to my outdated nodejs install since there isn’t anything I can do to fix that besides throw $ at it.

zenofmud · 25 June 2018 15:34

odd because it works fine for me.
osx high sierra - nr 0.18.7 node v8.9.4

zenofmud · 25 June 2018 15:36

you are on NR 0.18.4 according to the log, latest is 0.18.7

fahrvergnuugen · 25 June 2018 17:31

@zenofmud sorry for the confusion - i am definitely running 0.18.7. I had accidentally pasted the startup log from a few weeks back. I edited my post above and put today’s startup log.

zenofmud · 25 June 2018 18:37

Soooo I wonder if this is part of the problem

fahrvergnuugen · 25 June 2018 19:27

@zenofmud That has to be the problem - it would also confirm that my issue is due to my outdated nodejs. The question now is - what version of NodeJS is @knolleary running?

I edited the node-red-contrib-isy/nodes/isy/isy.js and replaced let with var just to see what would happen. It progresses past that error now but fails at this.REST = function (url, successCallback = dummyRESTCallback, errorCallback = dummyRESTCallback) { with SyntaxError: Unexpected token =

My guess is that this syntax was added to nodejs.

knolleary · 25 June 2018 19:35

I have tested with both node 4 and node 8 - recreates in both. But yes, the node does require a newer node.js version. However the error you’re hitting is in the browser and shouldn’t have anything to do with the version of node.

By all means continuing speculating, but given I can recreate it, I will have a definitive answer for you shortly.

fahrvergnuugen · 25 June 2018 19:47

@knolleary, @zenofmud I modified the code by replacing let with var and by removing the inline function declare syntax and replacing with something that is effectively similar but not as elegant. The configure pencil icon now appears in the node.

knolleary · 25 June 2018 19:58

Ah right - there are actually two bugs here.

The ISY WebSocket node’s label function assumes this.controller exists, which it won’t for a newly added node. That is the cause of the stack trace you see in the browser log. But it is completely separate to the lack of the pencil icon.
The ISY Controller node requires node 6 or later. If you’re running on an earlier version then that node won’t get loaded. That means when you edit the websocket node it can’t find the isy-controller type so doesn’t build the usual config node ui with the pencil button etc.

Topic		Replies	Views
Node Palette Question General	4	248	18 August 2019
Debugging and Notification issues General	7	251	2 August 2021
'</>' icon missing in the editor in browser? Forum Help	13	43	14 September 2024
Event Handler for Node Connection Developing Nodes	8	353	1 June 2021
Fa-microchip icon Developing Nodes http-request	19	468	11 June 2022

Configuration pencil icon missing from third party node. How to debug?

You should set your own key using the ‘credentialSecret’ option in your settings file. Node-RED will then re-encrypt your credentials file using your chosen key the next time you deploy a change.

Related Topics

You should set your own key using the ‘credentialSecret’ option in
your settings file. Node-RED will then re-encrypt your credentials
file using your chosen key the next time you deploy a change.