Completion doesn't work with referenced Object literal

[angelozerr]

Tern supports Object literal with this case :

requirejs.config({
  // here Ctrl+Space show baseUrl
});

But if you declare the object literal in a var, it doesn't work :

var config = {
  // PROBLEM : here Ctrl+Space doesn't show baseUrl
}
requirejs.config(config);

[marijnh]

Patches 71a7166 and c5db7a7 should resolve this.

[angelozerr]

@marijnh I have tested your patch and it works well if you type one character :

var config = {
  c // here Ctrl+Space show callback, config
}
requirejs.config(config);

but if you try to open the completion without characters, it doesn't work :

var config = {
  // PROBLEM : here Ctrl+Space doesn't show callback, etc
}
requirejs.config(config);

[marijnh]

This works for me (in the web demo).

[angelozerr]

The web demo http://ternjs.net/doc/demo.html doesn't work because of bad acorn path.

[marijnh]

Oops, forgot to upgrade Acorn on my server. Should be better now.

[angelozerr]

Oops, forgot to upgrade Acorn on my server. Should be better now.

Thanks @marijnh!

Indeed it works with HTML demo but in my case (execute tern inside node.js + lineCharPositions=false with Eclipse) it doesn't work. I will study why there is this problem.

[angelozerr]

@marijnh I have found my problem. It's because in my case query.omitObjectPrototype is set to true.

So I don't execute the following code https://github.com/marijnh/tern/blob/master/lib/tern.js#L694 because my completion is not empty

Tern demo works because query.omitObjectPrototype is set to false and completions result is empty, So https://github.com/marijnh/tern/blob/master/lib/tern.js#L694 can be executed to fill completion with properties of object literal.

To fix the problem, I think we should not provide methods of prototype (even if omitObjectPrototype is set to true) for object literal. See my PR #520