findDef (find definition) doesn't always work about tern

[marijnh]

Definition locations are currently only tracked for variables and function/object types. base.foo is not a variable, and holds a number, so Tern currently doesn't associate a location with it.

I could store locations in property definitions, but I'm not sure how useful that would be. Thoughts?

[jeffkenton]

Thanks. That was what I was seeing during debugging, but wasn't sure why. Our testers thought it was a natural thing to try.

[petetronic]

That would explain another scenario I believe this impacts - module exports... in the code below, usages of this module cannot jump to the exported title var definition because it is assigned a String. If you assign it an Object, jump to definition works.

/*global define */

define(function (require, exports, module) {
    "use strict";

    var title = "Apples";

    function getCompleteTitle() {
        return title + " and Oranges";
    }

    exports.getCompleteTitle = getCompleteTitle;
    exports.title = title;
});

[marijnh]

See commit aa3bcb8 , which cleans up the way properties and variables have their definition tracked, and fixes this issue in the process.