Skip to content
This repository has been archived by the owner on Sep 4, 2019. It is now read-only.

Corrected the variable name in jnext to prevent reference errors. #592

Open
wants to merge 1 commit into
base: next
Choose a base branch
from

Conversation

hoyoungjang
Copy link

No description provided.

@jeffheifetz
Copy link
Contributor

Has the webplatform updated this file at all?

@hoyoungjang
Copy link
Author

@jeffheifetz Webplatform does not have jnext module or .so.

@jeffheifetz
Copy link
Contributor

Unless things have changed wildly, thats still where the source for jnext lives

@hoyoungjang
Copy link
Author

I just checked that jnext under webplatform is using correct variable names.

@jeffheifetz
Copy link
Contributor

r+

@@ -138,7 +138,7 @@ function JNEXT_() {

var objNotify = self.m_arEvents[strObjId];
if( typeof (objNotify) == 'undefined') {
alert("Warning: No object with Id " + strId + " found for event " + strEvent);
alert("Warning: No object with Id " + strObjId + " found for event " + strEvent);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we change this from alert to console.log, its far less intrusive.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants