A bug at line 246 in org.fife.rsta.ac.js.SourceCompletionProvider - "this." is not implicit in JavaScript

[pvlasov]

I believe that line 246 in org.fife.rsta.ac.js.SourceCompletionProvider:
parseTextAndResolve(set, "this." + text);

shall be commented out because in JavaScript object properties shall be always resolved through this. which is different from Java behavior.

[stevenupton]

Hi pvlasov,

I believe I had to add this because otherwise this. was not recognised when parsing text. Perhaps I need to revisit it.

Steve

[pvlasov]

Steve,

I commented this line in my set-up and observed correct behavior in
which /this./ was not assumed and as such instance fields and methods
stopped from appearing in a list of completions after Ctrl-Space where
only locals, parameters, and globals belong. After the fix field and

methods appear only in the list after /this/.

Best regards, Pavel.

On 10/21/2013 5:20 AM, Steve Upton wrote:

Hi pvlasov,

I believe I had to add this because otherwise this. was not recognised
when parsing text. Perhaps I need to revisit it.

Steve

—
Reply to this email directly or view it on GitHub
#2 (comment).

[stevenupton]

OK, That is fine. I may need to move that code into a Rhino specific plugin. I needed it for some Server side scripting.