Repository navigation
EcmaScript 5 support (Express server.delete - Syntax error on token ".") #70
Description
Syntax error raised with this code:
server.delete('/accounts/:id/contact',function(req,res) {
});where server was instantiated like this:
var express = require('express');
var server = express(); server.get() or server.post() are doing fine but server.delete() is raising a syntax error in Nodeclipse which is hardly correct to me since delete() is a valid route for express objects (see: http://expressjs.com/api.html#app.VERB - the app.routes section)
Activity
paulvi commentedon Sep 9, 2013
The cause is that
deleteis reserved word.This issues seems to be a part of a bigger one.
paulvi commentedon Sep 26, 2013
Nodeclipse is using Eclipse standard JSDT (JavaScript Development Tools) that is not yet ES5 aware.
Before raising issue on Eclipse Bugzilla https://bugs.eclipse.org/bugs/enter_bug.cgi?product=JSDT
I would ask on Stackoverflow.com and do a bit research.
Frankly, I am not thinking in this direction, but I would follow. ES5 support is essential nowdays.
paulvi commentedon Sep 26, 2013
Good link about ES5 http://ejohn.org/blog/ecmascript-5-strict-mode-json-and-more/
Wikipedia http://en.wikipedia.org/wiki/ECMAScript
From http://wiki.eclipse.org/JSDT says, that there was work on es4 support in Eclipse that was thrown away, because es4 specification was abandoned. That why Eclipse has dropdown list with only 1 value.
paulvi commentedon Sep 26, 2013
Solution may be to disable JSDT validation and use JSHint Eclipse.
Here is options for JSHint http://www.jshint.com/docs/options/#strict, http://www.jshint.com/docs/options/#node
That solutions needs time to to try. @nodeleaf Could you?
Template to look at https://gist.github.com/haschek/2595796
paulvi commentedon Sep 28, 2013
Thanks, that was nice reading for me.
Please notice that what you find under JavaScript in Preferences is pure JSDT functionality.
No work have been done in Nodeclipse for extending validation/warnings.
I have voted for https://bugs.eclipse.org/bugs/show_bug.cgi?id=223131
Please do the same to show Eclipse that we care.
paulvi commentedon Oct 16, 2013
Nodeclipse 0.7 will switch to JSHint, where ES 5 is default,
jshint/jshint#1312 -> TODO ask on mailing list https://groups.google.com/forum/#!forum/jshint
However
deletewill still be as keyword.bcochran-shutterstock commentedon Oct 18, 2013
Hey Paul, does that mean it'll still highlight AND show an error, or just highlight?
paulvi commentedon Oct 20, 2013
This is not related to ES5. And as this is Eclipse platform behavior,
bugs should be raise on https://bugs.eclipse.org/bugs/ for Eclipse
Platform/Workbench.
On 18.10.2013 23:43, nodeleaf wrote:
paulvi commentedon Oct 20, 2013
Just try out 0.7 (
clone,mvn packagethen install from local .zip)@bcochran-shutterstock @nodeleaf Well, syntax highlight will stay the same. Error and warning will be JSHint configured as defined in project
.jshintrcSome related link http://wiki.eclipse.org/JSDT
http://wiki.eclipse.org/JSDTDevelopment - how to become a JSDT Committer
Interesting topic: Remove default JSDT in Luna [4.4] or Fix it?
http://www.eclipse.org/forums/index.php/t/544073/
Well as for implementing ES5 for JSDT I even don't know if will be 1 month or 3 months
And I don't have such order of magnitude spare time to devote for free to Nodeclipse.
So I don't start, I just do a little research every time someone asks about it (actually thanks to @nodeleaf ).
Also making ES5 support outside of Eclipse JSDT will become wasted time once Eclipse team does it.
So to start with, one should learn how JSDT is going and understand those people who drive the JSDT project now.
paulvi commentedon Dec 5, 2013
For those interested, should follow https://bugs.eclipse.org/bugs/show_bug.cgi?id=223131
It was just marked as planed for 3.6, that is Eclipse 4.4 Luna release summer 2014
JSDT team now has 6 commiters.
paulvi commentedon Dec 11, 2013
provided with wizards
.jshintrchas line"es5" : false, // Allow EcmaScript 5 syntax.TODO check if it helps
paulvi commentedon Nov 6, 2014
also asked on http://stackoverflow.com/questions/26768829/nodeclipse-enide-delete-keyword-issue
disable JSDT #70 #159 #194