Neo4j node orion editor problem

[Darrell Taylor]

Hi,

It seems after the recent changes around editing templates that the neo4j node is unable to edit the cypher query, the code remains intact in the underlying flows file and is executed ok, but I'm unable to edit it in the UI.  I wonder if somebody could give some pointers so I can have a go at fixing this.

The error seems to be thrown when this line is hit...

https://github.com/trivielldev/node-red-contrib-neo4j/blob/master/neo4j/89-neo4j-cf.html#L32

this is logged in the console : 'oneditprepare 64616647.9b9e98 neo4j ReferenceError: require is not defined'

Thanks

Darrell

[Nicholas O'Leary]

HI Darrell,

we stopped using the Orion editor last February and moved over to ACE. This was for a number of reasons at the time as we were hitting a number of issues.

We left Orion in place to give nodes time to migrate over and then finally removed the Orion library from the 0.14 release. But we did drop the ball in communicating this - our apologies for that.

When we made that change, we added a utility function for nodes to use to access the editor widget; this gives us some ability to hide node implementations of the specifics of the editor and gives us the ability to swap in a different editor in the future if the need arises again.

It should be very quick to move over - here's all you need:

https://github.com/node-red/node-red/blob/master/nodes/core/core/80-function.html#L91

Apologies again for the inconvenience - it was an oversight on our part.

Nick

--
http://nodered.org
 
Join us on Slack to continue the conversation: http://nodered.org/slack
---
You received this message because you are subscribed to the Google Groups "Node-RED" group.
To unsubscribe from this group and stop receiving emails from it, send an email to node-red+u...@googlegroups.com.
To post to this group, send email to node...@googlegroups.com.
Visit this group at https://groups.google.com/group/node-red.
For more options, visit https://groups.google.com/d/optout.

[Darrell Taylor]

Hi,

Thanks for the pointers.  I've updated the node and created a pull request so hopefully the original author will process that through so it ends up on npm soon.

https://github.com/trivielldev/node-red-contrib-neo4j/pull/1

I noticed there are not many ACE language modes installed by default.  I wanted to use the SQL mode, but it's not there.  Do you know if there is a way to point ACE at the mode file in the local node directory?

Thanks

Darrell

[Dave C-J]

I'm sure we could be persuaded to add mode-sql to the core as it's only 1800 bytes.

[Dave C-J]

Darrell,

ok added to base on master git. Where / how did you want to use it exactly ? Does the neo node have an edit panel ?

[Darrell Taylor]

Hi,

Yes, the neo4j node has an edit panel for the cypher query.  SQL mode is probably good enough, but strictly it should be a 'cypher' mode which is neo4j specific.  I've not looked to see if there is one out in the wild that can be used.

This was my reason for asking if the mode file could be referenced withing the node directory, as this is a very specific requirement for cypher/neo4j.

Cheers
Darrell

On 4 Jul 2016 10:04, "Dave C-J" <dce...@gmail.com> wrote:

Darrell,

ok added to base on master git. Where / how did you want to use it exactly ? Does the neo node have an edit panel ?

--

http://nodered.org
 
Join us on Slack to continue the conversation: http://nodered.org/slack
---

You received this message because you are subscribed to a topic in the Google Groups "Node-RED" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/node-red/LdEqUd1DZHs/unsubscribe.
To unsubscribe from this group and all its topics, send an email to node-red+u...@googlegroups.com.

[Dave C-J]

There doesn't appear to be a cypher one in the default ace package - nor found easily on google. So is SQL mode of use or not ?

[Darrell Taylor]

I'll give it a try later and see how it looks with cypher.

But I'm sure it's useful for other nodes that do 'real' SQL.

On 4 Jul 2016 10:25, "Dave C-J" <dce...@gmail.com> wrote:

There doesn't appear to be a cypher one in the default ace package - nor found easily on google. So is SQL mode of use or not ?

--

[Darrell Taylor]

I've tried it and it looks ok.  I pushed the change up to the neo4j node pull request as well.

Thanks

On Monday, 4 July 2016 10:32:21 UTC+1, Darrell Taylor wrote:

I'll give it a try later and see how it looks with cypher.

But I'm sure it's useful for other nodes that do 'real' SQL.

On 4 Jul 2016 10:25, "Dave C-J" <dce...@gmail.com> wrote:

There doesn't appear to be a cypher one in the default ace package - nor found easily on google. So is SQL mode of use or not ?

--
http://nodered.org
 
Join us on Slack to continue the conversation: http://nodered.org/slack
---
You received this message because you are subscribed to a topic in the Google Groups "Node-RED" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/node-red/LdEqUd1DZHs/unsubscribe.

To unsubscribe from this group and all its topics, send an email to node-red+unsubscribe@googlegroups.com.